@hiroleague/taskmanager 0.0.1 → 0.0.3

package/src/server/bootstrapDev.ts

@@ -1,6 +1,15 @@
 import { startTaskManagerServer } from "./index";
+import {
+  parseBootstrapDevFlagFromArgv,
+  parseBootstrapPortFromArgv,
+  parseBootstrapProfileFromArgv,
+} from "./parseBootstrapProfile";
+
+// --dev flag explicitly sets dev runtime; without it, defaults to installed.
+const isDev = parseBootstrapDevFlagFromArgv();
 
 await startTaskManagerServer({
-  kind: "dev",
-  profile: process.env.TASKMANAGER_PROFILE?.trim() || "dev",
+  kind: isDev ? "dev" : "installed",
+  profile: parseBootstrapProfileFromArgv(),
+  port: parseBootstrapPortFromArgv(),
 });

package/src/server/bootstrapInstalled.ts

@@ -1,6 +1,11 @@
 import { startTaskManagerServer } from "./index";
+import {
+  parseBootstrapPortFromArgv,
+  parseBootstrapProfileFromArgv,
+} from "./parseBootstrapProfile";
 
 await startTaskManagerServer({
   kind: "installed",
-  profile: process.env.TASKMANAGER_PROFILE?.trim() || undefined,
+  profile: parseBootstrapProfileFromArgv(),
+  port: parseBootstrapPortFromArgv(),
 });

package/src/server/index.ts

@@ -8,6 +8,11 @@ import {
   setRuntimeConfigSelection,
   type RuntimeKind,
 } from "../shared/runtimeConfig";
+import { resolveRuntimeSource } from "../shared/runtimeIdentity";
+import {
+  buildLocalServerUrl,
+  type RunningServerStatus,
+} from "../shared/serverStatus";
 import { authMiddleware, requireWebSession, type AppBindings } from "./auth";
 import { cliBoardReadError } from "./cliPolicyGuard";
 import { getDb, runMigrations } from "./db";
