Conversation
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…can indicator Settings' "Rescan apps" bypassed appStatus and, via onsaved(), unconditionally also triggered a repo rescan — its footer segment never reflected apps scanning at all. Wire Settings' rescan buttons to App.svelte's real rescan()/rescanAppsNow() so each acts (and reports) independently. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
One-hand operation: both defaults are now bare Ctrl + a key next to the arrow cluster, so open -> navigate -> Enter stays in one small area of the keyboard. Avoids Ctrl+/ since that's the near-universal editor comment-toggle shortcut and would steal it system-wide as a global hotkey. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.
dev→mainfor the next release (2 commits since the last merge).What ships
ca5ba08) — adds a demo GIF at the topof the README (first visual anyone sees landing on the repo), documents
the installed-app cache path (
<OS cache dir>/dev-prompt/apps.json)alongside the existing
repos.jsonmention in both the README anddocs/configuration.md, and drops a stray macOS config-path referencethe project doesn't target.
2076508)— standardizes every "scanning" message to
Scanning repos…/Scanning apps…(footer, Settings' save notes, the app-list emptystate). Also fixes a real bug: Settings' "Rescan apps" button bypassed
appStatusentirely and, viaonsaved(), unconditionally triggered arepo rescan too — so the Settings footer never reflected an apps scan
in progress. Settings' rescan buttons now call through to App.svelte's
real
rescan()/rescanAppsNow(), so each acts and reportsindependently.
Test plan
npm run check(svelte-check, 0 errors)npm run build(svelte-check + vite build, succeeds)npm run tauri dev: repo rescan, apps rescan,and Settings' Save/Rescan buttons each show the correct independent
status text without cross-triggering the other scan.
🤖 Generated with Claude Code