Repository navigation
Add the Muse (muse.ai) provider - #814
Merged
Merged
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 1 minute. View limit details
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 |
This was referenced Oct 11, 2026
# Conflicts: # apps/desktop-tauri/src/components/providers/providerIcons.ts # apps/desktop-tauri/src/test/providerCatalog.ts # docs/PROVIDERS.md # rust/src/core/provider.rs # rust/src/core/provider_factory.rs # rust/src/core/token_accounts.rs # rust/src/settings/api_keys.rs
# Conflicts: # apps/desktop-tauri/src/components/providers/providerIcons.ts # apps/desktop-tauri/src/test/providerCatalog.ts # docs/PROVIDERS.md # rust/src/core/provider.rs # rust/src/core/provider_factory.rs # rust/src/settings/api_keys.rs
# Conflicts: # README.md # apps/desktop-tauri/src/components/providers/providerIcons.ts # apps/desktop-tauri/src/test/providerCatalog.ts # rust/src/core/provider.rs # rust/src/core/provider_factory.rs # rust/src/core/token_accounts.rs
# Conflicts: # README.md # apps/desktop-tauri/src-tauri/src/commands/provider_settings.rs # apps/desktop-tauri/src-tauri/src/commands/session_cookie_scope_tests.rs # apps/desktop-tauri/src/components/providers/providerIcons.ts # apps/desktop-tauri/src/test/providerCatalog.ts # docs/PROVIDERS.md # rust/src/core/provider.rs # rust/src/core/provider_factory.rs # rust/src/core/token_accounts.rs # rust/src/settings.rs # rust/src/settings/tests.rs
# Conflicts: # README.md # apps/desktop-tauri/src-tauri/src/commands/provider_settings.rs # apps/desktop-tauri/src-tauri/src/commands/session_cookie_scope_tests.rs # apps/desktop-tauri/src/components/providers/providerIcons.ts # apps/desktop-tauri/src/test/providerCatalog.ts # docs/PROVIDERS.md # rust/src/core/provider.rs # rust/src/core/provider_factory.rs # rust/src/core/token_accounts.rs # rust/src/settings.rs # rust/src/settings/tests.rs
- Automatic cookie source: the provider now owns browser cookie resolution. It reads every detected browser's langdock.com cookies and uses the first one with a non-empty auth_token (upstream requiredCookies), instead of the shell's first browser with any langdock.com cookie. Manual headers and an empty Manual or Off source never read a browser. - No included limits: one informational primary labelled "Included limits" (upstream detail title) with no duplicate detail row, so the card no longer shows the message twice under "Session". - Correct the overage comment: RateWindow clamps when the window is built, not at display time.
- A missing key (shared resolve_api_key NotInstalled) now maps to upstream IBMBobUsageFetcher's "Missing IBM Bob API key. Add one in Settings or set BOBSHELL_API_KEY." instead of the generic "API key not found" text; before, only a blank key reached it. Other resolver errors still propagate. - New pure helper api_key_from; hermetic test a_missing_key_uses_the_ibm_bob_message.
- HTTP 401/403 ("ClawRouter rejected the API key...") now maps to
NeedsAuthentication via error_state_kind, matching upstream's
authenticationExpired; the message text is unchanged and other
errors keep the default mapping.
- HTTP tests no longer read CLAWROUTER_BASE_URL: the env fallback is an
injected lookup that the test constructor disables.
- Port upstream's budgeted golden fixture (0.024% used, $25 limit,
openai before anthropic, reset 2026-08-01Z) as a literal test.
- Omit the Prepaid credits cost block for a negative balance. The shared block floors balances at zero, so a debt showed "$0.00"; the primary line and Balance row keep "-$1.00" (upstream keeps debts visible). - Own browser cookie resolution: Automatic now tries every browser that holds both __Host-console_session and __Host-console_csrf, in order, and moves to the next after a 401 at any request (upstream rejectCookie). A pasted header is still the only session tried, an empty Manual source still fails closed, and Off is unchanged. - When no browser session is usable and the browser read itself failed (for example App-Bound Encryption), report that error instead of the generic sign-in hint.
- Stop attaching the weekly USD CostSnapshot to the fetch result. Upstream SyntheticProviderDescriptor hides the cost block on the card (ProviderCostPresentation(menuCardStyle: .hidden)), and core has no per-provider "hidden on card" flag. The parsed credits still drive the weekly regen line. As a result the weekly cost also leaves CLI JSON. QuotaCost.resets_at was only used for that snapshot and is removed. - Port upstream's "Synthetic fixture matches the cut-over golden" (ProviderPluginParityTests.swift) and "missing rolling lane keeps weekly and search slots" (SyntheticProviderTests.swift) as literal tests. Weekly used percent is 1.9411527777777593, not upstream's 1.9411527777777735: serde_json without float_roundtrip reads 98.05884722222223 one ULP high (1.4e-14, never visible). - Use the fixed NOW in the bearer-request test instead of Utc::now(). - docs/PROVIDERS.md: the weekly credits no longer give a USD cost.
- The top-up balance was mapped to an extra RateWindow ("topup"), so the
card drew it as a second quota bar with pace lines. Upstream museai.js
returns it as a detail row {label, value, progress}; map it to a
ProviderDisplayDetail with progress (used fraction of 1.0) instead.
- Tests assert extra_rate_windows stays empty and the detail row carries
"Additional tokens" / "1.5B tokens left" / progress 0.25, the "$80.00 left"
fallback, the verbatim label and the zero-total case.
- docs/PROVIDERS.md: describe the top-up as a detail row.
Synthetic's Mac brand color #141414 is near-black, so the palette's brandColorOnDark lifts it on dark surfaces like v0 and TypeSafe; list it in the registry test.
# Conflicts: # apps/desktop-tauri/src/components/providers/providerIcons.ts
# Conflicts: # apps/desktop-tauri/src/components/providers/providerIcons.ts
# Conflicts: # apps/desktop-tauri/src/components/providers/providerIcons.ts
# Conflicts: # apps/desktop-tauri/src/components/providers/providerIcons.ts
# Conflicts: # apps/desktop-tauri/src/components/providers/providerIcons.ts
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
1 participant
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.
Stacked on #821; merge in order 808 → 810 → 818 → 819 → 821 → 814 → 817; the diff shows predecessors until they merge.
What users get
A new Muse (muse.ai) provider for Meta's personal agent at muse.ai. It is separate from the existing Muse Code provider (
muse), and it is disabled by default.Cookie source. Settings → Providers → Muse (muse.ai) has a cookie-source picker that defaults to Automatic:
muse.aicookies from the first detected browser that has them, and Manual fails closed without reading a browser.The card shows:
Ported from upstream CodexBar v0.73.0
Resources/Plugins/museai.jsandProviders/MuseAI/MuseAIProviderDescriptor.swift. muse.ai has no usage API, so the provider posts the settings dialog's Next.js server actionfetchSubscriptionActiontohttps://muse.ai/with the session cookies. The action ID changes on every deploy. A404 Server action not found.starts a bounded discovery through the page's chunks: at most 96 page chunks, eight at a time, 8 MiB in total, then at most 32 settings chunks. Static chunks get no cookies. A 401, a 403 or a redirect toauth.muse.aiis reported as an expired session. Cookies and response bodies are never logged or echoed in errors.Files
rust/src/providers/museai/{mod.rs,model.rs,tests.rs}: fetch, action discovery, flight-response parse and card mappingWiring per the new-provider recipe:
core/provider.rs:ProviderId::MuseAI(climuseai, display "Muse (muse.ai)", cookie domainmuse.ai, colour #0668E1)core/provider_factory.rs: factory armcore/token_accounts.rs:Nonearmproviders/mod.rsFrontend:
ProviderIcon-museai.svg,providerIcons.ts,test/providerCatalog.tsDocs:
docs/PROVIDERS.md(new "Muse (muse.ai) weekly usage" section),README.md(provider table row)Cookie-source picker (second commit, following the Groq precedent in 5a44b63):
commands/provider_settings.rs: Automatic / Manual / Off options and the id mappingrust/src/settings.rs: the default cookie source is "auto" (before this, the default "manual" plus the provider's fail-closed policy meant Automatic could never be reached)rust/src/settings/tests.rsandcommands/session_cookie_scope_tests.rsNo i18n or dependency changes; the picker reuses the existing option labels.
Tests
26 tests in
museai/tests.rs(15 unit, 11 local-server). They use fixed fixtures and assert literal values:Two more tests cover the picker:
museai_cookie_source_defaults_to_automatic_session_import(settings): the default is "auto".museai_exposes_a_cookie_source_picker_and_routes_each_choice(tauri): the options are [auto, manual, off]; a stored header is used under Automatic and Manual; an empty Manual source is Web withmanual_cookie_missing; Off reaches the provider as Cli with no header.Commands
cargo fmt --all -- --checkcargo test --manifest-path rust/Cargo.tomlcargo clippy --manifest-path rust/Cargo.toml --all-targets -- -D warningscargo test --manifest-path apps/desktop-tauri/src-tauri/Cargo.tomlcargo clippy --manifest-path apps/desktop-tauri/src-tauri/Cargo.toml --all-targets -- -D warningspnpm testpnpm run lintpnpm run buildThe cargo rows ran on the head commit (8a4c5a3). The pnpm rows ran on the first commit (e584848); the picker commit touches no frontend files.
The branch is based on 44d5de3, and the checks ran on that base. It predates #813 (the Mac card anatomy), so the panel proof and the card-layout gaps below describe the card before #813. Main's later Providers-pane changes (the usage-details section and #816's Settings panes) don't touch the cookie-source or API-key sections, and
cookie_source_options_forand the API-key catalog are unchanged on main.Proof (Windows, synthetic data only)
build-proof.sh). The proof-shim patch is never committed.win_run.pywith the MuseAI scenario pack. It uses a synthetic manual cookie and a mocked muse.ai: the page, three chunks and the action POST, all 200.W:/mac-parity/report/provider-museai/panel.pngW:/mac-parity/report/provider-museai/COMPARISON.mdresult.json,mock.login the same folderwin_run.py --mode settings:providers --target settingswith a copy of the pack that lists this provider first (W:/mac-parity/report/provider-museai/settings-pack/).W:/mac-parity/report/provider-museai/settings-proof/settings-before.pngand.jsonW:/mac-parity/report/provider-museai/settings-proof/settings-after.pngand.jsonGaps vs the Mac card
muse.aicookies and does not fall through to the next after a failure.dataConfidenceequivalent (upstream marks the datapercentOnly).Sibling PR conflicts
This is one of seven provider PRs: Synthetic (#808), ClawRouter (#810), IBM Bob (#818), Langdock (#819), LithosAI (#821), MuseAI (#814) and WorkBuddy (#817). They all add lines at the same anchors, so expect trivial textual conflicts once one of them merges. Keep both sides. The shared anchors are:
ProviderIdlists incore/provider.rs(after Vercel), including theall().len()count in its testproviders/mod.rstoken_accounts.rsproviderCatalog.tsand theproviderIcons.tsregistrydocs/PROVIDERS.mdsection and README table rowsThe four cookie providers (Langdock #819, LithosAI #821, MuseAI #814 and WorkBuddy #817) also share the picker commit's anchors, so they conflict with each other there:
rust/src/settings.rs: each PR rewrites the default cookie-source armKimi | Hyper | Groq => "auto"to add its variant. Keep every variant in the arm.commands/provider_settings.rs: each PR adds lines after thegroqentries incookie_source_providerandcookie_source_options_for.rust/src/settings/tests.rs: each PR adds a default test after the Groq default test.commands/session_cookie_scope_tests.rs: each PR appends a test at the end of the file.