Skip to content

Diagnostics: Tighten DP086 for async assemble contracts - #328

Merged
Skymly merged 2 commits into
mainfrom
feature/dp086-async-assemble-contract-copy
Aug 12, 2026
Merged

Diagnostics: Tighten DP086 for async assemble contracts#328
Skymly merged 2 commits into
mainfrom
feature/dp086-async-assemble-contract-copy

Conversation

@Skymly

@Skymly Skymly commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes #325

Test plan

  • CI Ci / CiPack green
  • Confirm DP086 void-assemble Verify snapshot and descriptor tests pass
  • Spot-check message text names product T vs task transport clearly

Note

Low Risk
Diagnostic message/description and test snapshot updates only; no generator logic, auth, or data-handling changes.

Overview
Updates DP086 (GenerateBuilderAssembleContractMismatch) message and description so valid [BuilderAssemble] returns are explicitly sync product T or async Task<T> / ValueTask<T>, and illegal cases (void, bare Task/ValueTask, duplicate assemble, multiple CancellationToken) are called out.

Also refreshes the void-assemble Verify snapshot, descriptor unit assertions, and changelog entry. No generator validation behavior changes in this PR.

Reviewed by Cursor Bugbot for commit 92d8324. Configure here.

Name sync product T and async Task/ValueTask returns so Phase 2 diagnostics match Spec #325.

Co-authored-by: Cursor <cursoragent@cursor.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale comment

Autopilot status

Merge-ready for @Skymly — no action taken this pass.

Gate Status
Conflicts None (MERGEABLE)
Review threads None open
Required CI Green (build-test, build-test-net48, pack)
Bugbot Pass

Note: branch is behind main (mergeStateStatus: BEHIND). Update from base before merge if branch protection requires an up-to-date head; otherwise safe to merge when you choose.

Leaving merge / auto-merge / ready-for-review to you.

Open in Web View Automation 

Sent by Cursor Automation: DesignPatterns — Autopilot PR

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale comment

Autopilot status

Merge-ready for @Skymly — no code changes this pass.

Gate Status
Conflicts None (MERGEABLE)
Behind main 0 (2 commits ahead)
Review threads None open
Required CI Green (build-test, build-test-net48, pack)

Leaving merge to @Skymly.

Open in Web View Automation 

Sent by Cursor Automation: DesignPatterns — Autopilot PR

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autopilot status

Merge-ready for @Skymly — no further action this pass.

Gate Status
Conflicts None (MERGEABLE)
Base sync Up to date with main (0 behind / 2 ahead)
Review threads None open
Required CI Green (build-test, build-test-net48, pack)

Leaving merge / auto-merge / ready-for-review to you.

Open in Web View Automation 

Sent by Cursor Automation: DesignPatterns — Autopilot PR

@Skymly
Skymly merged commit e739821 into main Aug 12, 2026
4 checks passed
@Skymly
Skymly deleted the feature/dp086-async-assemble-contract-copy branch August 12, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Diagnostics: Tighten DP086 for async assemble contracts

1 participant