Skip to content

Bound Claude OAuth test gates - #593

Merged
cbusillo merged 1 commit into
mainfrom
audit/claude-token-test-gate
Aug 12, 2026
Merged

Bound Claude OAuth test gates#593
cbusillo merged 1 commit into
mainfrom
audit/claude-token-test-gate

Conversation

@cbusillo

Copy link
Copy Markdown
Owner

Summary

  • replace Claude OAuth polling helpers with a bounded continuation-backed start gate
  • keep release gates explicit so real locks and token POSTs cannot self-open
  • guarantee cleanup and preserve the original timeout reason through teardown cancellation
  • add a direct start-timeout regression test

Refs #578

Validation

  • four focused Claude OAuth gate tests passed
  • 25 repeated focused stability runs passed
  • lock-contention bypass mutation failed on expected error, HTTP request, and credential-save assertions
  • routine Swift lane passed 892 tests
  • scripts/commit-gate.sh
  • JetBrains changed-files inspection: GREEN, 0 findings
  • Opus and Gemini exact-SHA final reviews approved 5da35f2

Scope

  • test-only change in Tests/ContextPanelCoreTests/ProviderConnectorTests.swift
  • no production lock or OAuth behavior changed

@cbusillo
cbusillo merged commit 44f5982 into main Aug 12, 2026
4 checks passed
@cbusillo
cbusillo deleted the audit/claude-token-test-gate branch August 12, 2026 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant