@cargo-ai/cli 1.0.128 → 1.0.129

package/build/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { createRequire } from "node:module";
-import { availableCodingAgents, BRAND, canRunPalette, creditsLeftOf, declareSources, fetchSubscription, fetchWorkspaceName, findProjectDir, formatWorkspaceSuffix, isInteractive, isUnpaidPlan, printSetupIssues, registerCommands as registerCdkCommands, runPalette, SessionExpiredError, } from "@cargo-ai/cdk/cli";
+import { availableCodingAgents, BRAND, canRunPalette, creditsLeftOf, declareSources, fetchStateConnected, fetchSubscription, fetchWorkspaceName, findProjectDir, formatWorkspaceSuffix, isInteractive, isUnpaidPlan, printSetupIssues, registerCommands as registerCdkCommands, runPalette, SessionExpiredError, } from "@cargo-ai/cdk/cli";
 import { Command } from "commander";
 import { AGENT_SKILLS_HINT } from "./agentSkills.js";
 import { createApi } from "./api.js";
@@ -308,7 +308,7 @@ async function openPalette(startPath) {
     // Re-read after each command: signing in or out from inside the palette
     // changes both what the setup line says and which rows are worth pinning.
     // The cwd is re-tested too — a command can create a project in it.
-    const layoutFor = (unpaidPlan) => {
+    const layoutFor = (lookups) => {
         // Read once: a layout describing two different directories could offer to
         // deploy one and name the other.
         const cwd = process.cwd();
@@ -318,7 +318,14 @@ async function openPalette(startPath) {
             agents,
             projectDir: root,
             cwd,
-            unpaidPlan,
+            unpaidPlan: lookups.unpaidPlan,
+            // Only trusted for the project it was asked about: a command can `cd`
+            // the process or create a project, and an answer about another state
+            // would offer to connect one that is already connected.
+            stateConnected: lookups.stateConnected !== undefined &&
+                lookups.stateConnected.projectDir === root
+                ? lookups.stateConnected.connected
+                : undefined,
         });
     };
     await runPalette({
@@ -328,10 +335,17 @@ async function openPalette(startPath) {
         title: chrome.title,
         issues: collectIssues(),
         refreshIssues: collectIssues,
-        // First paint reuses the subscription already fetched for the title.
-        layout: layoutFor(chrome.unpaidPlan),
-        // After a command the plan can change (sign-in, upgrade); look it up again.
-        refreshLayout: async () => layoutFor(await resolveUnpaidPlan()),
+        // First paint reuses the lookups already made for the title.
+        layout: layoutFor(chrome),
+        // After a command the plan can change (sign-in, upgrade), and so can the
+        // state (the command may have been the connect); look both up again.
+        refreshLayout: async () => {
+            const [unpaidPlan, stateConnected] = await Promise.all([
+                resolveUnpaidPlan(),
+                resolveStateConnected(),
+            ]);
+            return layoutFor({ unpaidPlan, stateConnected });
+        },
     });
 }
 /**
@@ -342,19 +356,39 @@ async function openPalette(startPath) {
 async function paletteChrome() {
     const base = `${BRAND} cargo-ai ${version}`;
     if (isSignedIn() === false) {
-        return { title: base, unpaidPlan: false };
+        return { title: base, unpaidPlan: false, stateConnected: undefined };
     }
     const api = getApi();
-    const [name, subscription] = await Promise.all([
+    // The state lookup rides alongside the two the title already waits for, so
+    // it adds no round-trip of its own to the first paint.
+    const [name, subscription, stateConnected] = await Promise.all([
         fetchWorkspaceName(api),
         fetchSubscription(api),
+        resolveStateConnected(),
     ]);
     const creditsLeft = subscription === undefined ? undefined : creditsLeftOf(subscription);
     return {
         title: `${base}${formatWorkspaceSuffix(name, creditsLeft)}`,
         unpaidPlan: subscription === undefined ? false : isUnpaidPlan(subscription),
+        stateConnected,
     };
 }
+/**
+ * Best-effort, like the plan: whether the project the palette is standing in
+ * deploys from GitHub. Undefined outside a project, signed out, or when the
+ * lookup could not tell, all of which leave "Connect to GitHub" unpinned.
+ */
+async function resolveStateConnected() {
+    if (isSignedIn() === false) {
+        return undefined;
+    }
+    const projectDir = findProjectDir(process.cwd());
+    if (projectDir === undefined) {
+        return undefined;
+    }
+    const connected = await fetchStateConnected(getApi(), projectDir);
+    return connected === undefined ? undefined : { projectDir, connected };
+}
 /**
  * Best-effort: true only when we can see an Orb $0 plan. A failed lookup or a
  * signed-out session leaves the pin off rather than guessing.

package/build/paletteLayout.d.ts

@@ -15,6 +15,13 @@ export type PaletteLayoutInput = {
      * false leaves it in the last section only.
      */
     unpaidPlan?: boolean;
+    /**
+     * Whether the project's deploy state is connected to its GitHub repository.
+     * Only `false` pins "Connect to GitHub": undefined means the lookup could not
+     * say (no state yet, offline, timed out), and a row offered on a guess is a
+     * row that fails when picked.
+     */
+    stateConnected?: boolean;
 };
 export declare function buildPaletteLayout(input: PaletteLayoutInput): PaletteLayout;
 /**

package/build/paletteLayout.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"paletteLayout.d.ts","sourceRoot":"","sources":["../src/paletteLayout.ts"],"names":[],"mappings":"AAYA,OAAO,EAEL,KAAK,WAAW,EAChB,KAAK,aAAa,EAClB,KAAK,SAAS,EACf,MAAM,mBAAmB,CAAC;AA+E3B,MAAM,MAAM,kBAAkB,GAAG;IAC/B,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,EAAE,SAAS,WAAW,EAAE,CAAC;IAC/B,mFAAmF;IACnF,UAAU,EAAE,MAAM,GAAG,SAAS,CAAC;IAC/B;;;;OAIG;IACH,GAAG,EAAE,MAAM,CAAC;IACZ;;;OAGG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB,CAAC;AAEF,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,kBAAkB,GAAG,aAAa,CAS3E;AAED;;;;;GAKG;AACH,wBAAgB,UAAU,CAAC,KAAK,EAAE,kBAAkB,GAAG,SAAS,EAAE,CA+DjE"}
+{"version":3,"file":"paletteLayout.d.ts","sourceRoot":"","sources":["../src/paletteLayout.ts"],"names":[],"mappings":"AAYA,OAAO,EAEL,KAAK,WAAW,EAChB,KAAK,aAAa,EAClB,KAAK,SAAS,EACf,MAAM,mBAAmB,CAAC;AAgF3B,MAAM,MAAM,kBAAkB,GAAG;IAC/B,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,EAAE,SAAS,WAAW,EAAE,CAAC;IAC/B,mFAAmF;IACnF,UAAU,EAAE,MAAM,GAAG,SAAS,CAAC;IAC/B;;;;OAIG;IACH,GAAG,EAAE,MAAM,CAAC;IACZ;;;OAGG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB;;;;;OAKG;IACH,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B,CAAC;AAEF,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,kBAAkB,GAAG,aAAa,CAS3E;AAED;;;;;GAKG;AACH,wBAAgB,UAAU,CAAC,KAAK,EAAE,kBAAkB,GAAG,SAAS,EAAE,CAmFjE"}

package/build/paletteLayout.js

@@ -16,6 +16,7 @@ const CODE = " code";
 const ADD = " add";
 const DEPLOY = " deploy";
 const BOOK_DEMO = " book-demo";
+const CONNECT = " connect";
 /**
  * `start` runs most of these; the pinned row is the shortcut to one of them.
  * The exceptions are "Add" and "Deploy", which run `project add` and `project
@@ -139,7 +140,25 @@ export function pinnedRows(input) {
             },
         ]
         : [];
+    // Above everything, even Ask: a project deployed from a laptop has no review
+    // and no record of who changed what, and the step that fixes that is one
+    // command nobody guesses the name of. It disappears once taken (the layout
+    // is rebuilt after every command), so it is never a permanent tenant of the
+    // top row.
+    const connectRow = input.projectDir !== undefined && input.stateConnected === false
+        ? [
+            {
+                value: CONNECT,
+                label: "Connect to GitHub",
+                hint: "review every change in a pull request; merging deploys",
+                // `--dir` for the same reason as Add and Deploy: connect the
+                // project the palette found, not whatever the cwd happens to be.
+                argv: ["project", "state", "connect", "--dir", input.projectDir],
+            },
+        ]
+        : [];
     return [
+        ...connectRow,
         {
             value: ASK,
             label: "Ask Cargo",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cargo-ai/cli",
-  "version": "1.0.128",
+  "version": "1.0.129",
   "mcpName": "io.github.getcargohq/cargo",
   "private": false,
   "license": "UNLICENSED",
@@ -76,7 +76,7 @@
     "@ai-sdk/harness-claude-code": "1.0.148",
     "@cargo-ai/api": "^1.0.101",
     "@cargo-ai/app-sdk": "^1.0.11",
-    "@cargo-ai/cdk": "^1.0.121",
+    "@cargo-ai/cdk": "^1.0.122",
     "@cargo-ai/types": "^1.0.103",
     "@cargo-ai/utils": "^1.0.35",
     "@cargo-ai/worker-sdk": "^1.0.21",