Skip to content

wave8: status/error UX (#94, #100, #46) #123

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
AndresL230 merged 13 commits into from
May 24, 2026
Merged

wave8: status/error UX (#94, #100, #46) #123

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
fffdc3d
chore: gitignore tsbuildinfo build artifact
AndresL230 May 22, 2026
8f7e90c
docs(wave8): status/error UX design spec
AndresL230 May 23, 2026
962d2b3
docs(wave8): implementation plan for status/error UX
AndresL230 May 23, 2026
b617abb
fix(wave8): validateApiKey no longer treats 404 as valid (#94)
AndresL230 May 23, 2026
b785ebb
test(wave8): clean up stale null wording in api-client tests
AndresL230 May 23, 2026
ad34c8e
feat(wave8): plumb Retry-After seconds through apiFetch 429 errors
AndresL230 May 23, 2026
bc94f41
refactor(wave8): export ApiClientError type and cover HTTP-date Retry…
AndresL230 May 23, 2026
c11fcb8
feat(wave8): surface 429 with Retry-After and refresh status bar on s…
AndresL230 May 23, 2026
de559b0
refactor(wave8): require refreshStatusBar callback and use ApiClientE…
AndresL230 May 23, 2026
56c5cf9
fix(wave8): drop stale status-bar writes via generation counter (#46 …
AndresL230 May 23, 2026
39b5ebb
refactor(wave8): also guard lastValidationAt writes against stale cal…
AndresL230 May 23, 2026
e311fba
fix(wave8): debounce focus refresh to 60s and drop workspace-folders …
AndresL230 May 23, 2026
f9e5a2f
docs(wave8): correct CLAUDE.md auth-section naming and behavior
AndresL230 May 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ venv/

# Build artifacts
*.vsix
*.tsbuildinfo

# ReCost generated output files
.recost-context.md
Expand Down
2 changes: 1 addition & 1 deletion CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ Two separate key systems coexist:
**ReCost API key** (for scanning/remote API) — managed entirely in `extension.ts`:
- Stored in `context.secrets` under `"recost.apiKey"`
- Must begin with `rc-` prefix — validated before storing
- `validateRcApiKey()` in `api-client.ts` calls `GET /auth/me` with `Authorization: Bearer <key>`; returns `null` on 404 (dev mode), throws on 401 (invalid) or network error
- `validateApiKey()` in `api-client.ts` calls `GET /auth/me` with `Authorization: Bearer <key>`; returns the authenticated user on 200, throws on any non-2xx (including 401 invalid and 404)
- Status bar item reflects auth state with color: green (`testing.iconPassed`) when connected, `statusBarItem.warningForeground` when unreachable, no color when unconfigured; clicking it runs `recost.openKeys`
- `context.secrets.onDidChange` listener keeps status bar live without reload
- After key validation in the webview (`serviceId === "recost"`), `recost.keyOnline` context is also updated so the status bar stays in sync
Expand Down
Loading
Loading