Skip to content

feat: diagnostic help links and unified generated source headers (E3+E4)#115

Merged
Skymly merged 1 commit into
mainfrom
feat/e3-e4-diagnostic-header
Jul 8, 2026
Merged

feat: diagnostic help links and unified generated source headers (E3+E4)#115
Skymly merged 1 commit into
mainfrom
feat/e3-e4-diagnostic-header

Conversation

@Skymly

@Skymly Skymly commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • E3: Add shared DiagnosticHelpLink and populate description + helpLinkUri on all 44 generator/analyzer DiagnosticDescriptor instances (links to published diagnostics docs).
  • E4: Extend GeneratedSourceHeader with WritePrefix / ToSourceText string APIs; migrate SignalR, Mqtt, WebSocket, Grpc, Sse, Nats, and RestAPI emitters to emit // <auto-generated/> consistently.
  • Update Verify snapshots and mark ROADMAP E3/E4 complete.

Test plan

  • SignalR / Mqtt / WebSocket / Grpc / Sse / Nats R3 + Reactive generator tests
  • RestAPI generator tests
  • Observables.Analyzers.Tests
  • CI green on PR

Related

  • Observables.Docs PR for diagnostic table anchor IDs (help link targets)

Add DiagnosticHelpLink and description/helpLinkUri to all generator and
analyzer diagnostics (E3). Route IO domain emitters through GeneratedSourceHeader
string APIs for consistent auto-generated markers (E4).

Closes #114

@Skymly Skymly left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

E3+E4 looks good from a local pass: generator/analyzer tests and Verify snapshots updated for the unified // <auto-generated/> header. Diagnostic description + helpLinkUri wiring is consistent across domains; pairing with the Docs anchor PR should make IDE links land correctly.

@Skymly Skymly merged commit 3d70f42 into main Jul 8, 2026
20 checks passed
@Skymly Skymly deleted the feat/e3-e4-diagnostic-header branch July 8, 2026 12:21
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.

1 participant