support ignored globs & limit in file search & plugin-ext

- in current Theia, passing "ignored globs" or "limit" to plugins-api from plugins does not work, as the data is dropped before reaching the file search service. This change ensures the data goes through and handled by the service. Signed-off-by: elaihau <liang.huang@ericsson.com>
eclipse-theia · Feb 26, 2019 · f57cb98 · f57cb98
1 parent 41bbd0e
commit f57cb98
Show file tree

Hide file tree

Showing 3 changed files with 45 additions and 10 deletions.
diff --git a/packages/file-search/src/node/file-search-service-impl.spec.ts b/packages/file-search/src/node/file-search-service-impl.spec.ts
@@ -78,4 +78,12 @@ describe('search-service', function () {
         expect(matches).to.be.not.undefined;
         expect(matches.length).to.eq(2);
     });
+
+    it('should ignore globs passed through the search options', async () => {
+        const rootUri = FileUri.create(path.resolve(__dirname, '../../test-resources/subdir1/sub2')).toString();
+
+        const matches = await service.find('foo', { rootUris: [rootUri], defaultIgnorePatterns: ['foo'] });
+        expect(matches).to.be.not.undefined;
+        expect(matches.length).to.eq(0);
+    });
 });
diff --git a/packages/file-search/src/node/file-search-service-impl.ts b/packages/file-search/src/node/file-search-service-impl.ts
@@ -37,20 +37,14 @@ export class FileSearchServiceImpl implements FileSearchService {
             limit: Number.MAX_SAFE_INTEGER,
             useGitIgnore: true,
             defaultIgnorePatterns: [
-                '^.git$'
+                '.git'
             ],
             ...options
         };
-        const args: string[] = [
-            '--files',
-            '--sort-files',
-        ];
-        if (!options.useGitIgnore) {
-            args.push('-uu');
-        }
+
         const process = this.rawProcessFactory({
             command: rgPath,
-            args: [...args, ...opts.rootUris.map(r => FileUri.fsPath(r))]
+            args: this.getSearchArgs(opts)
         });
         const result: string[] = [];
         const fuzzyMatches: string[] = [];
@@ -94,4 +88,30 @@ export class FileSearchServiceImpl implements FileSearchService {
         return resultDeferred.promise;
     }
 
+    private getSearchArgs(options: FileSearchService.Options): string[] {
+        const args: string[] = [
+            '--sort-files'
+        ];
+        if (!options.useGitIgnore) {
+            args.push('-uu');
+        }
+        if (options && options.defaultIgnorePatterns) {
+            options.defaultIgnorePatterns.filter(p => p !== '')
+                .forEach(ignore => {
+                    if (!ignore.endsWith('*')) {
+                        ignore = `${ignore}*`;
+                    }
+                    if (!ignore.startsWith('*')) {
+                        ignore = `!*${ignore}`;
+                    } else {
+                        ignore = `!${ignore}`;
+                    }
+                    args.push('--glob');
+                    args.push(ignore);
+                });
+        }
+        args.push('--files');
+        args.push(...options.rootUris.map(r => FileUri.fsPath(r)));
+        return args;
+    }
 }
diff --git a/packages/plugin-ext/src/main/browser/workspace-main.ts b/packages/plugin-ext/src/main/browser/workspace-main.ts
@@ -154,7 +154,14 @@ export class WorkspaceMainImpl implements WorkspaceMain {
 
     async $startFileSearch(includePattern: string, excludePatternOrDisregardExcludes?: string | false,
         maxResults?: number, token?: theia.CancellationToken): Promise<UriComponents[]> {
-        const uriStrs = await this.fileSearchService.find(includePattern, { rootUris: this.roots.map(r => r.uri) });
+        const opts: FileSearchService.Options = { rootUris: this.roots.map(r => r.uri) };
+        if (typeof excludePatternOrDisregardExcludes === 'string') {
+            opts.defaultIgnorePatterns = [excludePatternOrDisregardExcludes];
+        }
+        if (maxResults !== undefined) {
+            opts.limit = maxResults;
+        }
+        const uriStrs = await this.fileSearchService.find(includePattern, opts);
         return uriStrs.map(uriStr => Uri.parse(uriStr));
     }