Claude/focused dirac 1qu5i0 - #27
Merged
Merged
Conversation
…s its store key On Pi 0.86.1+, Pi's built-in pi.dev `meta` catalog overlay refreshes before this extension through the same models-store entry. Persisted entries had no lastModified, so the overlay re-downloaded pi.dev on every refresh, a pi.dev failure skipped the Meta catalog refresh, and an overlay write left after a failed Meta refresh was later restored as the Meta catalog, cached baseUrl included. Persist with lastModified 0 and a source marker, restore only owned or legacy entries, republish the last good catalog after a failed Meta refresh, and pin restored models to https://api.meta.ai/v1. tests/model-runtime.test.ts drives the real Pi ModelRuntime and skips where the built-in provider is absent. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Dqv6ZT1fRfoPdsgQCydQVL
Device-authorization, token-poll, and key-mint requests had no timeout, so a stalled connection hung login past device-code expiry and hung Pi 0.83 refreshes (which pass no signal). Bound each request to 30 s, cap token polls at the deadline, and report timeouts and expiry explicitly. Report Pi 0.84+'s refresh timeout as a timeout rather than a cancellation, and accept verification and setup URLs only as normalized http(s) URLs, matching Pi's built-in Meta login, so control sequences cannot reach the terminal. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Dqv6ZT1fRfoPdsgQCydQVL
… aliases as missing ctx.model is the session's current model, so a model switch mid-request could strip reasoning and add prompt_cache_retention to another provider's payload. Apply the hints only when the payload's model is the Meta model. Pi treats an empty $META_API_KEY as unset, so an exported empty value used to block the MODEL_API_KEY alias. Treat empty and blank values as missing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Dqv6ZT1fRfoPdsgQCydQVL
The README installed the meta-oauth-only branch, which is pinned at v0.6.0 and is not an ancestor of main. Install from npm or main, guard against pinned git sources in tests, document the shared models-store entry, and align the test-driver agent's examples with its own hermetic-runner rule. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Dqv6ZT1fRfoPdsgQCydQVL
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 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.
Uh oh!
There was an error while loading. Please reload this page.