fix(cli): print remedy commands through getCliPrefix()

Author: theCodeDrift

.changeset/runnable-remedy-commands.md

@@ -0,0 +1,5 @@
+---
+"@taskless/cli": patch
+---
+
+Messages that tell you what to run now print a command you can run as shown. They used to hard-code `taskless`, which is usually not on `PATH` when the CLI was launched with `npx @taskless/cli` or `pnpm dlx`. They now use the same launcher-aware prefix the authentication and rule-recovery errors already used, for example `npx @taskless/cli@latest auth login`. This covers `auth`, `check`, `rule create`/`improve`/`meta`, `onboard`, `agent`, `feedback send`, the install wizard, the non-interactive banner, and the Vale and missing-fixture notices. `check` with no rules now also names the `--from` that `rule create` requires.

packages/cli/src/commands/agent.ts

@@ -19,6 +19,7 @@ import { withSurveyInvite } from "../survey/invite";
 import { applyCliInvocation } from "../util/invocation";
 import {
   detectCliInvocation,
+  getCliPrefix,
   processLauncherContext,
 } from "../util/package-manager";
 
@@ -162,8 +163,9 @@ export function createAgentCommand(subCommands: SubCommandsDef) {
           "\nAppend `--anonymous` to any rule/check command to skip the Taskless API"
         );
         console.log("and use local-only behavior.");
+        const prefix = getCliPrefix();
         console.log(
-          "\nRun `taskless agent <topic>` for the full recipe (e.g. `taskless agent create-sg-rule`)."
+          `\nRun \`${prefix} agent <topic>\` for the full recipe (e.g. \`${prefix} agent create-sg-rule\`).`
         );
         return;
       }
@@ -177,7 +179,7 @@ export function createAgentCommand(subCommands: SubCommandsDef) {
         telemetry.capture("cli_agent", { topic: positionals.join(" ") });
         console.error(`Too many arguments: ${positionals.join(" ")}`);
         console.error(
-          "A topic is a single token. Run `taskless agent` for the topic index."
+          `A topic is a single token. Run \`${getCliPrefix()} agent\` for the topic index.`
         );
         process.exitCode = 1;
         return;
@@ -238,7 +240,7 @@ export function createAgentCommand(subCommands: SubCommandsDef) {
         // cli_agent for an unknown topic — still the attempted topic string.
         telemetry.capture("cli_agent", { topic: key });
         console.error(`Unknown command: ${key}`);
-        console.error("Run `taskless agent` for available topics.");
+        console.error(`Run \`${getCliPrefix()} agent\` for available topics.`);
         process.exitCode = 1;
       }
     },

packages/cli/src/commands/auth.ts

@@ -8,6 +8,7 @@ import { fetchWhoami } from "../auth/whoami";
 import { getTelemetry } from "../telemetry";
 import { type CLIErrorCode, writeJsonError } from "../types/errors";
 import { splitRawArguments } from "../util/argv";
