Skip to content

codex-local 5/9: fork-surface enforcement - #9

Closed
ehrvs wants to merge 2 commits into
pr/04-resumefrom
pr/05-surface
Closed

ehrvs wants to merge 2 commits into
pr/04-resumefrom
pr/05-surface

Conversation

@ehrvs

@ehrvs ehrvs commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator

local/scripts/check_fork_surface.py diffs against upstream/<UPSTREAM_VERSION> and checks:

  • the path allowlist
  • the line/file budget, currently 122/11 against a budget of 135/12
  • via cargo metadata, that no upstream crate depends on codex-local-*, and that local crates are reachable only from codex-local-cli

This replaces the planned sc-lint records: sc-lint has no wildcard mode, so it would need a static list of 161 crates.

Tests: 3 unit tests.

Stack (merge bottom-up): 01-cli → 02-h8 → 03-models → 04-resume → 05-surface → 06-install → 07-ci → 08-sync → 09-docs. Plan: local/docs/plan-deltas.md; spike: local/docs/spike-responses.md.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RPCzgwU247gMtjybR5v4iR

ehrvs and others added 2 commits October 14, 2026 15:44
check_fork_surface.py diffs HEAD against the upstream tag and fails on
non-allowlisted upstream paths, budget overruns, or any upstream crate
depending on codex-local-* (via cargo metadata, no crate lists).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Qp4Gg5UQcjWVPZz3y58bWs
@ehrvs

ehrvs commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded: merged into codex-local/main via the integration PR #21 (3908b3d).

@ehrvs ehrvs closed this Oct 16, 2026
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