Skip to content

fix(macOS): bind JIT QA launches to reviewed cloud isolation receipts - #12811

Merged
Git-on-my-level merged 4 commits into
mainfrom
codex/jit-qa-cloud-routing-20260905
Sep 5, 2026
Merged

fix(macOS): bind JIT QA launches to reviewed cloud isolation receipts#12811
Git-on-my-level merged 4 commits into
mainfrom
codex/jit-qa-cloud-routing-20260905

Conversation

@Git-on-my-level

@Git-on-my-level Git-on-my-level commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Problem

The reserved omi-jit-qa launcher could route only to local or shared development endpoints. A cloud QA launch therefore had no atomic way to pair the isolated Python and desktop services, and an endpoint-shaped environment override could be mistaken for a reviewed deployment.

Change

Add a cloud-qa tuple backed by a reviewed deployment receipt. The launcher requires both QA Cloud Run URLs, validates the dev project, named jit-qa Firestore database, fixed services, gateway URL, revisions, image digests, source SHA, and dependency vector, rejects shared and production hosts, and binds Python/auth to backend-jit-qa while the child desktop agent uses desktop-backend-jit-qa. Use one producer-shaped receipt fixture and cover both supported Cloud Run URL forms, missing review, wrong database, mismatched hosts, and malformed receipts.

Validation

  • bash -n desktop/macos/scripts/jit-qa-target.sh desktop/macos/scripts/omi-jit-qa desktop/macos/tests/test-jit-qa-target.sh
  • bash desktop/macos/tests/test-jit-qa-target.sh
  • shellcheck desktop/macos/scripts/jit-qa-target.sh desktop/macos/scripts/omi-jit-qa
  • make preflight

Failure-Class: none

Review in cubic

Route the reserved omi-jit-qa bundle through an atomic cloud-qa endpoint pair backed by a deployment receipt. Validate isolated service identity, revisions, image digests, source SHA, and dependency-vector presence before launching.

Failure-Class: none

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 6 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread desktop/macos/scripts/jit-qa-target.sh Outdated
Comment thread desktop/macos/scripts/jit-qa-target.sh Outdated
Comment thread desktop/macos/tests/test-jit-qa-target.sh
@Git-on-my-level
Git-on-my-level merged commit 9a56953 into main Sep 5, 2026
25 checks passed
@Git-on-my-level
Git-on-my-level deleted the codex/jit-qa-cloud-routing-20260905 branch September 5, 2026 23:27
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