@alessandroraffa/tangyr 1.0.4 → 3.0.0

package/README.md

@@ -342,9 +342,9 @@ In `tangyr-kit.yaml`, `discipline` is optional: declare it to opt into the corpu
 
 ## Conflict policy
 
-Before writing, each destination is classified and resolved according to `onConflict` in `tangyr.config.yaml`: `ask` (default; prompts, and resolves to backup-and-replace in a non-interactive run), `skip`, `backup`, or `overwrite`.
+Before writing, each destination is classified and resolved according to `onConflict` in `tangyr.config.yaml`: `ask` (default; prompts, and resolves to **skip** when there is no one to ask), `skip`, `backup`, or `overwrite`.
 
-`--yes` supplies the confirmation a non-interactive run needs; it does **not** override the posture. Under `--yes`, `skip` leaves conflicting destinations unchanged and says which, `overwrite` replaces without a backup, and `backup` copies the existing file — or the existing directory, recursively — into `.tangyr/backups/<timestamp>/` and records it in the manifest so `uninstall` can put it back. A symlink is replaced without a backup: what it points at is untouched.
+`--yes` supplies the confirmation a non-interactive run needs; it does **not** override the posture. Under `--yes`, `skip` leaves conflicting destinations unchanged and says which, `overwrite` replaces without a backup, `ask` leaves the destination alone and names the postures that replace, and `backup` copies the existing file — or the existing directory, recursively — into `.tangyr/backups/<timestamp>/` and records it in the manifest so `uninstall` can put it back. A symlink is backed up as a symlink and restored as one: its target is untouched either way, but the link itself is the operator's artifact — often the one committed with the project — so replacing it without a record left nothing to put back.
 
 One decision is taken per destination, not per target, so a file several targets write — `AGENTS.md` is claimed by four — is backed up once.
 

package/dist/index.js

@@ -21980,6 +21980,25 @@ function realPathOrInput(filePath) {
     return filePath;
   }
 }
+function clearSymlinkAtDestination(destination) {
+  let stat;
+  try {
+    stat = fs23.lstatSync(destination);
+  } catch {
+    return null;
+  }
+  if (!stat.isSymbolicLink()) {
+    return null;
+  }
+  let target = null;
+  try {
+    target = fs23.readlinkSync(destination);
+  } catch {
+    target = null;
+  }
+  fs23.unlinkSync(destination);
+  return target ?? "(unreadable link target)";
+}
 
 // src/core/artifact-state.ts
 import fs24 from "fs";
