npm - @hiroleague/taskmanager - Versions diffs - 0.0.2 → 0.0.3 - Mend

@hiroleague/taskmanager 0.0.2 → 0.0.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (121) hide show

package/dist/index.html CHANGED Viewed

@@ -25,23 +25,27 @@
         } catch (e) {}
       })();
     </script>
-    <script type="module" crossorigin src="/assets/index-CZZuue3D.js"></script>
+    <script type="module" crossorigin src="/assets/index-oKG1C41_.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/chunk-CFjPhJqf.js">
     <link rel="modulepreload" crossorigin href="/assets/src-C47h8eAM.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-ICPOFSXX-BNR1V8rT.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-ICPOFSXX-DkUVjrLw.js">
     <link rel="modulepreload" crossorigin href="/assets/dist-BXMOO6xX.js">
-    <link rel="modulepreload" crossorigin href="/assets/identity-D4WOnl_h.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-5PVQY5BW-BrmXs2Gs.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-U2HBQHQK-DTJPeU7W.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-X2U36JSP-CrTnmMqG.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-ZZ45TVLE-D3I1kLlo.js">
-    <link rel="modulepreload" crossorigin href="/assets/rough.esm-DtEqI08j.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-5FUZZQ4R-XEga0hMC.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-BSJP7CBP-D8kBlJsf.js">
+    <link rel="modulepreload" crossorigin href="/assets/identity-Me9aart9.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-5PVQY5BW-cGfZCZGU.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-U2HBQHQK-DogcerR6.js">
+    <link rel="modulepreload" crossorigin href="/assets/isEmpty-Cu0k-j1j.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-X2U36JSP-CwVWdmZV.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-ZZ45TVLE-Bk1S1YtS.js">
+    <link rel="modulepreload" crossorigin href="/assets/rough.esm-DEh6Frf9.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-5FUZZQ4R-B_HuuUjf.js">
     <link rel="modulepreload" crossorigin href="/assets/path-CIRGLb_6.js">
     <link rel="modulepreload" crossorigin href="/assets/array-BCJ35G43.js">
-    <link rel="modulepreload" crossorigin href="/assets/line-DNzQATGr.js">
-    <link rel="modulepreload" crossorigin href="/assets/chunk-ENJZ2VHE-BgZKYo1l.js">
+    <link rel="modulepreload" crossorigin href="/assets/line-otOkzGl8.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-BSJP7CBP-B0LrXV9y.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-ENJZ2VHE-DrWzOrpd.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-336JU56O-BGQvSwLk.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-426QAEUC-Cl9nQN9c.js">
+    <link rel="modulepreload" crossorigin href="/assets/chunk-XPW4576I-DQpNCogT.js">
     <link rel="stylesheet" crossorigin href="/assets/index-hMFTu7sr.css">
   </head>
   <body>

package/package.json CHANGED Viewed

