Skip to content

fix: repair canonical snapshot pipeline imports - #200

Merged
Pigbibi merged 2 commits into
mainfrom
codex/qsl-safe-canonical-import-baseline-repair-v1
Jul 26, 2026
Merged

fix: repair canonical snapshot pipeline imports#200
Pigbibi merged 2 commits into
mainfrom
codex/qsl-safe-canonical-import-baseline-repair-v1

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update stale imports to canonical pipelines/research paths
  • remove obsolete test_run_codex_pr_review.py
  • align three entrypoint/dependency-pin assertions with current pyproject.toml

Validation

  • PYTHONPATH=src python -m pytest -q -> 704 passed, 4 skipped
  • python3 -m ruff check <11 changed paths> -> passed
  • python3 -m compileall -q src scripts -> passed
  • scope/diff check -> 11 authorized files only

No merge or review rerun performed.

Co-Authored-By: Codex <noreply@openai.com>
@github-actions

github-actions Bot commented Jul 26, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

Merge allowed: 1 finding(s) reported but none are blocking

ℹ️ Other Findings

1. 🟡 [MEDIUM] Security in tests/test_run_codex_pr_review.py

The new workflow-delegation test only checks that .github/workflows/codex_pr_review.yml references QuantStrategyLab/AIAuditBridge/.github/workflows/codex_pr_review.yml with some non-empty ref, so it passes even when the repository is using the mutable @main branch. That workflow is triggered by pull_request_target and grants id-token: write, issues: write, and pull-requests: write, so leaving it on a branch means future changes in AIAuditBridge/main can change privileged PR-review behavior without any pinning guardrail in this repository. (line 18)

Suggestion: Tighten the test to require an immutable 40-character commit SHA and update .github/workflows/codex_pr_review.yml to use that SHA instead of @main.


Review by Codex PR Review bot • PR

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit b0ae03a into main Jul 26, 2026
3 checks passed
@Pigbibi
Pigbibi deleted the codex/qsl-safe-canonical-import-baseline-repair-v1 branch July 26, 2026 03:16
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