create-principles-disciple 1.147.4 → 1.147.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-principles-disciple",
-  "version": "1.147.4",
+  "version": "1.147.5",
   "description": "Interactive CLI installer for Principles Disciple OpenClaw plugin",
   "type": "module",
   "bin": {
@@ -101,5 +101,5 @@
   "pd": {
     "bundledPluginVersion": "2.5.0"
   },
-  "gitHead": "49a91651ab6c755f9074a653998951f4f88c0f43"
+  "gitHead": "29d6fbfd56adcc2d71280438b0bcaf8ee8ef3bd8"
 }

package/release-manager/dist/i18n.js

@@ -105,10 +105,14 @@ const translations = {
         // Install failure / rollback result strings (catch block). EP-11.
         rollback_no_changes: 'No changes were made — the existing install was not modified (no backup was created).',
         rollback_restored: 'Previous install has been restored.',
+        // PRI-1102: the rollback target itself can be incomplete — say so instead
+        // of reporting a restore that leaves the operator with a broken install.
+        rollback_restored_incomplete: 'Rolled back to the state that existed before this run, but that state is NOT usable — incomplete: {paths}.',
         rollback_failed: 'CRITICAL: Rollback also failed — installation state is uncertain. {restoreError} Resolve manually: check {extDir} and {backupDir}',
         next_no_changes_lock: 'A file lock blocked the backup (EPERM). Stop the OpenClaw gateway (openclaw gateway stop, or re-run with --stop-gateway) and retry.',
         next_no_changes_other: 'Fix the error above and re-run the installer. The existing install was not modified.',
         next_restored: 'Check the error above. Previous install has been restored. Fix the issue and re-run the installer.',
+        next_restored_incomplete: 'Check the error above. Rollback returned the pre-run state, but it is incomplete: {paths}. Re-run the installer so the missing components are rebuilt.',
         next_restore_failed: 'Installation and rollback both failed. Check {extDir} and {backupDir} manually. Error: {errorMsg}',
         // CP-6: fresh-install failure mid-deployment (no pre-existing install → no backup).
         rollback_fresh_cleaned: 'The install failed before activation — the partially deployed runtime files were removed. Nothing else was modified.',
@@ -225,10 +229,12 @@ const translations = {
         // 安装失败 / 回滚结果字符串（catch 块）。EP-11。
         rollback_no_changes: '未做任何改动 —— 现有安装未被修改（未创建备份）。',
         rollback_restored: '已恢复到之前的安装。',
+        rollback_restored_incomplete: '已回滚到本次运行之前的状态，但该状态本身不可用 —— 不完整：{paths}。',
         rollback_failed: '严重：回滚也失败了 —— 安装状态不确定。{restoreError} 请手动处理：检查 {extDir} 和 {backupDir}',
         next_no_changes_lock: '文件锁阻塞了备份 (EPERM)。请停止 OpenClaw gateway（openclaw gateway stop，或用 --stop-gateway 重新运行）后重试。',
         next_no_changes_other: '请修复上方错误后重新运行安装器。现有安装未被修改。',
         next_restored: '请检查上方错误。已恢复到之前的安装。修复问题后重新运行安装器。',
+        next_restored_incomplete: '请检查上方错误。回滚返回的是本次运行之前的状态，但该状态不完整：{paths}。请重新运行安装器，让它重建缺失的组件。',
         next_restore_failed: '安装与回滚均失败。请手动检查 {extDir} 和 {backupDir}。错误：{errorMsg}',
         // CP-6：全新安装部署中途失败（无既有安装 → 无备份）。
         rollback_fresh_cleaned: '安装未激活即失败 —— 已移除本次部署的部分运行时文件。其余内容未被修改。',

package/release-manager/dist/installer.js

@@ -1553,10 +1553,79 @@ export async function checkBuiltPlugin(pluginDir) {
         }
     }
 }