+import { getCliPrefix } from "../util/package-manager";
 
 const loginCommand = defineCommand({
   meta: {
@@ -70,7 +71,9 @@ const loginCommand = defineCommand({
         case "already_logged_in": {
           if (!args.json) {
             console.log("You are already logged in.");
-            console.log("Run `taskless auth logout` first to re-authenticate.");
+            console.log(
+              `Run \`${getCliPrefix()} auth logout\` first to re-authenticate.`
+            );
           }
           return;
         }
@@ -174,15 +177,15 @@ export const authCommand = defineCommand({
     const token = await getToken(cwd);
     if (!token) {
       console.log("Not logged in.");
-      console.log("Run `taskless auth login` to authenticate.");
+      console.log(`Run \`${getCliPrefix()} auth login\` to authenticate.`);
       return;
     }
 
     const whoami = await fetchWhoami(token);
     if (!whoami) {
       console.log("Logged in, but unable to verify identity.");
       console.log(
-        "Your token may be invalid or expired. Run `taskless auth login` to re-authenticate."
+        `Your token may be invalid or expired. Run \`${getCliPrefix()} auth login\` to re-authenticate.`
       );
       return;
     }

packages/cli/src/commands/check.ts

@@ -19,6 +19,7 @@ import { planCheck } from "../rules/plan-check";
 import { openRun } from "../rules/run-directory";
 import { fromProjectRoot } from "../rules/snapshot";
 import { markNotice } from "../util/notices";
+import { getCliPrefix } from "../util/package-manager";
 
 async function pathExists(absolutePath: string): Promise<boolean> {
   try {
@@ -334,7 +335,7 @@ export const checkCommand = defineCommand({
           );
         } else {
           console.log(
-            "No rules configured. Create one with `taskless rule create`."
+            `No rules configured. Create one with \`${getCliPrefix()} rule create --from <file>\`.`
           );
         }
         return;

packages/cli/src/commands/feedback.ts

@@ -15,6 +15,7 @@ import {
 import { getTelemetry, isTelemetryEnabled } from "../telemetry";
 import { type CLIErrorCode, writeJsonError } from "../types/errors";
 import { CLIError } from "../util/cli-error";
+import { getCliPrefix } from "../util/package-manager";
 
 /**
  * What both verbs say under the telemetry opt-out. An agent should never
@@ -111,7 +112,7 @@ const sendCommand = defineCommand({
     // whether or not anything would be sent, and the agent should hear so.
     if (!args.from) {
       fail(
-        "--from is required. Provide a path to a JSON file.\n  Example: taskless feedback send --from .taskless/.tmp-feedback.json",
+        `--from is required. Provide a path to a JSON file.\n  Example: ${getCliPrefix()} feedback send --from .taskless/.tmp-feedback.json`,
         "INVALID_INPUT"
       );
     }

packages/cli/src/commands/onboard.ts

@@ -12,6 +12,7 @@ import { getTelemetry } from "../telemetry";
 import { CLIError } from "../util/cli-error";
 import {
   detectCliInvocation,
+  getCliPrefix,
   processLauncherContext,
 } from "../util/package-manager";
 
@@ -31,9 +32,9 @@ export function getOnboardTrailer(args: {
   commandsInstalled: boolean;
 }): string {
   if (args.commandsInstalled) {
-    return "Next: in your AI tool, run /tskl onboard or ask it to use the Taskless skill (or run `taskless onboard` from your terminal) to discover rule candidates from your codebase.";
+    return `Next: in your AI tool, run /tskl onboard or ask it to use the Taskless skill (or run \`${getCliPrefix()} onboard\` from your terminal) to discover rule candidates from your codebase.`;
   }
-  return "Next: in your AI tool, ask it to use the Taskless skill (or run `taskless onboard` from your terminal) to discover rule candidates from your codebase.";
+  return `Next: in your AI tool, ask it to use the Taskless skill (or run \`${getCliPrefix()} onboard\` from your terminal) to discover rule candidates from your codebase.`;
 }
 
 export const onboardCommand = defineCommand({
@@ -97,7 +98,7 @@ export const onboardCommand = defineCommand({
     if (alreadyOnboarded && !args.force) {
       console.log("Taskless onboarding is already marked complete.");
       console.log(
-        "Run `taskless onboard --force` to re-run the discovery recipe."
+        `Run \`${getCliPrefix()} onboard --force\` to re-run the discovery recipe.`
       );
       return;
     }

packages/cli/src/commands/rules.ts

@@ -43,6 +43,7 @@ import {
 import { getTelemetry } from "../telemetry";
 import { CLIError } from "../util/cli-error";
 import { type CLIErrorCode, writeJsonError } from "../types/errors";
+import { getCliPrefix } from "../util/package-manager";
 
 /** Why submitting a request or an iteration failed, as a message and a code. */
 function describeSubmitFailure(
@@ -251,8 +252,7 @@ const createCommand = defineCommand({
       // `create-sg-rule`: authoring an ast-grep rule on-device with no service
       // call is exactly what anonymous mode asks for, so it is the destination
       // rather than an `--anonymous` variant of the service recipe.
-      const message =
-        "Anonymous rule generation runs in the agent. Run `taskless agent create-sg-rule` to fetch the local-only recipe.";
+      const message = `Anonymous rule generation runs in the agent. Run \`${getCliPrefix()} agent create-sg-rule\` to fetch the local-only recipe.`;
       if (args.json) {
         writeJsonError("INVALID_INPUT", message);
       } else {
@@ -269,7 +269,7 @@ const createCommand = defineCommand({
       // 1. Read and validate --from file
       if (!args.from) {
         fail(
-          "--from is required. Provide a path to a JSON file.\n  Example: taskless rule create --from request.json",
+          `--from is required. Provide a path to a JSON file.\n  Example: ${getCliPrefix()} rule create --from request.json`,
           "INVALID_INPUT"
         );
       }
@@ -398,8 +398,7 @@ const improveCommand = defineCommand({
     }
 
     if (args.anonymous) {
-      const message =
-        "Anonymous rule improvement runs in the agent. Run `taskless agent improve-rule --anonymous` to fetch the local-only recipe.";
+      const message = `Anonymous rule improvement runs in the agent. Run \`${getCliPrefix()} agent improve-rule --anonymous\` to fetch the local-only recipe.`;
       if (args.json) {
         writeJsonError("INVALID_INPUT", message);
       } else {
@@ -416,7 +415,7 @@ const improveCommand = defineCommand({
       // 1. Read and validate --from file
       if (!args.from) {
         fail(
-          "--from is required. Provide a path to a JSON file.\n  Example: taskless rule improve --from request.json",
+          `--from is required. Provide a path to a JSON file.\n  Example: ${getCliPrefix()} rule improve --from request.json`,
           "INVALID_INPUT"
         );
       }
@@ -558,7 +557,7 @@ const metaCommand = defineCommand({
           `the rule service does not return the metadata block that ` +
           `.taskless/rule-metadata/ is written from. This is not specific to "${args.id}". ` +
           `To iterate on a rule, pass its id (the rule's directory name under .taskless/rules/<engine>/) ` +
-          `to \`taskless rule improve --from <file>\`, or use the local-only improve flow.`,
+          `to \`${getCliPrefix()} rule improve --from <file>\`, or use the local-only improve flow.`,
         "RULE_META_UNAVAILABLE"
       );
     }

packages/cli/src/index.ts

@@ -31,6 +31,7 @@ import {
 } from "./util/argv";
 import { shouldLaunchWizard } from "./util/interactive";
 import { showResolvedUsage } from "./util/help";
+import { getCliPrefix } from "./util/package-manager";
 import { CLIError } from "./util/cli-error";
 
 // `satisfies` against the name list (minus `agent`, which is constructed from
@@ -140,11 +141,12 @@ const main = defineCommand({
       return;
     }
 
+    const prefix = getCliPrefix();
     console.error(
       "Taskless CLI — non-interactive context detected.\n" +
         "  For interactive install, run from a terminal.\n" +
-        "  For scripted install, run `taskless init`.\n" +
-        "  For agent recipes, run `taskless agent` (no args) for the topic index.\n"
+        `  For scripted install, run \`${prefix} init\`.\n` +
+        `  For agent recipes, run \`${prefix} agent\` (no args) for the topic index.\n`
     );
     // Forward the parent's rawArgs (e.g. `-d <path>`) so the agent command
     // doesn't mis-parse them as positional topic names.

packages/cli/src/rules/deliver.ts

@@ -8,6 +8,7 @@ import {
 } from "./layout";
 import { ruleDirectory } from "./engines";
 import { isMissingDirectory } from "./errno";
+import { getCliPrefix } from "../util/package-manager";
 
 /** One file of a delivered rule, its path relative to the rule directory. */
 export interface DeliveredFile {
@@ -140,7 +141,7 @@ export function describeMissingFixtures(
   if (hasFixture) return undefined;
   return (
     `was delivered with no ${prefix} fixtures, so nothing exercises it: ` +
-    "`taskless test` reports a pass over zero cases, which is " +
+    `\`${getCliPrefix()} test\` reports a pass over zero cases, which is ` +
     "indistinguishable from a rule proven to work"
   );
 }

packages/cli/src/rules/dispatch.ts

@@ -10,6 +10,7 @@ import type { RuntimeRule } from "./runtime/discover";
 import { runAstGrepScan } from "./scan";
 import { runVale } from "./vale/run";
 import { collectNotices } from "../util/notices";
+import { getCliPrefix } from "../util/package-manager";
 
 /**
  * Whether `.taskless/rules/vale/` holds anything to run.
@@ -286,7 +287,7 @@ function describeValeRefusal(refused: RefusedValeConfig): string {
   );
   return [
     `Vale did not run: the config of ${rules} was rejected by the config schema. ` +
-      `Run \`taskless verify\` for the constraint behind each line.`,
+      `Run \`${getCliPrefix()} verify\` for the constraint behind each line.`,
     ...lines,
   ].join("\n");
 }