Repository navigation
Use Pi's official brand color and repository - #991
Conversation
Pi uses the blue from its official logo (#4D9ABF) instead of the generic purple, and its dashboard link points to earendil-works/pi. The pi-mono alias still parses. Pi joins the Mac parity accent table, so the contrast check covers it. Upstream steipete#4690.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Ui Changes Include Windows Proof |  |
The PR changes apps/desktop-tauri/src/components/providers/providerIcons.ts, which is a non-test UI file and changes the visible Pi brand color from #7c3aed to #4d9abf. The description contains … |
Add screenshots from a fresh Windows build, or add a clear proof note in the PR description that records the fresh Windows build and the visible Pi color/dashboard result. |
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title is a short imperative summary that accurately describes the Pi brand color and repository updates. Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Provider Data Stays Siloed PASS. The pull request changes only Pi metadata: brand color, dashboard URL, parity-test data, and the matching snapshot. The diff adds no identity, plan, account label, or email flow between provider… Secrets Handled Safely The pull request changes only Pi metadata, brand colors, a repository URL, and regression tests. The diff adds no token, cookie, API key, OAuth credential, authorization header, logging, error output,… No Unapproved Dependencies The pull request changes only provider source and test files. It does not modify any Cargo.toml or package.json, add an npm/yarn lockfile, or change the pinned pnpm packageManager version. No dependen… Full details: Provider Data Stays Siloed
-
Explanation
PASS. The pull request changes only Pi metadata: brand color, dashboard URL, parity-test data, and the matching snapshot. The diff adds no identity, plan, account label, or email flow between providers or accounts. It adds no cross-provider conditional or match branching. The new
ProviderId::Pireferences are table/test entries inrust/src/core/provider/id.rs, not branching logic.
Full details: Secrets Handled Safely
-
Explanation
The pull request changes only Pi metadata, brand colors, a repository URL, and regression tests. The diff adds no token, cookie, API key, OAuth credential, authorization header, logging, error output, credential-file operation, or keyring operation. The snapshot contains only the updated public URL and color.
Full details: No Unapproved Dependencies
-
Explanation
The pull request changes only provider source and test files. It does not modify any Cargo.toml or package.json, add an npm/yarn lockfile, or change the pinned pnpm packageManager version. No dependency approval is required.
-
Full details: Ui Changes Include Windows Proof
-
Explanation
The PR changes
apps/desktop-tauri/src/components/providers/providerIcons.ts, which is a non-test UI file and changes the visible Pi brand color from#7c3aedto#4d9abf. The description contains only a synthetic proof suggestion and no screenshots or proof note from a fresh Windows build.Resolution
Add screenshots from a fresh Windows build, or add a clear proof note in the PR description that records the fresh Windows build and the visible Pi color/dashboard result.
✨ Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Summary
#4D9ABFinstead of the earlier violet#7C3AED. The color is set in the provider spec (rust/src/core/provider/spec.rs) and in the frontend icon table (apps/desktop-tauri/src/components/providers/providerIcons.ts).https://github.com/earendil-works/pi.pi-monoCLI alias still resolves to Pi, so existing configs keep working.rust/src/core/provider/id.rs, with#7C3AEDas the accent and#4D9ABFas the brand color, and to the matching frontend test list.rust/src/core/provider_registry_snapshot.txtis updated for the two changed Pi lines.F09082/4D9ABF/F1BE58), its palette test anddocs/pi.md. Windows has no confetti palette code path or matching docs, so there is nothing to attach them to.Upstream reference:
steipete/CodexBarcommit591e6008c(steipete#4690),PiProviderDescriptor.swiftandProviderPaletteRegressionTests.swift.Checks
cargo fmt --all: clean.pi-monoalias, the new Pi test (pi_uses_its_official_brand_color_and_current_repository), the mac-parity accent tests (including the contrast check), the adopted-provider and CLI alias tests, and the brand-color tests.cargo clippy --manifest-path rust/Cargo.toml --all-targets -- -D warnings: clean.pnpm install --frozen-lockfile: ok (pnpm 11.25.0, local Node 22.22.0). CI pins Node 24.18.0; this run did not use it.pnpm test -- --maxWorkers=4: 164 files, 1617 tests passed.pnpm run build: succeeded (existing chunk-size warnings only).cargo test --manifest-path rust/Cargo.toml: 5280 passed, 0 failed, 2 ignored.pi_uses_its_official_brand_color_and_current_repositoryinrust/src/core/provider/id.rs;["pi","#4d9abf"]inproviderIcons.test.ts.Proof suggestion (fake data)
Open the Pi provider card with a synthetic snapshot. Expect the icon and usage bar in brand blue
#4D9ABF. Use the dashboard action and expecthttps://github.com/earendil-works/pi. Keep thepi-monoalias in a fake CLI config and expect it to resolve to Pi.Summary by CodeRabbit