@phren/cli 0.0.25 → 0.0.27

package/mcp/dist/init.js

@@ -85,19 +85,16 @@ function runSyncCommand(command, args) {
     }
 }
 function shouldUninstallCurrentGlobalPackage() {
-    const entryScript = process.argv[1];
-    if (!entryScript)
-        return false;
+    // Always attempt to remove the global package if it exists, regardless of
+    // whether the uninstaller was invoked from the global install or a local repo.
     const npmRootResult = runSyncCommand(getNpmCommand(), ["root", "-g"]);
     if (!npmRootResult.ok)
         return false;
     const npmRoot = npmRootResult.stdout.trim();
     if (!npmRoot)
         return false;
-    const resolvedEntryScript = path.resolve(entryScript);
-    const resolvedGlobalPackageRoot = path.resolve(path.join(npmRoot, PHREN_NPM_PACKAGE_NAME));
-    return resolvedEntryScript === resolvedGlobalPackageRoot
-        || resolvedEntryScript.startsWith(`${resolvedGlobalPackageRoot}${path.sep}`);
+    const globalPkgPath = path.join(npmRoot, PHREN_NPM_PACKAGE_NAME);
+    return fs.existsSync(globalPkgPath);
 }
 function uninstallCurrentGlobalPackage() {
     const result = runSyncCommand(getNpmCommand(), ["uninstall", "-g", PHREN_NPM_PACKAGE_NAME]);
@@ -1682,8 +1679,29 @@ function sweepSkillSymlinks(phrenPath) {
                     log(`  Removed skill symlink: ${fullPath}`);
                 }
             }
+            catch {
+                // Broken symlink (target no longer exists) — clean it up
+                try {
+                    fs.unlinkSync(fullPath);
+                    log(`  Removed broken skill symlink: ${fullPath}`);
+                }
+                catch (err2) {
+                    debugLog(`sweepSkillSymlinks: could not remove broken symlink ${fullPath}: ${errorMessage(err2)}`);
+                }
+            }
+        }
+        // Remove phren-generated manifest files from the skills parent directory
+        const parentDir = path.dirname(dir);
+        for (const manifestFile of ["skill-manifest.json", "skill-commands.json"]) {
+            const manifestPath = path.join(parentDir, manifestFile);
+            try {
+                if (fs.existsSync(manifestPath)) {
+                    fs.unlinkSync(manifestPath);
+                    log(`  Removed ${manifestFile} (${manifestPath})`);
+                }
+            }
             catch (err) {
-                debugLog(`sweepSkillSymlinks: could not check/remove ${fullPath}: ${errorMessage(err)}`);
+                debugLog(`sweepSkillSymlinks: could not remove ${manifestPath}: ${errorMessage(err)}`);
             }
         }
     }
@@ -2008,6 +2026,33 @@ export async function runUninstall(opts = {}) {
     if (shouldRemoveGlobalPackage) {
         uninstallCurrentGlobalPackage();
     }
+    // Remove VS Code extension if installed
+    try {
+        const codeResult = execFileSync("code", ["--list-extensions"], {
+            encoding: "utf8",
+            stdio: ["ignore", "pipe", "ignore"],
+            timeout: 10_000,
+        });
+        const phrenExts = codeResult.split("\n").filter((ext) => ext.toLowerCase().includes("phren"));
+        for (const ext of phrenExts) {
+            const trimmed = ext.trim();
+            if (!trimmed)
+                continue;
+            try {
+                execFileSync("code", ["--uninstall-extension", trimmed], {
+                    stdio: ["ignore", "pipe", "ignore"],
+                    timeout: 15_000,
+                });
+                log(`  Removed VS Code extension (${trimmed})`);
+            }
+            catch (err) {
+                debugLog(`uninstall: VS Code extension removal failed for ${trimmed}: ${errorMessage(err)}`);
+            }
+        }
+    }
+    catch {
+        // code CLI not available — skip
+    }
     log(`\nPhren config, hooks, and installed data removed.`);
     log(`Restart your agent(s) to apply changes.\n`);
 }

package/mcp/dist/link-doctor.js

@@ -17,6 +17,7 @@ import { repairPreexistingInstall } from "./init-setup.js";
 import { getMachineName, lookupProfile, findProfileFile, getProfileProjects, findProjectDir, } from "./link.js";
 import { claudeProjectKey } from "./link-context.js";
 import { getProjectOwnershipMode, readProjectConfig } from "./project-config.js";
+import { readInstallPreferences } from "./init-preferences.js";
 // ── Doctor ──────────────────────────────────────────────────────────────────
 function isWrapperActive(tool) {
     const wrapperPath = homePath(".local", "bin", tool);
@@ -35,7 +36,7 @@ function isWrapperActive(tool) {
         return false;
     }
 }
-function gitRemoteStatus(phrenPath) {
+function gitRemoteStatus(phrenPath, syncIntent) {
     try {
         execFileSync("git", ["-C", phrenPath, "rev-parse", "--is-inside-work-tree"], {
             stdio: ["ignore", "ignore", "ignore"],
@@ -45,19 +46,40 @@ function gitRemoteStatus(phrenPath) {
     catch {
         return { ok: false, detail: "phren path is not a git repository" };
     }
+    let remote;
     try {
-        const remote = execFileSync("git", ["-C", phrenPath, "remote", "get-url", "origin"], {
+        remote = execFileSync("git", ["-C", phrenPath, "remote", "get-url", "origin"], {
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],
             timeout: EXEC_TIMEOUT_QUICK_MS,
         }).trim();
-        return remote
-            ? { ok: true, detail: `origin=${remote}` }
-            : { ok: true, detail: "no remote configured (local-only sync mode)" };
     }
     catch {
+        // no remote configured
+    }
+    if (!remote) {
+        if (syncIntent === "sync") {
+            return {
+                ok: false,
+                detail: "sync configured but no git remote found. Run: cd ~/.phren && git remote add origin <YOUR_REPO_URL> && git push -u origin main",
+            };
+        }
         return { ok: true, detail: "no remote configured (local-only sync mode)" };
     }
+    // Remote exists — verify it's reachable
+    try {
+        execFileSync("git", ["-C", phrenPath, "ls-remote", "--exit-code", "origin"], {
+            stdio: ["ignore", "ignore", "ignore"],
+            timeout: 10_000,
+        });
+        return { ok: true, detail: `origin=${remote}` };
+    }
+    catch {
+        if (syncIntent === "sync") {
+            return { ok: false, detail: `origin=${remote} (unreachable) — check your network or SSH keys` };
+        }
+        return { ok: true, detail: `origin=${remote} (unreachable, local-only mode)` };
+    }
 }
 function pushSkillMirrorChecks(checks, scope, manifest, destDir) {
     const parentDir = path.dirname(destDir);
@@ -117,7 +139,8 @@ export async function runDoctor(phrenPath, fix = false, checkData = false) {
         ok: versionAtLeast(nodeVersion, 20),
         detail: nodeVersion || "node not found in PATH",
     });
-    const gitRemote = gitRemoteStatus(phrenPath);
+    const prefs = readInstallPreferences(phrenPath);
+    const gitRemote = gitRemoteStatus(phrenPath, prefs.syncIntent);
     checks.push({
         name: "git-remote",
         ok: gitRemote.ok,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phren/cli",
-  "version": "0.0.25",
+  "version": "0.0.27",
   "description": "Knowledge layer for AI agents. Phren learns and recalls.",
   "type": "module",
   "bin": {