@kntic/kntic 0.4.2 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kntic/kntic",
-  "version": "0.4.2",
+  "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/cli.js

@@ -30,7 +30,8 @@ if (!subcommand || subcommand === "usage") {
     process.exit(1);
   }
 } else if (subcommand === "update") {
-  commands.update().catch((err) => {
+  const updateOpts = { libOnly: args.includes("--lib-only") };
+  commands.update(updateOpts).catch((err) => {
     console.error(`Error: ${err.message}`);
     process.exit(1);
   });

package/src/commands/update.js

@@ -182,7 +182,53 @@ function extractLibOnly(tarball, destDir) {
   );
 }
 
-async function update() {
+/**
+ * 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 ".")
+ */
+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 lib and adrs (full replacement)
+  clearDirectory(libDir);
+  clearDirectory(adrsDir);
+
+  // 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 = {}) {
+  const libOnly = options.libOnly || false;
+
   // Resolve current version from the artifact metadata file
   const artifactFilename = await fetchText(BOOTSTRAP_ARTIFACT_URL);
   const version = extractVersion(artifactFilename);
@@ -192,8 +238,13 @@ async function update() {
   console.log(`Downloading KNTIC bootstrap archive… (v${version})`);
   await download(BOOTSTRAP_URL, tmpFile);
 
-  console.log("Updating .kntic/lib …");
-  extractLibOnly(tmpFile, ".");
+  if (libOnly) {
+    console.log("Updating .kntic/lib …");
+    extractLibOnly(tmpFile, ".");
+  } else {
+    console.log("Updating .kntic/lib, .kntic/adrs, and .kntic/hooks/gdr …");
+    extractUpdate(tmpFile, ".");
+  }
 
   // Update KNTIC_VERSION in .kntic.env
   updateEnvVersion(version);
@@ -202,11 +253,16 @@ async function update() {
   fs.unlinkSync(tmpFile);
 
   console.log(`@kntic/kntic@${version}`);
-  console.log("Done. .kntic/lib updated successfully.");
+  if (libOnly) {
+    console.log("Done. .kntic/lib updated successfully.");
+  } else {
+    console.log("Done. .kntic/lib, .kntic/adrs, and .kntic/hooks/gdr updated successfully.");
+  }
 }
 
 module.exports = update;
 module.exports.extractLibOnly = extractLibOnly;
+module.exports.extractUpdate = extractUpdate;
 module.exports.extractVersion = extractVersion;
 module.exports.clearDirectory = clearDirectory;
 module.exports.updateEnvVersion = updateEnvVersion;

package/src/commands/update.test.js

@@ -7,7 +7,7 @@ const path = require("path");
 const os = require("os");
 const { execSync } = require("child_process");
 
-const { extractLibOnly, extractVersion, clearDirectory, updateEnvVersion } = require("./update");
+const { extractLibOnly, extractUpdate, extractVersion, clearDirectory, updateEnvVersion } = require("./update");
 
 /**
  * Helper — create a tar.gz archive in `tmpDir` containing the given files.
@@ -202,6 +202,191 @@ describe("extractLibOnly", () => {
   });
 });
 
+describe("extractUpdate", () => {
+  let tmpDir;
+  let destDir;
+
+  beforeEach(() => {
+    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "kntic-test-extractupdate-"));
+    destDir = path.join(tmpDir, "dest");
+    fs.mkdirSync(destDir, { recursive: true });
+  });
+
+  afterEach(() => {
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+
+  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",
+    });
+
+    extractUpdate(tarball, destDir);
+
+    // lib files must be extracted
+    assert.equal(
+      fs.readFileSync(path.join(destDir, ".kntic", "lib", "orchestrator.py"), "utf8"),
+      "# orchestrator\n"
+    );
+    assert.equal(
+      fs.readFileSync(path.join(destDir, ".kntic", "lib", "skills", "validator.py"), "utf8"),
+      "# validator\n"
+    );
+
+    // adrs files must be extracted
+    assert.equal(
+      fs.readFileSync(path.join(destDir, ".kntic", "adrs", "ADR-001.md"), "utf8"),
+      "# ADR 001\n"
+    );
+    assert.equal(
+      fs.readFileSync(path.join(destDir, ".kntic", "adrs", "ADR-002.md"), "utf8"),
+      "# ADR 002\n"
+    );
+
+    // 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"
+    );
+    assert.ok(
+      !fs.existsSync(path.join(destDir, "README.md")),
+      "README.md must not be extracted"
+    );
+  });
+
+  it("replaces existing .kntic/lib and .kntic/adrs content", () => {
+    const libDir = path.join(destDir, ".kntic", "lib");
+    const adrsDir = path.join(destDir, ".kntic", "adrs");
+    fs.mkdirSync(libDir, { recursive: true });
+    fs.mkdirSync(adrsDir, { recursive: true });
+    fs.writeFileSync(path.join(libDir, "old_lib.py"), "# old lib\n");
+    fs.writeFileSync(path.join(adrsDir, "ADR-OLD.md"), "# old adr\n");
+
+    const tarball = createTarball(tmpDir, {
+      ".kntic/lib/new_lib.py": "# new lib\n",
+      ".kntic/adrs/ADR-NEW.md": "# new adr\n",
+    });
+
+    extractUpdate(tarball, destDir);
+
+    // Old files must be gone
+    assert.ok(!fs.existsSync(path.join(libDir, "old_lib.py")), "old lib file must be removed");
+    assert.ok(!fs.existsSync(path.join(adrsDir, "ADR-OLD.md")), "old adr file must be removed");
+
+    // New files must be present
+    assert.equal(
+      fs.readFileSync(path.join(libDir, "new_lib.py"), "utf8"),
+      "# new lib\n"
+    );
+    assert.equal(
+      fs.readFileSync(path.join(adrsDir, "ADR-NEW.md"), "utf8"),
+      "# new adr\n"
+    );
+  });
+
+  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");
+    fs.writeFileSync(path.join(destDir, "README.md"), "# My readme\n");
+
+    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",
+    });
+
+    extractUpdate(tarball, destDir);
+
+    assert.equal(
+      fs.readFileSync(path.join(knticDir, "MEMORY.MD"), "utf8"),
+      "# My memory\n",
+      "MEMORY.MD must not be overwritten"
+    );
+    assert.equal(
+      fs.readFileSync(path.join(destDir, "README.md"), "utf8"),
+      "# My readme\n",
+      "README.md must not be overwritten"
+    );
+  });
+});
+
 describe("updateEnvVersion", () => {
   let tmpDir;
 

package/src/commands/usage.js

@@ -9,7 +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 only");
+  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("");
 }