Skip to content

fix(strix): restore trusted binder for consumer scans - #2419

Merged
seonghobae merged 1 commit into
mainfrom
fix/strix-binder-runtime-hotfix-1220
Sep 27, 2026
Merged

seonghobae merged 1 commit into
mainfrom
fix/strix-binder-runtime-hotfix-1220

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Repair

Extract only the trusted evidence-binder path repair already owned by #2291. Do not carry its unrelated foundation or Job Analysis changes.

STRIX_REPO_ROOT identifies the consumer scan workspace; central executable helpers belong beside the trusted gate. Resolve the binder from SCRIPT_DIR, retaining missing-file and symlink rejection.

Evidence

  • contextual-orchestrator#1220 run 36246946213 / job 108478790557 completed scanning, then exited 2 because the consumer workspace had no central binder.
  • Exact main 5a0f00e regression: 2 failed.
  • Repair 841cb8f regression with warnings as errors: 2 passed, exit 0.
  • Bash syntax and git diff --check: pass.
  • Expanded binder tests: 36 passed, 1 failed under Python 3.14 warnings as errors. The HTTPError resource warning reproduces on untouched exact main; it is not suppressed or represented as a clean full suite.

The user explicitly authorized necessary bypass merges for CI repair on this session. This narrow infrastructure repair preserves scan severity and evidence validation. Product PR #1220 still needs actual current-head review and checks; this PR does not fabricate them.

@coderabbitai

coderabbitai Bot commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 58 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 01ae59a0-21e9-4125-b1a1-8f330a79c847

📥 Commits

Reviewing files that changed from the base of the PR and between 5ed0610 and 841cb8f.

📒 Files selected for processing (2)
  • scripts/ci/strix_quick_gate.sh
  • tests/test_strix_evidence_binder_trusted_path.py

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae
seonghobae merged commit 7597cc8 into main Sep 27, 2026
5 of 19 checks passed
@seonghobae
seonghobae deleted the fix/strix-binder-runtime-hotfix-1220 branch September 27, 2026 10:43
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