Skip to content

feat(app): relocate context tree from top panel to context side panel (amicode#264) - #149

Merged
jeonghun-jj-lee merged 1 commit into
local/amicodefrom
amico/issue-264-context-tree-to-side-panel
Aug 9, 2026
Merged

feat(app): relocate context tree from top panel to context side panel (amicode#264)#149
jeonghun-jj-lee merged 1 commit into
local/amicodefrom
amico/issue-264-context-tree-to-side-panel

Conversation

@jeonghun-jj-lee

Copy link
Copy Markdown
Contributor

Summary

Relocates the context tree canvas from the persistent 192px top panel into the context side panel (opened from the progress-circle ring button in the session header). The graph is now on-demand — visible when the researcher opens the context panel, hidden otherwise — reclaiming 192px of prime chat real estate.

Changes

  • Moved the context tree rendering logic (canvas, keyboard nav, hover-glance, vault browsability, node-click) into SessionContextTab as the first section (above the stats grid)
  • Deleted context-tree-panel.tsx — the top panel component and its usage in message-timeline.tsx
  • Removed the localStorage collapse toggle (amicode-context-tree-open); the panel itself is the dismiss gesture
  • Engine lifecycle follows component mount/unmount — pauses when panel closes, resumes when it opens
  • ADR 0004 status updated from Proposed → Accepted
  • context-tree-engine.ts and context-tree-data.ts remain untouched; all 21 existing tests pass

Testing

  • bun test on context-tree-engine and context-tree-data: 21/21 pass
  • tsc --noEmit on packages/app: clean
  • Pre-existing type error in packages/opencode (unrelated to these changes)

Closes harmoniqs/amicode#264

@jeonghun-jj-lee
jeonghun-jj-lee marked this pull request as ready for review August 8, 2026 23:53
… (amicode#264)

- Move the context tree canvas + keyboard nav + hover-glance into
  SessionContextTab (top section, above the stats grid)
- Delete the ContextTreePanel component and its top-panel rendering
  from message-timeline.tsx — 192px returns to the chat timeline
- Remove the localStorage collapse toggle (amicode-context-tree-open);
  the panel itself is the dismiss gesture
- Engine pause/resume follows component mount/unmount (the parent
  <Show when={activeTab() === 'context'}> gates the whole tab)
- sessionHasContextItems gates the graph section (no empty canvas)
- Node clicks, amicode:brain-hover events, and keyboard navigation
  continue to work identically
- ADR 0004 status updated from Proposed to Accepted
- context-tree-engine.ts and context-tree-data.ts remain unchanged;
  existing tests still pass (21/21)
@jeonghun-jj-lee
jeonghun-jj-lee force-pushed the amico/issue-264-context-tree-to-side-panel branch from ddc09e9 to d631fc9 Compare August 9, 2026 02:53
@jeonghun-jj-lee
jeonghun-jj-lee merged commit 12ce32a into local/amicode Aug 9, 2026
1 of 4 checks passed
@jeonghun-jj-lee
jeonghun-jj-lee deleted the amico/issue-264-context-tree-to-side-panel branch August 9, 2026 02:54
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.

Context tree: relocate from top panel to context side panel

1 participant