mirror of
https://github.com/windmill-labs/windmill.git
synced 2026-08-18 16:02:10 +00:00
docs(agents): state why a routine cleanup prompts, and where scratch goes
The guard hook already auto-allows a plain rm whose operands are under /tmp or inside a git checkout in $HOME, so deleting a temp dir or a stale .sqlx entry costs nothing. What prompts is the command shape: the hook's tokenizer defers on &&, ;, redirects, quotes and $VAR, so a chained cleanup falls through to the Bash(rm:*) ask rule. That was recorded only inside a paragraph about screenshot file paths in frontend/CLAUDE.md, where nobody looking for it would find it. Stated in Core Principles instead, alongside the rule that scratch belongs outside the tree — for the reason that actually applies, which is not committing junk rather than avoiding prompts. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -143,9 +143,11 @@ $NAV --root backend callees "X" # what does X call?
|
||||
|
||||
- **MUST `outline` before `Read`** on unfamiliar files — then `body` or `Read` with offset/limit for specifics
|
||||
- **Scratch stays outside the checkout.** Temp scripts, data dumps, cache backups and
|
||||
screenshots go in the session scratch directory or `/tmp`, never in the repo — a temp file
|
||||
in the tree has to be deleted again, and `rm` asks every time. When deletions are genuinely
|
||||
needed, do them together as one `rm` at the end rather than one call each.
|
||||
screenshots go in the session scratch directory or `/tmp`, so nothing temporary can end up
|
||||
committed. Write `rm`/`mv`/`cp` as one plain unchained command: a PreToolUse hook
|
||||
auto-allows those when every operand is under `/tmp` or inside this checkout, but it defers
|
||||
on `&&`, `;`, redirects, quotes and `$VAR` — that deferral, not the delete itself, is what
|
||||
turns a routine cleanup into a permission prompt.
|
||||
- Search for existing code to reuse before writing new code
|
||||
- Follow established patterns in the codebase
|
||||
- Keep changes focused — don't refactor beyond what's asked
|
||||
|
||||
Reference in New Issue
Block a user