Skip to content

Fail closed on untrusted AI signal provenance - #70

Merged
Pigbibi merged 2 commits into
mainfrom
codex/quant-advisor-manifest-v2-20260904
Sep 4, 2026
Merged

Fail closed on untrusted AI signal provenance#70
Pigbibi merged 2 commits into
mainfrom
codex/quant-advisor-manifest-v2-20260904

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • require ResearchSignalContextPipelines manifest v2 before AI context can affect advisory scoring
  • verify the signal and sibling manifest against the current upstream checkout HEAD blobs and expected origin repository
  • prevent untrusted AI inputs from expanding the market-confirmation universe
  • return one sanitized ai_signal_provenance_untrusted warning for missing, legacy, unknown, or mismatched provenance

Boundaries

  • no workflow changes
  • no GitHub API, provider, credential, deployment, or execution calls
  • no new dependencies or registry

Validation

  • RED first: 7 provenance/universe tests failed before implementation
  • python3 -m pytest -q tests/test_ai_ingestion_v6.py: 39 passed
  • python3 -m pytest -q tests/test_build_pipeline.py: 7 passed
  • focused Ruff, compileall, and diff check passed
  • full suite: 361 passed, 5 failed because existing tests outside the authorized write set still expect manifest-less/v1 AI inputs to affect scoring; production compatibility was intentionally not added because it would violate fail-closed

Migration note

CI remains blocked until those stale expectations are migrated to trusted manifest-v2 fixtures or no-op assertions in a separately authorized test-only follow-up.

Pigbibi and others added 2 commits September 4, 2026 21:47
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit c494c27 into main Sep 4, 2026
3 checks passed
@Pigbibi
Pigbibi deleted the codex/quant-advisor-manifest-v2-20260904 branch September 4, 2026 13:54
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