@@ -1,8 +1,8 @@
 {
   "name": "@hiroleague/taskmanager",
-  "version": "0.0.2",
+  "version": "0.0.3",
   "type": "module",
-  "description": "Local-only task board app (Bun + Hono + React)",
+  "description": "Task Manager for Solo Builders and AI Agents, working as a team",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -22,7 +22,6 @@
     "skills",
     "LICENSE",
     "README.md",
-    "scripts/postinstall-message.mjs",
     "scripts/stubs/node-domexception"
   ],
   "overrides": {
@@ -33,7 +32,6 @@
     "hirotm": "./src/cli/bin/hirotm.ts"
   },
   "scripts": {
-    "postinstall": "node scripts/postinstall-message.mjs",
     "app": "bun run src/cli/bin/hirotaskmanager.ts",
     "cli": "bun run src/cli/bin/hirotm.ts",
     "dev": "concurrently -k -n api,web -c blue,green \"npm run dev:server\" \"npm run dev:client\"",

package/src/cli/bootstrap/launcher.ts CHANGED Viewed

@@ -29,6 +29,8 @@ import {
   formatTextPrompt,
   isAuthInitialized,
   paintValue,
+  printSetupContinuePrompt,
+  printSetupNextSteps,
   printPassphraseHint,
   printRecoveryKey,
   printRecoveryKeyExitHint,
@@ -319,7 +321,7 @@ async function waitForRecoveryKeyFile(authDir: string): Promise<string> {
   return key;
 }
-async function waitForEnterToExitLauncher(): Promise<void> {
+async function waitForEnterKey(): Promise<void> {
   const rl = createInterface({
     input: process.stdin,
     output: process.stdout,
@@ -383,9 +385,21 @@ export function createHirotaskmanagerProgram(): Command {
               },
             };
-        // Safety net: copy bundled skills to ~/.taskmanager/skills/ if
-        // postinstall was skipped (--ignore-scripts, CI, bunx) or stale.
-        ensureBundledSkills();
+        // Registry installs can skip lifecycle hooks, so setup itself must
+        // print the exact `npx skills` commands users should run next.
+        const skillsInstalled = ensureBundledSkills();
+        if (shouldRunSetup) {
+          printSetupNextSteps({
+            profileName: selectedProfile,
+            skillsInstalled,
+          });
+          if (setupResult.setupMeta.justFinishedInteractiveSetup) {
+            // Keep setup blocked here so users see the required skills step
+            // before the server starts and the browser steals focus.
+            printSetupContinuePrompt();
+            await waitForEnterKey();
+          }
+        }
         const launcherConfig = setupResult.config;
@@ -471,7 +485,7 @@ export function createHirotaskmanagerProgram(): Command {
             const recoveryKey = await waitForRecoveryKeyFile(authDir);
             printRecoveryKey(recoveryKey);
             printRecoveryKeyExitHint(runningUrl);
-            await waitForEnterToExitLauncher();
+            await waitForEnterKey();
           }
         } finally {
           if (previousSilentStartup === undefined) {

package/src/cli/bootstrap/program.test.ts ADDED Viewed

@@ -0,0 +1,46 @@
+import { describe, expect, test } from "bun:test";
+import {
+  getHirotmLauncherSetupCommand,
+  shouldRequireLauncherSetupForHirotm,
+} from "./program";
+describe("shouldRequireLauncherSetupForHirotm", () => {
+  test("requires launcher setup for installed mode without a saved profile", () => {
+    expect(
+      shouldRequireLauncherSetupForHirotm({
+        runtimeKind: "installed",
+        hasInstalledProfileConfig: false,
+      }),
+    ).toBe(true);
+  });
+  test("allows installed mode once the profile config exists", () => {
+    expect(
+      shouldRequireLauncherSetupForHirotm({
+        runtimeKind: "installed",
+        hasInstalledProfileConfig: true,
+      }),
+    ).toBe(false);
+  });
+  test("never blocks dev mode on launcher setup", () => {
+    expect(
+      shouldRequireLauncherSetupForHirotm({
+        runtimeKind: "dev",
+        hasInstalledProfileConfig: false,
+      }),
+    ).toBe(false);
+  });
+});
+describe("getHirotmLauncherSetupCommand", () => {
+  test("uses the plain launcher command for the default profile", () => {
+    expect(getHirotmLauncherSetupCommand("default")).toBe("hirotaskmanager");
+  });
+  test("includes the selected profile for named profiles", () => {
+    expect(getHirotmLauncherSetupCommand("work")).toBe(
+      "hirotaskmanager --profile work",
+    );
+  });
+});

package/src/cli/bootstrap/program.ts CHANGED Viewed

@@ -8,8 +8,56 @@ import { registerStatusCommands } from "../commands/statuses";
 import { registerTaskCommands } from "../commands/tasks";
 import { registerTrashCommands } from "../commands/trash";
 import { createDefaultCliContext } from "../handlers/context";
+import { hasCliConfigFile, resolveProfileName, resolveRuntimeKind } from "../lib/config";
 import { syncCliOutputFormatFromGlobals } from "../lib/cliFormat";
-import { exitWithError, resetCliOutputFormat } from "../lib/output";
+import { CliError, exitWithError, resetCliOutputFormat } from "../lib/output";
+import { CLI_ERR } from "../types/errors";
+export function shouldRequireLauncherSetupForHirotm(options: {
+  runtimeKind: "installed" | "dev";
+  hasInstalledProfileConfig: boolean;
+}): boolean {
+  return (
+    options.runtimeKind === "installed" &&
+    options.hasInstalledProfileConfig === false
+  );
+}
+export function getHirotmLauncherSetupCommand(profileName: string): string {
+  return profileName === "default"
+    ? "hirotaskmanager"
+    : `hirotaskmanager --profile ${profileName}`;
+}
+function ensureInstalledProfileIsReadyForHirotm(): void {
+  const runtimeKind = resolveRuntimeKind();
+  const profileName = resolveProfileName({ kind: "installed" });
+  const hasInstalledProfileConfig = hasCliConfigFile({
+    profile: profileName,
+    kind: "installed",
+  });
+  if (
+    !shouldRequireLauncherSetupForHirotm({
+      runtimeKind,
+      hasInstalledProfileConfig,
+    })
+  ) {
+    return;
+  }
+  const setupCommand = getHirotmLauncherSetupCommand(profileName);
+  // Package managers can skip lifecycle scripts, so actionable `hirotm`
+  // commands must direct users back to the launcher for first-run setup.
+  throw new CliError(
+    `No installed TaskManager profile found for "${profileName}". Run \`${setupCommand}\` first to create and configure this profile.`,
+    2,
+    {
+      code: CLI_ERR.missingRequired,
+      profile: profileName,
+      hint: `Run \`${setupCommand}\` first.`,
+    },
+  );
+}
 /**
  * Build the hirotm Commander program.
@@ -45,6 +93,7 @@ export function createHirotmProgram(): Command {
       quiet?: boolean;
     };
     syncCliOutputFormatFromGlobals(opts);
+    ensureInstalledProfileIsReadyForHirotm();
   });
   const ctx = createDefaultCliContext();

package/src/cli/lib/launcherUi.test.ts ADDED Viewed

@@ -0,0 +1,74 @@
+import { describe, expect, test } from "bun:test";
+import { printSetupContinuePrompt, printSetupNextSteps } from "./launcherUi";
+async function captureConsoleLog(
+  run: () => Promise<void> | void,
+): Promise<string> {
+  const originalLog = console.log;
+  const lines: string[] = [];
+  console.log = (...args: unknown[]) => {
+    lines.push(args.map(String).join(" "));
+  };
+  try {
+    await run();
+  } finally {
+    console.log = originalLog;
+  }
+  return `${lines.join("\n")}\n`;
+}
+describe("printSetupNextSteps", () => {
+  test("prints a prominent table with required repo skills install guidance", async () => {
+    const stdout = await captureConsoleLog(async () => {
+      printSetupNextSteps({
+        profileName: "default",
+        skillsInstalled: false,
+      });
+    });
+    expect(stdout).toContain("+-");
+    expect(stdout).toContain("REQUIRED BEFORE USING hirotm");
+    expect(stdout).toContain("Install AI agent skills now. The hirotm CLI depends on these skills.");
+    expect(stdout).toContain("hirotm --help");
+    expect(stdout).toContain("1. Repo skills  : npx skills add hiro-league/hirotaskmanager");
+    expect(stdout).toContain("2. Update later : npx skills update");
+    expect(stdout).toContain("npx skills add hiro-league/hirotaskmanager");
+    expect(stdout).not.toContain("npx skills add \"$HOME/.taskmanager/skills\"");
+  });
+  test("prints local skills command when bundled skills were copied", async () => {
+    const stdout = await captureConsoleLog(async () => {
+      printSetupNextSteps({
+        profileName: "default",
+        skillsInstalled: true,
+      });
+    });
+    expect(stdout).toContain("npx skills add hiro-league/hirotaskmanager");
+    expect(stdout).toContain("2. Local skills : npx skills add \"$HOME/.taskmanager/skills\"");
+    expect(stdout).toContain("3. Update later : npx skills update");
+    expect(stdout).toContain("npx skills add \"$HOME/.taskmanager/skills\"");
+    expect(stdout).toContain("npx skills update");
+  });
+  test("prints profile-specific CLI guidance for named profiles", async () => {
+    const stdout = await captureConsoleLog(async () => {
+      printSetupNextSteps({
+        profileName: "work",
+        skillsInstalled: false,
+      });
+    });
+    expect(stdout).toContain("hirotm --profile work --help");
+  });
+});
+describe("printSetupContinuePrompt", () => {
+  test("prints a separate enter-to-continue message", async () => {
+    const stdout = await captureConsoleLog(() => {
+      printSetupContinuePrompt();
+    });
+    expect(stdout).toContain("Press Enter to continue and start TaskManager...");
+  });
+});

package/src/cli/lib/launcherUi.ts CHANGED Viewed

@@ -39,6 +39,18 @@ function paintWarning(text: string): string {
   return paint(out, text, ansi.bold + ansi.yellow);
 }
+function printAsciiTable(lines: readonly string[]): void {
+  const innerWidth = Math.max(76, ...lines.map((line) => line.length));
+  const border = `  +-${"-".repeat(innerWidth)}-+`;
+  line();
+  line(border);
+  for (const entry of lines) {
+    line(`  | ${entry.padEnd(innerWidth)} |`);
+  }
+  line(border);
+  line();
+}
 export type SpinnerHandle = {
   stop: (finalText?: string | null) => void;
 };
@@ -126,6 +138,41 @@ export function printSavedProfileSummary(opts: {
   );
 }
+export function printSetupNextSteps(opts: {
+  profileName: string;
+  skillsInstalled: boolean;
+}): void {
+  const cliHelpCommand =
+    opts.profileName === "default"
+      ? "hirotm --help"
+      : `hirotm --profile ${opts.profileName} --help`;
+  const skillLines = [
+    "1. Repo skills  : npx skills add hiro-league/hirotaskmanager",
+  ];
+  if (opts.skillsInstalled) {
+    skillLines.push("2. Local skills : npx skills add \"$HOME/.taskmanager/skills\"");
+    skillLines.push("3. Update later : npx skills update");
+  } else {
+    skillLines.push("2. Update later : npx skills update");
+  }
+  const lines = [
+    "REQUIRED BEFORE USING hirotm",
+    "Install AI agent skills now. The hirotm CLI depends on these skills.",
+    "",
+    ...skillLines,
+    "",
+    `After skills install: ${cliHelpCommand}`,
+  ];
+  // Package managers may block postinstall hooks, so setup must surface the
+  // exact skills commands in a standalone table before startup continues.
+  printAsciiTable(lines);
+}
+export function printSetupContinuePrompt(): void {
+  line("Press Enter to continue and start TaskManager...");
+}
 export async function printPassphraseHint(): Promise<void> {
   // Keep the browser handoff to one line because the recovery key prints later from the server.
   const text =

package/src/shared/skillsInstall.ts CHANGED Viewed

@@ -1,8 +1,7 @@
 /**
  * Copies bundled skills/ from the installed package into ~/.taskmanager/skills/.
- * Used as a safety net in the launcher when postinstall didn't run (e.g.
- * --ignore-scripts, CI).  The postinstall-message.mjs script has equivalent
- * plain-Node logic that runs at npm/bun install time.
+ * Used by the launcher because package-manager lifecycle scripts may be blocked
+ * or skipped entirely (trusted-package checks, --ignore-scripts, CI).
  */
 import {
   cpSync,

package/dist/assets/architecture-YZFGNWBL-C1MoQeSs.js DELETED Viewed

	@@ -1 +0,0 @@
1	- import"./chunk-K5T4RW27-BLIPdXaZ.js";import{n as e}from"./chunk-7N4EOEYR-D7mYFpz-.js";export{e as createArchitectureServices};