Skip to content

feat(lokee): History Original → Target version pickers - #258

Merged
huyplb merged 2 commits into
mainfrom
cursor/lokee-history-compare-chrome-847b
Aug 16, 2026
Merged

feat(lokee): History Original → Target version pickers#258
huyplb merged 2 commits into
mainfrom
cursor/lokee-history-compare-chrome-847b

Conversation

@huyplb

@huyplb huyplb commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

History now uses the same Original → Target model as Compare, so a version vs the current database (or an older version) is obvious.

What changed

  • Compare chrome is a bit smaller — tighter padding, smaller pills, compact connection cards and Compare button.
  • Workspace bar hides on History — Schema Sync / SQL Editor is hidden while the History pane is open (Compare / History / Snapshot stay). Switcher returns on Compare or SQL Editor.
  • History reuses Compare’s Original → Target cards
    • Original = pick a captured version (defaults to the previous one)
    • Target = Current database (latest snapshot) or an older version
    • Same database on both sides; swap still works
  • The graph filters to those two versions so the timeline matches the pickers.

How to try it

  1. Open Schema Sync → Compare (chrome should feel slightly tighter).
  2. Snapshot a Target, then open History.
  3. The top workspace row is gone; Original / Target version cards sit where the live connections were.
  4. Leave Target on Current database, or pick an older version.

Tests

  • historyCompare resolve/swap/label unit tests
  • HistoryCompareBar defaults + older target + swap
  • LokeeWeaveView embedded hides the in-graph database picker
  • Smoke e2e: workspace switcher hidden on History, compare bar visible
Open in Web Open in Cursor 

Compact the Compare chrome and hide the workspace switcher on History.
History reuses the same Original → Target cards: Original is a captured
version, Target is the current database (latest snapshot) or an older
version, and the graph filters to those two sides.

Co-authored-by: huy.phan9 <huyplb@users.noreply.github.com>
@huyplb
huyplb marked this pull request as ready for review August 16, 2026 04:02
@cursor

cursor Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_af9b1df9-c5e8-46ce-b26b-73f3049dbb76)

#259 landed the shared blueprint work first, which touched the same three
files. Resolved by keeping each side's better half:

- TopToolbar: main's comment explaining why the connection grid is hidden in
  History, plus this branch's <HistoryCompareBar /> in its place.
- LokeeWeavePage: main's compact one-line header — the tall title + subtitle +
  three-line totals card cost ~90px above the graph.
- LokeeWeaveView: this branch's store-backed version state wins. Main had grown
  a local useState pair and an inline picker bar for the same job; both are
  gone, so the sides live in lokeeHistoryStore and the picker is
  HistoryCompareBar alone. What stays from main is the compare *trigger* and
  the VersionCompareModal wiring, now reading the resolved pair out of the
  store — the bar chooses the versions, the button opens the diff for them.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@cursor

cursor Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_d2da52b9-3f63-4132-a1d2-2aeac28e09b9)

@huyplb
huyplb merged commit 308c56d into main Aug 16, 2026
11 checks passed
@huyplb
huyplb deleted the cursor/lokee-history-compare-chrome-847b branch August 16, 2026 14:52
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.

2 participants