@@ -22304,13 +22323,11 @@ async function runUninstallCommand(options, logger) {
     }
   }
   if (fullUninstall) {
-    const newestFirst = [...manifest.backups].reverse();
     const restored = /* @__PURE__ */ new Set();
-    for (const backup of newestFirst) {
+    const kept = [];
+    for (const backup of manifest.backups) {
       if (restored.has(backup.originalPath)) {
-        logger.verbose(
-          `  superseded by a newer backup, not restored: ${backup.backupLocation}`
-        );
+        kept.push(backup);
         continue;
       }
       restored.add(backup.originalPath);
@@ -22322,6 +22339,19 @@ async function runUninstallCommand(options, logger) {
         );
       }
     }
+    if (kept.length > 0) {
+      logger.warn(
+        `
+${String(kept.length)} later backup(s) were NOT restored: the earliest copy of each location is the one that comes back, because it is the content from before the first install.`
+      );
+      for (const backup of kept) {
+        logger.warn(`  ${backup.originalPath}`);
+        logger.warn(`    later copy left at: ${backup.backupLocation}`);
+      }
+      logger.warn(
+        "  The manifest is removed below, so nothing will reference these copies afterwards. Move what you need out of the backups directory now."
+      );
+    }
     const manifestPath = path24.join(scopePath, "manifest.json");
     if (fs27.existsSync(manifestPath)) {
       fs27.unlinkSync(manifestPath);
@@ -26598,6 +26628,11 @@ function syncDirectComponent(config, sourceRoot, options, logger, directComponen
     };
   } else if (artifactClass === "managed-symlink") {
     fs36.unlinkSync(destination);
+  } else if (artifactClass === "managed-compiled") {
+    logger.warn(
+      `${destination}: replacing another target's compiled instructions file with a link to the kit. A target that appends its rules here (opencode has no rules slot of its own) does not get them. Give it its own instructions path, or disable it.`
+    );
+    fs36.unlinkSync(destination);
   }
   ensureDir(path34.dirname(destination));
   createRelativeSymlink(source, destination, directComponent.type, {
@@ -28591,6 +28626,12 @@ function writeCompiledArtifact(component, destination, content, expectedHash, co
     return { component, status: "conflict", path: destination };
   }
   ensureDir(path37.dirname(destination));
+  const clearedLink = clearSymlinkAtDestination(destination);
+  if (clearedLink !== null) {
+    logger.verbose(
+      `  ${destination}: replaced a symlink to ${clearedLink} with generated content`
+    );
+  }
   fs38.writeFileSync(destination, content);
   return { component, status: "success", path: destination };
 }
@@ -29000,6 +29041,12 @@ function writeInstructionsFile(component, destination, content, contentHash, con
     return { component, status: "conflict", path: destination };
   }
   ensureDir(path39.dirname(destination));
+  const clearedLink = clearSymlinkAtDestination(destination);
+  if (clearedLink !== null) {
+    logger.verbose(
+      `  ${destination}: replaced a symlink to ${clearedLink} with generated content`
+    );
+  }
   fs39.writeFileSync(destination, content);
   return { component, status: "success", path: destination };
 }
@@ -29192,6 +29239,12 @@ function writeCompiledArtifact2(component, destination, content, expectedHash, c
     return { component, status: "conflict", path: destination };
   }
   ensureDir(path39.dirname(destination));
+  const clearedLink = clearSymlinkAtDestination(destination);
+  if (clearedLink !== null) {
+    logger.verbose(
+      `  ${destination}: replaced a symlink to ${clearedLink} with generated content`
+    );
+  }
   fs39.writeFileSync(destination, content);
   return { component, status: "success", path: destination };
 }
@@ -29827,6 +29880,12 @@ function writeCompiledArtifact3(component, destination, content, expectedHash, c
     return { component, status: "conflict", path: destination };
   }
   ensureDir(path40.dirname(destination));
+  const clearedLink = clearSymlinkAtDestination(destination);
+  if (clearedLink !== null) {
+    logger.verbose(
+      `  ${destination}: replaced a symlink to ${clearedLink} with generated content`
+    );
+  }
   fs40.writeFileSync(destination, content);
   return { component, status: "success", path: destination };
 }
@@ -30302,6 +30361,12 @@ function writeCompiledArtifact4(component, destination, content, expectedHash, c
     return { component, status: "conflict", path: destination };
   }
   ensureDir(path41.dirname(destination));
+  const clearedLink = clearSymlinkAtDestination(destination);
+  if (clearedLink !== null) {
+    logger.verbose(
+      `  ${destination}: replaced a symlink to ${clearedLink} with generated content`
+    );
+  }
   fs41.writeFileSync(destination, content);
   return { component, status: "success", path: destination };
 }
@@ -30795,6 +30860,12 @@ function writeCompiledArtifact5(component, destination, content, expectedHash, c
     return { component, status: "conflict", path: destination };
   }
   ensureDir(path42.dirname(destination));
+  const clearedLink = clearSymlinkAtDestination(destination);
+  if (clearedLink !== null) {
+    logger.verbose(
+      `  ${destination}: replaced a symlink to ${clearedLink} with generated content`
+    );
+  }
   fs42.writeFileSync(destination, content);
   return { component, status: "success", path: destination };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alessandroraffa/tangyr",
-  "version": "1.0.4",
+  "version": "3.0.0",
   "description": "CLI for the Tangyr discipline — install and manage operating kits for AI coding tools",
   "license": "MIT",
   "engines": {