@cargo-ai/cli 1.0.135 → 1.0.136

package/build/agentSkills.d.ts

@@ -19,7 +19,8 @@
 /** One line, for a footer where vertical space is somebody else's error. */
 export declare const AGENT_SKILLS_HINT = "Coding agents: install the Cargo skills with `npx skills add getcargohq/cargo-skills`";
 /**
- * Printed after a successful sign-in.
+ * Printed after a successful sign-in that did not end with the skills
+ * installed (see `offerAgentSkills`).
  *
  * Deliberately two lines and no more. A sign-in banner that scrolls is a banner
  * people configure away, and this fires at most a handful of times per machine
@@ -31,4 +32,23 @@ export declare const AGENT_SKILLS_HINT = "Coding agents: install the Cargo skill
  * somebody adds one, and nothing in CI was ever going to catch it here.
  */
 export declare const AGENT_SKILLS_NEXT_STEP: string;
+export type AgentSkillsOfferDeps = {
+    ensureSkillsInstalled: (cwd: string) => Promise<boolean>;
+    print: (message: string) => void;
+};
+/**
+ * Offer the skills install right after sign-in, falling back to printing the
+ * commands.
+ *
+ * Printing alone is what produced the gap above: the lines scroll past and the
+ * install never happens. Asking turns the one moment we know someone is
+ * watching into a single keypress. The prompt is the agent handoff's own, so a
+ * machine that already has the bundle (globally or in this repo) is not asked,
+ * and an agent or a CI run never is.
+ *
+ * Whatever stops the install — no terminal, a declined prompt, an installer
+ * that failed — still leaves the commands on screen, so nobody ends sign-in
+ * with less than they used to get.
+ */
+export declare function offerAgentSkills(deps?: AgentSkillsOfferDeps): Promise<void>;
 //# sourceMappingURL=agentSkills.d.ts.map

