@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
|
+
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.
|
|
@@ -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.
|
|
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
|
|
239
|
-
immutable
|
|
240
|
-
|
|
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.
|
|
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.
|
|
37
|
+
"@open-agent-toolkit/control-plane": "0.2.69"
|
|
38
38
|
},
|
|
39
39
|
"devDependencies": {
|
|
40
40
|
"@types/node": "^22.10.0",
|