@@ -23,6 +28,20 @@ import { ensureDataDir, entryByIdOrSlug } from "./storage";
 
 export function createTaskManagerApp(kind: RuntimeKind): Hono<AppBindings> {
   const app = new Hono<AppBindings>();
+  const source = resolveRuntimeSource(import.meta.url);
+  // Keep `/api/health` and CLI `server status` on the exact same payload so
+  // users can compare them directly without translating field names.
+  const healthStatus = (): RunningServerStatus => {
+    const port = resolvePort({ kind });
+    return {
+      pid: process.pid,
+      port,
+      running: true,
+      runtime: kind,
+      source,
+      url: buildLocalServerUrl(port),
+    };
+  };
 
   // Keep dev auth close to production, but allow the Vite origin to talk to the
   // API directly for EventSource and fetch during local development.
@@ -37,7 +56,7 @@ export function createTaskManagerApp(kind: RuntimeKind): Hono<AppBindings> {
   }
 
   app.use("/api/*", authMiddleware);
-  app.get("/api/health", (c) => c.json({ ok: true }));
+  app.get("/api/health", (c) => c.json(healthStatus()));
   app.get("/api/events", async (c) => {
     const rawBoardId = c.req.query("boardId");
     if (rawBoardId != null && !/^\d+$/.test(rawBoardId)) {
@@ -65,9 +84,10 @@ export function createTaskManagerApp(kind: RuntimeKind): Hono<AppBindings> {
   app.route("/api/notifications", notificationsRoute);
   app.route("/api/search", searchRoute);
 
-  if (kind === "installed") {
-    const distDir = resolveInstalledDistDir();
-
+  // Installed mode requires a built frontend; dev mode serves it as an
+  // optional fallback so the server URL is not a dead 404 if dist/ exists.
+  const distDir = resolveInstalledDistDir();
+  if (kind === "installed" || existsSync(path.join(distDir, "index.html"))) {
     app.use("/*", serveStatic({ root: distDir }));
 
     app.get("*", async () => {
@@ -100,10 +120,12 @@ function assertInstalledDistReady(): void {
 export async function startTaskManagerServer(options: {
   kind: RuntimeKind;
   profile?: string;
+  port?: number;
 }): Promise<ReturnType<typeof Bun.serve>> {
   setRuntimeConfigSelection({
     kind: options.kind,
     profile: options.profile,
+    port: options.port,
   });
 
   await ensureDataDir();
@@ -123,6 +145,7 @@ export async function startTaskManagerServer(options: {
     port: resolvePort({
       kind: options.kind,
       profile: options.profile,
+      port: options.port,
     }),
     fetch: app.fetch,
     // SSE keeps requests open by design, so raise the idle timeout above the
@@ -130,9 +153,12 @@ export async function startTaskManagerServer(options: {
     idleTimeout: 30,
   });
 
-  console.log(
-    `${options.kind === "installed" ? "TaskManager" : "TaskManager dev API"} server listening on http://localhost:${server.port}`,
-  );
+  // Let the installed launcher print its own startup status so first-run output stays compact.
+  if (process.env.TASKMANAGER_SILENT_STARTUP_LOG !== "1") {
+    console.log(
+      `${options.kind === "installed" ? "TaskManager" : "TaskManager dev API"} server listening on http://localhost:${server.port}`,
+    );
+  }
 
   return server;
 }

package/src/server/migrations/013_cli_policy_and_provenance.ts

@@ -18,10 +18,10 @@ ALTER TABLE task ADD COLUMN created_by_label TEXT;
 
 CREATE TABLE cli_global_policy (
   id INTEGER PRIMARY KEY CHECK (id = 1),
-  create_board INTEGER NOT NULL DEFAULT 0
+  create_board INTEGER NOT NULL DEFAULT 1
 );
 
-INSERT INTO cli_global_policy (id, create_board) VALUES (1, 0);
+INSERT INTO cli_global_policy (id, create_board) VALUES (1, 1);
 
 CREATE TABLE board_cli_policy (
   board_id INTEGER PRIMARY KEY REFERENCES board(id) ON DELETE CASCADE,

package/src/server/migrations/019_cli_global_create_board_default_on.ts

@@ -0,0 +1,14 @@
+import type { Database } from "bun:sqlite";
+import type { Migration } from "./types";
+
+/**
+ * Default global CLI `create_board` is now on (migration 013 originally seeded off).
+ * Updates existing databases that already applied the old seed.
+ */
+export const migration019: Migration = {
+  version: 19,
+  name: "019_cli_global_create_board_default_on",
+  up(db: Database): void {
+    db.run("UPDATE cli_global_policy SET create_board = 1 WHERE id = 1");
+  },
+};

package/src/server/migrations/registry.ts

@@ -1,41 +1,43 @@
-import { migration001 } from "./001_initial";
-import { migration002 } from "./002_closed_at";
-import { migration003 } from "./003_task_search_fts5";
-import { migration004 } from "./004_task_search_extended";
-import { migration005 } from "./005_task_priorities";
-import { migration006 } from "./006_task_group_emoji";
-import { migration007 } from "./007_task_emoji";
-import { migration008 } from "./008_list_emoji";
-import { migration009 } from "./009_board_emoji";
-import { migration010 } from "./010_celebration_sounds_muted";
-import { migration011 } from "./011_board_cli_access";
-import { migration012 } from "./012_notification_event";
-import { migration013 } from "./013_cli_policy_and_provenance";
-import { migration014 } from "./014_drop_board_cli_access";
-import { migration015 } from "./015_trash_deleted_at";
-import { migration016 } from "./016_task_group_sort_and_board_defaults";
-import { migration017 } from "./017_task_priority_none_default";
-import { migration018 } from "./018_board_releases";
-import type { Migration } from "./types";
-
-/** Numbered migrations, ascending by version. */
-export const migrations: Migration[] = [
-  migration001,
-  migration002,
-  migration003,
-  migration004,
-  migration005,
-  migration006,
-  migration007,
-  migration008,
-  migration009,
-  migration010,
-  migration011,
-  migration012,
-  migration013,
-  migration014,
-  migration015,
-  migration016,
-  migration017,
-  migration018,
-];
+import { migration001 } from "./001_initial";
+import { migration002 } from "./002_closed_at";
+import { migration003 } from "./003_task_search_fts5";
+import { migration004 } from "./004_task_search_extended";
+import { migration005 } from "./005_task_priorities";
+import { migration006 } from "./006_task_group_emoji";
+import { migration007 } from "./007_task_emoji";
+import { migration008 } from "./008_list_emoji";
+import { migration009 } from "./009_board_emoji";
+import { migration010 } from "./010_celebration_sounds_muted";
+import { migration011 } from "./011_board_cli_access";
+import { migration012 } from "./012_notification_event";
+import { migration013 } from "./013_cli_policy_and_provenance";
+import { migration014 } from "./014_drop_board_cli_access";
+import { migration015 } from "./015_trash_deleted_at";
+import { migration016 } from "./016_task_group_sort_and_board_defaults";
+import { migration017 } from "./017_task_priority_none_default";
+import { migration018 } from "./018_board_releases";
+import { migration019 } from "./019_cli_global_create_board_default_on";
+import type { Migration } from "./types";
+
+/** Numbered migrations, ascending by version. */
+export const migrations: Migration[] = [
+  migration001,
+  migration002,
+  migration003,
+  migration004,
+  migration005,
+  migration006,
+  migration007,
+  migration008,
+  migration009,
+  migration010,
+  migration011,
+  migration012,
+  migration013,
+  migration014,
+  migration015,
+  migration016,
+  migration017,
+  migration018,
+  migration019,
+];

package/src/server/parseBootstrapProfile.ts

@@ -0,0 +1,42 @@
+/**
+ * Reads `--profile <name>` from argv for server bootstrap entrypoints.
+ * Parent CLIs pass the resolved profile this way instead of any profile env var.
+ */
+export function parseBootstrapProfileFromArgv(): string | undefined {
+  const argv = process.argv.slice(2);
+  const i = argv.indexOf("--profile");
+  if (i >= 0) {
+    const next = argv[i + 1]?.trim();
+    if (next) return next;
+  }
+  const eq = argv.find((a) => a.startsWith("--profile="));
+  if (eq) {
+    const v = eq.slice("--profile=".length).trim();
+    if (v) return v;
+  }
+  return undefined;
+}
+
+/** Reads `--port <n>` from argv for server bootstrap entrypoints (no port env var). */
+export function parseBootstrapPortFromArgv(): number | undefined {
+  const argv = process.argv.slice(2);
+  const i = argv.indexOf("--port");
+  if (i >= 0) {
+    const next = argv[i + 1]?.trim();
+    if (next) {
+      const parsed = Number(next);
+      if (Number.isInteger(parsed) && parsed > 0) return parsed;
+    }
+  }
+  const eq = argv.find((a) => a.startsWith("--port="));
+  if (eq) {
+    const parsed = Number(eq.slice("--port=".length).trim());
+    if (Number.isInteger(parsed) && parsed > 0) return parsed;
+  }
+  return undefined;
+}
+
+/** Reads `--dev` flag from argv for server bootstrap entrypoints. */
+export function parseBootstrapDevFlagFromArgv(): boolean {
+  return process.argv.slice(2).includes("--dev");
+}

package/src/server/storage/cliPolicy.ts

@@ -61,7 +61,8 @@ export function readCliGlobalPolicy(): CliGlobalPolicy {
   const row = db
     .query("SELECT create_board FROM cli_global_policy WHERE id = 1")
     .get() as { create_board: number } | null;
-  return { createBoard: row ? row.create_board !== 0 : false };
+  // Missing row should not happen after migrations; default matches seeded `create_board` (on).
+  return { createBoard: row ? row.create_board !== 0 : true };
 }
 
 export function setCliGlobalCreateBoard(enabled: boolean): void {