Skip to content

fix: app binary name, composer focus ring, and ChatGPT Settings models - #289

Merged
tuckcode merged 9 commits into
mainfrom
fix/post-286-polish
Nov 2, 2026
Merged

tuckcode merged 9 commits into
mainfrom
fix/post-286-polish

Conversation

@tuckcode

@tuckcode tuckcode commented Nov 2, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Binary: the installed app now runs Contents/MacOS/Rhizome through mainBinaryName. The Cargo binary, bundle id, config folder, and keychain service do not change.
  • Composer: the focus ring now surrounds the whole composer box. The inner editor ring is removed. The caret and placeholder share one centered line.
  • Settings: the ChatGPT row shows the account's live models. It no longer falls back to gpt-4.1-mini, which a ChatGPT account cannot use.
  • Picker: an unlisted saved model keeps its provider name. Provider removed now means the provider is gone.

QA

  • Pre-push gates: passed on the latest commit.
  • Build: a test .app from the rename branch contains Contents/MacOS/Rhizome.
  • Focused vitest: pass.
  • Not done: native QA of the composer ring, the Settings row, and the picker label. No app with these changes was installed.

Completion notes

  • Localization: none — English only (C18).
  • PostHog: no event needed because these are visual and name fixes.
  • Codacy: not run locally. See the PR check.
  • ADRs: none.
  • Docs: docs/IDENTITY.md and the install page.
  • Demo vault dirt: clean.

🤖 Generated with Claude Code

tuckcode and others added 7 commits November 1, 2026 20:00
The inner editor showed the browser outline, and the outer box had no
focus state.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The bundle now runs Contents/MacOS/Rhizome. The Cargo binary, bundle id,
config folder, and keychain service do not change.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The row fell back to the catalog gpt-4.1-mini, which a ChatGPT account
cannot use.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 5 complexity · 0 duplication

Metric Results
Complexity 5
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

tuckcode and others added 2 commits November 1, 2026 21:01
The picker said Provider removed while the xAI provider still existed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The placeholder sat lower than the caret, and the shared editor ring
still drew inside the box.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tuckcode
tuckcode merged commit 8e70470 into main Nov 2, 2026
11 checks passed
@tuckcode
tuckcode deleted the fix/post-286-polish branch November 2, 2026 03:51
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