Skip to content

Ignore Codex local state during setup#452

Merged
NagyVikt merged 1 commit into
mainfrom
agent/codex/ignore-codex-in-setup-gitignore-2026-04-29-11-52
Apr 29, 2026
Merged

Ignore Codex local state during setup#452
NagyVikt merged 1 commit into
mainfrom
agent/codex/ignore-codex-in-setup-gitignore-2026-04-29-11-52

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

Guardex already appends a managed gitignore block during setup and doctor repair. Add .codex/ to that canonical path list so repo-local Codex state is excluded automatically anywhere the managed block is created or refreshed.

Constraint: setup and doctor both consume MANAGED_GITIGNORE_PATHS through the shared scaffold path.

Rejected: Patch only generated .gitignore fixtures | would not affect real gx setup or doctor runs

Confidence: high

Scope-risk: narrow

Directive: Keep gitignore behavior centralized in MANAGED_GITIGNORE_PATHS rather than duplicating setup/doctor-specific lists.

Tested: node --test --test-name-pattern "setup provisions workflow files and repo config|setup appends managed gitignore block without clobbering existing entries|setup and doctor explain .githooks file conflicts and still write managed gitignore first|shared context keeps the drift-prone help text, gitignore paths, and release repo root" test/setup.test.js test/cli-args-dispatch.test.js

Tested: npm test

Not-tested: GitHub Actions rerun
@NagyVikt NagyVikt merged commit 73a2d6d into main Apr 29, 2026
@NagyVikt NagyVikt deleted the agent/codex/ignore-codex-in-setup-gitignore-2026-04-29-11-52 branch April 29, 2026 09:58
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