files.exclude have extra globs

[studgeek]

Does this issue occur when all extensions are disabled?: Yes

Version: 1.99.3
Commit: 17baf84
Date: 2025-04-15T23:18:46.076Z
Electron: 34.3.2
ElectronBuildId: 11161073
Chromium: 132.0.6834.210
Node.js: 20.18.3
V8: 13.2.152.41-electron.0
OS: Darwin arm64 23.6.0

The default values for files.exclude include doublestar glob suffixes, but I since #137872 I think they are no longer needed? If so, they should be removed so that default serves as a better usage example.

Current default:

  "files.watcherExclude": {
    "**/.git/objects/**": true,
    "**/.git/subtree-cache/**": true,
    "**/.hg/store/**": true
  },

Suggested defaults:

  "files.watcherExclude": {
    "**/.git/objects": true,
    "**/.git/subtree-cache": true,
    "**/.hg/store": true
  },

If they are still needed then documentation should be updated to explain how watcherExclude globbing works differently from files.exclude.

[lramos15]

@bpasero Thoughts?

[bpasero]

Sorry, but what is the issue here exactly?

[lramos15]

I think the issue is they're stating that the file excludes don't require the extra /** and are wondering why the two exclude defaults differ

[vs-code-engineering]

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

[vs-code-engineering]

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

[vs-code-engineering]

🙁 In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.

Happy Coding!