Skip to content

fix: gate codex skip-git flag and harden continue validation#5

Merged
avifenesh merged 2 commits intomainfrom
consult-conditional-skip-git-flag
Mar 1, 2026
Merged

fix: gate codex skip-git flag and harden continue validation#5
avifenesh merged 2 commits intomainfrom
consult-conditional-skip-git-flag

Conversation

@avifenesh
Copy link
Copy Markdown
Contributor

@avifenesh avifenesh commented Mar 1, 2026

Summary

  • replace hardcoded Codex skip-git usage with conditional {SKIP_GIT_FLAG} templates
  • document deterministic trust-gate resolution (trusted git repo => empty, trusted non-repo => --skip-git-repo-check)
  • require explicit substitution of {SKIP_GIT_FLAG} and forbid inherited env-based expansion
  • harden --continue restore guidance with strict re-validation for tool, session_id, and model
  • strengthen regression checks to pin conditional behavior and prevent hardcoded skip-git regressions

Validation

  • npm test
  • npm run validate

@avifenesh avifenesh merged commit 80aba5a into main Mar 1, 2026
2 checks passed
@avifenesh avifenesh deleted the consult-conditional-skip-git-flag branch March 1, 2026 21:30
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