@node9/proxy 1.67.2 → 1.67.3

package/dist/cli.js

@@ -3202,9 +3202,34 @@ var init_dist = __esm({
       "vi",
       "emacs",
       "code",
-      "type"
+      "type",
+      // — the 22 that were missing —
+      "grep",
+      "egrep",
+      "fgrep",
+      "rg",
+      "ag",
+      "ack",
+      "awk",
+      "gawk",
+      "sed",
+      "cut",
+      "tr",
+      "jq",
+      "yq",
+      "od",
+      "xxd",
+      "hexdump",
+      "strings",
+      "sort",
+      "uniq",
+      "tac",
+      "nl",
+      "dd"
     ]);
-    FS_OP_PRESCREEN_RE = /(?:^|[\s|;&(`\n])(?:rm|cat|less|head|tail|bat|more|open|print|nano|vim|vi|emacs|code|type)\b/;
+    FS_OP_PRESCREEN_RE = new RegExp(
+      `(?:^|[\\s|;&(\`\\n])(?:rm|${[...FS_READ_TOOLS].map((t) => t.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|")})\\b`
+    );
     HOME_CACHE_ALLOWLIST = [
       ".cache",
       ".npm/_npx",
@@ -3243,9 +3268,37 @@ var init_dist = __esm({
         // for the canonical test-asserted contract.
         rule: "shield:project-jail:block-read-env",
         reason: "Reading .env files is blocked by project-jail shield",
-        match: (p) => /(?:^|[\\/])\.env(?:\.(?:local|production|staging|development|production\.local|staging\.local|development\.local))?$/i.test(
-          p
-        )
+        // Structural, not a list. The previous form enumerated seven suffixes and
+        // anchored on `$`, so `.env.prod`, `.env.ci` and `.env.local.bak` — all
+        // gitignored, all routinely holding real secrets — were never covered. A
+        // hand-written list of what to protect is only ever as complete as the day
+        // it was typed; this says "`.env` plus any suffix chain" and then names the
+        // exceptions, which is the direction that fails safe.
+        //
+        //   \.env          the segment itself
+        //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+        //                  files. Without it a flat suffix class swallows both.
+        //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+        //   [\w.-]*$       any suffix chain. Flat class, no nested quantifier —
+        //                  `(\.[\w-]+)*` reads the same but is rejected by
+        //                  safe-regex2, and this pattern runs on the hook hot path.
+        //
+        // The two exclusions are NOT the same shape, because the words do not mean
+        // the same thing:
+        //
+        //   (?!\.(?:example|sample|template)\b)  — "this file is a fixture", and it
+        //     stays a fixture whatever follows, so `.env.example.md` is allowed too.
+        //     These are checked into git by convention: already public, so blocking
+        //     them buys nothing and costs the most common legitimate agent read.
+        //
+        //   (?!\.test$)  — anchored, because `test` names an ENVIRONMENT, not a
+        //     fixture. `.env.test` is the committed template and stays allowed, but
+        //     `.env.test.local` is gitignored by the `.env*.local` convention and
+        //     holds real values, so it must block. Using `\b` here — the obvious
+        //     symmetry — silently exempts every `.env.test.*` file.
+        //
+        // shields.test.ts:983-995 is the canonical contract; keep both in step.
+        match: (p) => /(?:^|[\\/])\.env(?![\w-])(?!\.(?:example|sample|template)\b)(?!\.test$)[\w.-]*$/i.test(p)
       },
       {
         // verdict: 'review' (not 'block') is a deliberate design choice
@@ -4403,7 +4456,7 @@ var init_dist = __esm({
     REALTIME_PII_PATTERNS = ["SSN", "Credit Card"];
     MAX_PII_SCAN_BYTES = 1e5;
     LONG_OUTPUT_THRESHOLD_BYTES = 100 * 1024;
-    CANONICAL_EXTRACTOR_VERSION = "canonical-v8";
+    CANONICAL_EXTRACTOR_VERSION = "canonical-v9";
     DEDUPE_PREVIEW_LEN = 120;
     TERMINAL_ESCAPE_RE = // eslint-disable-next-line no-control-regex
     /\x1b\[[0-9;?]*[A-Za-z]|\x1b\][^\x07\x1b]*(?:\x07|\x1b\\)|\x1b[@-_]|[\x00-\x08\x0b\x0c\x0e-\x1f\x7f]/g;

package/dist/cli.mjs

@@ -3206,9 +3206,34 @@ var init_dist = __esm({
       "vi",
       "emacs",
       "code",
-      "type"
+      "type",
+      // — the 22 that were missing —
+      "grep",
+      "egrep",
+      "fgrep",
+      "rg",
+      "ag",
+      "ack",
+      "awk",
+      "gawk",
+      "sed",
+      "cut",
+      "tr",
+      "jq",
+      "yq",
+      "od",
+      "xxd",
+      "hexdump",
+      "strings",
+      "sort",
+      "uniq",
+      "tac",
+      "nl",
+      "dd"
     ]);
-    FS_OP_PRESCREEN_RE = /(?:^|[\s|;&(`\n])(?:rm|cat|less|head|tail|bat|more|open|print|nano|vim|vi|emacs|code|type)\b/;
+    FS_OP_PRESCREEN_RE = new RegExp(
+      `(?:^|[\\s|;&(\`\\n])(?:rm|${[...FS_READ_TOOLS].map((t) => t.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|")})\\b`
+    );
     HOME_CACHE_ALLOWLIST = [
       ".cache",
       ".npm/_npx",
@@ -3247,9 +3272,37 @@ var init_dist = __esm({
         // for the canonical test-asserted contract.
         rule: "shield:project-jail:block-read-env",
         reason: "Reading .env files is blocked by project-jail shield",
-        match: (p) => /(?:^|[\\/])\.env(?:\.(?:local|production|staging|development|production\.local|staging\.local|development\.local))?$/i.test(
-          p
-        )
+        // Structural, not a list. The previous form enumerated seven suffixes and
+        // anchored on `$`, so `.env.prod`, `.env.ci` and `.env.local.bak` — all
+        // gitignored, all routinely holding real secrets — were never covered. A
+        // hand-written list of what to protect is only ever as complete as the day
+        // it was typed; this says "`.env` plus any suffix chain" and then names the
+        // exceptions, which is the direction that fails safe.
+        //
+        //   \.env          the segment itself
+        //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+        //                  files. Without it a flat suffix class swallows both.
+        //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+        //   [\w.-]*$       any suffix chain. Flat class, no nested quantifier —
+        //                  `(\.[\w-]+)*` reads the same but is rejected by
+        //                  safe-regex2, and this pattern runs on the hook hot path.
+        //
+        // The two exclusions are NOT the same shape, because the words do not mean
+        // the same thing:
+        //
+        //   (?!\.(?:example|sample|template)\b)  — "this file is a fixture", and it
+        //     stays a fixture whatever follows, so `.env.example.md` is allowed too.
+        //     These are checked into git by convention: already public, so blocking
+        //     them buys nothing and costs the most common legitimate agent read.
+        //
+        //   (?!\.test$)  — anchored, because `test` names an ENVIRONMENT, not a
+        //     fixture. `.env.test` is the committed template and stays allowed, but
+        //     `.env.test.local` is gitignored by the `.env*.local` convention and
+        //     holds real values, so it must block. Using `\b` here — the obvious
+        //     symmetry — silently exempts every `.env.test.*` file.
+        //
+        // shields.test.ts:983-995 is the canonical contract; keep both in step.
+        match: (p) => /(?:^|[\\/])\.env(?![\w-])(?!\.(?:example|sample|template)\b)(?!\.test$)[\w.-]*$/i.test(p)
       },
       {
         // verdict: 'review' (not 'block') is a deliberate design choice
@@ -4407,7 +4460,7 @@ var init_dist = __esm({
     REALTIME_PII_PATTERNS = ["SSN", "Credit Card"];
     MAX_PII_SCAN_BYTES = 1e5;
     LONG_OUTPUT_THRESHOLD_BYTES = 100 * 1024;
-    CANONICAL_EXTRACTOR_VERSION = "canonical-v8";
+    CANONICAL_EXTRACTOR_VERSION = "canonical-v9";
     DEDUPE_PREVIEW_LEN = 120;
     TERMINAL_ESCAPE_RE = // eslint-disable-next-line no-control-regex
     /\x1b\[[0-9;?]*[A-Za-z]|\x1b\][^\x07\x1b]*(?:\x07|\x1b\\)|\x1b[@-_]|[\x00-\x08\x0b\x0c\x0e-\x1f\x7f]/g;

package/dist/dashboard.mjs

@@ -1239,9 +1239,34 @@ var init_dist = __esm({
       "vi",
       "emacs",
       "code",
-      "type"
+      "type",
+      // — the 22 that were missing —
+      "grep",
+      "egrep",
+      "fgrep",
+      "rg",
+      "ag",
+      "ack",
+      "awk",
+      "gawk",
+      "sed",
+      "cut",
+      "tr",
+      "jq",
+      "yq",
+      "od",
+      "xxd",
+      "hexdump",
+      "strings",
+      "sort",
+      "uniq",
+      "tac",
+      "nl",
+      "dd"
     ]);
-    FS_OP_PRESCREEN_RE = /(?:^|[\s|;&(`\n])(?:rm|cat|less|head|tail|bat|more|open|print|nano|vim|vi|emacs|code|type)\b/;
+    FS_OP_PRESCREEN_RE = new RegExp(
+      `(?:^|[\\s|;&(\`\\n])(?:rm|${[...FS_READ_TOOLS].map((t) => t.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|")})\\b`
+    );
     HOME_CACHE_ALLOWLIST = [
       ".cache",
       ".npm/_npx",
@@ -1280,9 +1305,37 @@ var init_dist = __esm({
         // for the canonical test-asserted contract.
         rule: "shield:project-jail:block-read-env",
         reason: "Reading .env files is blocked by project-jail shield",
-        match: (p) => /(?:^|[\\/])\.env(?:\.(?:local|production|staging|development|production\.local|staging\.local|development\.local))?$/i.test(
-          p
-        )
+        // Structural, not a list. The previous form enumerated seven suffixes and
+        // anchored on `$`, so `.env.prod`, `.env.ci` and `.env.local.bak` — all
+        // gitignored, all routinely holding real secrets — were never covered. A
+        // hand-written list of what to protect is only ever as complete as the day
+        // it was typed; this says "`.env` plus any suffix chain" and then names the
+        // exceptions, which is the direction that fails safe.
+        //
+        //   \.env          the segment itself
+        //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+        //                  files. Without it a flat suffix class swallows both.
+        //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+        //   [\w.-]*$       any suffix chain. Flat class, no nested quantifier —
+        //                  `(\.[\w-]+)*` reads the same but is rejected by
+        //                  safe-regex2, and this pattern runs on the hook hot path.
+        //
+        // The two exclusions are NOT the same shape, because the words do not mean
+        // the same thing:
+        //
+        //   (?!\.(?:example|sample|template)\b)  — "this file is a fixture", and it
+        //     stays a fixture whatever follows, so `.env.example.md` is allowed too.
+        //     These are checked into git by convention: already public, so blocking
+        //     them buys nothing and costs the most common legitimate agent read.
+        //
+        //   (?!\.test$)  — anchored, because `test` names an ENVIRONMENT, not a
+        //     fixture. `.env.test` is the committed template and stays allowed, but
+        //     `.env.test.local` is gitignored by the `.env*.local` convention and
+        //     holds real values, so it must block. Using `\b` here — the obvious
+        //     symmetry — silently exempts every `.env.test.*` file.
+        //
+        // shields.test.ts:983-995 is the canonical contract; keep both in step.
+        match: (p) => /(?:^|[\\/])\.env(?![\w-])(?!\.(?:example|sample|template)\b)(?!\.test$)[\w.-]*$/i.test(p)
       },
       {
         // verdict: 'review' (not 'block') is a deliberate design choice

package/dist/index.js

@@ -1371,9 +1371,34 @@ var FS_READ_TOOLS = /* @__PURE__ */ new Set([
   "vi",
   "emacs",
   "code",
-  "type"
+  "type",
+  // — the 22 that were missing —
+  "grep",
+  "egrep",
+  "fgrep",
+  "rg",
+  "ag",
+  "ack",
+  "awk",
+  "gawk",
+  "sed",
+  "cut",
+  "tr",
+  "jq",
+  "yq",
+  "od",
+  "xxd",
+  "hexdump",
+  "strings",
+  "sort",
+  "uniq",
+  "tac",
+  "nl",
+  "dd"
 ]);
-var FS_OP_PRESCREEN_RE = /(?:^|[\s|;&(`\n])(?:rm|cat|less|head|tail|bat|more|open|print|nano|vim|vi|emacs|code|type)\b/;
+var FS_OP_PRESCREEN_RE = new RegExp(
+  `(?:^|[\\s|;&(\`\\n])(?:rm|${[...FS_READ_TOOLS].map((t) => t.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|")})\\b`
+);
 var HOME_CACHE_ALLOWLIST = [
   ".cache",
   ".npm/_npx",
@@ -1412,9 +1437,37 @@ var SENSITIVE_PATH_RULES = [
     // for the canonical test-asserted contract.
     rule: "shield:project-jail:block-read-env",
     reason: "Reading .env files is blocked by project-jail shield",
-    match: (p) => /(?:^|[\\/])\.env(?:\.(?:local|production|staging|development|production\.local|staging\.local|development\.local))?$/i.test(
-      p
-    )
+    // Structural, not a list. The previous form enumerated seven suffixes and
+    // anchored on `$`, so `.env.prod`, `.env.ci` and `.env.local.bak` — all
+    // gitignored, all routinely holding real secrets — were never covered. A
+    // hand-written list of what to protect is only ever as complete as the day
+    // it was typed; this says "`.env` plus any suffix chain" and then names the
+    // exceptions, which is the direction that fails safe.
+    //
+    //   \.env          the segment itself
+    //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+    //                  files. Without it a flat suffix class swallows both.
+    //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+    //   [\w.-]*$       any suffix chain. Flat class, no nested quantifier —
+    //                  `(\.[\w-]+)*` reads the same but is rejected by
+    //                  safe-regex2, and this pattern runs on the hook hot path.
+    //
+    // The two exclusions are NOT the same shape, because the words do not mean
+    // the same thing:
+    //
+    //   (?!\.(?:example|sample|template)\b)  — "this file is a fixture", and it
+    //     stays a fixture whatever follows, so `.env.example.md` is allowed too.
+    //     These are checked into git by convention: already public, so blocking
+    //     them buys nothing and costs the most common legitimate agent read.
+    //
+    //   (?!\.test$)  — anchored, because `test` names an ENVIRONMENT, not a
+    //     fixture. `.env.test` is the committed template and stays allowed, but
+    //     `.env.test.local` is gitignored by the `.env*.local` convention and
+    //     holds real values, so it must block. Using `\b` here — the obvious
+    //     symmetry — silently exempts every `.env.test.*` file.
+    //
+    // shields.test.ts:983-995 is the canonical contract; keep both in step.
+    match: (p) => /(?:^|[\\/])\.env(?![\w-])(?!\.(?:example|sample|template)\b)(?!\.test$)[\w.-]*$/i.test(p)
   },
   {
     // verdict: 'review' (not 'block') is a deliberate design choice

package/dist/index.mjs

@@ -1341,9 +1341,34 @@ var FS_READ_TOOLS = /* @__PURE__ */ new Set([
   "vi",
   "emacs",
   "code",
-  "type"
+  "type",
+  // — the 22 that were missing —
+  "grep",
+  "egrep",
+  "fgrep",
+  "rg",
+  "ag",
+  "ack",
+  "awk",
+  "gawk",
+  "sed",
+  "cut",
+  "tr",
+  "jq",
+  "yq",
+  "od",
+  "xxd",
+  "hexdump",
+  "strings",
+  "sort",
+  "uniq",
+  "tac",
+  "nl",
+  "dd"
 ]);
-var FS_OP_PRESCREEN_RE = /(?:^|[\s|;&(`\n])(?:rm|cat|less|head|tail|bat|more|open|print|nano|vim|vi|emacs|code|type)\b/;
+var FS_OP_PRESCREEN_RE = new RegExp(
+  `(?:^|[\\s|;&(\`\\n])(?:rm|${[...FS_READ_TOOLS].map((t) => t.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|")})\\b`
+);
 var HOME_CACHE_ALLOWLIST = [
   ".cache",
   ".npm/_npx",
@@ -1382,9 +1407,37 @@ var SENSITIVE_PATH_RULES = [
     // for the canonical test-asserted contract.
     rule: "shield:project-jail:block-read-env",
     reason: "Reading .env files is blocked by project-jail shield",
-    match: (p) => /(?:^|[\\/])\.env(?:\.(?:local|production|staging|development|production\.local|staging\.local|development\.local))?$/i.test(
-      p
-    )
+    // Structural, not a list. The previous form enumerated seven suffixes and
+    // anchored on `$`, so `.env.prod`, `.env.ci` and `.env.local.bak` — all
+    // gitignored, all routinely holding real secrets — were never covered. A
+    // hand-written list of what to protect is only ever as complete as the day
+    // it was typed; this says "`.env` plus any suffix chain" and then names the
+    // exceptions, which is the direction that fails safe.
+    //
+    //   \.env          the segment itself
+    //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+    //                  files. Without it a flat suffix class swallows both.
+    //   (?![\w-])      a boundary, so `.environment` and `.envrc` are NOT .env
+    //   [\w.-]*$       any suffix chain. Flat class, no nested quantifier —
+    //                  `(\.[\w-]+)*` reads the same but is rejected by
+    //                  safe-regex2, and this pattern runs on the hook hot path.
+    //
+    // The two exclusions are NOT the same shape, because the words do not mean
+    // the same thing:
+    //
+    //   (?!\.(?:example|sample|template)\b)  — "this file is a fixture", and it
+    //     stays a fixture whatever follows, so `.env.example.md` is allowed too.
+    //     These are checked into git by convention: already public, so blocking
+    //     them buys nothing and costs the most common legitimate agent read.
+    //
+    //   (?!\.test$)  — anchored, because `test` names an ENVIRONMENT, not a
+    //     fixture. `.env.test` is the committed template and stays allowed, but
+    //     `.env.test.local` is gitignored by the `.env*.local` convention and
+    //     holds real values, so it must block. Using `\b` here — the obvious
+    //     symmetry — silently exempts every `.env.test.*` file.
+    //
+    // shields.test.ts:983-995 is the canonical contract; keep both in step.
+    match: (p) => /(?:^|[\\/])\.env(?![\w-])(?!\.(?:example|sample|template)\b)(?!\.test$)[\w.-]*$/i.test(p)
   },
   {
     // verdict: 'review' (not 'block') is a deliberate design choice

package/dist/scan-ink.mjs

@@ -818,6 +818,48 @@ function assertBuiltinPatternsAreSafe() {
 assertBuiltinPatternsAreSafe();
 var { syntax } = mvdanSh;
 var sharedParser = syntax.NewParser();
+var FS_READ_TOOLS = /* @__PURE__ */ new Set([
+  "cat",
+  "less",
+  "head",
+  "tail",
+  "bat",
+  "more",
+  "open",
+  "print",
+  "nano",
+  "vim",
+  "vi",
+  "emacs",
+  "code",
+  "type",
+  // — the 22 that were missing —
+  "grep",
+  "egrep",
+  "fgrep",
+  "rg",
+  "ag",
+  "ack",
+  "awk",
+  "gawk",
+  "sed",
+  "cut",
+  "tr",
+  "jq",
+  "yq",
+  "od",
+  "xxd",
+  "hexdump",
+  "strings",
+  "sort",
+  "uniq",
+  "tac",
+  "nl",
+  "dd"
+]);
+var FS_OP_PRESCREEN_RE = new RegExp(
+  `(?:^|[\\s|;&(\`\\n])(?:rm|${[...FS_READ_TOOLS].map((t) => t.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|")})\\b`
+);
 function deriveRedirOp(sample) {
   try {
     const f = sharedParser.Parse(sample, "cmd");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@node9/proxy",
-  "version": "1.67.2",
+  "version": "1.67.3",
   "description": "The Sudo Command for AI Agents. Execution Security for Claude Code, Codex, Gemini, Cursor, Opencode, Pi, and any MCP server.",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",