Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Go to File" limited to recently opened files #39061

Closed
BeKnowDo opened this issue Nov 23, 2017 · 12 comments
Closed

"Go to File" limited to recently opened files #39061

BeKnowDo opened this issue Nov 23, 2017 · 12 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues verified Verification succeeded
Milestone

Comments

@BeKnowDo
Copy link

  • VSCode Version: Code 1.18.1 (929bacb, 2017-11-16T18:23:26.125Z)
  • OS Version: Darwin x64 17.2.0
  • Extensions:
Extension Author (truncated) Version
html-class-suggestions And 1.0.7
EditorConfig Edi 0.11.1
ReactSnippetsStandard Tim 1.1.0
html-css-class-completion Zig 1.14.0
vscode-caniuse aka 0.5.3
npm-intellisense chr 1.3.0
path-intellisense chr 1.4.2
vscode-eslint dba 1.4.3
vscode-instant-markdown dba 1.3.0
vscode-new-file dku 4.0.0
vscode-html-css ecm 0.1.8
auto-close-tag for 0.5.3
auto-rename-tag for 0.0.15
vscode-styled-components jpo 0.0.10
vscode-colorize kam 0.6.9
atom-keybindings ms- 3.0.2
vscode-template-literal-editor pli 0.8.4
vscode-icons rob 7.18.1
ignore-gitignore stu 1.0.1
vscode-import-cost wix 2.5.1
JavaScriptSnippets xab 1.5.0
ReactSnippets xab 1.7.0

(4 theme extensions excluded)


Steps to Reproduce:

  1. CMD + P
  2. Search for any file that has not been recently opened/accessed

Reproduces without extensions: Yes

@bpasero bpasero added the search Search widget and operation issues label Nov 25, 2017
@bpasero bpasero assigned roblourens and unassigned bpasero Nov 25, 2017
@bpasero bpasero added the info-needed Issue requires more information from poster label Nov 25, 2017
@bpasero
Copy link
Member

bpasero commented Nov 25, 2017

@VeteranPatriot what is the issue?

@iamstevendao
Copy link

@VeteranPatriot were you trying to search a file which is in your .gitignore and you enabled search.useIgnoreFiles : true?

@itomek
Copy link

itomek commented Nov 30, 2017

Hi, I am having the same exact issue. For instance I have the following files in the current open folder:

  • vault-dashboard
    • vault-dashboard.component.html
    • vault-dashboard.component.scss
    • vault-dashboard.component.spec.ts
    • vault-dashboard.component.ts
  • vault-item-share
    • vault-item-share.component.html
    • vault-item-share.component.scss
    • vault-item-share.component.spec.ts
    • vault-item-share.component.ts

and I recently opened vault-dashboard.component.ts but NOT vault-item-share.component.ts...

When I use Go to File to search and type in "vault" I only see "vault-dashboard.component...." files listed. This happens for all files in open folder.

@bpasero Benjamin, for me the issue is that I am only seeing files when using "Go to file" that have been recently open instead of all the files available in the open folder.

@itomek
Copy link

itomek commented Nov 30, 2017

Here is a video of the issue being reproduced: https://1drv.ms/v/s!AsWzMHwD7gfb8RwKTBdVTs6c64f7

@roblourens
Copy link
Member

roblourens commented Nov 30, 2017

Does it still happen if you set "search.useRipgrep": false?

Does search work?

@chrmarti
Copy link
Contributor

@itomek Could you share your settings? Does this only occur with these two subfolders or also others?

@itomek
Copy link

itomek commented Nov 30, 2017

@roblourens that seemed to have fixed it.

@chrmarti
VSCode settings
{ "editor.minimap.enabled": false, "workbench.colorTheme": "One Dark Pro", "terminal.integrated.shell.osx": "/usr/local/bin/bash" }

Workspace Settings
// Place your settings in this file to overwrite default and user settings. { "files.exclude": { "**/.git": true, "**/.svn": true, "**/.hg": true, "**/CVS": true, "**/.DS_Store": true, "node_modules": true, ".vscode/chrome": true, "coverage": true, "dist": true, "**.*.trx": true } }

Ok I think I found the culprit... its this last line for me: "**.*.trx": true. Once I remove it, the search is working well.

I have just learned that the files.exclude also works on the "Go to File"? Is this intentional?

@roblourens
Copy link
Member

roblourens commented Nov 30, 2017

"**.*.trx": true That's right, ripgrep doesn't allow patterns like that.

@chrmarti In text search, an error message would pop up, but we don't have anywhere to surface messages like that in quickopen.

I have just learned that the files.exclude also works on the "Go to File"? Is this intentional?

Yes

@itomek
Copy link

itomek commented Nov 30, 2017

Thank you everyone!

@chrmarti chrmarti added this to the November 2017 milestone Nov 30, 2017
@chrmarti
Copy link
Contributor

chrmarti commented Dec 1, 2017

@roblourens Ripgrep's exit code has changed to 1 in this case. We ignore that exit code because we assume it means no results were found.

@chrmarti chrmarti added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Dec 1, 2017
@roblourens
Copy link
Member

The handling for this in text search is: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/search/node/ripgrepTextSearch.ts#L124

If there were no results, and there was some stderr output, and the stderr matches a whitelist of error messages, then return that as an error.

@roblourens
Copy link
Member

roblourens commented Dec 7, 2017

I only see the message in a single folder workspace... I can investigate

@roblourens roblourens reopened this Dec 7, 2017
roblourens added a commit that referenced this issue Dec 7, 2017
@RMacfarlane RMacfarlane added the verified Verification succeeded label Dec 8, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

7 participants