@phren/cli 0.0.22 → 0.0.24

package/mcp/dist/init-setup.js

@@ -290,9 +290,10 @@ export function repairPreexistingInstall(phrenPath) {
 function isExpectedVerifyFailure(phrenPath, check) {
     if (check.ok)
         return false;
-    if (check.name === "git-remote")
-        return true;
     const prefs = readInstallPreferences(phrenPath);
+    // git-remote failure is only expected when the user chose local-only (no clone URL)
+    if (check.name === "git-remote")
+        return prefs.syncIntent !== "sync";
     if (check.name === "mcp-config" && prefs.mcpEnabled === false)
         return true;
     if (check.name === "hooks-registered" && prefs.hooksEnabled === false)
@@ -339,17 +340,30 @@ 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: false, detail: "git origin remote not configured" };
+        if (!remote)
+            return { ok: false, detail: "git origin remote not configured" };
     }
     catch {
         return { ok: false, detail: "git origin remote not configured" };
     }
+    // Connectivity test: verify the remote is reachable (10s timeout)
+    try {
+        execFileSync("git", ["-C", phrenPath, "ls-remote", "--exit-code", "origin"], {
+            stdio: ["ignore", "ignore", "ignore"],
+            timeout: 10_000,
+        });
+        return { ok: true, detail: `origin=${remote}` };
+    }
+    catch {
+        return { ok: false, detail: `origin=${remote} (configured but unreachable)` };
+    }
 }
 function copyStarterFile(phrenPath, src, dest) {
     fs.mkdirSync(path.dirname(dest), { recursive: true });
@@ -1147,14 +1161,22 @@ export function runPostInitVerify(phrenPath) {
     }
     else {
         const gitRemote = gitRemoteStatus(phrenPath);
+        const wantSync = prefs.syncIntent === "sync";
         const gitRemoteDetail = gitRemote.ok
             ? gitRemote.detail
-            : `${gitRemote.detail} (optional unless you want cross-machine sync)`;
+            : wantSync
+                ? `${gitRemote.detail} — sync was configured but remote is missing or unreachable`
+                : `${gitRemote.detail} (optional unless you want cross-machine sync)`;
+        const gitRemoteFix = gitRemote.ok
+            ? undefined
+            : wantSync
+                ? `Your clone URL didn't work. Fix: cd ${phrenPath} && git remote add origin <URL> && git push -u origin main`
+                : "Optional: initialize a repo and add an origin remote for cross-machine sync.";
         checks.push({
             name: "git-remote",
             ok: gitRemote.ok,
             detail: gitRemoteDetail,
-            fix: gitRemote.ok ? undefined : "Optional: initialize a repo and add an origin remote for cross-machine sync.",
+            fix: gitRemoteFix,
         });
         const settingsPath = hookConfigPath("claude");
         const configWritable = nearestWritableTarget(settingsPath);

package/mcp/dist/init.js

@@ -104,9 +104,18 @@ function parseRiskySectionsAnswer(raw, fallback) {
     return Array.from(new Set(parsed));
 }
 function hasInstallMarkers(phrenPath) {
-    return fs.existsSync(phrenPath) && (fs.existsSync(path.join(phrenPath, "machines.yaml")) ||
-        fs.existsSync(path.join(phrenPath, ".config")) ||
-        fs.existsSync(path.join(phrenPath, "global")));
+    // Require at least two markers to consider this a real install.
+    // A partial clone or failed init may create one directory but not finish.
+    if (!fs.existsSync(phrenPath))
+        return false;
+    let found = 0;
+    if (fs.existsSync(path.join(phrenPath, "machines.yaml")))
+        found++;
+    if (fs.existsSync(path.join(phrenPath, ".config")))
+        found++;
+    if (fs.existsSync(path.join(phrenPath, "global")))
+        found++;
+    return found >= 2;
 }
 function resolveInitPhrenPath(opts) {
     const raw = opts._walkthroughStoragePath || (process.env.PHREN_PATH) || DEFAULT_PHREN_PATH;
@@ -1028,9 +1037,23 @@ export async function runInit(opts = {}) {
         }
         catch (e) {
             log(`  Clone failed: ${e instanceof Error ? e.message : String(e)}`);
-            log(`  Continuing with fresh install instead.`);
+            log("");
+            log("  ┌──────────────────────────────────────────────────────────────────┐");
+            log("  │  WARNING: Sync is NOT configured. Your phren data is local-only. │");
+            log("  │                                                                  │");
+            log("  │  To fix later:                                                   │");
+            log(`  │    cd ${phrenPath}`);
+            log("  │    git remote add origin <YOUR_REPO_URL>                         │");
+            log("  │    git push -u origin main                                       │");
+            log("  └──────────────────────────────────────────────────────────────────┘");
+            log("");
+            log(`  Continuing with fresh local-only install.`);
         }
     }
+    // Record sync intent: "sync" if a clone URL was provided (regardless of success), "local" otherwise.
+    // On re-runs of existing installs, preserve the existing syncIntent unless the user provided a new clone URL.
+    const existingSyncIntent = hasExistingInstall ? readInstallPreferences(phrenPath).syncIntent : undefined;
+    const syncIntent = opts._walkthroughCloneUrl ? "sync" : (existingSyncIntent ?? "local");
     const mcpEnabled = opts.mcp ? opts.mcp === "on" : getMcpEnabledPreference(phrenPath);
     const hooksEnabled = opts.hooks ? opts.hooks === "on" : getHooksEnabledPreference(phrenPath);
     const skillsScope = opts.skillsScope ?? "global";
@@ -1185,7 +1208,7 @@ export async function runInit(opts = {}) {
                 log(`  No starter template updates were applied (starter files not found).`);
             }
         }
-        writeInstallPreferences(phrenPath, { mcpEnabled, hooksEnabled, skillsScope, installedVersion: VERSION });
+        writeInstallPreferences(phrenPath, { mcpEnabled, hooksEnabled, skillsScope, installedVersion: VERSION, syncIntent });
         if (repaired.removedLegacyProjects > 0) {
             log(`  Removed ${repaired.removedLegacyProjects} legacy starter project entr${repaired.removedLegacyProjects === 1 ? "y" : "ies"} from profiles.`);
         }
@@ -1363,7 +1386,7 @@ export async function runInit(opts = {}) {
     // Configure MCP for all detected AI coding tools and hooks
     configureMcpTargets(phrenPath, { mcpEnabled, hooksEnabled }, "Configured");
     configureHooksIfEnabled(phrenPath, hooksEnabled, "Configured");
-    writeInstallPreferences(phrenPath, { mcpEnabled, hooksEnabled, skillsScope, installedVersion: VERSION });
+    writeInstallPreferences(phrenPath, { mcpEnabled, hooksEnabled, skillsScope, installedVersion: VERSION, syncIntent });
     // Post-init verification
     log(`\nVerifying setup...`);
     const verify = runPostInitVerify(phrenPath);

package/mcp/dist/mcp-ops.js

@@ -163,15 +163,52 @@ export function register(server, ctx) {
             if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] healthCheck taskMode: ${errorMessage(err)}\n`);
         }
+        let syncIntent;
         try {
             const { readInstallPreferences } = await import("./init-preferences.js");
             const prefs = readInstallPreferences(phrenPath);
             proactivity = prefs.proactivity || "high";
+            syncIntent = prefs.syncIntent;
         }
         catch (err) {
             if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] healthCheck proactivity: ${errorMessage(err)}\n`);
         }
