Skip to content

fix(file-explorer): align default exclusions with VS Code - #1633

Merged
wsp1911 merged 1 commit into
GCWing:mainfrom
wsp1911:dev
Jul 19, 2026
Merged

fix(file-explorer): align default exclusions with VS Code#1633
wsp1911 merged 1 commit into
GCWing:mainfrom
wsp1911:dev

Conversation

@wsp1911

@wsp1911 wsp1911 commented Jul 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Align default file-tree exclusions with VS Code's built-in baseline.
  • Show dotfiles, IDE metadata, dependency directories, and build outputs by default.
  • Keep .git, .svn, .hg, .DS_Store, and Thumbs.db excluded.
  • Add a Rust contract test for the default visibility behavior.

Fixes #

Type and Areas

Type:

bug fix / UI/UX / test

Areas:

Rust core, desktop/Tauri, web UI

Motivation / Impact

The workspace explorer previously hid a broad hard-coded set of project directories such as target, node_modules, dist, and .vscode. It now follows VS Code's narrower built-in exclusion baseline, making project files and generated outputs visible by default.

The follow-up work for user-configurable glob exclusions, .gitignore handling, persistence, and remote-rule unification is intentionally out of scope.

Verification

  • pnpm run fmt:rs
  • cargo test -p bitfun-services-core default_tree_visibility_matches_vscode_baseline
  • pnpm run type-check:web
  • git diff --check

Reviewer Notes

This change only updates default visibility. Search filtering and remote SSH directory filtering remain unchanged.

Checklist

  • This PR is focused and does not include secrets, temporary prompts, generated scratch files, or unrelated artifacts.
  • Relevant verification is recorded above, or skipped checks are explained.
  • User-facing strings, docs, and locales are updated where applicable.

@wsp1911
wsp1911 marked this pull request as ready for review July 19, 2026 11:13
@wsp1911
wsp1911 merged commit be58b3a into GCWing:main Jul 19, 2026
5 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