dsh-openwrite 0.2.26 → 0.2.28

package/dog-tools.mjs

@@ -4,8 +4,10 @@ export function apply(ctx) {
   for (const tool of ctx.openwriteDog.tools) ctx.effect(() => ctx.tools.register(tool))
   ctx.effect(() => ctx.tools.guard(execution => {
     if (!execution.name.startsWith('dog_') || ['dog_status', 'dog_wait', 'dog_validate'].includes(execution.name)) return
-    if (!execution.agent) return 'DoG requires an Agent session'
-    if (execution.agent.ctx.get('planMode')?.get(execution.agent).active) return 'DoG execution is disabled in plan mode'
-    if (!ctx.get('sandboxPolicy') || ctx.get('sandboxPolicy').resolve({ session: execution.agent.session }).mode === 'read-only') return 'DoG execution is disabled in read-only mode'
+    if (!execution.agent) return 'DOG_AGENT_SESSION_REQUIRED: DoG execution requires an Agent session; run this tool inside a writing Agent session.'
+    if (execution.agent.ctx.get('planMode')?.get(execution.agent)?.active) return 'DOG_PLAN_MODE: DoG execution is disabled in plan mode; leave plan mode before running it.'
+    const policy = ctx.get('sandboxPolicy')
+    if (!policy) return 'DOG_SANDBOX_POLICY_UNAVAILABLE: The host sandbox policy service is unavailable; check the host configuration.'
+    if (policy.resolve({ session: execution.agent.session }).mode === 'read-only') return 'DOG_READ_ONLY: DoG execution requires a writable session; the current sandbox is read-only.'
   }))
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-openwrite",
-  "version": "0.2.26",
+  "version": "0.2.28",
   "description": "OpenWrite 长篇小说领域后端接入 DeepSeek Harness：统一创作 Agent 预设 + 原生创作工作台",
   "type": "module",
   "scripts": {
@@ -14,14 +14,15 @@
     "install-dsh": "bash scripts/install.sh",
     "dev": "bash scripts/dev.sh",
     "doctor": "node scripts/plugin-doctor.mjs",
-    "test:maintenance": "node --test scripts/plugin-doctor.test.mjs scripts/lifecycle.test.mjs packages/openwrite-bridge/scripts/runtime-lifecycle.test.mjs scripts/maintenance.test.mjs scripts/preset-lifecycle.test.mjs scripts/safe-upgrade.test.mjs",
+    "test:maintenance": "node --test scripts/plugin-doctor.test.mjs scripts/lifecycle.test.mjs packages/openwrite-bridge/scripts/runtime-lifecycle.test.mjs scripts/maintenance.test.mjs scripts/preset-lifecycle.test.mjs scripts/safe-upgrade.test.mjs scripts/dog-tools.test.mjs",
     "check": "npm run check:plugin && npm run test:contracts",
     "test:lifecycle": "node --test scripts/lifecycle.test.mjs",
     "test:runtime": "node --test packages/openwrite-bridge/scripts/runtime-lifecycle.test.mjs",
-    "check:plugin": "npm run build && npm run doctor && npm run test:maintenance && npm run test:profiles && npm run test:preset && npm --prefix packages/openwrite-bridge run smoke && npm --prefix packages/studio-panel run smoke && npm run test:epochs && npm run test:components",
+    "check:plugin": "npm run build && npm run docs:check && npm run doctor && npm run test:maintenance && npm run test:profiles && npm run test:preset && npm --prefix packages/openwrite-bridge run smoke && npm --prefix packages/studio-panel run smoke && npm run test:epochs && npm run test:components",
+    "docs:check": "node scripts/gen-tools-md.mjs --check && node scripts/check-editor-coordinates.mjs",
     "test:profiles": "node scripts/profile-smoke.mjs",
     "prepare": "node scripts/prepare.mjs",
-    "test:managed": "node --test packages/openwrite-bridge/scripts/managed-runtime.test.mjs packages/openwrite-bridge/scripts/tool-policy.test.mjs",
+    "test:managed": "node --test packages/openwrite-bridge/scripts/managed-runtime.test.mjs packages/openwrite-bridge/scripts/tool-policy.test.mjs packages/openwrite-bridge/scripts/followup-tools.test.mjs",
     "test:release": "node scripts/release-smoke.mjs",
     "test:client-boot": "node scripts/client-boot-smoke.mjs"
   },

package/packages/openwrite-bridge/lib/contracts-generated.js

@@ -726,6 +726,12 @@ const SCHEMAS = {
                     "secret"
                 ],
                 "properties": {
+                    "auth_source": {
+                        "enum": [
+                            "api_key",
+                            "opencode_console"
+                        ]
+                    },
                     "capabilities": {
                         "properties": {
                             "chat": {
@@ -772,6 +778,9 @@ const SCHEMAS = {
                             ]
                         },
                         "type": "array"
+                    },
+                    "workspace_id": {
+                        "type": "string"
                     }
                 },
                 "required": [

package/packages/openwrite-bridge/lib/dog-delivery.js

@@ -216,7 +216,7 @@ export async function materializeChapterDelivery(workspaceValue, chapterId, thre
     });
     let reviewStage;
     if (Object.keys(review).length === 0) {
-        reviewStage = stage(chapterId, 'review', 'inconclusive', 'missing', {});
+        reviewStage = stage(chapterId, 'review', 'inconclusive', 'not_reviewed', {});
     }
     else if (!reviewCurrent) {
         reviewStage = stage(chapterId, 'review', 'inconclusive', reviewStale ? 'stale' : 'unverifiable', {
@@ -225,9 +225,13 @@ export async function materializeChapterDelivery(workspaceValue, chapterId, thre
     }
     else {
         const v2 = record(review['review_v2']);
-        reviewStage = stage(chapterId, 'review', reviewPassed ? 'pass' : ['inconclusive', 'stale'].includes(currentDeliveryStatus) ? 'inconclusive' : 'fail', 'current', {
+        const diagnostics = record(v2['delivery_diagnostics']);
+        const reviewState = !reviewPassed && currentDeliveryStatus === 'inconclusive' ? String(diagnostics['code'] ?? v2['delivery_reason'] ?? 'review_inconclusive') : 'current';
+        reviewStage = stage(chapterId, 'review', reviewPassed ? 'pass' : ['inconclusive', 'stale'].includes(currentDeliveryStatus) ? 'inconclusive' : 'fail', reviewState, {
             score: review['score'] ?? null, threshold: effectiveThreshold, passedGate: reviewPassed,
-            qualityScore: v2['quality_score'] ?? review['score'] ?? null, coverage: v2['coverage'] ?? 1,
+            qualityScore: hasV2 ? v2['quality_score'] ?? null : review['score'] ?? null, coverage: v2['coverage'] ?? 1,
+            scoreValid: v2['score_valid'] ?? (hasV2 ? v2['quality_score'] != null : review['score'] != null),
+            scopeStatus: v2['scope_status'] ?? 'full', deliveryDiagnostics: diagnostics,
             gateStatus: v2['gate_status'] ?? (hardIssueIds.length > 0 ? 'blocked' : 'pass'),
             deliveryStatus: currentDeliveryStatus,
             issueIds, hardIssueIds, sourceRevision: reviewSourceRevision,

package/packages/openwrite-bridge/lib/managed-runtime.d.ts

@@ -81,6 +81,7 @@ export declare class ManagedRuntime {
     private connection?;
     private closed;
     private allowRevive;
+    private cancelEpoch;
     private generation;
     private restartAttempts;
     private recovered;

package/packages/openwrite-bridge/lib/managed-runtime.js

@@ -173,6 +173,7 @@ export class ManagedRuntime {
     connection;
     closed = false;
     allowRevive = true;
+    cancelEpoch = 0;
     generation = 0;
     restartAttempts = 0;
     recovered = false;
@@ -190,15 +191,33 @@ export class ManagedRuntime {
     }
     status() { return { ...this.state }; }
     update(phase, message, extra = {}) {
-        this.state = { phase, message, ...extra };
+        const recoveryError = this.recovered && ['recovering', 'waiting', 'downloading', 'installing'].includes(phase)
+            ? this.state.error : undefined;
+        this.state = { phase, message, ...(recoveryError ? { error: recoveryError } : {}), ...extra };
     }
     ensure() {
         if (this.closed)
             return Promise.reject(new Error('OpenWrite was removed'));
         if (this.connection)
             return this.verifyConnection(this.connection);
-        if (this.pending)
+        if (this.pending) {
+            if (['error', 'cancelled', 'stopped'].includes(this.state.phase)) {
+                // A ready child can exit while prepare is still saving active.json.
+                // The terminal state is visible before that promise settles; returning
+                // it would hand a manual retry the dead connection or its old error.
+                const prior = this.pending;
+                const epoch = this.cancelEpoch;
+                const retry = () => {
+                    if (epoch !== this.cancelEpoch)
+                        throw new Error('Setup was cancelled; retry when ready');
+                    if (this.pending === prior)
+                        this.pending = undefined;
+                    return this.ensure();
+                };
+                return prior.then(retry, retry);
+            }
             return this.pending;
+        }
         this.allowRevive = true;
         if (this.state.phase === 'error' || this.state.phase === 'cancelled' || this.state.phase === 'stopped') {
             this.restartAttempts = 0;
@@ -261,6 +280,7 @@ export class ManagedRuntime {
         }
     }
     async cancel() {
+        this.cancelEpoch += 1;
         this.allowRevive = false;
         this.controller?.abort();
         await this.pending?.catch(() => { });

package/packages/openwrite-bridge/lib/tool-policy.js

@@ -31,6 +31,8 @@ export function novelEffect(name, args) {
     if (reads.has(name))
         return 'read';
     const values = args && typeof args === 'object' ? args : {};
+    if (name === 'novel_write_chapter' && values.dry_run === true)
+        return 'read';
     if (readActions[name]?.includes(String(values.action)))
         return 'read';
     if (name === 'novel_model_benchmark' && values.action === 'run')

package/packages/openwrite-bridge/lib/tools.js

@@ -703,8 +703,10 @@ export function registerNovelTools(ctx, clientFactory, options) {
         name: 'novel_write_chapter',
         description: 'Queue a chapter write and immediately return a durable task_id. Poll novel_task_get for progress/result. Omit chapter_id to write ' +
             'the outline-recommended next chapter; a given chapter_id must match the current recommendation. ' +
-            'Preview context first with novel_context_preview and consult novel_outline_read for the recommendation.',
+            'Preview context first with novel_context_preview and consult novel_outline_read for the recommendation. ' +
+            'Cancel with novel_task_cancel. Generated partial prose is retained; resume_from_draft continues it in a new run when the source/context revisions still match.',
         parameters: {
+            dry_run: { type: 'boolean', description: 'Only assemble the current writer input and per-document sizes; no model call and no writing task. Guidance and target_words are included in this preview.' },
             run_id_v2: { type: 'string', description: 'Resume this completed draft without generating it again.' },
             resume_from_draft: { type: 'boolean', description: 'Resume the latest uncommitted draft for this chapter.' },
             target_words: { type: 'integer', description: 'Target word count, 200-50000. When omitted use the outline target, then default 3000.' },
@@ -735,6 +737,8 @@ export function registerNovelTools(ctx, clientFactory, options) {
                 body['temperature'] = args.temperature;
             if (args.outline_revision !== undefined)
                 body['outline_revision'] = args.outline_revision;
+            if (args.dry_run === true)
+                return await client.postJson('/api/write', { ...body, dry_run: true }, exec.signal);
             return await client.postJson('/api/tasks', { type: 'chapter_write', input: body }, exec.signal);
         },
     }));
@@ -749,6 +753,8 @@ export function registerNovelTools(ctx, clientFactory, options) {
             chapter_id: { type: 'string', description: 'Chapter id (ch_<digits>), resolved client-side to data/manuscript/<id>.md.' },
             strict: { type: 'boolean', description: 'Enable strict review mode (default false).' },
             dimensions: { type: 'array', items: { type: 'integer' }, description: 'Optional list of review dimension numbers (1-37) to restrict to.' },
+            depth: { type: 'string', enum: ['quick', 'full'], description: 'Default full. Quick reviews coherence/canon/prose and safety only; scope_status=partial cannot approve full-chapter delivery. Do not combine quick with dimensions.' },
+            domains: { type: 'array', items: { type: 'string' }, description: 'Review domain ids: coherence, character, plot, pacing, prose, canon. Use this or dimensions. Omitted domains remain explicitly unreviewed.' },
             dog_threshold: { type: 'integer', description: 'DoG aggregate score threshold (default 70); this does not change OpenWrite review.' },
         },
         output: { schema: JSON_OUTPUT_SCHEMA, render: (_args, value) => renderJson(value) },
@@ -769,6 +775,10 @@ export function registerNovelTools(ctx, clientFactory, options) {
                 body['strict'] = args.strict;
             if (args.dimensions !== undefined)
                 body['dimensions'] = args.dimensions;
+            if (args.depth !== undefined)
+                body['depth'] = args.depth;
+            if (args.domains !== undefined)
+                body['domains'] = args.domains;
             const response = await client.postJson('/api/review', body, exec.signal);
             const value = response !== null && typeof response === 'object' && !Array.isArray(response)
                 ? response
@@ -887,7 +897,9 @@ export function registerNovelTools(ctx, clientFactory, options) {
             'to receive an immutable preview_token and real before/after summary. Use "apply" only after author acceptance, ' +
             'passing that token and confirm: true; "reject" discards it, "retry" rebuilds it against the current source, and "undo" safely ' +
             'applies an undo_preview_token returned by a prior apply. Each edit uses old_text/new_text or unique start_text/' +
-            'end_text/new_text anchors. The server rechecks the source revision and writes exactly the stored preview.',
+            'end_text/new_text anchors. Preview returns applied:false; changed/would_change describe only the proposed difference. ' +
+            'OLD_TEXT_NOT_FOUND is an anchor mismatch (400), while revision conflicts are 409; use the exact suggested bytes and reread the source before retrying. ' +
+            'The server rechecks the source revision and writes exactly the stored preview.',
         parameters: {
             action: { type: 'string', required: true, enum: ['preview', 'apply', 'reject', 'retry', 'undo'], description: 'Plan lifecycle action.' },
             confirm: { type: 'boolean', description: 'Required true for apply and undo after author acceptance.' },
@@ -1207,13 +1219,14 @@ export function registerNovelTools(ctx, clientFactory, options) {
     ctx.tools.register(defineTool({
         name: 'novel_revision_create_selection',
         description: 'Create a revision proposal for a text selection in a chapter (LONG-RUNNING, model-generated). ' +
-            'start/end are character offsets; pass the selected text as original_text. The proposal is only staged — ' +
+            'start/end are Unicode code-point offsets in the exact saved text, including original CR/LF characters; pass the selected text as original_text and the current expected_document_revision. The proposal is only staged — ' +
             'apply it with novel_revision_apply.',
         parameters: {
             chapter_id: { type: 'string', required: true, description: 'Chapter id (ch_<digits>).' },
             start: { type: 'integer', required: true, description: 'Selection start offset.' },
             end: { type: 'integer', required: true, description: 'Selection end offset.' },
             original_text: { type: 'string', description: 'The selected original text.' },
+            expected_document_revision: { type: 'string', description: 'Current exact manuscript revision; reject before generation if the source changed.' },
             instruction: { type: 'string', description: 'Revision instruction (max 4000 characters).' },
             action: { type: 'string', enum: ['rewrite', 'expand', 'compress', 'pace', 'dialogue', 'naturalize', 'custom'], description: 'Revision mode (default "rewrite").' },
             target_units: { type: 'integer', description: 'Target length in writing units (0 = keep similar).' },
@@ -1228,6 +1241,8 @@ export function registerNovelTools(ctx, clientFactory, options) {
             const body = { chapter_id: args.chapter_id, start: args.start, end: args.end };
             if (args.original_text !== undefined)
                 body['original_text'] = args.original_text;
+            if (args.expected_document_revision !== undefined)
+                body['expected_document_revision'] = args.expected_document_revision;
             if (args.instruction !== undefined)
                 body['instruction'] = args.instruction;
             if (args.action !== undefined)
@@ -1242,10 +1257,15 @@ export function registerNovelTools(ctx, clientFactory, options) {
     ctx.tools.register(defineTool({
         name: 'novel_revision_create_from_review',
         description: 'Create a revision proposal that addresses selected issues from a chapter review (LONG-RUNNING, model-generated). ' +
-            'Issue ids come from novel_review_chapter\'s issue_details. The proposal is only staged — apply it with novel_revision_apply.',
+            'Issue ids come from novel_review_chapter\'s issue_details. Pass selection to constrain rewriting to an exact manuscript range; overlapping issue evidence never expands that range. The proposal is only staged — apply it with novel_revision_apply.',
         parameters: {
             chapter_id: { type: 'string', required: true, description: 'Chapter id (ch_<digits>).' },
             issue_ids: { type: 'array', required: true, items: { type: 'string' }, description: 'Review issue ids to fix.' },
+            selection: { type: 'object', additionalProperties: false, description: 'Optional exact saved-text range in Unicode code points; preserve raw CR/LF characters.', properties: {
+                    start: { type: 'integer', required: true, description: 'Inclusive source code-point offset.' },
+                    end: { type: 'integer', required: true, description: 'Exclusive source code-point offset.' },
+                    original_text: { type: 'string', required: true, description: 'Exact saved text inside this range.' },
+                } },
             instruction: { type: 'string', description: 'Additional revision instruction.' },
             target_units: { type: 'integer', description: 'Target length in writing units (0 = keep similar).' },
             expected_review_revision: { type: 'string', description: 'Exact review file revision from the chapter work brief.' },
@@ -1260,6 +1280,8 @@ export function registerNovelTools(ctx, clientFactory, options) {
             if (args.issue_ids.length === 0)
                 throw new Error('issue_ids must be a non-empty array');
             const body = { chapter_id: args.chapter_id, issue_ids: args.issue_ids };
+            if (args.selection !== undefined)
+                body['selection'] = args.selection;
             if (args.instruction !== undefined)
                 body['instruction'] = args.instruction;
             if (args.target_units !== undefined)
@@ -2027,6 +2049,7 @@ export function registerNovelTools(ctx, clientFactory, options) {
             'confirm: true), "cancel" (run_id + revision, optional reason). All mutating actions are revision-gated: ' +
             'pass the run\'s current revision from a list/get call.',
         parameters: {
+            include_archived: { type: 'boolean', description: 'For list: include historical runs moved to chapter_deletions when a chapter was deleted. Archived runs are read-only and include archive_directory.' },
             action: { type: 'string', required: true, enum: ['list', 'get', 'record_intervention', 'update_intervention', 'cancel', 'archive'], description: 'The chapter-run operation.' },
             run_id: { type: 'string', description: 'Run id (required for get/record_intervention/update_intervention/cancel).' },
             revision: { type: 'string', description: 'Current run revision (required for mutating actions).' },
@@ -2053,7 +2076,7 @@ export function registerNovelTools(ctx, clientFactory, options) {
         async execute(args, exec) {
             const client = clientFor(exec);
             const body = { action: args.action };
-            for (const key of ['run_id', 'revision', 'chapter_id', 'statuses', 'since', 'limit', 'scope', 'risk', 'request', 'affected_items', 'rewrite_required', 'intervention_id', 'state', 'facts_revision', 'impact', 'proposal', 'confirm', 'reason']) {
+            for (const key of ['run_id', 'revision', 'chapter_id', 'statuses', 'since', 'limit', 'include_archived', 'scope', 'risk', 'request', 'affected_items', 'rewrite_required', 'intervention_id', 'state', 'facts_revision', 'impact', 'proposal', 'confirm', 'reason']) {
                 const value = args[key];
                 if (value !== undefined)
                     body[key] = value;