fix(generation): derive mindmaps from nested output - #312
Merged
Conversation
Collaborator
Author
How to use the Graphite Merge QueueAdd either label to this PR to merge it via 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. |
This was referenced Sep 1, 2026
anandpant
force-pushed
the
gen/gen-mindmap-nested
branch
from
September 1, 2026 18:38
47cc19f to
c35165e
Compare
anandpant
force-pushed
the
gen/gen-prompt-rules
branch
2 times, most recently
from
September 1, 2026 18:52
0565fe2 to
ded1ec6
Compare
anandpant
force-pushed
the
gen/gen-mindmap-nested
branch
2 times, most recently
from
September 1, 2026 19:27
779498f to
308fe6e
Compare
anandpant
force-pushed
the
gen/gen-prompt-rules
branch
from
September 1, 2026 20:15
ded1ec6 to
efd5be9
Compare
anandpant
force-pushed
the
gen/gen-mindmap-nested
branch
from
September 1, 2026 20:15
308fe6e to
0cac637
Compare
Merge activity
|
## Scope - Replace model-authored flat mindmap IDs, edges, and depth metadata with a typed recursive tree contract. - Derive canonical flat mindmap IR deterministically in code while retaining the existing flat parser for callers. - Update mindmap instructions and compact example to require the top-level diagram ID while forbidding only derived topic/node/edge IDs. ## Proof - `pnpm nx run-many -t typecheck,test,build` - passed at this branch. - `pnpm nx build-storybook diagram-ui` - passed at this branch. - Added recursive property coverage proving arbitrary generated trees derive into valid flat mindmap IR. - Added deterministic ID, edge, depth, sibling-order, and prompt-contract coverage. This is PR 4 of 6 in the generation-reliability stack.
graphite-app
Bot
force-pushed
the
gen/gen-prompt-rules
branch
from
September 1, 2026 21:27
efd5be9 to
01a254a
Compare
graphite-app
Bot
force-pushed
the
gen/gen-mindmap-nested
branch
from
September 1, 2026 21:27
0cac637 to
f014afc
Compare
Contributor
Sketchi Playground / Studio PreviewStatus:
Preview Worker cleanup has completed. |
Contributor
Sketchi Icons PreviewStatus:
Preview Worker cleanup has completed. |
Contributor
Sketchi Excalidraw Workspace PreviewStatus:
Preview Worker cleanup has completed. |
Contributor
Sketchi Eval Harness PreviewStatus:
Preview Worker cleanup has completed. |
Contributor
Sketchi Web PreviewStatus:
Preview Worker cleanup has completed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Proof
pnpm nx run-many -t typecheck,test,build- passed at this branch.pnpm nx build-storybook diagram-ui- passed at this branch.This is PR 4 of 6 in the generation-reliability stack.