docs: forbid unbounded home directory searches in agent notes - #1492
Merged
Conversation
…pipeline Add prominent rule at the start of NOTES explaining why find /Users/coder searches wedge the pipeline for 19-96 minutes, with concrete guidance on: - Tools resolution via mise (mise which, mise exec) - Cluster credentials (kubeconfig, talosconfig) expected absence in fresh worktrees - Reporting missing tools instead of searching This generalizes beyond PR #1482's kustomize-specific patch to prevent recurrence with other tools or credentials. The rule reads as a hazard rather than a style preference. Measured occurrences: - 2026-08-28: kustomize searches (19 + 6 minutes) - 2026-08-29: kubeconfig searches (96 minutes + recurrence)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Intent
Add prominent rule to AGENTS.md forbidding unbounded find searches of /Users/coder and /Users/coder. Explains the concrete consequence (19-96 minute wedges), shows how to find tools properly (mise), and clarifies that cluster credentials like kubeconfig don't exist in fresh worktrees. Generalizes PR #1482's kustomize-specific patch to prevent recurrence. Documentation only, no changes to workflows or tooling.
What Changed
AGENTS.mdbanning unboundedfindof/Users/coder,$HOME, and parent paths, citing 19–96 minute validation wedges and requiring agents to report missing tools/files instead of hunting for them.mise which/mise exec, plus the expectation thatkubeconfig/talosconfigare absent in fresh worktrees and must not be searched for.Risk Assessment
✅ Low: Documentation-only AGENTS.md guidance that matches the stated intent with no workflow, tooling, or behavioral code changes.
Testing
Verified the agent-facing NOTES surface and CLAUDE.md symlink against the full intent checklist, confirmed only AGENTS.md changed, and demonstrated expected-missing cluster credentials plus the mise lookup path without home-directory find; all acceptance checks passed.
Evidence: Intent acceptance checklist (all PASS)
intent_acceptance: forbids_find_Users_coder: PASS forbids_find_HOME: PASS forbids_unbounded_home_or_parent: PASS consequence_19_96_minutes: PASS mise_which_pattern: PASS mise_install_path: PASS kubeconfig_absent_expected: PASS talosconfig_mentioned: PASS report_and_continue: PASS generalized_not_kustomize_only: PASS cross_ref_from_cli_bullet: PASS rule_is_first_notes_bullet: PASS overall: PASSEvidence: Agent-facing NOTES opening with new first bullet
Evidence: Fresh worktree credentials absent + mise path (no home find)
absent (expected): kubeconfig absent (expected): talos/talosconfigEvidence: CLAUDE.md symlink surfaces identical rule
CLAUDE.md -> AGENTS.md identical_bytes: trueEvidence: Full acceptance summary
Pipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
✅ **Review** - passed
✅ No issues found.
✅ **Test** - passed
✅ No issues found.
git diff --name-only acfcde3592f00e26a570e678e8dd84325eebeb58..HEAD(docs-only: AGENTS.md)intent acceptance checklist over AGENTS.md NOTES (12 required constraints)readlink/shasumCLAUDE.md symlink identity with AGENTS.mdfresh-worktree credential absence check forkubeconfigandtalos/talosconfiggit check-ignore -v kubeconfig talos/talosconfigdoc-recommended tool lookup viamise whichwithout unboundedfind $HOME✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.