Skip to content

fix(generation): surface validator diagnostics - #309

Merged
graphite-app[bot] merged 1 commit into
mainfrom
gen/gen-diagnostics
Sep 1, 2026
Merged

fix(generation): surface validator diagnostics#309
graphite-app[bot] merged 1 commit into
mainfrom
gen/gen-diagnostics

Conversation

@anandpant

@anandpant anandpant commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Scope

  • Capture every schema and flowchart semantic validation issue, including repair hints.
  • Return a bounded first eight diagnostics in the existing 422 malformed_output contract.
  • Surface every returned API issue message and hint in CLI failure details.

Proof

  • pnpm nx run-many -t typecheck,test,build - passed (52 tasks; 50 executed, 2 cache hits)
  • pnpm nx build-storybook diagram-ui - passed
  • Added generation, Playground API, and CLI diagnostic regression coverage.

This is PR 1 of 6 in the generation-reliability stack.

Copy link
Copy Markdown
Collaborator Author

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge - adds this PR to the back of the merge queue
  • fast - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Sketchi Web Preview

Status: deleted

Preview Worker cleanup has completed.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Sketchi Icons Preview

Status: deleted

Preview Worker cleanup has completed.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Sketchi Excalidraw Workspace Preview

Status: deleted

  • Surface: internal preview; not linked from public navigation
  • Project: excalidraw
  • Worker identity: sketchi-excalidraw
  • Route policy: internal canvas workspace; no public product domain
  • Preview Worker: sketchi-excalidraw-pr-309
  • Commit: d6a3d8faef11
  • Workflow run: https://github.com/shpitdev/sketchi/actions/runs/33561360245

Preview Worker cleanup has completed.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Sketchi Eval Harness Preview

Status: deleted

  • Surface: internal preview; not linked from public navigation
  • Project: eval-harness
  • Worker identity: sketchi-playground
  • Route policy: internal eval harness; no public product domain
  • Preview Worker: sketchi-playground-pr-309
  • Commit: d6a3d8faef11
  • Workflow run: https://github.com/shpitdev/sketchi/actions/runs/33561360245

Preview Worker cleanup has completed.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Sketchi Playground / Studio Preview

Status: deleted

  • Surface: public product preview
  • Project: playground
  • Worker identity: sketchi-studio
  • Route policy: playground.sketchi.app product surface; authenticated Studio remains unexposed
  • Preview Worker: sketchi-studio-pr-309
  • Commit: d6a3d8faef11
  • Workflow run: https://github.com/shpitdev/sketchi/actions/runs/33561360245

Preview Worker cleanup has completed.

@graphite-app

graphite-app Bot commented Sep 1, 2026

Copy link
Copy Markdown

Merge activity

## Scope

- Capture every schema and flowchart semantic validation issue, including repair hints.
- Return a bounded first eight diagnostics in the existing 422 `malformed_output` contract.
- Surface every returned API issue message and hint in CLI failure details.

## Proof

- `pnpm nx run-many -t typecheck,test,build` - passed (52 tasks; 50 executed, 2 cache hits)
- `pnpm nx build-storybook diagram-ui` - passed
- Added generation, Playground API, and CLI diagnostic regression coverage.

This is PR 1 of 6 in the generation-reliability stack.
@graphite-app
graphite-app Bot force-pushed the gen/gen-diagnostics branch from 00736da to d6a3d8f Compare September 1, 2026 21:25
@graphite-app
graphite-app Bot merged commit d6a3d8f into main Sep 1, 2026
10 checks passed
@graphite-app
graphite-app Bot deleted the gen/gen-diagnostics branch September 1, 2026 21:29
@graphite-app
graphite-app Bot deployed to production September 1, 2026 21:29 Active
@graphite-app
graphite-app Bot deployed to production September 1, 2026 21:29 Active
@graphite-app
graphite-app Bot deployed to production September 1, 2026 21:29 Active
@graphite-app
graphite-app Bot deployed to production September 1, 2026 21:29 Active
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