Conversation
Co-authored-by: WillBooster (Codex CLI) <agent@willbooster.com>
There was a problem hiding this comment.
Code Review
This pull request updates the agent instruction generator to include a guideline for placing temporary files in the .tmp or /tmp directories. The review feedback suggests clarifying that the .tmp directory should be at the project root to prevent clutter in monorepos and ensure files are covered by root-level ignore rules.
| - If not specified, make sure to add a new line at the end of your commit message${rootConfig.isWillBoosterRepo ? ` with: \`Co-authored-by: WillBooster (${toolName}) <agent@willbooster.com>\`` : ''}. | ||
| - Always create new commits. Avoid using \`--amend\`. | ||
| - Always use heredoc syntax when passing multi-line content to any command. | ||
| - Put temporary files in the \`.tmp\` or \`/tmp\` directory. |
There was a problem hiding this comment.
The generated files (e.g., AGENTS.md, CLAUDE.md, GEMINI.md, and .cursor/rules/general.mdc) appear to be out of sync with this change. Please ensure you run the generator to update these files in the repository.
Additionally, it's better to clarify that the .tmp directory should be at the project root. This helps avoid cluttering sub-packages in monorepos and ensures the files are more likely to be covered by a root-level .gitignore.
| - Put temporary files in the \`.tmp\` or \`/tmp\` directory. | |
| - Put temporary files in the project root's \`.tmp\` or the system \`/tmp\` directory. |
Summary
.tmpor/tmpwbfyagent instruction generatorWhy
Testing
yarn verify