feat(index): add --hidden flag for dot-prefixed files#97
Closed
peterkc wants to merge 1 commit into
Closed
Conversation
Add `--hidden` CLI flag to include hidden (dot-prefixed) files and directories in indexing and search operations. Changes: - Add --hidden flag to CLI (ck-cli) - Add show_hidden field to FileCollectionOptions (ck-core) - Add hidden field to SearchOptions (ck-core) - Make WalkBuilder.hidden() conditional on config (ck-index) - Wire flag through MCP server, TUI, and engine - Add integration test for --hidden functionality - Update CHANGELOG.md The flag defaults to false (hidden files excluded) to maintain backward compatibility. When enabled, hidden directories like .claude/ can be indexed and searched. Closes: BeaconBay/ck#XX
Contributor
|
Hi @peterkc 👋 This PR is well-implemented (clean Heads-up: Would you like to:
No pressure, just nudging since it's a small, useful feature. |
runonthespot
added a commit
that referenced
this pull request
Jul 9, 2026
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>
Contributor
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
2 participants
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.
Summary
Add
--hiddenCLI flag to include hidden (dot-prefixed) files and directories in indexing and search operations.Use case: Users want to index configuration directories like
.claude/,.config/, or.github/that are hidden by default.Changes
--hiddenflag to CLI (ck-cli/src/main.rs)show_hiddenfield toFileCollectionOptions(ck-core)hiddenfield toSearchOptions(ck-core)WalkBuilder.hidden()conditional on config (ck-index)--hiddenfunctionalityBehavior
ck --index .ck --hidden --index .ck --hidden --sem "query" .The flag defaults to
falseto maintain backward compatibility.Test plan
cargo test --workspacepasses.hidden-test/directorytest_hidden_flag_includes_hidden_files--hidden+--no-ignorecombination works correctly