kandev 0.15.0 → 0.16.0

package/dist/constants.js

@@ -3,14 +3,17 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.DATA_DIR = exports.CACHE_DIR = exports.HEALTH_TIMEOUT_MS_DEV = exports.HEALTH_TIMEOUT_MS_RELEASE = exports.RANDOM_PORT_RETRIES = exports.RANDOM_PORT_MAX = exports.RANDOM_PORT_MIN = exports.DEFAULT_MCP_PORT = exports.DEFAULT_AGENTCTL_PORT = exports.DEFAULT_WEB_PORT = exports.DEFAULT_BACKEND_PORT = void 0;
+exports.DEV_KANDEV_DOTDIR = exports.DATA_DIR = exports.CACHE_DIR = exports.KANDEV_TASKS_DIR = exports.KANDEV_HOME_DIR = exports.KANDEV_DOTDIR = exports.HEALTH_TIMEOUT_MS_DEV = exports.HEALTH_TIMEOUT_MS_RELEASE = exports.RANDOM_PORT_RETRIES = exports.RANDOM_PORT_MAX = exports.RANDOM_PORT_MIN = exports.DEFAULT_MCP_PORT = exports.DEFAULT_AGENTCTL_PORT = exports.DEFAULT_WEB_PORT = exports.DEFAULT_BACKEND_PORT = void 0;
+exports.devKandevHome = devKandevHome;
 const node_os_1 = __importDefault(require("node:os"));
 const node_path_1 = __importDefault(require("node:path"));
 // Default service ports (will auto-fallback if busy).
-exports.DEFAULT_BACKEND_PORT = 8080;
+// Clustered near the web port (37429) to avoid collisions with commonly used
+// ports (8080, 9090, 9999, etc.) while keeping the numbers memorable.
+exports.DEFAULT_BACKEND_PORT = 38429;
 exports.DEFAULT_WEB_PORT = 37429;
-exports.DEFAULT_AGENTCTL_PORT = 9999;
-exports.DEFAULT_MCP_PORT = 9090;
+exports.DEFAULT_AGENTCTL_PORT = 39429;
+exports.DEFAULT_MCP_PORT = 40429;
 // Random fallback range for port selection.
 exports.RANDOM_PORT_MIN = 10000;
 exports.RANDOM_PORT_MAX = 60000;
@@ -18,6 +21,19 @@ exports.RANDOM_PORT_RETRIES = 10;
 // Backend healthcheck timeout during startup.
 exports.HEALTH_TIMEOUT_MS_RELEASE = 45000;
 exports.HEALTH_TIMEOUT_MS_DEV = 600000;
+// Kandev root directory. Single source of truth for the dotdir name and
+// everything derived from it (data, tasks, bin). Dev mode uses a separate
+// root under the repo (see DEV_KANDEV_DOTDIR).
+exports.KANDEV_DOTDIR = ".kandev";
+exports.KANDEV_HOME_DIR = node_path_1.default.join(node_os_1.default.homedir(), exports.KANDEV_DOTDIR);
+exports.KANDEV_TASKS_DIR = node_path_1.default.join(exports.KANDEV_HOME_DIR, "tasks");
 // Local user cache/data directories for release bundles and DB.
-exports.CACHE_DIR = node_path_1.default.join(node_os_1.default.homedir(), ".kandev", "bin");
-exports.DATA_DIR = node_path_1.default.join(node_os_1.default.homedir(), ".kandev", "data");
+exports.CACHE_DIR = node_path_1.default.join(exports.KANDEV_HOME_DIR, "bin");
+exports.DATA_DIR = node_path_1.default.join(exports.KANDEV_HOME_DIR, "data");
+// Dev-mode root: an isolated kandev home inside the repo so that running
+// `make dev` from inside a kandev-spawned task workspace does not touch the
+// user's production state.
+exports.DEV_KANDEV_DOTDIR = ".kandev-dev";
+function devKandevHome(repoRoot) {
+    return node_path_1.default.join(repoRoot, exports.DEV_KANDEV_DOTDIR);
+}

package/dist/dev.js

@@ -4,21 +4,18 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.runDev = runDev;
+exports.resolveDevBackendEnv = resolveDevBackendEnv;
 const node_child_process_1 = require("node:child_process");
 const node_path_1 = __importDefault(require("node:path"));
 const constants_1 = require("./constants");
 const health_1 = require("./health");
