feat(highlights): only-notes filter, file export, shortcuts, spotlight shuffle#123
Merged
Conversation
…ts, shuffle The four polish items deferred from the original commonplace-book release (PR #76), plus a real bug the verification pass surfaced. - Only-notes filter: a toggle next to "On this day" showing just the highlights that carry a note (server: only_notes param on GET /api/annotations; composes with q and on_this_day). - File export: download all matching highlights as a Markdown file (tome-highlights-YYYY-MM-DD.md), alongside the existing clipboard copy. - Keyboard shortcuts on /highlights: "/" search, Esc clear, "c" collapse all, "n" only-notes, "e" export — registered in the "?" help modal. - Spotlight shuffle: a re-roll button on the Home tab's highlight card. GET /annotations/spotlight?exclude=<id> returns a different highlight whenever more than one candidate exists (on-this-day preference kept; a sole highlight comes back rather than emptying the card). Fixed along the way: "copy all as Markdown" has been broken since the page shipped — it requested limit=2000 against the endpoint's le=1000 cap, got a 422, and failed with no feedback (no catch, no toast). Cap raised to 10000, both exports stay under it, and failures now toast. Verification: 849 tests green (3 new: only-notes filtering + both exclude edge cases). Playwright drive on the live dev app: filter 28→18→28 highlights, "/" focuses search, the export downloads a real .md file (the step that caught the 422), and shuffle swaps the home quote. Frontend build clean. Pre-existing hydration warning in HomeGoalRings (<div> in <p>) noted but out of scope here. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Owner
Author
|
Closed pending hands-on testing — opened prematurely; will reopen once the branch is verified on dev. |
bndct-devops
pushed a commit
that referenced
this pull request
Jul 5, 2026
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
1 participant
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.
The four polish items deferred from the original commonplace-book release (PR #76), plus a real bug the verification pass surfaced.
only_notesonGET /api/annotations; composes with search and on-this-day).tome-highlights-YYYY-MM-DD.md, alongside the existing clipboard copy./search,Escclear,ccollapse all,nonly-notes,eexport — listed in the?help modal.GET /annotations/spotlight?exclude=<id>returns a different highlight whenever more than one exists (a sole highlight comes back rather than emptying the card).Fixed along the way: "copy all as Markdown" has been broken since the page shipped — it requested
limit=2000against the endpoint'sle=1000cap, got a 422, and failed with no feedback (no catch, no toast). Cap raised to 10000, both exports stay under it, failures now toast.Verification: suite green (3 new endpoint tests); Playwright drive on the live dev app — filter 28→18→28 highlights,
/focuses search, the export downloads a real .md file (the step that caught the 422), shuffle swaps the home quote.