fix: restore Claude usage session key settings to settings panel - #90
Merged
Conversation
The Session Key and Organization ID fields were removed from the settings panel UI at some point, while the backend plumbing (settings.ts, routes/settings.ts, scanner.ts) remained intact. Re-adds both fields under a "Claude Usage" section header above the Anthropic API Override.
nox-0x
approved these changes
Mar 28, 2026
nox-0x
left a comment
nox-0x
left a comment
Collaborator
There was a problem hiding this comment.
Clean restore of the Claude Usage settings UI. The two SettingRow additions follow the existing pattern correctly — null-coalescing on read, functional pending state update on write, and secret correctly applied only to the session key. Backend plumbing was already in place, so no surface area concerns. Ship it.
aterrylu
marked this pull request as ready for review
March 28, 2026 08:24
- Toggle handler: replace silent catch with error display + console.error - Toggle handler: surface non-2xx responses instead of ignoring them - handleSave: log actual error to console for debugging - Simplify onChange callbacks to expression-bodied form - Extract button label helpers for readability
nox-0x
approved these changes
Mar 28, 2026
nox-0x
left a comment
nox-0x
left a comment
Collaborator
There was a problem hiding this comment.
Clean update on top of the previous review. The catch blocks now log errors properly and surface HTTP status codes on toggle failures — both real improvements. Helper functions for button labels are a nice touch for readability. No new issues introduced. Ship it.
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
2 participants
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.
Summary
settings.ts,routes/settings.ts,scanner.ts) remained intactTest plan
🤖 Generated with Claude Code