@navels/neal 0.2.0 → 0.3.1

package/examples/compat/add-edit-verify/src/add.js

@@ -1,2 +1,2 @@
-// Trivial compat fixture: this addition helper is intentionally buggy.
+// Trivial compat fixture: an arithmetic helper exercised by test/add.test.js.
 export const add = (a, b) => a - b;

package/examples/compat/manifest.json

@@ -2,51 +2,47 @@
   "fixtures": [
     {
       "id": "add-edit-verify",
-      "roles": ["coder", "reviewer"],
+      "roles": [
+        "coder",
+        "reviewer"
+      ],
       "projectDir": "add-edit-verify",
       "planDoc": "add-edit-verify/PLAN.md",
       "verifyCommand": "node --test test/add.test.js",
-      "referenceFix": { "file": "src/add.js", "from": "a - b", "to": "a + b" },
-      "reviewer": {
-        "goodDiff": "add-edit-verify/good.diff",
-        "brokenDiff": "add-edit-verify/broken.diff"
-      }
-    },
-    {
-      "id": "reverse-grep-edit",
-      "roles": ["coder", "reviewer"],
-      "projectDir": "reverse-grep-edit",
-      "planDoc": "reverse-grep-edit/PLAN.md",
-      "verifyCommand": "node --test test/strings.test.js",
       "referenceFix": {
-        "file": "src/strings.js",
-        "from": "s.split('').join('')",
-        "to": "s.split('').reverse().join('')"
+        "file": "src/add.js",
+        "from": "a - b",
+        "to": "a + b"
       },
       "reviewer": {
-        "goodDiff": "reverse-grep-edit/good.diff",
-        "brokenDiff": "reverse-grep-edit/broken.diff"
+        "goodDiff": "add-edit-verify/good.diff",
+        "brokenDiff": "add-edit-verify/broken.diff"
       }
     },
     {
-      "id": "is-even-add-test",
-      "roles": ["coder", "reviewer"],
-      "projectDir": "is-even-add-test",
-      "planDoc": "is-even-add-test/PLAN.md",
-      "verifyCommand": "node --test test/is-even.test.js",
+      "id": "sum-grep-edit",
+      "roles": [
+        "coder",
+        "reviewer"
+      ],
+      "projectDir": "sum-grep-edit",
+      "planDoc": "sum-grep-edit/PLAN.md",
+      "verifyCommand": "node --test test/numbers.test.js",
       "referenceFix": {
-        "file": "test/is-even.test.js",
-        "from": "assert.ok(false, 'TODO: replace this placeholder with a real test for isEven');",
-        "to": "assert.equal(isEven(4), true);"
+        "file": "src/numbers.js",
+        "from": "(total, n) => total",
+        "to": "(total, n) => total + n"
       },
       "reviewer": {
-        "goodDiff": "is-even-add-test/good.diff",
-        "brokenDiff": "is-even-add-test/broken.diff"
+        "goodDiff": "sum-grep-edit/good.diff",
+        "brokenDiff": "sum-grep-edit/broken.diff"
       }
     },
     {
       "id": "plan-greeting",
-      "roles": ["planner"],
+      "roles": [
+        "planner"
+      ],
       "projectDir": "plan-greeting",
       "issuePrompt": "plan-greeting/ISSUE.md",
       "verifyCommand": "node --test test/greet.test.js",

package/examples/compat/sum-grep-edit/PLAN.md

@@ -0,0 +1,33 @@
+# Compat fixture: fix the number helper that never adds
+
+> Fixture note: this plan describes the pre-fix state and is intentionally not updated after the fix lands.
+
+## Execution Shape
+
+executionShape: one_shot
+
+## Objective
+
+`src/numbers.js` exports several small number helpers. One of them is faulty:
+`sumAll(ns)` is supposed to return the total of the list, but it always returns
+`0`. The `sumAll` test in `test/numbers.test.js` fails because of this.
+
+Read `src/numbers.js`, locate the faulty `sumAll` helper, and make the smallest
+complete change so it returns the sum of the list. Do not touch the other
+helpers, which already pass their tests.
+
+## Boundaries
+
+Allowed paths:
+
+- `src/numbers.js`
+
+Forbidden:
+
+- Do not edit the test file.
+- Do not change the other (already-correct) helpers.
+- Do not add dependencies or new files.
+
+## Verification
+
+Run `node --test test/numbers.test.js`; it must exit `0`.

package/examples/compat/sum-grep-edit/broken.diff

@@ -0,0 +1,12 @@
+diff --git a/src/numbers.js b/src/numbers.js
+index e6677cc..6a5102c 100644
+--- a/src/numbers.js
++++ b/src/numbers.js
+@@ -4,6 +4,6 @@ export const double = (n) => n * 2;
+ 
+ export const negate = (n) => -n;
+ 
+-export const sumAll = (ns) => ns.reduce((total, n) => total, 0);
++export const sumAll = (ns) => ns.reduce((total, n) => total + 1, 0);
+ 
+ export const countOf = (ns) => ns.length;

package/examples/compat/sum-grep-edit/good.diff

@@ -0,0 +1,12 @@
+diff --git a/src/numbers.js b/src/numbers.js
+index e6677cc..89f58ba 100644
+--- a/src/numbers.js
++++ b/src/numbers.js
+@@ -4,6 +4,6 @@ export const double = (n) => n * 2;
+ 
+ export const negate = (n) => -n;
+ 
+-export const sumAll = (ns) => ns.reduce((total, n) => total, 0);
++export const sumAll = (ns) => ns.reduce((total, n) => total + n, 0);
+ 
+ export const countOf = (ns) => ns.length;

package/examples/compat/sum-grep-edit/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "@navels/neal-compat-sum-grep-edit",
+  "private": true,
+  "type": "module"
+}

package/examples/compat/sum-grep-edit/src/numbers.js

@@ -0,0 +1,9 @@
+// Trivial compat fixture: small number helpers. test/numbers.test.js is the
+// source of truth for their behavior.
+export const double = (n) => n * 2;
+
+export const negate = (n) => -n;
+
+export const sumAll = (ns) => ns.reduce((total, n) => total, 0);
+
+export const countOf = (ns) => ns.length;

package/examples/compat/sum-grep-edit/test/numbers.test.js

@@ -0,0 +1,15 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+
+import { countOf, double, negate, sumAll } from '../src/numbers.js';
+
+test('double/negate/countOf helpers are correct', () => {
+  assert.equal(double(4), 8);
+  assert.equal(negate(3), -3);
+  assert.equal(countOf([1, 2, 3]), 3);
+});
+
+test('sumAll returns the total of the list', () => {
+  assert.equal(sumAll([1, 2, 3]), 6);
+  assert.equal(sumAll([]), 0);
+});

package/examples/issue-triage-js/README.md

@@ -59,18 +59,3 @@ guarantee that a live provider-backed run will complete successfully.
 
 Normal repository CI validates this example through non-live tests and package
 checks without running providers.
-
-## Manual Canary Workflow
-
-Maintainers can dispatch the `Example Canary` GitHub Actions workflow when they
-want a live provider-backed smoke test for this example. The workflow is
-manual-only, uses the `example-canary` environment, and requires the
-environment to provide `OPENAI_API_KEY`, `ANTHROPIC_API_KEY`, or both depending
-on the selected coder and reviewer providers.
-
-The canary builds Neal, writes a temporary `~/.neal/config.yml` from the
-selected provider roles, runs the example baseline tests, runs
-`node ../../dist/neal/index.js run PLAN.md --no-squash` from this directory,
-and uploads only curated markdown support artifacts from the example-local
-`.neal/` directory. Scheduled canaries are deferred until cost and flakiness are
-understood from intentional manual runs.

package/examples/reviewer-eval/clean-doc/base/src/clamp.ts

@@ -0,0 +1,3 @@
+export function clamp(value: number, lo: number, hi: number): number {
+  return Math.max(lo, Math.min(hi, value));
+}

package/examples/reviewer-eval/clean-doc/change.diff

@@ -0,0 +1,9 @@
+diff --git a/src/clamp.ts b/src/clamp.ts
+index 58793c4..c7a624d 100644
+--- a/src/clamp.ts
++++ b/src/clamp.ts
+@@ -1,3 +1,4 @@
++/** Restrict `value` to the inclusive range [lo, hi]. */
+ export function clamp(value: number, lo: number, hi: number): number {
+   return Math.max(lo, Math.min(hi, value));
+ }

package/examples/reviewer-eval/clean-extract/base/src/greet.ts

@@ -0,0 +1,3 @@
+export function greet(name: string): string {
+  return 'Hello, ' + name.trim() + '!';
+}

package/examples/reviewer-eval/clean-extract/change.diff

@@ -0,0 +1,13 @@
+diff --git a/src/greet.ts b/src/greet.ts
+index 1c1e070..b768208 100644
+--- a/src/greet.ts
++++ b/src/greet.ts
+@@ -1,3 +1,7 @@
++function normalize(name: string): string {
++  return name.trim();
++}
++
+ export function greet(name: string): string {
+-  return 'Hello, ' + name.trim() + '!';
++  return 'Hello, ' + normalize(name) + '!';
+ }

package/examples/reviewer-eval/dropped-error/base/src/load.ts

@@ -0,0 +1,4 @@
+export async function loadConfig(read: () => Promise<string>): Promise<string> {
+  const raw = await read();
+  return raw.trim();
+}

package/examples/reviewer-eval/dropped-error/change.diff

@@ -0,0 +1,15 @@
+diff --git a/src/load.ts b/src/load.ts
+index 3da1e73..d7835c0 100644
+--- a/src/load.ts
++++ b/src/load.ts
+@@ -1,4 +1,8 @@
+ export async function loadConfig(read: () => Promise<string>): Promise<string> {
+-  const raw = await read();
+-  return raw.trim();
++  try {
++    const raw = await read();
++    return raw.trim();
++  } catch {
++    return '';
++  }
+ }

package/examples/reviewer-eval/manifest.json

@@ -0,0 +1,45 @@
+{
+  "description": "Labeled reviewer-recall eval fixtures. Each fixture is a small before-tree (base/) plus a change.diff; defective fixtures plant a known defect in a labeled file, clean fixtures are behavior-preserving. scripts/eval-reviewer.mjs applies each diff in a throwaway repo, runs the configured reviewer, and scores its findings against these labels. This is a starter set — add fixtures by following the same shape.",
+  "fixtures": [
+    {
+      "id": "off-by-one",
+      "kind": "defective",
+      "diff": "off-by-one/change.diff",
+      "baseDir": "off-by-one/base",
+      "expectedFindings": [
+        {
+          "file": "src/paginate.ts",
+          "defectClass": "off-by-one",
+          "description": "The page slice end is start + size + 1, so each page overruns into the next page's first item."
+        }
+      ]
+    },
+    {
+      "id": "dropped-error",
+      "kind": "defective",
+      "diff": "dropped-error/change.diff",
+      "baseDir": "dropped-error/base",
+      "expectedFindings": [
+        {
+          "file": "src/load.ts",
+          "defectClass": "swallowed-error",
+          "description": "A bare catch turns any read failure into an empty config string, hiding the error from the caller."
+        }
+      ]
+    },
+    {
+      "id": "clean-extract",
+      "kind": "clean",
+      "diff": "clean-extract/change.diff",
+      "baseDir": "clean-extract/base",
+      "expectedFindings": []
+    },
+    {
+      "id": "clean-doc",
+      "kind": "clean",
+      "diff": "clean-doc/change.diff",
+      "baseDir": "clean-doc/base",
+      "expectedFindings": []
+    }
+  ]
+}

package/examples/reviewer-eval/off-by-one/base/src/paginate.ts

@@ -0,0 +1,5 @@
+export function pageSlice<T>(items: T[], page: number, size: number): T[] {
+  const start = page * size;
+  const end = start + size;
+  return items.slice(start, end);
+}

package/examples/reviewer-eval/off-by-one/change.diff

@@ -0,0 +1,11 @@
+diff --git a/src/paginate.ts b/src/paginate.ts
+index d122c4f..b9b0f92 100644
+--- a/src/paginate.ts
++++ b/src/paginate.ts
+@@ -1,5 +1,5 @@
+ export function pageSlice<T>(items: T[], page: number, size: number): T[] {
+   const start = page * size;
+-  const end = start + size;
++  const end = start + size + 1;
+   return items.slice(start, end);
+ }

package/{config.yml → neal.yml}

@@ -57,7 +57,7 @@
 #
 #     # Or run the coder against any OpenAI-compatible tool-calling endpoint
 #     # via the providers.openai_compatible block (see docs/providers.md):
-#     # provider: generic-agentic
+#     # provider: openai-compatible
 #
 #     # Optional default model override for the coder role.
 #     # Leave null to let the provider choose its default model.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@navels/neal",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "description": "A source-first multi-agent CLI for planning, executing, reviewing, and resuming scoped code changes.",
   "license": "MIT",
   "publishConfig": {
@@ -8,7 +8,7 @@
   },
   "author": "Lee Nave",
   "type": "module",
-  "packageManager": "pnpm@11.12.0",
+  "packageManager": "pnpm@11.13.1",
   "homepage": "https://github.com/navels/neal#readme",
   "repository": {
     "type": "git",
@@ -35,16 +35,16 @@
     "README.md",
     "SECURITY.md",
     "LICENSE",
-    "config.yml",
+    "neal.yml",
     "docs",
     "examples"
   ],
   "engines": {
-    "node": ">=22.13",
-    "pnpm": ">=11"
+    "node": ">=24.18.0",
+    "pnpm": ">=11.13.1"
   },
   "scripts": {
-    "build": "rm -rf dist && tsc -p tsconfig.json && chmod +x dist/neal/index.js",
+    "build": "rm -rf dist && node node_modules/typescript-7/bin/tsc -p tsconfig.json && chmod +x dist/neal/index.js",
     "pack:local": "pnpm build && pnpm pack",
     "verify:package": "pnpm build && node scripts/verify-package.mjs",
     "validate:release": "node scripts/validate-release.mjs",
@@ -52,23 +52,24 @@
     "lint": "eslint src test",
     "start": "tsx src/neal/index.ts",
     "test": "NEAL_NOTIFY_BIN= node --import tsx --test test/*.test.ts",
-    "typecheck": "tsc --noEmit -p tsconfig.json && tsc -p tsconfig.test.json"
+    "typecheck": "node node_modules/typescript-7/bin/tsc --noEmit -p tsconfig.json && node node_modules/typescript-7/bin/tsc -p tsconfig.test.json"
   },
   "dependencies": {
-    "@ai-sdk/openai-compatible": "2.0.49",
-    "@anthropic-ai/claude-agent-sdk": "0.3.163",
-    "@openai/codex-sdk": "0.144.1",
-    "ai": "6.0.202",
+    "@ai-sdk/openai-compatible": "3.0.12",
+    "@anthropic-ai/claude-agent-sdk": "0.3.218",
+    "@openai/codex-sdk": "0.145.0",
+    "ai": "7.0.31",
     "dotenv": "^17.4.2",
-    "yaml": "^2.8.3",
+    "yaml": "^2.9.0",
     "zod": "4.4.3"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
-    "@types/node": "^22.19.19",
+    "@types/node": "^24.13.3",
     "eslint": "^10.7.0",
-    "tsx": "^4.21.0",
+    "tsx": "^4.23.1",
     "typescript": "^6.0.3",
+    "typescript-7": "npm:typescript@^7.0.2",
     "typescript-eslint": "^8.64.0"
   }
 }