Skip to content

feat(viewer): show active observer auth path in settings UI#232

Merged
kunickiaj merged 1 commit into
mainfrom
03-15-feat_show_active_observer_auth_in_settings
Mar 15, 2026
Merged

feat(viewer): show active observer auth path in settings UI#232
kunickiaj merged 1 commit into
mainfrom
03-15-feat_show_active_observer_auth_in_settings

Conversation

@kunickiaj
Copy link
Copy Markdown
Owner

@kunickiaj kunickiaj commented Mar 15, 2026

Description

Add an observer status banner at the top of the Connection settings panel showing the actual resolved auth state (provider, model, auth method, token validity) and available credentials per provider (OAuth, API key, env var). Fetches from the /api/observer-status endpoint in parallel with config load.

Uses safe DOM construction (createElement + textContent + append) — no innerHTML with unsanitized data.

Type of Change

  • 🚀 Feature (new functionality)

Testing

  • Tests pass locally (pytest)
  • Manually verified changes work as expected
  • UI bundle rebuilt (bun run build)

Checklist

  • Code follows project style (ruff check and ruff format pass)
  • Self-review completed
  • No new warnings introduced

Closes: codemem-4uc

Stack: 3/3 (depends on #231)

Copy link
Copy Markdown
Owner Author

kunickiaj commented Mar 15, 2026

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8d19fd002e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread viewer_ui/src/tabs/settings.ts Outdated
@kunickiaj kunickiaj force-pushed the 03-15-feat_add_observer_status_api_endpoint branch from 5a8325e to 657d70f Compare March 15, 2026 18:30
@kunickiaj kunickiaj force-pushed the 03-15-feat_show_active_observer_auth_in_settings branch from 8d19fd0 to 9c72563 Compare March 15, 2026 18:30
Copy link
Copy Markdown
Owner Author

kunickiaj commented Mar 15, 2026

Merge activity

  • Mar 15, 6:33 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 15, 6:37 PM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 15, 6:39 PM UTC: @kunickiaj merged this pull request with Graphite.

@kunickiaj kunickiaj changed the base branch from 03-15-feat_add_observer_status_api_endpoint to graphite-base/232 March 15, 2026 18:34
@kunickiaj kunickiaj changed the base branch from graphite-base/232 to main March 15, 2026 18:36
Add an observer status banner at the top of the Connection settings panel
showing the actual resolved auth state (provider, model, auth method, token
validity) and available credentials per provider. Fetches from the new
/api/observer-status endpoint on settings load.

Closes: codemem-4uc
@kunickiaj kunickiaj force-pushed the 03-15-feat_show_active_observer_auth_in_settings branch from 9c72563 to 6fa3735 Compare March 15, 2026 18:37
@kunickiaj kunickiaj merged commit fa32432 into main Mar 15, 2026
8 checks passed
@kunickiaj kunickiaj deleted the 03-15-feat_show_active_observer_auth_in_settings branch March 15, 2026 18:39
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.

1 participant