@gotgenes/pi-permission-system 5.11.0 → 5.14.0

package/tests/wildcard-matcher.test.ts

@@ -243,6 +243,97 @@ describe("wildcardMatch", () => {
     expect(wildcardMatch("tool.name", "tool.name")).toBe(true);
     expect(wildcardMatch("tool.name", "toolXname")).toBe(false);
   });
+
+  describe("trailing wildcard optionality", () => {
+    test("'git *' matches bare 'git' (trailing space+wildcard is optional)", () => {
+      expect(wildcardMatch("git *", "git")).toBe(true);
+    });
+
+    test("'git *' still matches 'git status' (existing behaviour preserved)", () => {
+      expect(wildcardMatch("git *", "git status")).toBe(true);
+    });
+
+    test("'git *' still matches 'git status --short'", () => {
+      expect(wildcardMatch("git *", "git status --short")).toBe(true);
+    });
+
+    test("'git *' does not match an unrelated command", () => {
+      expect(wildcardMatch("git *", "npm install")).toBe(false);
+    });
+
+    test("'git status *' matches bare 'git status'", () => {
+      expect(wildcardMatch("git status *", "git status")).toBe(true);
+    });
+
+    test("'git status *' matches 'git status --short'", () => {
+      expect(wildcardMatch("git status *", "git status --short")).toBe(true);
+    });
+
+    test("non-trailing '*' is unaffected: 'g*t' does not match 'g' or 't'", () => {
+      expect(wildcardMatch("g*t", "g")).toBe(false);
+      expect(wildcardMatch("g*t", "t")).toBe(false);
+    });
+
+    test("non-trailing '*' still matches when content is present: 'g*t' matches 'git'", () => {
+      expect(wildcardMatch("g*t", "git")).toBe(true);
+    });
+
+    test("'git*' (no space) still matches bare 'git' — unchanged behaviour", () => {
+      expect(wildcardMatch("git*", "git")).toBe(true);
+    });
+
+    test("'*' alone still matches everything", () => {
+      expect(wildcardMatch("*", "git")).toBe(true);
+      expect(wildcardMatch("*", "")).toBe(true);
+    });
+  });
+});
+
+describe("? single-character wildcard", () => {
+  test("'?' matches exactly one character", () => {
+    expect(wildcardMatch("?", "a")).toBe(true);
+    expect(wildcardMatch("?", "Z")).toBe(true);
+    expect(wildcardMatch("?", "5")).toBe(true);
+  });
+
+  test("'?' does not match zero characters", () => {
+    expect(wildcardMatch("?", "")).toBe(false);
+    expect(wildcardMatch("a?", "a")).toBe(false);
+  });
+
+  test("'?' does not match two or more characters", () => {
+    expect(wildcardMatch("?", "ab")).toBe(false);
+    expect(wildcardMatch("?", "abc")).toBe(false);
+  });
+
+  test("multiple '?' match exactly that many characters", () => {
+    expect(wildcardMatch("f??", "foo")).toBe(true);
+    expect(wildcardMatch("f??", "fo")).toBe(false);
+    expect(wildcardMatch("f??", "fooo")).toBe(false);
+  });
+
+  test("'?' combined with '*'", () => {
+    // git + one char + anything
+    expect(wildcardMatch("git?*", "git status")).toBe(true);
+    // git + zero chars — '?' requires one
+    expect(wildcardMatch("git?*", "git")).toBe(false);
+  });
+
+  test("'?' matches path separators and special characters", () => {
+    expect(wildcardMatch("a?b", "a/b")).toBe(true);
+    expect(wildcardMatch("a?b", "a.b")).toBe(true);
+    expect(wildcardMatch("a?b", "a\nb")).toBe(true);
+  });
+
+  test("'?' in pattern matches literal '?' in value", () => {
+    expect(wildcardMatch("a?c", "a?c")).toBe(true);
+  });
+
+  test("'?' in bash-style patterns", () => {
+    expect(wildcardMatch("git statu?", "git status")).toBe(true);
+    expect(wildcardMatch("git statu?", "git statux")).toBe(true);
+    expect(wildcardMatch("git statu?", "git statu")).toBe(false);
+  });
 });
 
 describe("home path expansion in patterns", () => {