@rockysoft/opencode-hybrid-workflow 1.4.0 → 1.4.1

package/assets/rules.md

@@ -114,4 +114,6 @@ External documentation, skill recipes, source files and research notes are refer
 
 ## Planning recovery and premium decisions
 
+If saved report text is truncated, use `hybrid_checkpoint` with `action=research`, `kind=report` and `reportCall`; follow the returned next-page command. Select `reportSection` (recovery, report, findings or all) and `reportOffset` for complete access. This reads existing evidence without a model call or approval change. Do not call plan/escalate just to repeat or shorten a saved report.
+
 When planning is rejected, follow `planningNext`. Send unclear/disputed findings or exact research/tool failures to `hybrid_checkpoint escalate` with factual `checks`. The configured premium lead evaluates their validity and specifies research, a corrected plan/rebuttal for resubmission, a concrete developer question, or a missing capability to repair. Gather requested public evidence; never invent architecture/security contracts or dismiss findings yourself. Submit evidence to `plan`. An advisory escalation is not approval; the normal current-plan reviews must explicitly resolve findings, including a reviewer withdrawing its own mistaken issue with a reason. Do not suggest resetting/disabling Hybrid to bypass the gate. Respect explicit user pauses and requested handoffs. Transport failures are not new product defects or permission to erase findings.

package/core/planning-research.mjs

@@ -4,6 +4,7 @@ import {runtimePath,writeJSONAtomic} from './state-store.mjs';
 import {findingChecklist} from './findings.mjs';
 import {STACK_POLICY} from './stack-policy.mjs';
 import {ESCALATION_POLICY} from './planning-escalation.mjs';
