Skip to content

feat: show .herdr directories in tree, search, and watch#19

Merged
exilis merged 1 commit into
masterfrom
feat/show-herdr-dir
Jul 12, 2026
Merged

feat: show .herdr directories in tree, search, and watch#19
exilis merged 1 commit into
masterfrom
feat/show-herdr-dir

Conversation

@exilis

@exilis exilis commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add .herdr to the isNoiseDir dot-directory allowlist so herdr worktree directories appear in the file tree, search, and live-reload watching — same treatment as the existing .allium and .claude exceptions.
  • Extend TestListDir_AlliumFilesAndDir to cover .herdr visibility.
  • CHANGELOG entry under [Unreleased].

Testing

  • go test ./... — all pass (test was written first and observed failing before the fix)
  • Verified live against a running instance: /api/tree lists .herdr at the root and /api/tree?path=.herdr lists its contents.

🤖 Generated with Claude Code

Add .herdr to the isNoiseDir dot-directory allowlist so herdr worktrees
are browsable like .allium and .claude.

Co-Authored-By: Claude Fable 5 <[email protected]>
@exilis
exilis merged commit f669120 into master Jul 12, 2026
3 checks passed
@exilis
exilis deleted the feat/show-herdr-dir branch July 12, 2026 10:05
exilis added a commit that referenced this pull request Jul 25, 2026
Resolves the tree.go / tree_test.go conflict between the .herdr
directory visibility added on master (#19) and the .worktrees
listing added on this branch.

Both special-cased the same dot-directory gate in isNoiseDir: master
extended an inline != chain, this branch replaced that chain with a
map allowlist. Kept the map (isUnfiltered depends on it) and folded
.herdr into it, so both directories are listed. Search and watch both
route through isNoiseDir, so they inherit the fix.

Verified: gofmt clean, go build, go vet, go test ./... (38 passed).

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
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