Skip to content

fix: make shim validation safe and accurate - #16

Open
rudijetson wants to merge 2 commits into
codex/codex-standalone-ownershipfrom
fix/15-shim-validation
Open

fix: make shim validation safe and accurate#16
rudijetson wants to merge 2 commits into
codex/codex-standalone-ownershipfrom
fix/15-shim-validation

Conversation

@rudijetson

Copy link
Copy Markdown
Contributor

Summary

  • resolve assigned shim targets before validating them
  • accept only executable files, not executable directories or documentation
  • use explicit command and argument boundaries for shim checks and repairs
  • preserve unmanaged wrappers that cannot be validated safely
  • add focused core and command-level regression coverage

Closes #15.

Verification

  • focused shim tests: 8 passed, 0 failed
  • pnpm test: 655 passed, 0 failed
  • pnpm build: passed; tracked bundle reproduced cleanly after commit
  • debt scan: 0 findings
  • npm pack --dry-run --json: passed with 6 expected package files
  • git diff --check: passed

Stack

This PR is intentionally based on codex/codex-standalone-ownership (#14), because both concerns regenerate dist/index.cjs. After #14 lands, retarget this PR to main; its two-commit diff is otherwise independent.

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