@m13v/s4l 1.6.197-rc.8 → 1.6.198

package/mcp/dist/index.js

@@ -20,7 +20,7 @@ import os from "node:os";
 import path from "node:path";
 import fs from "node:fs";
 import { repoDir, runPython, run, readPlan, writePlan, planPath, } from "./repo.js";
-import { applySetup, resolveProject, personaReady, listManagedProjectStatus, ensureShortLinksDefault, ensurePersonaProject, findPersonaProject, REQUIRED_FIELDS, RECOMMENDED_FIELDS, configPath, normalizeStringList, } from "./setup.js";
+import { applySetup, resolveProject, personaReady, listManagedProjectStatus, listProjectSettings, ensureShortLinksDefault, ensurePersonaProject, findPersonaProject, REQUIRED_FIELDS, RECOMMENDED_FIELDS, configPath, normalizeStringList, } from "./setup.js";
 import { xStatus, xConnect, xDetectSources, xScanProfile, summarizeXAuth } from "./twitterAuth.js";
 import { startProvisioning, isProvisioning, readProgress, runtimeReady, readRuntime, resolvePython, resolveChrome, ensureMenubar, menubarRunning, clearMenubarStop, ensurePipelineCurrent, ensureRuntimeProvisioned, } from "./runtime.js";
 import { blockOnboardingMilestone, completeOnboardingMilestone, ensureDoctorPhase, onboardingLedger, onboardingSnapshot, recordOnboardingAttempt, runDoctorPhase, } from "./onboarding.js";