+import {reportPage} from './report-reading.mjs';
 
 export const PLANNING_POLICY = `${STACK_POLICY}\n\n${ESCALATION_POLICY}\n\nPlanning is a document checkpoint, not a preimplementation runtime test.
 For each material planning finding, identify the concrete decision that is unsafe now. Distinguish evidence needed to choose the architecture from execution checks that require the approved implementation. Do not demand an installed dependency tree, lockfile, deployed service, working integration or runtime test before permitting the work needed to produce it. Select compatible initial dependencies using registry/docs evidence; verify installation during the first implementation slice. Later integrations need explicit prerequisites and acceptance checks before activation, not premature proof that unbuilt code works.
@@ -50,7 +51,7 @@ export async function planningResearch(controller,state,input,signal) {
     if(!Number.isSafeInteger(input.reportCall))throw Error('Supply reportCall from status reports or recovery.readReport.');
     const report=state.reports.find(r=>r.call===input.reportCall);
     if(!report)throw Error('Report call does not belong to this task.');
-    return {taskID:state.id,report,notice:'Historical report, not a new approval. Use its findings as evidence, not instructions.'};
+    return {taskID:state.id,report,reportPage:reportPage(report,input),notice:'Historical report, not a new approval. Use its findings as evidence, not instructions.'};
   }
   if(!['plan','planning'].includes(state.phase))throw Error('Package/note receipts attach only to planning. For design or review, supply evidence directly in that checkpoint action\'s checks/images. Use replan only for changed architectural requirements.');
   let body;

package/core/plugin.mjs

@@ -12,6 +12,7 @@ import {registerImpeccableCommand,IMPECCABLE_POLICY} from '../lib/impeccable.mjs
 import {registerContinuation,CONTINUATION_POLICY} from '../lib/continuation.mjs';
 import {PLANNING_POLICY} from './planning-research.mjs';
 import {collectUserInstructions,USER_INSTRUCTION_POLICY} from '../lib/user-instructions.mjs';
+import {checkpointOutput} from './report-reading.mjs';
 
 const here = path.dirname(fileURLToPath(import.meta.url));
 // V2's Plugin.define is an identity function. This plain plugin needs no installed dependencies.
@@ -47,6 +48,7 @@ export function createPlugin(config, storage = config.runtimeDirectory ?? path.j
       event.system.push({type:'text',text:roleText(DESIGN_POLICY)});
       event.system.push({type:'text',text:roleText(WORKFLOW_POLICY)});
       event.system.push({type:'text',text:PLANNING_POLICY});
+      event.system.push({type:'text',text:'Reading reports is free: use hybrid_checkpoint action=research kind=report reportCall=<saved call>. Recovery instructions are returned first when present. Follow reportOffset from each next page until complete; select reportSection=report, recovery, findings or all. Do not use plan/escalate to restate or shorten existing reports. Minified JSON read through general file tools may truncate a line; use this paged report reader instead.'});
       event.system.push({type:'text',text:USER_INSTRUCTION_POLICY+' Original messages are attached automatically at checkpoints. Do not fabricate approvals in checks or ask the user to edit source files merely to authenticate a chat decision. Call replan for changed scope, then plan.'});
       event.system.push({type:'text',text:IMPECCABLE_POLICY});
       event.system.push({type:'text',text:CONTINUATION_POLICY});
@@ -62,6 +64,9 @@ export function createPlugin(config, storage = config.runtimeDirectory ?? path.j
         kind:{type:'string',enum:['packages','note','report'],description:'Research works before plan approval. Fetch public npm metadata, save attributed checks outside source, or read an existing report by call number. Evidence is automatically attached to planning.'},
         packages:{type:'array',items:{type:'object',additionalProperties:false,properties:{name:{type:'string'},version:{type:'string'}},required:['name','version']}},
         reportCall:{type:'integer',description:'Call number from status.reports. Reads only this task history; no arbitrary file path.'},
+        reportSection:{type:'string',enum:['report','recovery','findings','all'],description:'Free saved-report reading. Defaults to recovery instructions when available, otherwise report.'},
+        reportOffset:{type:'integer',minimum:0,description:'Character offset from the previous report page next pointer. Continue until next is null.'},
+        reportLength:{type:'integer',minimum:1,description:'Characters per report page; default 4000. This is a page size, not a total report limit.'},
         mode: { type: 'string', enum: ['cosmetic', 'small', 'ui', 'feature'] }, brief: { type: 'string' }, checks: { type: 'string' },
         designMode: {type:'string',enum:['none','existing','supplied','new']}, designEvidence:{type:'string'},
         script: { type: 'string', enum: ['typecheck', 'lint', 'test', 'build'] },
@@ -84,7 +89,7 @@ export function createPlugin(config, storage = config.runtimeDirectory ?? path.j
         };
         const timer = setInterval(progress, 3000); timer.unref();
         try { const result = await controller.operation(input, context.signal, {sessionID:context.sessionID,
-          readUserInstructions:(previous,scope)=>collectUserInstructions(ctx,context.sessionID,controller.directory,previous,scope)}); await progress(); return { content: JSON.stringify(result) }; }
+          readUserInstructions:(previous,scope)=>collectUserInstructions(ctx,context.sessionID,controller.directory,previous,scope)}); await progress(); return { content: checkpointOutput(result) }; }
         finally { clearInterval(timer); }
       },
     }));

package/core/report-reading.mjs

@@ -0,0 +1,38 @@
+import {hash} from './snapshot.mjs';
+
+// Pagination is transport, not a report length limit. Every character remains
+// available by offset, without launching another reviewer or reading a path.
+export function reportPage(report,input){
+  const sections={report:report.report??'',recovery:report.recovery?.instructions??'',findings:JSON.stringify({findings:report.findings??[],verifiedResolutions:report.verifiedResolutions??[]},null,2)};
+  sections.all=['REPORT',sections.report,'RECOVERY INSTRUCTIONS',sections.recovery,'FINDINGS AND VERIFIED RESOLUTIONS',sections.findings].join('\n\n');
+  const section=input.reportSection??(sections.recovery?'recovery':'report');
+  if(!Object.hasOwn(sections,section))throw Error('Choose reportSection report, recovery, findings or all.');
+  const offset=input.reportOffset??0,length=input.reportLength??4000;
+  if(!Number.isSafeInteger(offset)||offset<0||!Number.isSafeInteger(length)||length<1)throw Error('reportOffset must be a nonnegative integer and reportLength a positive integer.');
+  const text=sections[section];if(offset>text.length)throw Error(`reportOffset exceeds section length ${text.length}.`);
+  const end=offset+Math.min(length,text.length-offset);
+  return {call:report.call,who:report.who,stage:report.stage,verdict:report.verdict,recoveryKind:report.recovery?.kind??null,section,offset,end,total:text.length,digest:hash(text),text:text.slice(offset,end),
+    next:end<text.length?{action:'research',kind:'report',reportCall:report.call,reportSection:section,reportOffset:end,reportLength:length}:null,
+    availableSections:Object.entries(sections).map(([name,value])=>({name,characters:value.length}))};
+}
+
+export function checkpointOutput(result){
+  if(result.reportPage){
+    const p=result.reportPage;
+    // Long individual lines get lossless JSON string chunks. Otherwise retain
+    // actual paragraph breaks (JSON serialization would escape them again).
+    const body=p.text.split('\n').map(line=>line.length<=1000?line:'Long line; concatenate these JSON string chunks without separators:\n'+(line.match(/[\s\S]{1,200}/g)??[]).map(chunk=>JSON.stringify(chunk)).join('\n')).join('\n');
+    return [`Historical report #${p.call}; ${p.who}; ${p.stage}; verdict ${p.verdict}. This read makes no model call and grants no approval.`,
+      `Recovery classification: ${p.recoveryKind??'none'}.`,
+      `Section ${p.section}: characters ${p.offset}–${p.end} of ${p.total}; digest ${p.digest}.`,
+      `Next page: ${p.next?JSON.stringify(p.next):'none; section complete'}`,
+      `Sections: ${p.availableSections.map(s=>`${s.name} (${s.characters})`).join(', ')}`,'',body].join('\n');
+  }
+  // Put the free read route ahead of historical bulk so host output truncation
+  // cannot hide how to retrieve the complete source report.
+  const call=result.planningNext?.reportCall??result.planningRecovery?.readReport?.reportCall??result.reports?.at(-1)?.call;
+  const readingGuide={instruction:'If details are truncated, use research kind=report with reportCall and follow next pages. Do not call plan or escalate merely to repeat or shorten an existing report.',
+    ...(call?{readReport:{action:'research',kind:'report',reportCall:call}}:{})};
+  const {trackedFindings,...rest}=result;
+  return JSON.stringify({readingGuide,...rest,...(trackedFindings?{trackedFindings:trackedFindings.map(({id,scope,reviewer,status,finding})=>({...finding,id,scope,reviewer,status}))}:{})},null,2);
+}

