Repository navigation
Route Planner visual authoring through Jev - #415
Merged
Merged
Conversation
MaggieAppleton
force-pushed
the
maggie/seecode-document-diagrams
branch
from
October 8, 2026 20:10
fbb37ff to
0cbfd1e
Compare
MaggieAppleton
marked this pull request as ready for review
October 9, 2026 12:03
MaggieAppleton
enabled auto-merge (rebase)
October 9, 2026 16:52
MaggieAppleton
force-pushed
the
maggie/planner-visual-authoring
branch
2 times, most recently
from
October 9, 2026 18:15
5a13eda to
c782385
Compare
MaggieAppleton
force-pushed
the
maggie/planner-visual-authoring
branch
from
October 9, 2026 20:36
c782385 to
bd786af
Compare
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
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.
Jev visual routing update
PLANNER_VISUALS=on) for explicit member requests of at most 1,000 characters undercopilot-sdkorpi. It sends one explanatory passage to Jev for three small decisions: can it be visualized, would that aid comprehension, and which of Chopin’s 42 SeeCode types, table, or none fits. Accepted-comment turns, longer requests, background jobs, and Atomic retain their ordinary authoring path.edit_planaccepts only the assessed paragraph plus its selected diagram or table; human edits and Stop invalidate pending routes.Visual routing flow
Landing review · 9 October 2026
main, preserving all 48 feature files byte for byte before review fixes. The complete reconstruction diff from the old head exactly matched incomingmainchanges; no previously merged feature commits were replayed.5a13eda4is a single linear commit onmainat9623e25d. Local verification passed:bun test(4,326 passed, four environment-dependent skips, zero failures),bun run ci,bun run types, the web build, and 51 exact-case documentation links. GitHub runs the PostgreSQL, browser, and container checks on this head in CI run 37969587305. No assertions were loosened.New verification
noneroute. This probe did not run a live Planner authoring turn.flowchart(confidence 0.77). Planner attempted five diagram edits, all rejected asinvalid-visual; an offline renderer check found a 43-character node label over the 40-character limit. A fourth assessment was refused before HTTP dispatch. Planner saved prose and finished its Chat turn, but the runner stopped before a fresh browser reopen. E1 and D1 did not run. This is partial failure evidence, not a live diagram acceptance pass. The batch used one hosted Planner turn and three successful Jev calls.b4047aebreturns bounded field and constraint diagnostics fromedit_plan. An unavailable Jev assessment now returnsok: falsewithout minting or replacing a write route; a genuine Jev prose decision still permits prose. Focused fake-provider tests cover repair on the same route, initial provider failure, and a rejected diagram followed by failed reassessment. The repair itself made no hosted Planner or Jev calls.b4047aebcompleted in one Planner turn and three Jev calls. Jev answered possible 0.90, helpful 0.78, andflowchart(confidence 0.62). The first diagram edit received a precise 40-character label error; Planner shortened that label and saved the eight-node, nine-edge flowchart with the same route and no new assessment. Saved MDX matched a fresh reopen byte for byte, and the browser rendered one SVG without errors. The graph preserves the RFC's creation, polling,Pending,Ready, and completion transitions. Its right side needs horizontal scrolling in the split document pane. This is one R1 success, not a three-case or general acceptance claim. E1 and D1 remain unrun. Cumulative usage is 40 hosted turns and 20 successful Jev calls.b4047aeb: container, validation with unit and PostgreSQL tests, and browser integration. Browser E2E reported 476 passed, 5 skipped, and 2 flaky retries. The earlier headfd204251also passed CI.bun run typesandbun run cipassed. The ordinary default-off Planner prompt and instructions were verified byte-for-byte identical to the prior commit.Prior guidance change
askdecisions on the existing native authoring path. No new component or runtime is introduced.Paired development result
The same frozen R1, A1, E1, D1, and S1 inputs ran once with the baseline guide and once with the candidate guide, using
gpt-6-lunathrough Copilot SDK in fresh channels. Only the committed guidance and phase label changed. These were ten real Planner turns and zero Jev calls, with no case rerun for a better result.E1 changed from dense prose to a legible Markdown table comparing the compatibility and CLI/inline coverage arguments. R1, A1, and D1 remained prose; S1 remained an ordinary
ask. No diagram, Callout, or Tabs appeared in either arm. All ten documents saved and reopened without a render error. Both S1 questions added a fifth defer option beyond the four candidates named by the source. Rejected tool attempts were 4 in baseline and 7 in candidate; the small paired run does not establish that the guide caused this difference.This is a five-case development diagnostic, not a general accuracy claim. The guide shows one useful local improvement and no clear broader shift.
Live R1 UI after repair
The reopened document shows the new explanation and the start of the flowchart; the split pane requires horizontal scrolling to see its right side.
Real UI after reopen
The comparison shows the actual authored E1 document section in the same browser layout before and after the guide change.
Evaluation flow
Prior trial checks
bun run typesandbun run cipassed on the candidate commit;bun run fixapplied and its diff was inspected.AGENT=offbrowser passes showed the authored sections and S1 decisions after a fresh reopen.8f445cc8: container build, validation, and browser integration.Final integration validation: the feature was rebased onto merged
mainat1d788ad7, with source identical to the verified combined branch. With Atomic 0.9.31 installed, the full unit suite passed (4,405 passed, four skipped), focused Atomic/visual/graph/connector tests passed (172 passed, one skipped), andbun run ciandbun run typespassed. The merged connector change’s compact 390px touch Build regression passed. Earlier live R1 evidence retains its stated scope; this integration made no hosted Planner or Jev calls.Landing corrections: the fake Jev browser regression now selects the exact accessible flowchart group, so the merged diagram toolbar icons do not make its SVG assertions ambiguous. Real Chromium reproduced the failure and passed after the test-only correction, retaining edit/save/reopen checks. CI fetches the same pinned Bun and PostgreSQL images through Google’s Docker Hub mirror after two confirmed anonymous pull-limit failures; both mirrored index digests matched Docker Hub. Every required test remains enabled and production container configuration is unchanged. Fresh full CI runs against
5df1105a.