@osovv/vv-opencode 1.1.5 → 1.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@osovv/vv-opencode",
-  "version": "1.1.5",
+  "version": "1.2.0",
   "description": "A curated, opinionated set of OpenCode plugins for spec-first, review-driven, safer agentic development.",
   "type": "module",
   "main": "./dist/index.js",
@@ -69,6 +69,10 @@
     "./plugins/secrets-redaction": {
       "types": "./dist/plugins/secrets-redaction/index.d.ts",
       "import": "./dist/plugins/secrets-redaction/index.js"
+    },
+    "./plugins/web-tools": {
+      "types": "./dist/plugins/web-tools/index.d.ts",
+      "import": "./dist/plugins/web-tools/index.js"
     }
   },
   "scripts": {
@@ -80,7 +84,7 @@
     "fmt:check": "oxfmt --check src",
     "test": "bun test",
     "check": "bun run typecheck && bun run lint && bun run fmt:check && bun test",
-    "pack:check": "bun run build && bun -e \"const root = await import('./dist/index.js'); if (!('GuardianPlugin' in root)) throw new Error('dist root export missing GuardianPlugin'); if (!('HashlineEditPlugin' in root)) throw new Error('dist root export missing HashlineEditPlugin'); if (!('ModelRolesPlugin' in root)) throw new Error('dist root export missing ModelRolesPlugin'); if (!('SystemContextInjectionPlugin' in root)) throw new Error('dist root export missing SystemContextInjectionPlugin'); if (!('WorkflowPlugin' in root)) throw new Error('dist root export missing WorkflowPlugin'); if (!('SecretsRedactionPlugin' in root)) throw new Error('dist root export missing SecretsRedactionPlugin'); const tui = await import('./dist/tui.js'); if (!tui.default || typeof tui.default.tui !== 'function') throw new Error('dist TUI export missing default tui module'); await import('./dist/plugins/guardian/index.js'); await import('./dist/plugins/hashline-edit/index.js'); await import('./dist/plugins/model-roles/index.js'); await import('./dist/plugins/system-context-injection/index.js'); await import('./dist/plugins/workflow/index.js'); await import('./dist/plugins/secrets-redaction/index.js')\" && npm pack --dry-run",
+    "pack:check": "bun run build && bun -e \"const root = await import('./dist/index.js'); if (!('GuardianPlugin' in root)) throw new Error('dist root export missing GuardianPlugin'); if (!('HashlineEditPlugin' in root)) throw new Error('dist root export missing HashlineEditPlugin'); if (!('ModelRolesPlugin' in root)) throw new Error('dist root export missing ModelRolesPlugin'); if (!('SystemContextInjectionPlugin' in root)) throw new Error('dist root export missing SystemContextInjectionPlugin'); if (!('WorkflowPlugin' in root)) throw new Error('dist root export missing WorkflowPlugin'); if (!('SecretsRedactionPlugin' in root)) throw new Error('dist root export missing SecretsRedactionPlugin'); if (!('WebToolsPlugin' in root)) throw new Error('dist root export missing WebToolsPlugin'); const tui = await import('./dist/tui.js'); if (!tui.default || typeof tui.default.tui !== 'function') throw new Error('dist TUI export missing default tui module'); await import('./dist/plugins/guardian/index.js'); await import('./dist/plugins/hashline-edit/index.js'); await import('./dist/plugins/model-roles/index.js'); await import('./dist/plugins/system-context-injection/index.js'); await import('./dist/plugins/workflow/index.js'); await import('./dist/plugins/secrets-redaction/index.js'); await import('./dist/plugins/web-tools/index.js')\" && npm pack --dry-run",
     "tui:local": "bun scripts/tui-local.ts",
     "release:check": "bun scripts/release-check.ts",
     "release:bump": "bun scripts/release-bump.ts",
@@ -92,22 +96,25 @@
     "@opencode-ai/sdk": "1.18.2",
     "ajv": "^8.17.1",
     "citty": "0.2.2",
-    "jsonc-parser": "3.3.1"
+    "htmlparser2": "^12.0.0",
+    "jsonc-parser": "3.3.1",
+    "turndown": "^7.2.4"
   },
   "devDependencies": {
-    "@types/node": "22.13.9",
+    "@commitlint/cli": "^19.0.0",
+    "@commitlint/config-conventional": "^19.0.0",
     "@opentui/core": "0.4.3",
     "@opentui/keymap": "0.4.3",
     "@opentui/solid": "0.4.3",
+    "@types/node": "22.13.9",
+    "@types/turndown": "^5.0.6",
     "bun-types": "1.3.11",
+    "conventional-changelog-cli": "^5.0.0",
     "lefthook": "2.1.5",
     "oxfmt": "0.43.0",
     "oxlint": "1.58.0",
     "solid-js": "1.9.12",
-    "typescript": "5.8.2",
-    "conventional-changelog-cli": "^5.0.0",
-    "@commitlint/cli": "^19.0.0",
-    "@commitlint/config-conventional": "^19.0.0"
+    "typescript": "5.8.2"
   },
   "peerDependencies": {
     "@opentui/core": ">=0.4.3",

package/schemas/vvoc/v3.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://json-schema.org/draft/2020-12/schema",
-  "$id": "https://cdn.jsdelivr.net/npm/@osovv/vv-opencode@1.1.5/schemas/vvoc/v3.json",
+  "$id": "https://cdn.jsdelivr.net/npm/@osovv/vv-opencode@1.2.0/schemas/vvoc/v3.json",
   "title": "vvoc config",
   "description": "Canonical vvoc configuration document.",
   "type": "object",
@@ -138,6 +138,48 @@
       "additionalProperties": {
         "type": "boolean"
       }
+    },
+    "web": {
+      "type": "object",
+      "additionalProperties": false,
+      "properties": {
+        "search": {
+          "type": "object",
+          "additionalProperties": false,
+          "properties": {
+            "provider": { "type": "string", "enum": ["exa", "brave", "zai"] },
+            "region": { "type": "string", "enum": ["international", "china"] },
+            "apiKey": { "type": "string", "minLength": 1 }
+          },
+          "allOf": [
+            {
+              "if": {
+                "properties": { "provider": { "const": "zai" } },
+                "required": ["provider"]
+              },
+              "then": { "required": ["region"] }
+            }
+          ]
+        },
+        "fetch": {
+          "type": "object",
+          "additionalProperties": false,
+          "properties": {
+            "provider": { "type": "string", "enum": ["native", "spider", "zai"] },
+            "region": { "type": "string", "enum": ["international", "china"] },
+            "apiKey": { "type": "string", "minLength": 1 }
+          },
+          "allOf": [
+            {
+              "if": {
+                "properties": { "provider": { "const": "zai" } },
+                "required": ["provider"]
+              },
+              "then": { "required": ["region"] }
+            }
+          ]
+        }
+      }
     }
   }
 }

