fix: externalize packages in CLI bundle (#47)

Author: V3RON

package.json

@@ -43,7 +43,7 @@
   "scripts": {
     "build": "npm run build:runtime && npm run build:declarations",
     "build:declarations": "tsc -p ./tsconfig.build.json",
-    "build:runtime": "esbuild ./src/index.ts ./src/cli.ts --bundle --platform=node --format=esm --target=node22 --outdir=./dist",
+    "build:runtime": "esbuild ./src/index.ts ./src/cli.ts --bundle --packages=external --platform=node --format=esm --target=node22 --outdir=./dist",
     "fmt": "oxfmt .",
     "fmt:check": "oxfmt --check .",
     "prepare": "husky",
@@ -52,7 +52,7 @@
     "skillgym:help": "tsx ./index.ts help",
     "skillgym:example:basic": "tsx ./index.ts run ./examples/basic-suite.ts",
     "skillgym:example:skill-selection": "tsx ./index.ts run ./examples/skill-selection-suite.ts",
-    "test": "vitest run",
+    "test": "npm run build:runtime && vitest run",
     "typecheck": "tsc -p ./tsconfig.json --noEmit"
   },
   "dependencies": {

test/cli.test.ts

@@ -7,7 +7,6 @@ import { execFileCapture } from "../src/utils/process.js";
 import { accumulateTagOptionValues, parseTagOption } from "../src/cli/tag-options.js";
 
 const repoRoot = process.cwd();
-const tsxLoaderPath = path.join(repoRoot, "node_modules", "tsx", "dist", "loader.mjs");
 const tempDirs: string[] = [];
 
 afterEach(async () => {
@@ -880,12 +879,8 @@ test("cli run passes retryFailed through to execution", async () => {
 });
 
 async function execCli(args: string[], cwd = repoRoot) {
-  return execFileCapture(
-    process.execPath,
-    ["--import", tsxLoaderPath, path.join(repoRoot, "index.ts"), ...args],
-    {
-      cwd,
-      timeoutMs: 30_000,
-    },
-  );
+  return execFileCapture(process.execPath, [path.join(repoRoot, "bin.js"), ...args], {
+    cwd,
+    timeoutMs: 30_000,
+  });
 }

test/runner/execute-suite.reporter.test.ts

@@ -1483,6 +1483,8 @@ test("executeSuite raises process max listeners for parallel runs and restores i
           passed: true,
           durationMs: 10,
           executionArtifactDir: options.artifactDir,
+          outputTokens: 0,
+          observedReads: 0,
         });
       },
     });

test/runner/model-rejection.test.ts

@@ -102,6 +102,7 @@ async function createResultWithLogs(options: {
     passed: false,
     status: "failed",
     durationMs: 1,
+    executionArtifactDir: artifactDir,
     artifactDir,
     error: {
       name: "Error",