Skip to content

fix(artifacts): scope coordinator and writer write_artifact schemas to their kinds - #76

Merged
TheGreatAxios merged 5 commits into
mainfrom
cl-11952-scope-coordinator-and-writer-write_artifact-schemas-to-the
Nov 2, 2026
Merged

TheGreatAxios merged 5 commits into
mainfrom
cl-11952-scope-coordinator-and-writer-write_artifact-schemas-to-the

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-11952

The coordinator's write_artifact tool now uses BriefWrite (kind: "problem_brief", title, content, optional openQuestions, with no approaches field or other document kinds). The writer's write_artifact tool uses WriterDocumentWrite, an object schema scoped to the writer kinds (constraints, approaches, product_requirements, build_plan) that narrows approaches to kind === "approaches" while keeping top-level type: "object" in toJsonSchema() for provider tool registration. Both @corbits-builder/coordinator (0.35.1) and @corbits-builder/writer (0.7.1) are patch-bumped.

Verified:

  • bun --conditions=intx-src test apps/hub/src/artifacts/tools.test.ts apps/hub/src/projects/errors.test.ts (5 pass, 0 fail, including top-level inputSchema.type === "object" across all roles)
  • pre-commit: bun fmt --no-script --check, oxlint --deny-warnings, and bun check (exit 0)

@linear-code

linear-code Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

CL-11952

CL-11928

@corbits-triage

corbits-triage Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Triage will approve, request review.

@corbits-triage corbits-triage Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Triage requests changes:

  • CI green: required checks are failing
  • Conventions: added code matches a code pattern

@TheGreatAxios
TheGreatAxios force-pushed the cl-11952-scope-coordinator-and-writer-write_artifact-schemas-to-the branch from af5345b to b99a7e5 Compare November 2, 2026 03:48
@corbits-triage
corbits-triage Bot dismissed their stale review November 2, 2026 03:52

Nothing triage asked to change still fails, so this request no longer stands.

corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 2, 2026
WriterDocumentWrite is a union so approaches is a field only when kind is approaches. The writer tool rejects the key on the other kinds before POST.
constraints, product_requirements, and build_plan each accept only kind, title, content, and openQuestions. approaches is the only arm that also accepts approaches.
corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 2, 2026
@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Currently the removal of top level object is causing the head of this PR to be incompatible with Anthropic, fixing as it's being used in relation to the Designer Improvement PR

corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 2, 2026
@TheGreatAxios
TheGreatAxios added this pull request to the merge queue Nov 2, 2026
Merged via the queue into main with commit ea054ea Nov 2, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant