Put Code cards on the canvas and confirm recent deletes - #88
Conversation
Code view now opens selected and connected files as cards on the infinite canvas, linked by the existing graph lines, instead of a fixed right-hand split capped at four. Recents delete requires a second Confirm? click so a misclick does not drop a cached analysis. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Cards no longer snap back after a drag, and a remounted Code view starts from identity zoom so overlay cards are not offset. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5fc2dcee02
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
High-degree files now open at most 12 cards, preferring neighbors that already have source, so a cached GitHub reopen does not fire one getFile per edge. A leftover click after dragging a card is consumed so repositioning does not change the selection. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Addressed both Codex P2s on this branch:
Still one HTML file. Tests pass. Not merging. |
Cards grow to the full file so the canvas does the scrolling. Opened files auto-align in directory groups using the same parent folder frames the graph already draws. Symbols sit in a vertical list. Click-to-open connected files stays the default; other nodes fade instead of becoming more cards. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Applied eorroe’s #82 notes on this branch:
The two Codex P2s (12-card cap + ignore post-drag click) were already on this branch. Still one HTML file. Tests pass. Not merging. |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f470afdcf3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Code no longer waits for a click, which made it look like Graph. It now seeds the most-connected file plus neighbors immediately, keeps that set if the canvas is clicked, and fits the cards in view. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Graph vs Code looking the same was a real product bug: Code only opened cards after a click, so at rest it was the same node graph (sometimes faded). Pushed After Vercel rebuilds, analyze a repo and toggle Graph vs Code. They should be obviously different at rest. |
Color By is a small canvas control on Graph/3D Graph instead of a tall sidebar block. The left rail is now full-height Overview, Files, and Recents tabs so the file tree can use the whole column. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Pushed
Not merged. |
First paint and a newly loaded analysis open Overview. Files and Recents stay one click away; last tab is not persisted across visits. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Pushed First paint and a newly loaded analysis select Overview, not Files or Recents. Switching still works. Last tab is not persisted across visits. Not merged. |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6334bd90d6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Auto-fit now runs at most once when entering Code. Opening files, hydrating source, or resizing cards no longer refits or regrids the canvas. The user pan/zoom stays put and cards keep their readable size. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Pushed The unlabeled hypothesis was right — a fit-to-cards path was firing on graph rebuilds (including source hydration) and on every card-set change, which zoomed the world out and made cards/nodes look tiny. Auto-fit now runs at most once when first entering Code. Opening more files, dragging, and zooming leave the viewport where you put it. Cards keep their full-file size. Fit View still works if you ask for it. Not merged. |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 013558bedb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Cards are HTML overlays. Wrapping them in foreignObject would not make a real standalone SVG, so Code view disables SVG/PDF in the export modal and keeps reports and share links. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Landed
|
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9a5da0198e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Folder filters no longer delete placements for still-open cards, so dragged cards restore when the filter clears. Far-zoom clipped bodies are not native-scroll targets, so unmodified wheel pans the canvas. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Landed
|
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Routing the tree through goToFile closed the props object too early, so Babel died and the page stayed blank. The remaining keys are back inside that object. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Landed Routing Files-tree clicks through
|
|
Blank-screen parse error from #82 is fixed in |
Loading babel.min.js in a VM contended with the 3k-file analysis budget on CI. Node's parser still catches the TreeNode syntax error that blank-screened the page. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Landed The new Babel VM smoke test itself passed, but it loaded
|
Folder hulls now read live node pins and the painted card boxes, so they follow a card after drag instead of only refreshing on layout. Code view gets top-right Scroll/Expand and word-wrap toggles. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Pulled eorroe’s #82 retest (comment 5374655600). Hulls now wrap the painted card boxes (and leftover nodes in the same folder) from live pins, so they follow a drag instead of only refreshing on layout/spread. Code view also has top-right prefs, same style as Color By:
Line-thickness and 3D moving links still later. Same PR, not merged. |
Code view now always shows a top-right Expand All / Wrap Text bar, even when both prefs are off, and Expand+Unwrap keeps a horizontal scrollbar. Cards get right/bottom resize handles. Folder hulls wrap live card boxes, ignore far leftover nodes, and sit on Graph-like folder centers so sibling hulls stay apart after a drag. Co-authored-by: Braedon Saunders <braedonsaunders@users.noreply.github.com>
|
Pulled eorroe’s latest #82 notes (comment 5383189364). On 88 now (
Zoom-out color blocks, mini map, and a root-file-count gate are still later. Line-number pills unchanged. Same PR, not merged. |
2 participants
What this PR does
Two follow-ups from issues #82 and #85, plus layout fixes from live preview, all in the same one-file app.
Code cards live on the infinite canvas (#82). Switching to Code no longer opens a fixed right pane capped at four files. Code seeds one card on enter (current selection, or the most-connected file in the folder filter) so Graph vs Code is obvious. After that, files get onto the canvas only when you open them (leftover node or Files tree). Cards are full source panes: uniform width, height follows line count, folder group frames, line-level bezier edges when we have a symbol/line. Opened cards auto-append to a per-directory column. Existing cards do not move or resize when you click another node.
Code camera stays put. Auto-fit runs at most once when first entering Code. After that the camera is user-owned. Opening a new file may fly to that card only. Clicking a card, symbol, or leftover node does not refit. Wheel pans; Ctrl+wheel zooms about the cursor.
Cards replace their nodes. An open file card does not also show a tiny circle underneath. Remaining nodes stay Graph-sized, stay clickable, and are pushed out from under cards. Selection is overlay-only (highlight + symbol pills on the selected card). Click empty canvas to deselect; open cards stay.
Right-rail subtabs stay level. FILE / PATTERNS / SECURITY / ACTIONS sit on one baseline. Count pills are inline and do not wrap or shift alignment.
Confirm before deleting a recent analysis (#85). Delete is two clicks on the same control: Delete, then red Confirm?. Escape, a 4s timeout, or choosing another row cancels. No modal.
Left rail and Color By. Color By (Folder / Layer / Churn) is a small control on the Graph / 3D Graph canvas, not a tall sidebar block. The left rail is full-height tabs: Overview (health/stats), Files (folder tree), and Recents (cached analyses). After analyze, and on first paint, Overview is selected. Files and Recents stay one click away.
Out of scope
CLI diffs stay out of this PR. No second product, no extra HTML file. Do not merge until Braedon reviews.
How to verify