package/templates/skills/vv-execute/SKILL.md

@@ -182,22 +182,17 @@ The implementer writes code, runs tests, and returns a status. This controller v
 <step name="handle-status">
   <case name="done">
     Implementer returned DONE. Use task-file to verify files exist. Run the test command from the plan (if specified). Verify each acceptance criterion.
-    Optionally dispatch vv-spec-reviewer to confirm contract compliance.
-    If verification fails: re-dispatch implementer with failure details.
-    If verification passes: proceed to close.
+    If verification fails: re-dispatch implementer with failure details (the work item is still awaiting implementation).
+    If verification passes: proceed to the review step.
   </case>
   <case name="done-with-concerns">
-    Read the concerns before proceeding. If concerns are about correctness or scope, address them by updating the packet and re-dispatching. If they are observations (e.g. "file is getting large"), note them and proceed with verification as DONE.
+    Read the concerns before proceeding. If concerns are about correctness or scope, address them by updating the packet and re-dispatching the implementer. If they are observations (e.g. "file is getting large"), note them and proceed with verification as DONE, then to the review step.
   </case>
   <case name="needs-context">
-    The implementer lacked context. Provide the missing information in a revised packet and re-dispatch the SAME implementer type. Do not force them to proceed without the missing context.
+    NEEDS_CONTEXT is a hard stop. The runtime will not allow another tracked launch for this work item, so do NOT re-dispatch the implementer. Surface the preserved context (work_item_list shows the captured excerpt) to the user and require an explicit recovery decision — typically open a fresh work item with the missing context supplied.
   </case>
   <case name="blocked">
-    The implementer cannot complete the task. Assess:
-    1. Context problem → provide more context, re-dispatch
-    2. Task too complex for chosen model → re-dispatch with smarter model
-    3. Plan is wrong → escalate to the user
-    Never force the same model to retry without changes. If the implementer said it is stuck, something needs to change.
+    BLOCKED is a hard stop. The runtime will not allow another tracked launch for this work item, so do NOT re-dispatch the implementer. Escalate to the user with the captured blocker context and ask for a decision (a different approach, a different model via a fresh work item, or a plan change).
   </case>
 </step>
 
@@ -207,10 +202,20 @@ Run the acceptance criteria. For each criterion:
 - Does the test pass?
 - Did the implementer miss any edge cases?
 
-If all criteria pass → proceed to commit.
+If all criteria pass → proceed to review.
 If criteria fail → re-dispatch implementer with specific failure details.
 </step>
 
+<step name="review">
+A DONE implementer moves the work item to awaiting_reviews with a review round that requires EVERY role in requiredReviewers (spec and code). Dispatch and collect ALL required reviewers before closing:
+- Dispatch vv-spec-reviewer with the VVOC_WORK_ITEM_ID header and a spec-compliance packet.
+- Dispatch vv-code-reviewer with the VVOC_WORK_ITEM_ID header and the changed code/diff.
+Collect both results.
+- Both PASS → the work item becomes ready_to_close → proceed to commit.
+- Any FAIL → the work item returns to awaiting_implementer. Re-dispatch the implementer with the normalized reviewer findings, then repeat handle-status → verify → review (bounded by the runtime review-round limit).
+A reviewer returning NEEDS_CONTEXT is a hard stop: surface it to the user instead of re-dispatching.
+</step>
+
 <step name="commit">
 After all acceptance criteria pass, commit the task's changes to git.
 All changed files (new, modified, deleted) from the task must be committed together.
@@ -236,7 +241,7 @@ If the commit fails (e.g. nothing to commit, hook rejection), report the failure
 </step>
 
 <step name="close">
-The task's changes are already committed. Mark the task complete in TodoWrite. Close the work item with work_item_close.
+The task's changes are committed and all required reviewers passed, so the work item is ready_to_close. Mark the task complete in TodoWrite. Close the work item with work_item_close.
 If all tasks are done → proceed to completion.
 Otherwise → move to the next task in dependency order.
 </step>