+        // Determine sync status from intent + git remote state
+        let syncStatus = "local-only";
+        let syncDetail = "no git remote configured";
+        try {
+            const { execFileSync } = await import("child_process");
+            const remote = execFileSync("git", ["-C", phrenPath, "remote", "get-url", "origin"], {
+                encoding: "utf8",
+                stdio: ["ignore", "pipe", "ignore"],
+                timeout: 5_000,
+            }).trim();
+            if (remote) {
+                try {
+                    execFileSync("git", ["-C", phrenPath, "ls-remote", "--exit-code", "origin"], {
+                        stdio: ["ignore", "ignore", "ignore"],
+                        timeout: 10_000,
+                    });
+                    syncStatus = "synced";
+                    syncDetail = `origin=${remote}`;
+                }
+                catch {
+                    syncStatus = syncIntent === "sync" ? "broken" : "local-only";
+                    syncDetail = `origin=${remote} (unreachable)`;
+                }
+            }
+            else if (syncIntent === "sync") {
+                syncStatus = "broken";
+                syncDetail = "sync was configured but no remote found";
+            }
+        }
+        catch {
+            if (syncIntent === "sync") {
+                syncStatus = "broken";
+                syncDetail = "sync was configured but no remote found";
+            }
+        }
         const lines = [
             `Phren v${version}`,
             `Profile: ${activeProfile || "(default)"}`,
@@ -182,6 +219,7 @@ export function register(server, ctx) {
             `Hooks: ${hooksEnabled ? "enabled" : "disabled"}`,
             `Proactivity: ${proactivity}`,
             `Task mode: ${taskMode}`,
+            `Sync: ${syncStatus}${syncStatus !== "synced" ? ` (${syncDetail})` : ""}`,
             `Path: ${phrenPath}`,
         ].filter(Boolean);
         return mcpResponse({
@@ -197,6 +235,8 @@ export function register(server, ctx) {
                 hooksEnabled,
                 proactivity,
                 taskMode,
+                syncStatus,
+                syncDetail,
                 phrenPath,
             },
         });