Skip to content

feat(index): add --hidden flag for dot-prefixed files - #176

Merged
runonthespot merged 1 commit into
mainfrom
fix/hidden-flag
Jul 9, 2026
Merged

feat(index): add --hidden flag for dot-prefixed files#176
runonthespot merged 1 commit into
mainfrom
fix/hidden-flag

Conversation

@runonthespot

Copy link
Copy Markdown
Contributor

Re-implements #97 (original by @peterkc) cleanly on current main — the original branch was too stale to rebase (touched since-removed files).

Changes

  • New --hidden flag: "Include hidden (dot-prefixed) files and directories"
  • Threads through both the search and index walkers via FileCollectionOptions.show_hidden / SearchOptions.hiddenWalkBuilder.hidden(!show_hidden)
  • Independent of --no-ignore/--no-ckignore (dot-visibility vs ignore-file honoring are separate)
  • README + CHANGELOG updated

Validation

  • 2 new integration tests (regex search path + lexical index path) pass
  • Full workspace compiles; ck-index suite green; clippy clean

🤖 Generated with Claude Code

Add a `--hidden` CLI flag that includes hidden (dot-prefixed) files and
directories in both search and indexing. Off by default to preserve the
current "skip hidden" behavior.

The flag threads through `SearchOptions.hidden` and
`FileCollectionOptions.show_hidden` to the `ignore` crate's
`WalkBuilder.hidden(!show_hidden)` in `ck-index::collect_files`, which is
the single walker shared by the search path (`ck-engine`) and the
index-building path (embedding + lexical corpus). Composes independently
with `--no-ignore` / `--no-ckignore`.

Adds two integration tests (regex search path and lexical index path),
README docs, and a CHANGELOG entry.

Re-implements #97 (original by @peterkc), cleanly on current main.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@runonthespot
runonthespot merged commit f4caaf5 into main Jul 9, 2026
14 checks passed
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