Address Copilot review comments on PR #120

- types.ts: make isTemplate optional (isTemplate?: boolean) to keep
  existing builders/fixtures valid without changes
- output.ts: shell-quote groupByTeamPrefix in buildReplayCommand
  to prevent replay command breakage when the value contains spaces
- output.test.ts: add tests for excludeTemplates flag in replay command;
  update groupByTeamPrefix assertion to reflect new shell-quoting
- docs/usage/filtering.md: fix intro count (three → four filtering
  options); reword template info box to clarify filtering applies in
  both interactive and non-interactive mode

Author: shouze

docs/usage/filtering.md

@@ -1,6 +1,6 @@
 # Filtering
 
-`github-code-search` provides three pre-query filtering options so you can exclude noise before results ever appear in the TUI or output.
+`github-code-search` provides four pre-query filtering options so you can exclude noise before results ever appear in the TUI or output.
 
 ## `--exclude-repositories`
 
@@ -73,7 +73,7 @@ github-code-search "useFeatureFlag" --org fulll --exclude-template-repositories
 ```
 
 ::: info
-Template repositories (marked as templates on GitHub) are filtered out in the aggregation step before the TUI is shown. Useful when your organisation uses template repos for boilerplate that should not appear in search results.
+Template repositories (marked as templates on GitHub) are filtered out in the aggregation step — both in interactive and non-interactive mode. Useful when your organisation uses template repos for boilerplate that should not appear in search results.
 :::
 
 ## Combining filters

src/output.test.ts

@@ -169,11 +169,25 @@ describe("buildReplayCommand", () => {
     expect(cmd).not.toContain("--include-archived");
   });
 
+  it("includes --exclude-template-repositories when excludeTemplates is true", () => {
+    const groups = [makeGroup("myorg/repoA", ["a.ts"])];
+    const opts: ReplayOptions = { excludeTemplates: true };
+    const cmd = buildReplayCommand(groups, QUERY, ORG, new Set(), new Set(), opts);
+    expect(cmd).toContain("--exclude-template-repositories");
+  });
+
+  it("does not include --exclude-template-repositories when excludeTemplates is false (default)", () => {
+    const groups = [makeGroup("myorg/repoA", ["a.ts"])];
+    const opts: ReplayOptions = { excludeTemplates: false };
+    const cmd = buildReplayCommand(groups, QUERY, ORG, new Set(), new Set(), opts);
+    expect(cmd).not.toContain("--exclude-template-repositories");
+  });
+
   it("includes --group-by-team-prefix when groupByTeamPrefix is set", () => {
     const groups = [makeGroup("myorg/repoA", ["a.ts"])];
     const opts: ReplayOptions = { groupByTeamPrefix: "squad-,chapter-" };
     const cmd = buildReplayCommand(groups, QUERY, ORG, new Set(), new Set(), opts);
-    expect(cmd).toContain("--group-by-team-prefix squad-,chapter-");
+    expect(cmd).toContain("--group-by-team-prefix 'squad-,chapter-'");
   });
 
   it("does not include --group-by-team-prefix when groupByTeamPrefix is empty (default)", () => {
@@ -567,6 +581,6 @@ describe("buildOutput", () => {
       groupByTeamPrefix: "squad-",
     });
     const parsed = JSON.parse(out);
-    expect(parsed.replayCommand).toContain("--group-by-team-prefix squad-");
+    expect(parsed.replayCommand).toContain("--group-by-team-prefix 'squad-'");
   });
 });

src/output.ts

@@ -96,7 +96,7 @@ export function buildReplayCommand(
     parts.push("--exclude-template-repositories");
   }
   if (groupByTeamPrefix) {
-    parts.push(`--group-by-team-prefix ${groupByTeamPrefix}`);
+    parts.push(`--group-by-team-prefix ${shellQuote(groupByTeamPrefix)}`);
   }
   if (regexHint) {
     parts.push(`--regex-hint ${shellQuote(regexHint)}`);

src/types.ts

@@ -21,7 +21,7 @@ export interface CodeMatch {
   htmlUrl: string;
   textMatches: TextMatch[];
   archived: boolean;
-  isTemplate: boolean;
+  isTemplate?: boolean;
 }
 
 export interface RepoGroup {