Skip to content

Report frozen OSV findings without failing the advisory check - #880

Merged
seathatflowsinourveins merged 2 commits into
mainfrom
codex/ns2604-opt6-frozen-risk-report-only-20261008
Oct 8, 2026
Merged

seathatflowsinourveins merged 2 commits into
mainfrom
codex/ns2604-opt6-frozen-risk-report-only-20261008

Conversation

@seathatflowsinourveins

Copy link
Copy Markdown
Collaborator

Scope

  • When the frozen OSV scan completes with findings (native exit 1), its advisory check currently fails on unrelated PRs. This change reports those findings successfully while keeping scanner and report errors failing.
  • Base commit: 89e89852161d8157e464f4bd07c2b5c747ad4706
  • Lane: lane:foundation
  • Owned paths touched: frozen report workflow, its existing recording-double test, dated decision, measurement receipt and their evidence registrations.

SOTA sources

Evidence-class table

Claim Evidence class Command / receipt
Hosted before fails only on frozen findings native_proven Retained GitHub artifact digest 07cbf215…, native run/job and input binding in the receipt
Finding-only report exits become 0; raw data/codes/logs stay identical synthetic Eight extracted-shell before/after cases; additional vendor 128/129 error cases
Coverage, error and privilege controls pass local_integration 254 focused tests (one existing skip); final expanded method separately passes; actionlint 1.17.0
Inverse restores base bytes and forward patch restores candidate local_integration Native inverse round trip, both owned file digests verified
Hosted after Unobserved Await this exact PR's native report check and explicit CC workflow landing cue

Local commands run

python3 -m unittest tests.test_frozen_evidence_reporting tests.test_osv_lockfile_coverage tests.test_frozen_macos_variant_no_use tests.test_workflow_hardening tests.test_workflow_policy tests.test_workflow_security_coverage -q
# 254 tests, rc 0, one existing skip; before the final vendor-code test extension
python3 -m unittest tests.test_frozen_evidence_reporting.FrozenReportWorkflowTests.test_native_findings_errors_and_both_logs_are_retained -v
# final 128/129 extension: rc 0
actionlint .github/workflows/frozen-evidence-risk.yml
# rc 0
python3 scripts/validate.py
# rc 0: 70 components, 10878 hashed files, 4 profiles, 235 receipts
git apply --check <owned inverse patch>
# inverse and forward round trip passed; base/candidate file hashes matched

Decision record

docs/decisions/2026-10-08-frozen-risk-report-result.md and
evidence/artifacts/github-ci-opt6-report-status-20261008.json.

Intentional kind 3 contract change: only
FrozenReportWorkflowTests.test_native_findings_errors_and_both_logs_are_retained.
Running all 13 unchanged base reporting methods on this candidate produces
exactly its two declared finding-only subtest failures, JSON1/SARIF0 and
JSON0/SARIF1; zero errors. Its original invocation/code/log assertions remain.

Host evidence

No new host receipt or status flip. The actual hosted before artifact is retained
separately from synthetic after controls. The archived input, empty report config,
ordinary required scan, raw serialization, artifact retention and trusted write
guards keep their existing bytes/contracts.

Checklist

  • Action pins and version comments preserved.
  • Empty default permissions and existing least-privilege job grants preserved.
  • No credential values, new required secret, provider or billing surface.
  • Peer work preserved; isolated owned worktree.
  • Inverse tested and receipt retained.
  • Native hosted after recorded.
  • Explicit CC workflow landing cue through 5f.

Revert this change through a reviewed inverse PR; findings stay visible in native
JSON/SARIF/logs and review alerts. Scanner errors and unavailable/malformed reports
remain failures. This PR remains draft while exact-head review and hosted checks finish.

@seathatflowsinourveins seathatflowsinourveins added the lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers label Oct 8, 2026
@socket-security

Copy link
Copy Markdown

Dependency limit exceeded — report not shown.

This pull request scan exceeded the 10,000-dependency limit applied to this scan, so the results are incomplete and may be inaccurate. To avoid reporting false positives, Socket has not posted a report.

Upgrade your plan to raise the dependency limit and get complete reports, or view the partial scan in the dashboard.

Socket is always free for open source. If this is a non-commercial open source project, contact us to request a free Team account.

@seathatflowsinourveins
seathatflowsinourveins marked this pull request as ready for review October 8, 2026 19:37
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@seathatflowsinourveins
seathatflowsinourveins enabled auto-merge (squash) October 8, 2026 19:40
@seathatflowsinourveins
seathatflowsinourveins merged commit 586a31e into main Oct 8, 2026
36 checks passed
@seathatflowsinourveins
seathatflowsinourveins deleted the codex/ns2604-opt6-frozen-risk-report-only-20261008 branch October 8, 2026 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant