fix(planner): regenerate the stale plan-pane-bodies graph record - #4283
Open
kevinthelago wants to merge 2 commits into
Open
fix(planner): regenerate the stale plan-pane-bodies graph record#4283kevinthelago wants to merge 2 commits into
kevinthelago wants to merge 2 commits into
Conversation
…ssion Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PRyWeiBv8xpzk5VdfAQkVy
The graph-parity guard caught real drift: FocusedBodies.tsx moved on to FeaturesStageBody/PlannerLibraryPane (#4265) but the plan-pane-bodies seed record (its provides-tagged graph twin, #4227) was never regenerated, so the runtime loader was still vendoring the old FeaturesBody/PlannerComponentsPane switch wherever the pane composes FocusedBodies from the graph. Re-ran scripts/gen-pane-graph.cjs to bring the record and its derived graphPlatform.ts back in sync with the live file, and regenerated platform-modules.json (UPDATE_KITS=1) to match the updated registration set. Also confirms #3787's full scope (Planning.tsx / FocusedShell / ProjectPane / ProjectsList into the graph) is already delivered and live-mounted via already-merged #3874 + #4224 + #4227 + #4230; #3989 already merged via #4011. Decision notes recorded in DECISIONS.md, including ~15 stale DB-only duplicate component records found during verification that the planner role cannot prune itself (bsc ui is read-only for this role) — flagged for a session with UI write access. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PRyWeiBv8xpzk5VdfAQkVy
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.
Summary
graphParity.test.ts) caught real drift:FocusedBodies.tsxmoved on toFeaturesStageBody/PlannerLibraryPane(The project planner shows neither library — present both the components and algorithms panes #4265), but theplan-pane-bodiesseed record — itsprovides-tagged graph twin authored in Migrate the focused project pane into the graph (epic #3604) #4227 — was never regenerated, so the runtime loader was still vendoring the oldFeaturesBody/PlannerComponentsPaneswitch wherever the pane composesFocusedBodiesfrom the graph.scripts/gen-pane-graph.cjsto bringplan-pane-bodies.jsonand its derivedgraphPlatform.tsback in sync with the live file (verified every other pane/bodies/session/list record is still byte-identical — regenerating them would only have stripped their derivedcomposes/propsmetadata for no content change, so those were left untouched).platform-modules.json(UPDATE_KITS=1) to match the updated registration set.Status of owned issues
discovery/outage_response.md) #3989 (Discovery cloud-outage-response dimension) — already merged (Discovery: add a cloud outage response plan (discovery/outage_response.md) #4011).Also logged in
DECISIONS.md: ~15 stale DB-only duplicate/draft component records were found in the local component graph during verification (not seed-backed, so they don't affect the shipped app). The planner role'sbsc uiscope is read-only, so I can't prune them myself — flagged for a session with UI write access.Test plan
npm run typecheck— cleannpm run lint— 0 errors (225 pre-existing warnings outside this lane)npm test— 728/729 files pass; the 1 remaining failure (reactUiKit.gen.test.ts, 2 tests) is pre-existing ondevelopbefore this change, unrelated to this lane (src/features/designskit-hash drift)cargo test -p plandb -p bsc-blueprint— 150/150 passcargo clippy -p plandb -p bsc-blueprint --all-targets -- -D warnings— cleangraphParity.test.tsspecifically re-verified green after the fixCo-Authored-By: Claude Sonnet 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01PRyWeiBv8xpzk5VdfAQkVy