Skip to content

fix(glob): match explicitly requested dot directories#32907

Closed
dannyward630 wants to merge 1 commit into
anomalyco:devfrom
dannyward630:fix/glob-dot-directories
Closed

fix(glob): match explicitly requested dot directories#32907
dannyward630 wants to merge 1 commit into
anomalyco:devfrom
dannyward630:fix/glob-dot-directories

Conversation

@dannyward630

Copy link
Copy Markdown

Issue for this PR

Closes #32669

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Allows the Glob tool to match files under dot directories when the user explicitly asks for one, such as .ai/**/*.md or a search path inside .ai.

The tool now enables ripgrep hidden-file scanning only when the requested glob pattern or search path includes a dot-directory segment, so ordinary patterns like *.md keep the previous default behavior.

How did you verify your code works?

  • npx --yes bun@1.3.14 test test/tool/glob.test.ts
  • npx --yes bun@1.3.14 run typecheck

Screenshots / recordings

Not a UI change.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@github-actions

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Potential Duplicate Found:

Recommendation: Check if PR #32866 is already merged or still open. If it's still open, one of these PRs may be redundant and should be closed in favor of the other.

@dannyward630

Copy link
Copy Markdown
Author

Closing as duplicate of #32866, which already covers explicit dot-directory glob support.

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.

Glob tool does not match files under dot directories

1 participant