@open-agent-toolkit/cli 0.2.68 → 0.2.69

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -1,6 +1,6 @@
1
1
  ---
2
2
  name: oat-reviewer
3
- version: 1.2.3
3
+ version: 1.2.4
4
4
  description: Unified reviewer for OAT projects - mode-aware verification of requirements/design alignment and code quality. Writes a review artifact to disk by default, or returns structured findings in-memory when dispatched in structured-output mode.
5
5
  tools: Read, Bash, Grep, Glob, Write, Task
6
6
  color: yellow
@@ -235,6 +235,25 @@ Treat the artifact as a product deliverable. Verify it is:
235
235
  - Coverage of design/discovery: every in-scope design component or discovery decision is mapped to at least one task or explicitly deferred/out of scope.
236
236
  - Parallelism-claim sanity: any parallel phase group or parallelism statement is consistent with declared file boundaries and dependency order.
237
237
 
238
+ 6. **Necessary**
239
+ - Evaluate whether the proposed approach materially serves the requested
240
+ outcome. For substantial added machinery — a script, schema, persisted
241
+ record, gate, provider seam, or new document that someone will have to
242
+ own — identify its consumer (an agent acting on a named instruction, a
243
+ human reading a named surface, or code at a named call site) and consider
244
+ whether a smaller approach preserves the requirements.
245
+ - Judge prose by the behavior it meaningfully guides. An instruction whose
246
+ consumer is the agent that reads it is a real consumer; missing automated
247
+ enforcement of that instruction is not proof of uselessness.
248
+ - A mechanism whose only consumer is its own tests, its own plan, or a
249
+ deferred future reader is grounds to investigate, not an automatic
250
+ severity: grade the finding by wasted effort, maintenance burden, or
251
+ failure to meet the requested outcome, and say which.
252
+ - Report justified concerns in the existing findings sections. Do not
253
+ produce an inventory of unproblematic mechanisms, and do not override an
254
+ explicitly stated user requirement with a preference for simplicity;
255
+ name the conflict instead.
256
+
238
257
  ### Step 5: Verify Design Alignment
239
258
 
240
259
  This step applies to **code reviews** only.
@@ -1,4 +1,4 @@
1
1
  {
2
2
  "schemaVersion": 1,
3
- "oatVersion": "0.2.68"
3
+ "oatVersion": "0.2.69"
4
4
  }
@@ -1,6 +1,6 @@
1
1
  {
2
- "cli": "0.2.68",
3
- "docs-config": "0.2.68",
4
- "docs-theme": "0.2.68",
5
- "docs-transforms": "0.2.68"
2
+ "cli": "0.2.69",
3
+ "docs-config": "0.2.69",
4
+ "docs-theme": "0.2.69",
5
+ "docs-transforms": "0.2.69"
6
6
  }
@@ -6,7 +6,7 @@ disable-model-invocation: false
6
6
  user-invocable: true
7
7
  allowed-tools: Read, Write, Edit, Bash, Grep, Glob, Task
8
8
  metadata:
9
- version: 1.9.2
9
+ version: 1.9.3
10
10
  ---
11
11
 
12
12
  # Execute a Wave of External Plans
@@ -235,9 +235,18 @@ writes and stay on the orchestrator's own model class, the same rule
235
235
 
236
236
  Run the cross-runtime artifact gate with a **bounded** prompt (rule 6): review the
237
237
  wrapper artifacts for plan invariants, contract consistency, frontmatter validity,
238
- and whether any task restates/narrows its source plan — the external plans are
239
- immutable inputs, NOT review targets. Disposition findings in-artifact
240
- (gate-invoked artifact review) and commit. A plan gate MAY PROCEED at
238
+ and whether any task restates/narrows its source plan. The external plans'
239
+ content is immutable to the wave (the gate never rewrites a plan), but a plan is
240
+ a legitimate target for one question: whether its proposed mechanism materially
241
+ serves the requested outcome. A reviewer who finds substantial machinery with no
242
+ identifiable consumer — an agent acting on a named instruction, a human reading
243
+ a named surface, or code at a named call site — or a smaller approach that
244
+ preserves the requirements, reports it as a necessity finding. The root returns
245
+ that plan to its author (`oat-repo-improve`) for disposition and defers it from
246
+ this wave until dispositioned; it never silently drops or narrows it, and an
247
+ explicitly stated user requirement is honored over a preference for simplicity.
248
+ Disposition the remaining findings in-artifact (gate-invoked artifact review)
249
+ and commit. A plan gate MAY PROCEED at
241
250
  `fixes_completed` per the wave-0/1 precedent, but that is a proceed point, not a
242
251
  terminal state. Every gate row MUST flip to `passed` once all fix dispositions
243
252
  carry the stored verification records required by the fix-disposition contract below;
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@open-agent-toolkit/cli",
3
- "version": "0.2.68",
3
+ "version": "0.2.69",
4
4
  "private": false,
5
5
  "description": "Open Agent Toolkit CLI",
6
6
  "homepage": "https://github.com/voxmedia/open-agent-toolkit/tree/main/packages/cli",
@@ -34,7 +34,7 @@
34
34
  "ora": "^9.0.0",
35
35
  "yaml": "2.8.2",
36
36
  "zod": "^3.25.76",
37
- "@open-agent-toolkit/control-plane": "0.2.68"
37
+ "@open-agent-toolkit/control-plane": "0.2.69"
38
38
  },
39
39
  "devDependencies": {
40
40
  "@types/node": "^22.10.0",