Repository navigation
Harden profile isolation and avoid redundant refresh work - #33
Merged
Merged
Conversation
euforic
marked this pull request as ready for review
October 23, 2026 18:26
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
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.
Claude profile discovery previously trusted helper command lines without verifying the helper executable, and lifecycle polling repeatedly scanned the ASAR archive. Verify helper publisher, executable containment, and stable process identity, revalidate profile ownership before focus/close, and keep contract probes at operation boundaries.
Separate stale quota retries from local statistics refreshes, and release the profile-mutation busy state before the follow-up status read so subsequent edits are accepted. Add real bundled Codex app-server coverage and alternate installed-app paths for compatibility checks.
Validation:
swift testpassed (376 tests reported, 20 opt-in skips, zero failures). Six installed-provider checks passed, including two temporary Codex/Claude profiles remaining independent through close and restart. Staged secret scan and publication metadata audit passed. An optimized synthetic benchmark confirmed cached history reuse and zero query errors over 50,000 sessions/two million log rows.Latest downloaded Claude 3.6556.2 passed signature and startup-contract validation; its separate live run reported a temporary-directory cleanup permission error, so full lifecycle compatibility for that build is not yet established.
The requested direct push to main was rejected because the required Test, package, and privacy audit check has not run. This draft publishes the exact validated commit for that check without bypassing branch protection.