Skip to content

fix(quarantine): harden provenance, symlink layout, and write integrity - #60

Merged
ManintheCrowds merged 4 commits into
mainfrom
fix/quarantine-provenance-hardening
Oct 9, 2026
Merged

ManintheCrowds merged 4 commits into
mainfrom
fix/quarantine-provenance-hardening

Conversation

@ManintheCrowds

Copy link
Copy Markdown
Owner

Summary

  • Consolidate unique AppSec fixes from draft Cursor PRs Fix MCP install and registry fetch merge guards #44–Fix registry quarantine provenance and write integrity #56 into one non-draft branch: reject self-certifying JSON sidecars, refuse symlinked registry_fetch layouts, fail closed on quarantine write errors, and allocate full UUID hex ids (no 8-char prefix collisions).
  • Split validate-before-write vs enforce-after-commit so failed temp writes do not evict existing quarantine evidence.
  • Also lands the SCP pipeline diagram HTML/SVG previously staged on this branch (docs only).

Test plan

  • pytest tests/test_security_regressions.py tests/test_registry_ssot.py tests/test_registry_fetch_r4.py tests/test_contract_document_hash.py tests/test_mcp_contract_v1.py (61 passed)
  • GitHub job CI green on this PR
  • Human merge (no auto-merge)

Supersedes

Closes the overlapping draft pile once this lands: #44 #45 #46 #47 #48 #49 #50 #51 #52 #53 #54 #55 #56 (close-as-superseded with comments). Does not pin mcp<2 (contract jobs already pass on mcp 2.x). No src/scp_mcp.egg-info/ commits.

Related

ManintheCrowds and others added 4 commits October 6, 2026 23:29
Reject symlink layouts and forged .json sidecars; stage atomic
writes with full UUID ids; fail closed before eviction on pressure.

Co-authored-by: Cursor <[email protected]>
Dedupe _safe_existing_layout_dir; skip redundant mkdir; short-circuit
validate when eviction is deferred to enforce.

Co-authored-by: Cursor <[email protected]>
@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, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

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