@t09tanaka/test-gates 0.5.1 → 0.5.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.5.2
+
+### Fixed
+
+- `selfcheck --jobs` really no longer keeps every other control apart because of the word `require` in a string, a template, a comment or a regular expression. 0.5.1 fixed the reason given for a spec, but the plan of which controls run on their own still looked at the whole text.
+
 ## 0.5.1
 
 ### Fixed

package/dist/cli.js

@@ -5447,6 +5447,9 @@ var REACHES = [
 ];
 var COMPUTED_LOAD = /(?:require|import)\((?!['"])|\brequire(?!\(|\.resolve\b|[\w$])|\bimport\.meta\.glob\b/;
 var COMPUTED = "loads a module whose name is computed";
+function loadsComputed(source) {
+  return COMPUTED_LOAD.test(withoutLiterals(source).replace(/\s/g, ""));
+}
 var STRING = /(^|[:,([={?+&|;])(\s*)(['"])((?:\\.|(?!\3)[^\\\n])*)\3(?=\s*(?:[,}\]);:+?&|]|$))/gm;
 function withoutStrings(source) {
   return source.replace(
@@ -5537,13 +5540,12 @@ var PURE_BUILTINS = [
 ];
 var RUNNER_PACKAGES = ["vitest", "@jest/globals"];
 function reachesOutside(file, trusted) {
-  const dense = withoutLiterals(file.source).replace(/\s/g, "");
   const code = withoutStrings(file.source).replace(/\s/g, "");
   const mark = REACHES.find((text) => code.includes(text));
   if (mark !== void 0) {
     return `uses ${mark.replace(/[.?[(:]+$/, "")}`;
   }
-  if (COMPUTED_LOAD.test(dense)) {
+  if (loadsComputed(file.source)) {
     return COMPUTED;
   }
   for (const { specifier, builtin } of file.imports) {
@@ -5583,7 +5585,7 @@ function jobsPlan(input) {
     const reaches = loaded.flatMap((file) => {
       const reach = reachesOutside(file, input.trusted);
       const said = reach === null ? [] : [`${file.path} ${reach}`];
-      if (!COMPUTED_LOAD.test(file.source.replace(/\s/g, ""))) {
+      if (!loadsComputed(file.source)) {
         return said;
       }
       computed = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@t09tanaka/test-gates",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "Test gates for the code that must not be wrong: per-file 100% coverage as the floor, mutation testing with a per-mutant allow list as the real check.",
   "author": "t09tanaka",
   "license": "MIT",