@@ -1397,9 +1397,14 @@ tool("project_config", {
     inputSchema: {
         status: z.boolean().optional(),
         action: z
-            .enum(["connect_x", "detect_x_sources", "profile_scan"])
+            .enum(["get", "connect_x", "detect_x_sources", "profile_scan"])
             .optional()
-            .describe("connect_x = import/validate your X session in the autoposter's managed browser. " +
+            .describe("get = read the CURRENT SAVED VALUES of every project's editable fields (website, " +
+            "description, icp, voice, differentiator, search_topics, get_started_link, " +
+            "content_guardrails, content_angle) plus readiness — the read companion to editing. " +
+            "Use it before tweaking part of a nested value; the panel's Project settings section " +
+            "is built on it. " +
+            "connect_x = import/validate your X session in the autoposter's managed browser. " +
             "With an explicit setup/connect request, warn about possible keychain prompts and call " +
             "with confirm:true without waiting for another yes/no reply. Without confirm:true it " +
             "only previews the operation for users who asked to inspect it rather than run it. " +
@@ -1469,12 +1474,25 @@ tool("project_config", {
             "weight, platform, voice_relationship, booking_link, qualification, subreddit_bans, " +
             "short_links_host, short_links_live, content_angle, messaging, landing_pages, posthog. " +
             "Pass {name:'<project>', fields:{<key>:<value>, ...}}; each key SHALLOW-merges onto the " +
-            "project, REPLACING that key's whole value (read the current value via status:true first if " +
+            "project, REPLACING that key's whole value (read the current value via action:'get' first if " +
             "you only want to tweak part of a nested object, then pass the full new value). A value of " +
             "null DELETES the key. 'name' is ignored here (can't rename through this path). This is how " +
             "you edit advanced config without any raw whole-file overwrite."),
     },
 }, async (args) => {
+    // ---- Read current saved values (the panel's Project settings source) ---
+    // Whitelisted field values + readiness for every managed project and the
+    // persona. Read-only; the write path stays the validated merge below.
+    if (args.action === "get") {
+        return jsonContent({
+            action: "get",
+            projects: listProjectSettings(),
+            config_path: configPath(),
+            note: "Current saved values of each project's editable fields. To change one, call project_config " +
+                "with {name, <field>: <new value>} — it merges onto what's saved and re-seeds topics. " +
+                "extra_keys lists advanced keys editable only via the `fields` escape hatch.",
+        });
+    }
     // ---- List import sources (for the panel dropdown) ---------------------
     // Read-only browser/profile detection. Never reads the keychain or decrypts
     // a cookie, so it shows no macOS Safe Storage prompt. Lets the user pick the
@@ -1724,11 +1742,20 @@ tool("project_config", {
     // Apply mode (incremental): merge whatever fields were supplied onto the
     // named project, then report whether it's now ready or still missing fields.
     try {
-        recordOnboardingAttempt("project_ready", {
-            missing_count: 0,
-        });
+        // Editing the persona project must not touch the PRODUCT onboarding
+        // milestone (a persona is never "project_ready" in the product sense; it
+        // validates against PERSONA_REQUIRED_FIELDS inside applySetup).
+        const editingPersona = findPersonaProject()?.name === args.name;
+        if (!editingPersona) {
+            recordOnboardingAttempt("project_ready", {
+                missing_count: 0,
+            });
+        }
         const result = applySetup(args);
-        if (result.ready) {
+        if (result.persona) {
+            // no-op on the onboarding ledger; readiness is reported below as usual.
+        }
+        else if (result.ready) {
             completeOnboardingMilestone("project_ready", { missing_count: 0 });
         }
         else {
@@ -1809,6 +1836,7 @@ tool("project_config", {
             ok: true,
             project: result.project,
             action: result.created ? "created" : "updated",
+            persona: result.persona,
             ready: result.ready,
             missing_required: result.missing_required,
             topics_seeded: topicsSeeded,
@@ -1820,16 +1848,19 @@ tool("project_config", {
             fields_removed: result.fields_removed,
             config_path: configPath(),
             onboarding: onboardingSnapshot(),
-            note: (result.ready
-                ? `Project '${result.project}' is fully configured.${seedNote} Next: if X is not connected, ` +
-                    `detect sources, warn about keychain prompts, and call project_config with ` +
-                    `action:'connect_x', confirm:true immediately. Once X is connected, schedule the autopilot ` +
-                    `(queue_setup + create_scheduled_task per task); the autopilot then drafts on its own. Call the ` +
-                    `dashboard to confirm the schedule is firing (schedule_state 'ok') — that is the final step, ` +
-                    `no need to wait for or verify a draft card.`
-                : `Saved what you provided for '${result.project}'. Still need: ${result.missing_required.join(", ")}. ` +
-                    `First derive those fields from existing context, profile_scan, and website research, then ` +
-                    `call project_config again with name='${result.project}'. Ask only if a required field is genuinely unknowable.`) +
+            note: (result.persona
+                ? `Persona '${result.project}' updated.${seedNote}` +
+                    (result.ready ? "" : ` Still needs: ${result.missing_required.join(", ")}.`)
+                : result.ready
+                    ? `Project '${result.project}' is fully configured.${seedNote} Next: if X is not connected, ` +
+                        `detect sources, warn about keychain prompts, and call project_config with ` +
+                        `action:'connect_x', confirm:true immediately. Once X is connected, schedule the autopilot ` +
+                        `(queue_setup + create_scheduled_task per task); the autopilot then drafts on its own. Call the ` +
+                        `dashboard to confirm the schedule is firing (schedule_state 'ok') — that is the final step, ` +
+                        `no need to wait for or verify a draft card.`
+                    : `Saved what you provided for '${result.project}'. Still need: ${result.missing_required.join(", ")}. ` +
+                        `First derive those fields from existing context, profile_scan, and website research, then ` +
+                        `call project_config again with name='${result.project}'. Ask only if a required field is genuinely unknowable.`) +
                 advancedNote,
         });
     }
@@ -4004,7 +4035,7 @@ tool("show_browser_to_user", {
         const message = ensured.error === "no_browser"
             ? "No managed Chrome is running right now. Start a draft cycle or autopilot so there's a live browser session to show."
             : ensured.error === "no_websocket"
-                ? "This Node runtime has no WebSocket support (needs Node 21+), so a screencast can't be opened."
+                ? "This runtime has no WebSocket support, so a live screencast can't be opened. Use 'Bring to front' to see the browser window instead."
                 : "Couldn't attach to the browser: " + String(ensured.error);
         return jsonContent({ ok: false, running: false, frame: null, message });
     }