@cardor/agent-harness-kit 1.10.1 → 1.10.2

package/README.md

@@ -142,7 +142,7 @@ Then run the interactive setup inside your project:
 npx ahk init
 ```
 
-> **Local install required.** `ahk` needs to resolve your project's `agent-harness-kit.config.ts` and its dependencies relative to your project's own `node_modules`. A **global-only** install (`npm install -g @cardor/agent-harness-kit`) cannot do this reliably, so every command except `--version`/`--help` will detect a global-only install and exit with an error telling you to run `npm install --save-dev @cardor/agent-harness-kit` (or the `pnpm`/`yarn`/`bun` equivalent) in your project root.
+> **Local install recommended.** The generated `agent-harness-kit.config.ts` uses `import type`, which is erased at compile time, so `ahk` no longer needs to resolve the package from your project's `node_modules` at runtime. If it detects a **global-only** install (`npm install -g @cardor/agent-harness-kit`), it prints a non-blocking warning recommending `npm install --save-dev @cardor/agent-harness-kit` (or the `pnpm`/`yarn`/`bun` equivalent) — the command still runs and exits normally either way. This is only about keeping the CLI version pinned and reproducible across your team and CI, not a functional requirement.
 >
 > This check also works with **Yarn Berry (PnP)** projects, which never create a `node_modules` folder — `ahk` detects `.pnp.cjs`/`.pnp.loader.mjs` and falls back to checking that the package is declared in `package.json` instead of requiring a `node_modules` entry.
 
@@ -522,9 +522,9 @@ The `tasks` table includes an `updated_at` timestamp column, set on creation and
 Everything in the config file is yours to change:
 
 ```ts
-import { defineHarness } from '@cardor/agent-harness-kit'
+import type { HarnessConfig } from '@cardor/agent-harness-kit'
 
-export default defineHarness({
+const config: HarnessConfig = {
   project: {
     name: 'My App',
     description: 'What this project does',
@@ -578,9 +578,13 @@ export default defineHarness({
     mcp: { enabled: true, port: 3742 },
     scripts: { enabled: true, outputDir: './.harness/scripts' },
   },
-})
+}
+
+export default config
 ```
 
+> `defineHarness()` is still exported for anyone who prefers the value-import form (`import { defineHarness } from '@cardor/agent-harness-kit'` + `export default defineHarness({ ... })`) — it's an identity function kept for backward compatibility, and `loadConfig()` supports both shapes.
+
 ### `health.sh`
 
 This is the most important file to implement. Agents will not start or close tasks until this script exits 0. Examples:

package/dist/cli.js

@@ -530,12 +530,9 @@ If orchestrating: Agent definition files in .claude/agents/
 function modelField(model) {
   return model ? `, model: ${JSON.stringify(model)}` : "";
 }
-function configTs(params) {
+function configObjectBody(params) {
   const models = params.models ?? {};
-  return `import { defineHarness } from '@cardor/agent-harness-kit'
-
-export default defineHarness({
-  project: {
+  return `  project: {
     name: ${JSON.stringify(params.name)},
     description: ${JSON.stringify(params.description)},
     docsPath: '${params.docsPath}',
@@ -583,64 +580,29 @@ export default defineHarness({
     mcp:     { enabled: true, port: ${params.port} },
     scripts: { enabled: true, outputDir: './.harness/scripts' },
   },
-})
 `;
 }
-var configMjs = configTs;
-function configCjs(params) {
-  const models = params.models ?? {};
-  return `const { defineHarness } = require('@cardor/agent-harness-kit')
-
-module.exports = defineHarness({
-  project: {
-    name: ${JSON.stringify(params.name)},
-    description: ${JSON.stringify(params.description)},
-    docsPath: '${params.docsPath}',
-  },
-
-  provider: '${params.provider}',
-
-  agents: {
-    lead:     { instructionsPath: null${modelField(models.lead)} },
-    explorer: { instructionsPath: null, allowedPaths: ['${params.docsPath}', './src']${modelField(models.explorer)} },
-    builder:  { instructionsPath: null, writablePaths: ['./src', './tests']${modelField(models.builder)} },
-    reviewer: { instructionsPath: null${modelField(models.reviewer)} },
-    ${models.consultant ? `consultant: { instructionsPath: null${modelField(models.consultant)} },
-    ` : ""}custom:   [],
-  },
+function configTs(params) {
+  return `import type { HarnessConfig } from '@cardor/agent-harness-kit'
 
-  // SQLite (default). Switch to postgres/mysql by changing database.type.
-  // database: { type: 'postgres', connectionString: process.env.DATABASE_URL },
-  // database: { type: 'mysql',    connectionString: process.env.DATABASE_URL },
-  database: { type: 'sqlite', path: '.harness/harness.db' },
+const config: HarnessConfig = {
+${configObjectBody(params)}}
 
-  storage: {
-    dir:    '.harness',
-    tasks:  { adapter: '${params.tasksAdapter}' },
-    sections: {
-      toolsUsed:     true,
-      filesModified: true,
-      result:        true,
-      blockers:      true,
-      nextSteps:     false,
-    },
-    markdownFallback: { enabled: true, path: '.harness/current.md' },
-    // 'local' \u2014 DB lives in .harness/ (project-relative). 'global' \u2014 DB lives
-    // under ~/.harness/dbs/<projectId>/, outside the project tree.
-    scope:     '${params.scope}',
-    projectId: '${params.projectId}',
-  },
+export default config
+`;
+}
+function configMjs(params) {
+  return `const config = {
+${configObjectBody(params)}}
 
-  health: {
-    scriptPath: './health.sh',
-    required:   true,
-  },
+export default config
+`;
+}
+function configCjs(params) {
+  return `const config = {
+${configObjectBody(params)}}
 
-  tools: {
-    mcp:     { enabled: true, port: ${params.port} },
-    scripts: { enabled: true, outputDir: './.harness/scripts' },
-  },
-})
+module.exports = config
 `;
 }
 function agentLead(vars) {
@@ -3750,7 +3712,10 @@ function isLocalInstallSatisfied(cwd2) {
   return false;
 }
 function printLocalInstallWarning() {
-  console.error(pc17.red(`\u2717 ${pkg.name} must be installed locally in this project.`));
+  console.error(pc17.yellow(`\u26A0 ${pkg.name} is not installed locally in this project.`));
+  console.error(pc17.dim("  This is only a recommendation for reproducibility: pinning a local"));
+  console.error(pc17.dim("  version keeps behavior consistent across your team and CI, instead of"));
+  console.error(pc17.dim("  drifting with whatever version is installed globally on each machine."));
   console.error(pc17.dim(`  Run: npm install --save-dev ${pkg.name}`));
   console.error(pc17.dim("  (or the equivalent for your package manager: pnpm add -D, yarn add --dev, bun add -d)"));
 }
@@ -3853,7 +3818,6 @@ program.command("doctor").description("Check lib version, agent files, and harne
 program.hook("preAction", () => {
   if (!isLocalInstallSatisfied(cwd)) {
     printLocalInstallWarning();
-    process.exit(1);
   }
 });
 program.hook("postAction", async () => {