premanmcp 0.15.2 → 0.15.3

package/bin/integrations.js

@@ -615,7 +615,9 @@ export async function onboardCommand(
                 // why they are looking at a login screen.
                 process.stdout.write("Opened PreMan \u2014 sign in with the account you just used.\n");
               }
-              return installed;
+              // Whether the app is on screen decides what the last line of the walk
+              // should say, and only this closure knows.
+              return { ...installed, appOpened: opened.state !== "not-installed" };
             } finally {
               stopOpening?.();
             }
@@ -665,10 +667,18 @@ export async function onboardCommand(
     }
 
     try {
-      // A step that caught its own failure and already explained it says so, and
-      // is not summarised as done; anything else that returns is taken as done.
+      // A step that caught its own failure and already explained it says so; a step
+      // with nothing to do says that; anything else that returns is done.
+      //
+      // Translated rather than spread over, because the summary knows three words
+      // and a step may report any of its own. Spreading put `Desktop app: undefined`
+      // at the end of a successful install: the install reports state "installed",
+      // which is none of the three, so it printed as a failure with no detail.
       const reported = await step.run();
-      outcome.set(step.name, { state: "done", ...(reported || {}) });
+      const state =
+        { failed: "failed", skipped: "skipped", unsupported: "skipped" }[reported?.state] ||
+        "done";
+      outcome.set(step.name, { ...(reported || {}), state });
     } catch (err) {
       // Report and carry on: a failed Slack install must not cost the customer
       // the AWS connection they just finished.
@@ -687,7 +697,16 @@ export async function onboardCommand(
     else if (result.state === "skipped") process.stdout.write(`${MARK.skip()} ${step.name} (skipped)\n`);
     else process.stdout.write(`${MARK.fail()} ${step.name}: ${result.detail}\n`);
   }
-  process.stdout.write(`\nOpen ${frontendUrl(args)} to see your logs and endpoints.\n`);
+  // The app is on screen by the time this prints, so telling them to go and open
+  // a website is a step backwards. The website still earns its mention -- it is
+  // the way in from a machine that has no app, and the account is the same one --
+  // but as an alternative, not as the next thing to do.
+  const appOpened = outcome.get("Desktop app")?.appOpened === true;
+  process.stdout.write(
+    appOpened
+      ? `\nPreMan is open. You can also sign in at ${frontendUrl(args)}.\n`
+      : `\nOpen ${frontendUrl(args)} to see your logs and endpoints.\n`
+  );
 }
 
 export const INTEGRATIONS_HELP = `

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "premanmcp",
-  "version": "0.15.2",
+  "version": "0.15.3",
   "description": "Turn APIs into agent-callable MCP tools with auth, testing, and audit logs",
   "type": "module",
   "bin": {