+const kandev_env_1 = require("./kandev-env");
 const process_1 = require("./process");
 const shared_1 = require("./shared");
 const web_1 = require("./web");
 async function runDev({ repoRoot, backendPort, webPort }) {
     const ports = await (0, shared_1.pickPorts)(backendPort, webPort);
-    const dbPath = process.env.KANDEV_DATABASE_PATH || node_path_1.default.join(repoRoot, "apps", "backend", "kandev.db");
-    const extra = {
-        KANDEV_DATABASE_PATH: dbPath,
-        KANDEV_MOCK_AGENT: process.env.KANDEV_MOCK_AGENT || "true",
-        KANDEV_DEBUG_PPROF_ENABLED: "true",
-    };
+    const { dbPath, extra } = resolveDevBackendEnv(repoRoot);
     const backendEnv = (0, shared_1.buildBackendEnv)({ ports, extra });
     const webEnv = (0, shared_1.buildWebEnv)({ ports, debug: true });
     const logLevel = process.env.KANDEV_LOGGING_LEVEL?.trim() || process.env.KANDEV_LOG_LEVEL?.trim() || "info";
@@ -55,3 +52,48 @@ async function runDev({ repoRoot, backendPort, webPort }) {
     console.log(`[kandev] web ready at ${webUrl}`);
     (0, web_1.openBrowser)(webUrl);
 }
+// Computes the dev-mode backend env. Dev mode always roots kandev under
+// <repo>/.kandev-dev so state is isolated from the user's production ~/.kandev
+// and so `make clean-db` (which removes .kandev-dev/) matches what `make dev`
+// writes.
+//
+// When invoked from inside a kandev task workspace, any KANDEV_DATABASE_PATH
+// is assumed to be leaked from the parent backend and is ignored. In a normal
+// shell, an explicit KANDEV_DATABASE_PATH is honored as an escape hatch.
+function resolveDevBackendEnv(repoRoot) {
+    const baseExtra = {
+        KANDEV_MOCK_AGENT: process.env.KANDEV_MOCK_AGENT || "true",
+        KANDEV_DEBUG_PPROF_ENABLED: "true",
+    };
+    const devHome = (0, constants_1.devKandevHome)(repoRoot);
+    // Display only; the backend derives its own DB path from KANDEV_HOME_DIR
+    // via ResolvedDataDir(). Both resolve to the same location.
+    const devDbPath = node_path_1.default.join(devHome, "data", "kandev.db");
+    if ((0, kandev_env_1.isInsideKandevTask)(repoRoot)) {
+        console.log("[kandev] task workspace detected → using local dev state");
+        return {
+            dbPath: devDbPath,
+            extra: {
+                ...baseExtra,
+                KANDEV_HOME_DIR: devHome,
+                // Clear a parent-leaked DB path so the backend uses the HomeDir-derived default.
+                KANDEV_DATABASE_PATH: "",
+            },
+        };
+    }
+    const override = process.env.KANDEV_DATABASE_PATH;
+    if (override) {
+        return {
+            dbPath: override,
+            extra: { ...baseExtra, KANDEV_DATABASE_PATH: override },
+        };
+    }
+    return {
+        dbPath: devDbPath,
+        extra: {
+            ...baseExtra,
+            KANDEV_HOME_DIR: devHome,
+            KANDEV_DATABASE_PATH: "",
+        },
+    };
+}

package/dist/kandev-env.js

@@ -0,0 +1,28 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.isInsideKandevTask = isInsideKandevTask;
+const node_path_1 = __importDefault(require("node:path"));
+const constants_1 = require("./constants");
+/**
+ * Returns true when the current process looks like it was spawned inside a
+ * kandev-created task workspace. Two signals:
+ *   1. The parent kandev backend exports KANDEV_TASK_ID into every task shell.
+ *   2. Task worktrees live under ~/.kandev/tasks/ (see KANDEV_TASKS_DIR).
+ *
+ * Used by dev mode to auto-isolate the backend onto a local dev root so that
+ * `make dev` never mutates the user's production state.
+ *
+ * Note: the path-prefix fallback is a defensive secondary signal for nested
+ * shells where KANDEV_TASK_ID was stripped. It is case-sensitive and does not
+ * resolve symlinks, so a realpath'd repoRoot may miss a symlinked HOME on
+ * macOS / Windows. KANDEV_TASK_ID remains the primary guarantee.
+ */
+function isInsideKandevTask(repoRoot) {
+    if (process.env.KANDEV_TASK_ID) {
+        return true;
+    }
+    return repoRoot.startsWith(constants_1.KANDEV_TASKS_DIR + node_path_1.default.sep);
+}

package/dist/start.js

@@ -15,6 +15,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.resolveStandaloneServerPath = resolveStandaloneServerPath;
 exports.runStart = runStart;
 const node_child_process_1 = require("node:child_process");
 const node_fs_1 = __importDefault(require("node:fs"));
@@ -25,6 +26,50 @@ const platform_1 = require("./platform");
 const process_1 = require("./process");
 const shared_1 = require("./shared");
 const web_1 = require("./web");
+/**
+ * Locates the standalone Next.js `server.js` inside `apps/web/.next/standalone/`.
+ *
+ * Normally this sits at `.next/standalone/web/server.js`, but Turbopack may
+ * place it under a deeper path (e.g. `.next/standalone/Users/.../web/server.js`)
+ * when it detects the wrong project root (typically caused by a stray
+ * `package-lock.json` in a parent directory). We look for any `web/server.js`
+ * below the standalone directory so `kandev start` keeps working.
+ *
+ * @returns absolute path to `server.js`, or null if it cannot be found.
+ */
+function resolveStandaloneServerPath(repoRoot) {
+    const standaloneDir = node_path_1.default.join(repoRoot, "apps", "web", ".next", "standalone");
+    const expected = node_path_1.default.join(standaloneDir, "web", "server.js");
+    if (node_fs_1.default.existsSync(expected))
+        return expected;
+    if (!node_fs_1.default.existsSync(standaloneDir))
+        return null;
+    return findWebServerJs(standaloneDir);
+}
+// Walks `dir` manually instead of relying on `Dirent.parentPath` (Node 20.12+),
+// so the CLI keeps working on older Node 20.x runtimes installed via npx.
+function findWebServerJs(dir) {
+    const stack = [dir];
+    while (stack.length > 0) {
+        const current = stack.pop();
+        let entries;
+        try {
+            entries = node_fs_1.default.readdirSync(current, { withFileTypes: true });
+        }
+        catch {
+            continue;
+        }
+        for (const entry of entries) {
+            if (entry.isDirectory() && entry.name !== "node_modules") {
+                stack.push(node_path_1.default.join(current, entry.name));
+            }
+            else if (entry.isFile() && entry.name === "server.js" && node_path_1.default.basename(current) === "web") {
+                return node_path_1.default.join(current, entry.name);
+            }
+        }
+    }
+    return null;
+}
 /**
  * Runs the application in production mode using local builds.
  *
@@ -45,9 +90,16 @@ async function runStart({ repoRoot, backendPort, webPort, verbose = false, debug
         throw new Error("Backend binary not found. Run `make build` first.");
     }
     // Check for standalone build (Next.js standalone output)
-    const webServerPath = node_path_1.default.join(repoRoot, "apps", "web", ".next", "standalone", "web", "server.js");
-    if (!node_fs_1.default.existsSync(webServerPath)) {
-        throw new Error("Web standalone build not found. Run `make build` first.");
+    const webServerPath = resolveStandaloneServerPath(repoRoot);
+    if (!webServerPath) {
+        const standaloneDir = node_path_1.default.join(repoRoot, "apps", "web", ".next", "standalone");
+        if (!node_fs_1.default.existsSync(standaloneDir)) {
+            throw new Error("Web standalone build not found. Run `make build` first.");
+        }
+        throw new Error(`Web standalone build is missing server.js under ${standaloneDir}. ` +
+            "This can happen when Next.js/Turbopack detects a different project root " +
+            "(e.g. a stray package-lock.json in a parent directory). " +
+            "Remove the stray lockfile and re-run `make build`.");
     }
     const webStandaloneDir = node_path_1.default.dirname(webServerPath);
     const webStaticDir = node_path_1.default.join(repoRoot, "apps", "web", ".next", "static");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kandev",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "private": false,
   "description": "NPX launcher for Kandev",
   "license": "AGPL-3.0-only",
@@ -24,7 +24,7 @@
     "vitest": "^1.6.0"
   },
   "scripts": {
-    "dev": "tsx src/cli.ts",
+    "dev": "unset npm_config_prefix && tsx src/cli.ts",
     "build": "tsc -p tsconfig.json",
     "start": "node dist/cli.js",
     "test": "vitest run",