test: improve code coverage from 63% to 74% (+179 tests)

src/commands/upgrade.ts

@@ -391,4 +391,6 @@ export {
   isProtoInstall as _isProtoInstall,
   isLocalInstall as _isLocalInstall,
   detectInstallMethod as _detectInstallMethod,
+  formatBytes as _formatBytes,
+  createDownloadProgress as _createDownloadProgress,
 };

src/engine/rule-scanner.ts

@@ -201,8 +201,6 @@ const IMPORTED_BLOCKED_GLOBALS = new Set(["require", "WebSocket"]);
  *   - environment variable reads via process
  *   - `require()` calls
  *   - `WebSocket` usage
- *
- * @internal
  */
 export function scanImportedRuleSource(source: string): ScanViolation[] {
   const transpiler = new Bun.Transpiler({ loader: "ts" });

tests/commands/adr/create.test.ts

@@ -183,4 +183,134 @@ describe("adr create action handler", () => {
 
     expect(exitSpy).toHaveBeenCalledWith(1);
   });
+
+  test("parses comma-separated --files patterns into frontmatter", async () => {
+    const adrsDir = join(tempDir, ".archgate", "adrs");
+    mkdirSync(adrsDir, { recursive: true });
+
+    process.chdir(tempDir);
+    const parent = makeProgram();
+    await parent.parseAsync([
+      "node",
+      "adr",
+      "create",
+      "--title",
+      "Scoped Rule",
+      "--domain",
+      "architecture",
+      "--files",
+      "src/**/*.ts, tests/**/*.ts",
+      "--body",
+      "## Context\nScoped to specific files.",
+    ]);
+
+    const createdFile = join(adrsDir, "ARCH-001-scoped-rule.md");
+    expect(existsSync(createdFile)).toBe(true);
+    const content = await Bun.file(createdFile).text();
+    expect(content).toContain("src/**/*.ts");
+    expect(content).toContain("tests/**/*.ts");
+  });
+
+  test("sets rules: true in frontmatter when --rules flag is passed", async () => {
+    const adrsDir = join(tempDir, ".archgate", "adrs");
+    mkdirSync(adrsDir, { recursive: true });
+
+    process.chdir(tempDir);
+    const parent = makeProgram();
+    await parent.parseAsync([
+      "node",
+      "adr",
+      "create",
+      "--title",
+      "Enforced Rule",
+      "--domain",
+      "general",
+      "--rules",
+      "--body",
+      "## Context\nThis ADR has rules.",
+    ]);
+
+    const createdFile = join(adrsDir, "GEN-001-enforced-rule.md");
+    expect(existsSync(createdFile)).toBe(true);
+    const content = await Bun.file(createdFile).text();
+    expect(content).toContain("rules: true");
+  });
+
+  test("generates companion .rules.ts file when --rules flag is passed", async () => {
+    const adrsDir = join(tempDir, ".archgate", "adrs");
+    mkdirSync(adrsDir, { recursive: true });
+
+    process.chdir(tempDir);
+    const parent = makeProgram();
+    await parent.parseAsync([
+      "node",
+      "adr",
+      "create",
+      "--title",
+      "With Rules",
+      "--domain",
+      "backend",
+      "--rules",
+      "--body",
+      "## Context\nHas companion rules.",
+    ]);
+
+    const rulesFile = join(adrsDir, "BE-001-with-rules.rules.ts");
+    expect(existsSync(rulesFile)).toBe(true);
+  });
+
+  test("does not generate .rules.ts file when --rules is omitted", async () => {
+    const adrsDir = join(tempDir, ".archgate", "adrs");
+    mkdirSync(adrsDir, { recursive: true });
+
+    process.chdir(tempDir);
+    const parent = makeProgram();
+    await parent.parseAsync([
+      "node",
+      "adr",
+      "create",
+      "--title",
+      "No Rules",
+      "--domain",
+      "backend",
+      "--body",
+      "## Context\nNo rules needed.",
+    ]);
+
+    const rulesFile = join(adrsDir, "BE-001-no-rules.rules.ts");
+    expect(existsSync(rulesFile)).toBe(false);
+  });
+
+  test("increments ADR ID when existing ADRs are present", async () => {
+    const adrsDir = join(tempDir, ".archgate", "adrs");
+    mkdirSync(adrsDir, { recursive: true });
+
+    // Create a first ADR
+    process.chdir(tempDir);
+    const parent1 = makeProgram();
+    await parent1.parseAsync([
+      "node",
+      "adr",
+      "create",
+      "--title",
+      "First ADR",
+      "--domain",
+      "backend",
+    ]);
+
+    // Create a second ADR in the same domain
+    const parent2 = makeProgram();
+    await parent2.parseAsync([
+      "node",
+      "adr",
+      "create",
+      "--title",
+      "Second ADR",
+      "--domain",
+      "backend",
+    ]);
+
+    expect(existsSync(join(adrsDir, "BE-001-first-adr.md"))).toBe(true);
+    expect(existsSync(join(adrsDir, "BE-002-second-adr.md"))).toBe(true);
+  });
 });