Skip to content

ci(dx): require @example on the authoring surface#717

Merged
blove merged 1 commit into
mainfrom
ci/dx-guard-examples
Jun 20, 2026
Merged

ci(dx): require @example on the authoring surface#717
blove merged 1 commit into
mainfrom
ci/dx-guard-examples

Conversation

@blove

@blove blove commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

What

Tightens the DX-coverage guard (#715) so that authoring-surface functions — the ones developers call to wire/use the framework — must carry an @example, not just a summary.

Authoring surface = provide* / inject* / mock* + the client-tool builders (tools/action/view/ask) + the view-registry helpers (views/withViews/withoutViews/overrideViews/toRenderRegistry) + defineAngularRegistry/createAgentRef/signalStateStore. (Plumbing/utility functions still only need a summary — examples there would be noise.)

Documented the stragglers

Added @example to the authoring-surface functions that lacked one: render views/withViews/withoutViews/overrideViews/toRenderRegistry + injectRenderHost, ag-ui + langgraph provideFakeAgent, langgraph mockLangGraphAgent.

Result

✓ DX-coverage: all 65 public functions have a summary; all 29 authoring-surface functions have an @example.

render/ag-ui/langgraph build green; api-docs regenerated/committed.

🤖 Generated with Claude Code

…gglers

Tightens the DX-coverage guard: functions developers call to wire/use the
framework (provide*/inject*/mock*, the client-tool builders, and the view-
registry helpers views/withViews/withoutViews/overrideViews/toRenderRegistry)
must now carry an @example, not just a summary.

Added @example to the authoring-surface functions that lacked one: render
views/withViews/withoutViews/overrideViews/toRenderRegistry + injectRenderHost,
ag-ui + langgraph provideFakeAgent, and langgraph mockLangGraphAgent.

Guard: "all 65 public functions have a summary; all 29 authoring-surface
functions have an @example." render/ag-ui/langgraph build green; api-docs regenerated.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@blove blove enabled auto-merge (squash) June 20, 2026 02:14
@vercel

vercel Bot commented Jun 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment Jun 20, 2026 3:02am

Request Review

@github-actions github-actions 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.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Claude encountered an error after 1m 52s —— View job


PR Review

  • Gather context
  • Read diff and changed files
  • Review check-dx-coverage.mjs logic
  • Review added @example blocks in source files
  • Post findings

@blove blove merged commit d50b946 into main Jun 20, 2026
62 of 63 checks passed
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