@kntic/kntic 0.4.3 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kntic/kntic",
-  "version": "0.4.3",
+  "version": "0.4.4",
   "author": "Thomas Robak <contact@kntic.ai> (https://kntic.ai)",
   "description": "KNTIC CLI — bootstrap and manage KNTIC projects",
   "main": "src/index.js",

package/src/commands/update.js

@@ -183,8 +183,12 @@ function extractLibOnly(tarball, destDir) {
 }
 
 /**
- * Extract .kntic/lib/** and .kntic/adrs/** from a tarball into the destination
- * directory. Clears both directories first, then extracts fresh content.
+ * Extract .kntic/lib/**, .kntic/adrs/**, and .kntic/hooks/gdr/** from a tarball
+ * into the destination directory.
+ *
+ * - .kntic/lib and .kntic/adrs are **replaced** (cleared first, then extracted).
+ * - .kntic/hooks/gdr is **updated** (existing files are overwritten or new files
+ *   added, but files not present in the archive are preserved).
  *
  * @param {string} tarball  – path to the .tar.gz file
  * @param {string} destDir  – target directory (usually ".")
@@ -192,20 +196,34 @@ function extractLibOnly(tarball, destDir) {
 function extractUpdate(tarball, destDir) {
   const libDir = path.join(destDir, ".kntic", "lib");
   const adrsDir = path.join(destDir, ".kntic", "adrs");
+  const gdrDir = path.join(destDir, ".kntic", "hooks", "gdr");
 
   // Ensure target directories exist
   fs.mkdirSync(libDir, { recursive: true });
   fs.mkdirSync(adrsDir, { recursive: true });
+  fs.mkdirSync(gdrDir, { recursive: true });
 
-  // Clear existing contents
+  // Clear lib and adrs (full replacement)
   clearDirectory(libDir);
   clearDirectory(adrsDir);
 
-  // Extract both .kntic/lib/ and .kntic/adrs/ from the archive.
+  // Note: gdr is NOT cleared — update semantics (overwrite/add, preserve others)
+
+  // Extract .kntic/lib/ and .kntic/adrs/ (guaranteed to be in the archive)
   execSync(
     `tar xzf "${tarball}" -C "${destDir}" "./.kntic/lib/" "./.kntic/adrs/"`,
     { stdio: "pipe" }
   );
+
+  // Extract .kntic/hooks/gdr/ separately — archive may not contain it
+  try {
+    execSync(
+      `tar xzf "${tarball}" -C "${destDir}" "./.kntic/hooks/gdr/"`,
+      { stdio: "pipe" }
+    );
+  } catch (_) {
+    // No .kntic/hooks/gdr/ in the archive — that's fine
+  }
 }
 
 async function update(options = {}) {
@@ -224,7 +242,7 @@ async function update(options = {}) {
     console.log("Updating .kntic/lib …");
     extractLibOnly(tmpFile, ".");
   } else {
-    console.log("Updating .kntic/lib and .kntic/adrs …");
+    console.log("Updating .kntic/lib, .kntic/adrs, and .kntic/hooks/gdr …");
     extractUpdate(tmpFile, ".");
   }
 
@@ -238,7 +256,7 @@ async function update(options = {}) {
   if (libOnly) {
     console.log("Done. .kntic/lib updated successfully.");
   } else {
-    console.log("Done. .kntic/lib and .kntic/adrs updated successfully.");
+    console.log("Done. .kntic/lib, .kntic/adrs, and .kntic/hooks/gdr updated successfully.");
   }
 }
 

package/src/commands/update.test.js

@@ -216,12 +216,13 @@ describe("extractUpdate", () => {
     fs.rmSync(tmpDir, { recursive: true, force: true });
   });
 
-  it("extracts both .kntic/lib and .kntic/adrs from the archive", () => {
+  it("extracts .kntic/lib, .kntic/adrs, and .kntic/hooks/gdr from the archive", () => {
     const tarball = createTarball(tmpDir, {
       ".kntic/lib/orchestrator.py": "# orchestrator\n",
       ".kntic/lib/skills/validator.py": "# validator\n",
       ".kntic/adrs/ADR-001.md": "# ADR 001\n",
       ".kntic/adrs/ADR-002.md": "# ADR 002\n",
+      ".kntic/hooks/gdr/check.sh": "#!/bin/sh\necho check\n",
       ".kntic/MEMORY.MD": "# Memory\n",
       "README.md": "# Hello\n",
     });
@@ -248,7 +249,13 @@ describe("extractUpdate", () => {
       "# ADR 002\n"
     );
 
-    // Files outside lib/adrs must NOT be extracted
+    // hooks/gdr files must be extracted
+    assert.equal(
+      fs.readFileSync(path.join(destDir, ".kntic", "hooks", "gdr", "check.sh"), "utf8"),
+      "#!/bin/sh\necho check\n"
+    );
+
+    // Files outside lib/adrs/hooks/gdr must NOT be extracted
     assert.ok(
       !fs.existsSync(path.join(destDir, ".kntic", "MEMORY.MD")),
       "MEMORY.MD must not be extracted"
@@ -289,7 +296,69 @@ describe("extractUpdate", () => {
     );
   });
 
-  it("preserves files outside .kntic/lib and .kntic/adrs", () => {
+  it("updates .kntic/hooks/gdr without clearing existing files (update semantics)", () => {
+    const gdrDir = path.join(destDir, ".kntic", "hooks", "gdr");
+    fs.mkdirSync(gdrDir, { recursive: true });
+    fs.writeFileSync(path.join(gdrDir, "existing-hook.sh"), "#!/bin/sh\necho existing\n");
+    fs.writeFileSync(path.join(gdrDir, "shared-hook.sh"), "#!/bin/sh\necho old\n");
+
+    const tarball = createTarball(tmpDir, {
+      ".kntic/lib/orchestrator.py": "# orchestrator\n",
+      ".kntic/adrs/ADR-001.md": "# ADR 001\n",
+      ".kntic/hooks/gdr/shared-hook.sh": "#!/bin/sh\necho new\n",
+      ".kntic/hooks/gdr/new-hook.sh": "#!/bin/sh\necho added\n",
+    });
+
+    extractUpdate(tarball, destDir);
+
+    // Existing file NOT in archive must be preserved (update semantics, not replace)
+    assert.equal(
+      fs.readFileSync(path.join(gdrDir, "existing-hook.sh"), "utf8"),
+      "#!/bin/sh\necho existing\n",
+      "existing hook not in archive must be preserved"
+    );
+
+    // File in both archive and disk must be overwritten
+    assert.equal(
+      fs.readFileSync(path.join(gdrDir, "shared-hook.sh"), "utf8"),
+      "#!/bin/sh\necho new\n",
+      "shared hook must be overwritten with archive version"
+    );
+
+    // New file from archive must be added
+    assert.equal(
+      fs.readFileSync(path.join(gdrDir, "new-hook.sh"), "utf8"),
+      "#!/bin/sh\necho added\n",
+      "new hook from archive must be added"
+    );
+  });
+
+  it("works when archive has no .kntic/hooks/gdr directory", () => {
+    const gdrDir = path.join(destDir, ".kntic", "hooks", "gdr");
+    fs.mkdirSync(gdrDir, { recursive: true });
+    fs.writeFileSync(path.join(gdrDir, "my-hook.sh"), "#!/bin/sh\necho mine\n");
+
+    const tarball = createTarball(tmpDir, {
+      ".kntic/lib/orchestrator.py": "# orchestrator\n",
+      ".kntic/adrs/ADR-001.md": "# ADR 001\n",
+    });
+
+    // Should not throw
+    extractUpdate(tarball, destDir);
+
+    // Existing gdr hook must be preserved
+    assert.equal(
+      fs.readFileSync(path.join(gdrDir, "my-hook.sh"), "utf8"),
+      "#!/bin/sh\necho mine\n",
+      "existing gdr hook must be preserved when archive has no gdr"
+    );
+
+    // lib and adrs must still be extracted
+    assert.ok(fs.existsSync(path.join(destDir, ".kntic", "lib", "orchestrator.py")));
+    assert.ok(fs.existsSync(path.join(destDir, ".kntic", "adrs", "ADR-001.md")));
+  });
+
+  it("preserves files outside .kntic/lib, .kntic/adrs, and .kntic/hooks/gdr", () => {
     const knticDir = path.join(destDir, ".kntic");
     fs.mkdirSync(knticDir, { recursive: true });
     fs.writeFileSync(path.join(knticDir, "MEMORY.MD"), "# My memory\n");
@@ -298,6 +367,7 @@ describe("extractUpdate", () => {
     const tarball = createTarball(tmpDir, {
       ".kntic/lib/orchestrator.py": "# orchestrator\n",
       ".kntic/adrs/ADR-001.md": "# ADR 001\n",
+      ".kntic/hooks/gdr/check.sh": "#!/bin/sh\n",
       ".kntic/MEMORY.MD": "# Archive memory\n",
       "README.md": "# Archive readme\n",
     });

package/src/commands/usage.js

@@ -9,8 +9,8 @@ function usage() {
   console.log("  start    Build and start KNTIC services via docker compose (uses kntic.yml + .kntic.env)");
   console.log("             --screen   Run inside a GNU screen session");
   console.log("  stop     Stop KNTIC services via docker compose");
-  console.log("  update   Download the latest KNTIC bootstrap and update .kntic/lib and .kntic/adrs");
-  console.log("             --lib-only   Update only .kntic/lib (skip .kntic/adrs)");
+  console.log("  update   Download the latest KNTIC bootstrap and update .kntic/lib, .kntic/adrs, and .kntic/hooks/gdr");
+  console.log("             --lib-only   Update only .kntic/lib (skip .kntic/adrs and .kntic/hooks/gdr)");
   console.log("");
 }