package/build/agentSkills.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agentSkills.d.ts","sourceRoot":"","sources":["../src/agentSkills.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,4EAA4E;AAC5E,eAAO,MAAM,iBAAiB,0FAC2D,CAAC;AAE1F;;;;;;;;;;;GAWG;AACH,eAAO,MAAM,sBAAsB,QAIvB,CAAC"}
+{"version":3,"file":"agentSkills.d.ts","sourceRoot":"","sources":["../src/agentSkills.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAMH,4EAA4E;AAC5E,eAAO,MAAM,iBAAiB,0FAC2D,CAAC;AAE1F;;;;;;;;;;;;GAYG;AACH,eAAO,MAAM,sBAAsB,QAIvB,CAAC;AAEb,MAAM,MAAM,oBAAoB,GAAG;IACjC,qBAAqB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IACzD,KAAK,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;CAClC,CAAC;AAOF;;;;;;;;;;;;;GAaG;AACH,wBAAsB,gBAAgB,CACpC,IAAI,GAAE,oBAAkC,GACvC,OAAO,CAAC,IAAI,CAAC,CAQf"}

package/build/agentSkills.js

@@ -16,10 +16,13 @@
  * Keep the strings here rather than at each call site, so a change to how the
  * bundles are named or installed cannot land in `--help` and miss sign-in.
  */
+import { ensureSkillsInstalled } from "@cargo-ai/cdk/cli";
+import { info } from "./commands/runHandler.js";
 /** One line, for a footer where vertical space is somebody else's error. */
 export const AGENT_SKILLS_HINT = "Coding agents: install the Cargo skills with `npx skills add getcargohq/cargo-skills`";
 /**
- * Printed after a successful sign-in.
+ * Printed after a successful sign-in that did not end with the skills
+ * installed (see `offerAgentSkills`).
  *
  * Deliberately two lines and no more. A sign-in banner that scrolls is a banner
  * people configure away, and this fires at most a handful of times per machine
@@ -35,3 +38,28 @@ export const AGENT_SKILLS_NEXT_STEP = [
     "  npx skills add getcargohq/cargo-skills     every command, as agent skills",
     "  npx skills add getcargohq/gtm-skills       standalone one-job GTM skills",
 ].join("\n");
+const machineDeps = {
+    ensureSkillsInstalled,
+    print: info,
+};
+/**
+ * Offer the skills install right after sign-in, falling back to printing the
+ * commands.
+ *
+ * Printing alone is what produced the gap above: the lines scroll past and the
+ * install never happens. Asking turns the one moment we know someone is
+ * watching into a single keypress. The prompt is the agent handoff's own, so a
+ * machine that already has the bundle (globally or in this repo) is not asked,
+ * and an agent or a CI run never is.
+ *
+ * Whatever stops the install — no terminal, a declined prompt, an installer
+ * that failed — still leaves the commands on screen, so nobody ends sign-in
+ * with less than they used to get.
+ */
+export async function offerAgentSkills(deps = machineDeps) {
+    const installed = await deps.ensureSkillsInstalled(process.cwd());
+    if (installed === true) {
+        return;
+    }
+    deps.print(AGENT_SKILLS_NEXT_STEP);
+}

package/build/commands/auth/email.js

@@ -1,4 +1,4 @@
-import { AGENT_SKILLS_NEXT_STEP } from "../../agentSkills.js";
+import { offerAgentSkills } from "../../agentSkills.js";
 import { getCredentialsPath } from "../../credentials.js";
 import { askQuestion, openPromptInput } from "../../utils/prompt.js";
 import { ExitCodes, failWith, info, outputJson, startSpinner, success, } from "../runHandler.js";
@@ -52,7 +52,7 @@ export async function runEmailLogin(opts) {
         return;
     }
     success(`Signed in as ${result.email}. Workspace: ${result.workspace.name}.`);
-    info(AGENT_SKILLS_NEXT_STEP);
+    await offerAgentSkills();
     outputJson({
         ok: true,
         status: "signedIn",

package/build/commands/auth/index.js

@@ -1,6 +1,6 @@
 import { declarePrompt, revokeSession } from "@cargo-ai/cdk/cli";
 import { Option } from "commander";
-import { AGENT_SKILLS_NEXT_STEP } from "../../agentSkills.js";
+import { offerAgentSkills } from "../../agentSkills.js";
 import { getConfig } from "../../config.js";
 import { clearCredentials, CredentialsNotClearedError, getCredentialsPath, loadCredentials, saveCredentials, } from "../../credentials.js";
 import { parseBooleanOption } from "../../utils/booleanOption.js";
@@ -140,7 +140,7 @@ async function runLogin(opts) {
             baseUrl: opts.baseUrl,
         });
         success("Signed in with API token.");
-        info(AGENT_SKILLS_NEXT_STEP);
+        await offerAgentSkills();
         outputJson({
             ok: true,
             status: "signedIn",
@@ -159,7 +159,7 @@ async function runLogin(opts) {
         workspaceName: opts.workspaceName,
     });
     success(`Signed in. Default workspace: ${workspace.name}.`);
-    info(AGENT_SKILLS_NEXT_STEP);
+    await offerAgentSkills();
     outputJson({
         ok: true,
         status: "signedIn",

package/build/commands/doctor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"doctor.d.ts","sourceRoot":"","sources":["../../src/commands/doctor.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA+KzC,wBAAgB,qBAAqB,CACnC,OAAO,EAAE,OAAO,EAChB,cAAc,EAAE,MAAM,GACrB,IAAI,CAgFN"}
+{"version":3,"file":"doctor.d.ts","sourceRoot":"","sources":["../../src/commands/doctor.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA+KzC,wBAAgB,qBAAqB,CACnC,OAAO,EAAE,OAAO,EAChB,cAAc,EAAE,MAAM,GACrB,IAAI,CAkFN"}

package/build/commands/doctor.js

@@ -125,7 +125,8 @@ export function registerDoctorCommand(program, currentVersion) {
 Runs every check and reports all results as one JSON object, then exits with the
 most severe failure:
 
-  0  healthy (an outdated CLI or a pin mismatch is reported but never fails the check)
+  0  healthy (an outdated CLI, a pin mismatch or missing agent skills is
+     reported but never fails the check)
   1  API unreachable or returned an unexpected error
   3  not authenticated (no credentials, or the API rejected them)
   4  permission denied for the configured workspace
@@ -167,6 +168,7 @@ Examples:
             currentVersion,
             latestVersion,
             latestSkillsPin,
+            cwd: process.cwd(),
         });
         const apiIssue = describeApiIssue(api);
         const issues = apiIssue === undefined

package/build/index.js

@@ -300,10 +300,13 @@ async function openPalette(startPath) {
     const agents = availableCodingAgents();
     // Read after the refresh above, which fetches the skills pin alongside.
     const latestSkillsPin = cachedLatestSkillsPin();
+    // The cwd is read per call: a command run from the palette can install the
+    // skills into the project it is standing in.
     const collectIssues = () => collectSetupIssues({
         currentVersion: version,
         latestVersion,
         latestSkillsPin,
+        cwd: process.cwd(),
     });
     // 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.

package/build/setupIssues.d.ts

@@ -19,15 +19,25 @@ export type SetupIssueInput = {
      * stale pin and the warning has nothing to offer.
      */
     latestSkillsPin?: string;
+    /**
+     * Where a coding agent would be opened. Passing one opts this caller in to
+     * reporting skills that are not installed at all, which only the palette and
+     * `doctor` do. The pre-command header does not: it prints before every
+     * command, and someone working the CLI by hand, with no coding agent, would
+     * see a line they could only clear by installing something they do not use.
+     * Agents, who do need the skills, run without a TTY and never see the header.
+     */
+    cwd?: string;
 };
 /**
- * The two facts this reads off the machine. Injected so the rules — which side
- * of a pin mismatch to update, when an outdated CLI is this line's business at
+ * The facts this reads off the machine. Injected so the rules — which side of
+ * a pin mismatch to update, when an outdated CLI is this line's business at
  * all — can be tested without a home directory.
  */
 export type SetupIssueDeps = {
     isSignedIn: () => boolean;
     readSkillsPin: () => SkillsPin | undefined;
+    skillsInstalledFor: (cwd: string) => boolean;
 };
 export declare function collectSetupIssues(input: SetupIssueInput, deps?: SetupIssueDeps): SetupIssue[];
 //# sourceMappingURL=setupIssues.d.ts.map

package/build/setupIssues.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"setupIssues.d.ts","sourceRoot":"","sources":["../src/setupIssues.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAC/D,OAAO,EACL,UAAU,EACV,aAAa,EAEd,MAAM,mBAAmB,CAAC;AAM3B,YAAY,EAAE,SAAS,EAAE,CAAC;AAC1B,OAAO,EAAE,UAAU,EAAE,aAAa,EAAE,CAAC;AAErC,MAAM,MAAM,eAAe,GAAG;IAC5B,cAAc,EAAE,MAAM,CAAC;IACvB;;;;;OAKG;IACH,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;;;;;OAMG;IACH,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B,CAAC;AAEF;;;;GAIG;AACH,MAAM,MAAM,cAAc,GAAG;IAC3B,UAAU,EAAE,MAAM,OAAO,CAAC;IAC1B,aAAa,EAAE,MAAM,SAAS,GAAG,SAAS,CAAC;CAC5C,CAAC;AAIF,wBAAgB,kBAAkB,CAChC,KAAK,EAAE,eAAe,EACtB,IAAI,GAAE,cAA4B,GACjC,UAAU,EAAE,CAgDd"}
+{"version":3,"file":"setupIssues.d.ts","sourceRoot":"","sources":["../src/setupIssues.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAC/D,OAAO,EACL,UAAU,EACV,aAAa,EAGd,MAAM,mBAAmB,CAAC;AAM3B,YAAY,EAAE,SAAS,EAAE,CAAC;AAC1B,OAAO,EAAE,UAAU,EAAE,aAAa,EAAE,CAAC;AAErC,MAAM,MAAM,eAAe,GAAG;IAC5B,cAAc,EAAE,MAAM,CAAC;IACvB;;;;;OAKG;IACH,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;;;;;OAMG;IACH,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB;;;;;;;OAOG;IACH,GAAG,CAAC,EAAE,MAAM,CAAC;CACd,CAAC;AAEF;;;;GAIG;AACH,MAAM,MAAM,cAAc,GAAG;IAC3B,UAAU,EAAE,MAAM,OAAO,CAAC;IAC1B,aAAa,EAAE,MAAM,SAAS,GAAG,SAAS,CAAC;IAC3C,kBAAkB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC;CAC9C,CAAC;AAQF,wBAAgB,kBAAkB,CAChC,KAAK,EAAE,eAAe,EACtB,IAAI,GAAE,cAA4B,GACjC,UAAU,EAAE,CAwDd"}

package/build/setupIssues.js

@@ -1,14 +1,18 @@
 // What is wrong with this machine's Cargo setup, and the command that fixes
 // each one.
 //
-// This runs before every command, so every check here is a small file read or
-// cheaper — no network. `cargo-ai doctor` is the same list with the online
+// This runs before every command (and in the palette), so every check here is
+// a small file read or cheaper — no network. `cargo-ai doctor` is the same list with the online
 // checks added.
-import { isSignedIn, readSkillsPin, SKILLS_INSTALL_COMMAND, } from "@cargo-ai/cdk/cli";
+import { isSignedIn, readSkillsPin, SKILLS_INSTALL_COMMAND, skillsInstalledFor, } from "@cargo-ai/cdk/cli";
 import { isOutdated, UPDATE_COMMAND } from "./version.js";
 const LOGIN_COMMAND = "cargo-ai login --oauth";
 export { isSignedIn, readSkillsPin };
-const machineDeps = { isSignedIn, readSkillsPin };
+const machineDeps = {
+    isSignedIn,
+    readSkillsPin,
+    skillsInstalledFor,
+};
 export function collectSetupIssues(input, deps = machineDeps) {
     const issues = [];
     if (deps.isSignedIn() === false) {
@@ -46,5 +50,12 @@ export function collectSetupIssues(input, deps = machineDeps) {
             });
         }
     }
+    if (input.cwd !== undefined && deps.skillsInstalledFor(input.cwd) === false) {
+        issues.push({
+            key: "skills-missing",
+            label: "no Cargo skills for coding agents",
+            fix: SKILLS_INSTALL_COMMAND,
+        });
+    }
     return issues;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cargo-ai/cli",
-  "version": "1.0.135",
+  "version": "1.0.136",
   "mcpName": "io.github.getcargohq/cargo",
   "private": false,
   "license": "UNLICENSED",
@@ -76,11 +76,11 @@
     "@ai-sdk/harness-claude-code": "1.0.148",
     "@cargo-ai/api": "^1.0.102",
     "@cargo-ai/app-sdk": "^1.0.11",
-    "@cargo-ai/cdk": "^1.0.126",
+    "@cargo-ai/cdk": "^1.0.127",
     "@cargo-ai/types": "^1.0.104",
     "@cargo-ai/utils": "^1.0.35",
     "@cargo-ai/worker-sdk": "^1.0.21",
-    "@modelcontextprotocol/sdk": "^1.31.0",
+    "@modelcontextprotocol/sdk": "^1.32.0",
     "ai": "^7.0.133",
     "ai-sdk-sandbox-local": "^0.1.2",
     "commander": "^12.1.0",