Skip to content

fix: update agent temp file guidance#749

Merged
exKAZUu merged 1 commit intomainfrom
codex/update-agents-generator
May 5, 2026
Merged

fix: update agent temp file guidance#749
exKAZUu merged 1 commit intomainfrom
codex/update-agents-generator

Conversation

@exKAZUu
Copy link
Copy Markdown
Member

@exKAZUu exKAZUu commented May 5, 2026

Summary

  • Add agent instruction guidance to place temporary files under .tmp or /tmp
  • Keep the change scoped to the wbfy agent instruction generator

Why

  • Makes generated agent instructions clearer about where temporary files should live
  • Avoids leaving throwaway files in arbitrary repository locations

Testing

  • yarn verify

Co-authored-by: WillBooster (Codex CLI) <agent@willbooster.com>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

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.

Suggested change
- Put temporary files in the \`.tmp\` or \`/tmp\` directory.
- Put temporary files in the project root's \`.tmp\` or the system \`/tmp\` directory.

@exKAZUu exKAZUu changed the title docs: update agent temp file guidance fix: update agent temp file guidance May 5, 2026
@exKAZUu exKAZUu enabled auto-merge (squash) May 5, 2026 02:22
@exKAZUu exKAZUu merged commit c697cc8 into main May 5, 2026
8 checks passed
@exKAZUu exKAZUu deleted the codex/update-agents-generator branch May 5, 2026 02:22
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