@phren/cli 0.0.24 → 0.0.26

package/mcp/dist/init.js

@@ -5,7 +5,7 @@
 import * as fs from "fs";
 import * as path from "path";
 import * as crypto from "crypto";
-import { execFileSync } from "child_process";
+import { execFileSync, spawnSync } from "child_process";
 import { configureAllHooks } from "./hooks.js";
 import { getMachineName, machineFilePath, persistMachineName } from "./machine-identity.js";
 import { atomicWriteText, debugLog, isRecord, hookConfigPath, homeDir, homePath, expandHomePath, findPhrenPath, getProjectDirs, readRootManifest, writeRootManifest, } from "./shared.js";
@@ -24,6 +24,7 @@ import { DEFAULT_PHREN_PATH, STARTER_DIR, VERSION, log, confirmPrompt } from "./
 import { PROJECT_OWNERSHIP_MODES, getProjectOwnershipDefault, } from "./project-config.js";
 import { getWorkflowPolicy, updateWorkflowPolicy } from "./shared-governance.js";
 import { addProjectToProfile } from "./profile-store.js";
+const PHREN_NPM_PACKAGE_NAME = "@phren/cli";
 function parseVersion(version) {
     const match = version.trim().match(/^(\d+)\.(\d+)\.(\d+)(?:-(.+))?/);
     if (!match)
@@ -58,6 +59,56 @@ export function isVersionNewer(current, previous) {
         return true;
     return c.pre > p.pre;
 }
+function getNpmCommand() {
+    return process.platform === "win32" ? "npm.cmd" : "npm";
+}
+function runSyncCommand(command, args) {
+    try {
+        const result = spawnSync(command, args, {
+            encoding: "utf8",
+            stdio: ["ignore", "pipe", "pipe"],
+        });
+        return {
+            ok: result.status === 0,
+            status: result.status,
+            stdout: typeof result.stdout === "string" ? result.stdout : "",
+            stderr: typeof result.stderr === "string" ? result.stderr : "",
+        };
+    }
+    catch (err) {
+        return {
+            ok: false,
+            status: null,
+            stdout: "",
+            stderr: errorMessage(err),
+        };
+    }
+}
+function shouldUninstallCurrentGlobalPackage() {
+    const entryScript = process.argv[1];
+    if (!entryScript)
+        return false;
+    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}`);
+}
+function uninstallCurrentGlobalPackage() {
+    const result = runSyncCommand(getNpmCommand(), ["uninstall", "-g", PHREN_NPM_PACKAGE_NAME]);
+    if (result.ok) {
+        log(`  Removed global npm package (${PHREN_NPM_PACKAGE_NAME})`);
+        return;
+    }
+    const detail = result.stderr.trim() || result.stdout.trim() || (result.status === null ? "failed to start command" : `exit code ${result.status}`);
+    log(`  Warning: could not remove global npm package (${PHREN_NPM_PACKAGE_NAME})`);
+    debugLog(`uninstall: global npm cleanup failed: ${detail}`);
+}
 export function parseMcpMode(raw) {
     if (!raw)
         return undefined;
@@ -1732,6 +1783,7 @@ export async function runUninstall(opts = {}) {
         return;
     }
     log("\nUninstalling phren...\n");
+    const shouldRemoveGlobalPackage = shouldUninstallCurrentGlobalPackage();
     // Confirmation prompt (shared-mode only — project-local is low-stakes)
     if (!opts.yes) {
         const confirmed = phrenPath
@@ -1924,6 +1976,16 @@ export async function runUninstall(opts = {}) {
     catch (err) {
         debugLog(`uninstall: cleanup failed for ${machineFile}: ${errorMessage(err)}`);
     }
+    const contextFile = homePath(".phren-context.md");
+    try {
+        if (fs.existsSync(contextFile)) {
+            fs.unlinkSync(contextFile);
+            log(`  Removed machine context file (${contextFile})`);
+        }
+    }
+    catch (err) {
+        debugLog(`uninstall: cleanup failed for ${contextFile}: ${errorMessage(err)}`);
+    }
     // Sweep agent skill directories for symlinks pointing into the phren store
     if (phrenPath) {
         try {
@@ -1943,6 +2005,9 @@ export async function runUninstall(opts = {}) {
             log(`  Warning: could not remove phren root (${phrenPath})`);
         }
     }
+    if (shouldRemoveGlobalPackage) {
+        uninstallCurrentGlobalPackage();
+    }
     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.24",
+  "version": "0.0.26",
   "description": "Knowledge layer for AI agents. Phren learns and recalls.",
   "type": "module",
   "bin": {