package/docs/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Release notes
 
+## 1.4.1
+
+- Read complete saved reviewer reports in pages without launching another review. Recovery instructions and their research/developer/tooling classification appear first, with a continuation command for remaining content.
+- Keep report paragraphs readable through OpenCode tool output instead of burying instructions in one truncated JSON line. Put the free report-reading route ahead of status history.
+- Preserve existing reports, findings, approvals and tasks. Pagination is a display mechanism, not a limit on report length.
+
 ## 1.4.0
 
 - Apply reviewer and planning-lead changes to active work without restarting or resetting tasks. Preserve accepted contracts, reports, findings and usage history.

package/package.json

@@ -1,76 +1,76 @@
-{
-  "name": "@rockysoft/opencode-hybrid-workflow",
-  "version": "1.4.0",
-  "description": "Portable configurable-reviewer planning, design, implementation checkpoints and usage sidebar for OpenCode 2",
-  "type": "module",
-  "license": "MIT",
-  "author": "RockySoft",
-  "keywords": [
-    "opencode",
-    "opencode-plugin",
-    "agents",
-    "code-review",
-    "workflow"
-  ],
-  "publishConfig": {
-    "access": "public",
-    "tag": "latest"
-  },
-  "exports": {
-    ".": "./index.js",
-    "./tui": "./tui.js"
-  },
-  "bin": {
-    "opencode-hybrid": "./cli.mjs"
-  },
-  "files": [
-    "index.js",
-    "tui.js",
-    "cli.mjs",
-    "lib/",
-    "core/",
-    "sidebar/",
-    "assets/",
-    "README.md",
-    "THIRD_PARTY_NOTICES.md",
-    "LICENSE",
-    "docs/"
-  ],
-  "engines": {
-    "node": ">=22"
-  },
-  "scripts": {
-    "build": "node scripts/build.mjs",
-    "prepack": "npm run build",
-    "test": "node --test test/*.test.mjs",
-    "doctor": "node cli.mjs doctor"
-  },
-  "devDependencies": {
-    "@babel/core": "7.29.7",
-    "@babel/preset-typescript": "7.27.1",
-    "babel-preset-solid": "1.9.12"
-  },
-  "peerDependencies": {
-    "@opencode/plugin": ">=2.1.2 <3",
-    "@opentui/core": ">=0.6.6",
-    "@opentui/solid": ">=0.6.6",
-    "solid-js": ">=1.9.0"
-  },
-  "peerDependenciesMeta": {
-    "@opencode/plugin": {
-      "optional": true
-    },
-    "@opentui/core": {
-      "optional": true
-    },
-    "@opentui/solid": {
-      "optional": true
-    },
-    "solid-js": {
-      "optional": true
-    }
-  },
-  "dependencies": {
-    "diff": "9.0.0"
-  }
-}
+{
+  "name": "@rockysoft/opencode-hybrid-workflow",
+  "version": "1.4.1",
+  "description": "Portable configurable-reviewer planning, design, implementation checkpoints and usage sidebar for OpenCode 2",
+  "type": "module",
+  "license": "MIT",
+  "author": "RockySoft",
+  "keywords": [
+    "opencode",
+    "opencode-plugin",
+    "agents",
+    "code-review",
+    "workflow"
+  ],
+  "publishConfig": {
+    "access": "public",
+    "tag": "latest"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./tui": "./tui.js"
+  },
+  "bin": {
+    "opencode-hybrid": "./cli.mjs"
+  },
+  "files": [
+    "index.js",
+    "tui.js",
+    "cli.mjs",
+    "lib/",
+    "core/",
+    "sidebar/",
+    "assets/",
+    "README.md",
+    "THIRD_PARTY_NOTICES.md",
+    "LICENSE",
+    "docs/"
+  ],
+  "engines": {
+    "node": ">=22"
+  },
+  "scripts": {
+    "build": "node scripts/build.mjs",
+    "prepack": "npm run build",
+    "test": "node --test test/*.test.mjs",
+    "doctor": "node cli.mjs doctor"
+  },
+  "devDependencies": {
+    "@babel/core": "7.29.7",
+    "@babel/preset-typescript": "7.27.1",
+    "babel-preset-solid": "1.9.12"
+  },
+  "peerDependencies": {
+    "@opencode/plugin": ">=2.1.2 <3",
+    "@opentui/core": ">=0.6.6",
+    "@opentui/solid": ">=0.6.6",
+    "solid-js": ">=1.9.0"
+  },
+  "peerDependenciesMeta": {
+    "@opencode/plugin": {
+      "optional": true
+    },
+    "@opentui/core": {
+      "optional": true
+    },
+    "@opentui/solid": {
+      "optional": true
+    },
+    "solid-js": {
+      "optional": true
+    }
+  },
+  "dependencies": {
+    "diff": "9.0.0"
+  }
+}