+/**
+ * PRI-1102: a target that exists as a DANGLING link (an interrupted install
+ * left a reparse point whose target is gone) is not a directory the installer
+ * can copy into. `existsSync` follows the link and reports "missing", so the
+ * backup step never moves it, and `ensureDir` then treats it as present —
+ * the follow-up mkdir fails with ENOENT and aborts the whole install.
+ *
+ * Absent targets are created. A dangling link is removed (the link itself only,
+ * never a target), and the caller recreates the real directory. Everything
+ * else — a real directory, or a link that still resolves — is left untouched,
+ * so an existing install is never destroyed by this step.
+ */
+export async function preparePluginInstallTarget(targetDir) {
+    let stats;
+    try {
+        stats = lstatSync(targetDir, { throwIfNoEntry: false });
+    }
+    catch (error) {
+        throw new Error(`Cannot inspect plugin install target ${targetDir}: ${error instanceof Error ? error.message : String(error)}`, { cause: error });
+    }
+    if (stats !== undefined && stats.isSymbolicLink()) {
+        let resolves = true;
+        try {
+            realpathSync(targetDir);
+        }
+        catch {
+            resolves = false;
+        }
+        if (!resolves) {
+            // fs-extra's remove unlinks the reparse point instead of traversing it.
+            await fse.remove(targetDir);
+            logger.warn(`Removed dangling link at ${targetDir} (target missing) — rebuilding the plugin install directory.`);
+        }
+    }
+    await fse.ensureDir(targetDir);
+}
+/**
+ * PRI-1102: a rollback is only as good as the state it returns to. The
+ * pre-run state can itself be incomplete (a component directory the previous
+ * run never finished, a link whose target is gone), so "restored" must not be
+ * reported as a healthy install without checking what the restored paths
+ * actually resolve to.
+ */
+export function describeRestoredStateGaps() {
+    const gaps = [];
+    for (const dir of [getPluginExtDir(), getPdRuntimeDir()]) {
+        let stats;
+        try {
+            stats = lstatSync(dir, { throwIfNoEntry: false });
+        }
+        catch (error) {
+            gaps.push(`${dir} (unreadable: ${error instanceof Error ? error.message : String(error)})`);
+            continue;
+        }
+        if (stats === undefined) {
+            gaps.push(`${dir} (missing)`);
+            continue;
+        }
+        if (stats.isSymbolicLink()) {
+            try {
+                realpathSync(dir);
+            }
+            catch {
+                gaps.push(`${dir} (dangling link)`);
+            }
+        }
+    }
+    return gaps;
+}
 export async function installPluginToStaging(pluginDir, language) {
     const builtPluginDir = path.join(pluginDir, 'plugin');
     for (const targetDir of pluginInstallDirs()) {
-        await fse.ensureDir(targetDir);
+        await preparePluginInstallTarget(targetDir);
         // PRI-1037 payload fix: npm-era payloads carried a ~16k-file node_modules
         // (the whole pi-ai/openai/anthropic tree, dragged in through the plugin's
         // historical file:./core dependency) plus a `core` junction, in BOTH plugin
@@ -4355,6 +4424,10 @@ transaction) {
         // pre-date this run, so it is NOT wholesale-removed. Report its retention
         // honestly instead of claiming "nothing else was modified".
         const workspaceTouched = !hasBackup && mutationStarted && journal?.lastState !== 'planned';
+        // PRI-1102: `restored` describes the rollback operation, not the health of
+        // the state it returned to. Name the gaps so "restored" is never read as
+        // "usable install".
+        const restoredGaps = hasBackup && restoreResult.restored ? describeRestoredStateGaps() : [];
         const rollbackSuffix = !hasBackup
             ? freshCleanup
                 ? freshCleanup.failed.length === 0
@@ -4362,7 +4435,9 @@ transaction) {
                     : t('rollback_fresh_clean_failed').replace('{dirs}', freshCleanup.failed.map((f) => f.dir).join(', '))
                 : t('rollback_no_changes')
             : restoreResult.restored
-                ? t('rollback_restored')
+                ? restoredGaps.length === 0
+                    ? t('rollback_restored')
+                    : t('rollback_restored_incomplete').replace('{paths}', restoredGaps.join(', '))
                 : t('rollback_failed')
                     .replace('{restoreError}', restoreResult.error ?? '')
                     .replace('{extDir}', extDir)
@@ -4379,7 +4454,9 @@ transaction) {
                     ? t('next_no_changes_lock')
                     : t('next_no_changes_other'))
             : restoreResult.restored
-                ? t('next_restored')
+                ? restoredGaps.length === 0
+                    ? t('next_restored')
+                    : t('next_restored_incomplete').replace('{paths}', restoredGaps.join(', '))
                 : t('next_restore_failed')
                     .replace('{extDir}', extDir)
                     .replace('{backupDir}', backupDir ?? runtimeBackupDir ?? '')

package/release-manager/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-principles-disciple",
-  "version": "1.147.4",
+  "version": "1.147.5",
   "description": "Interactive CLI installer for Principles Disciple OpenClaw plugin",
   "type": "module",
   "bin": {