fix(sdk): allow parallel ChatGPT scans - #440
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d35ff64529
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e05019fe3f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Summary
ChatGPT scans currently keep the managed credential-home lock for the full scan, so a second scan can wait without starting workers or reporting usage. Narrow the lock to shared startup work and keep mutable scan configuration isolated so compatible scans can execute concurrently.
Refs #228. This is complementary to #229, which handles malformed stale-lock owner PIDs.
Changes
Testing
pnpm run typespnpm dlx bun@1.3.13 test --reporter=dot(1132 passed, 11 skipped)pnpm dlx --package=bun@1.3.13 node scripts/run-windows-ci-tests.mjs 1(5 passed)pnpm run buildpnpm run formatnode scripts/check-package.mjs <tarball>Risk and rollout
The managed credential identity and plugin installation remain shared, and startup mutations remain serialized. Compatible plugins receive isolated Deep Scan settings and can run concurrently. Legacy or custom plugins without the isolated-config protocol continue to use the shared configuration under the full credential-home lock, preserving their existing settings behavior. Existing stale-lock recovery behavior is unchanged, so #229 remains independently useful for malformed owner metadata.
Public disclosure review