fixup! repl: add auto‑completion for dynamic import calls

ExE-Boss · ExE-Boss · commit 4264d4f32772 · 2021-02-02T20:58:58.000+01:00
diff --git a/lib/repl.js b/lib/repl.js
@@ -1319,9 +1319,9 @@ function complete(line, callback) {
           return;
         }
 
-        const extension = path.extname(name);
         if (!dirent.isDirectory()) {
-          if (StringPrototypeIncludes(extensions, extension) && !subdir) {
+          const extension = path.extname(name);
+          if (StringPrototypeIncludes(extensions, extension)) {
             ArrayPrototypePush(group, `${subdir}${name}`);
           }
           return;
@@ -1345,7 +1345,7 @@ function complete(line, callback) {
     }
 
     if (!subdir) {
-      ArrayPrototypePush(completionGroups, nodeSchemeBuiltinLibs, _builtinLibs);
+      ArrayPrototypePush(completionGroups, _builtinLibs, nodeSchemeBuiltinLibs);
     }
   } else if (RegExpPrototypeTest(fsAutoCompleteRE, line) &&
              this.allowBlockingCompletions) {

Original file line number	Diff line number	Diff line change
`@@ -1319,9 +1319,9 @@ function complete(line, callback) {`
`1319`	`1319`	`return;`
`1320`	`1320`	`}`
`1321`	`1321`
`1322`		`- const extension = path.extname(name);`
`1323`	`1322`	`if (!dirent.isDirectory()) {`
`1324`		`- if (StringPrototypeIncludes(extensions, extension) && !subdir) {`
	`1323`	`+ const extension = path.extname(name);`
	`1324`	`+ if (StringPrototypeIncludes(extensions, extension)) {`
`1325`	`1325`	ArrayPrototypePush(group, `${subdir}${name}`);
`1326`	`1326`	`}`
`1327`	`1327`	`return;`
`@@ -1345,7 +1345,7 @@ function complete(line, callback) {`
`1345`	`1345`	`}`
`1346`	`1346`
`1347`	`1347`	`if (!subdir) {`
`1348`		`- ArrayPrototypePush(completionGroups, nodeSchemeBuiltinLibs, _builtinLibs);`
	`1348`	`+ ArrayPrototypePush(completionGroups, _builtinLibs, nodeSchemeBuiltinLibs);`
`1349`	`1349`	`}`
`1350`	`1350`	`} else if (RegExpPrototypeTest(fsAutoCompleteRE, line) &&`
`1351`	`1351`	`this.allowBlockingCompletions) {`