@north-light/crouter 0.3.317 → 0.3.319

package/dist/core/review/document.js

@@ -1,138 +1,2 @@
-import { createHash } from 'node:crypto';
-import { readFileSync, statSync } from 'node:fs';
-import { documentLines } from '../document-lines.js';
-import { COMMENT_QUOTE_MAX, ReviewOperationError, } from './types.js';
-const REVIEW_SOURCE_MAX_BYTES = 2 * 1024 * 1024;
-function sourceUnreadable(file, phase, reason) {
-    throw new ReviewOperationError('source_unreadable', 'review source is unreadable', { phase, reason, file });
-}
-/** Read the reviewed document once, preserving the exact bytes used for its digest. */
-export function readReviewSource(file, phase = 'submit') {
-    let size;
-    try {
-        const stat = statSync(file);
-        if (!stat.isFile())
-            return sourceUnreadable(file, phase, 'unreadable');
-        size = stat.size;
-    }
-    catch (error) {
-        return sourceUnreadable(file, phase, error.code === 'ENOENT' ? 'missing' : 'unreadable');
-    }
-    if (size > REVIEW_SOURCE_MAX_BYTES)
-        return sourceUnreadable(file, phase, 'too_large');
-    let bytes;
-    try {
-        bytes = readFileSync(file);
-    }
-    catch (error) {
-        return sourceUnreadable(file, phase, error.code === 'ENOENT' ? 'missing' : 'unreadable');
-    }
-    if (bytes.length > REVIEW_SOURCE_MAX_BYTES)
-        return sourceUnreadable(file, phase, 'too_large');
-    if (bytes.includes(0))
-        return sourceUnreadable(file, phase, 'binary');
-    let text;
-    try {
-        text = new TextDecoder('utf-8', { fatal: true }).decode(bytes);
-    }
-    catch {
-        return sourceUnreadable(file, phase, 'binary');
-    }
-    return {
-        bytes,
-        sha256: createHash('sha256').update(bytes).digest('hex'),
-        lines: documentLines(text),
-    };
-}
-function requireDocumentSource(review, expected_doc_hash) {
-    const source = readReviewSource(review.file, 'comment');
-    if (source.sha256 !== expected_doc_hash) {
-        throw new ReviewOperationError('stale_document', 'review document changed since it was rendered', {
-            reason: 'document_changed',
-            expected_sha256: expected_doc_hash,
-            actual_sha256: source.sha256,
-        });
-    }
-    if (source.sha256 !== review.anchor_base_sha256) {
-        throw new ReviewOperationError('stale_document', 'review comment ranges have not been remapped to the current document', {
-            reason: 'anchor_base_stale',
-            expected_sha256: review.anchor_base_sha256,
-            actual_sha256: source.sha256,
-        });
-    }
-    return source;
-}
-/** Require both the caller's rendered source and the persisted range coordinate base to be current. */
-export function requireCurrentDocument(review, expected_doc_hash) {
-    return requireDocumentSource(review, expected_doc_hash);
-}
-/** Verify a create-time source precondition from the review row held by the caller's canvas write transaction. */
-export function requireCurrentDocumentLocked(db, review_id, expected_doc_hash) {
-    const review = db.prepare("SELECT file, anchor_base_sha256 FROM human_requests WHERE request_id = ? AND kind = 'review' AND state = 'open'").get(review_id);
-    if (review === undefined) {
-        throw new ReviewOperationError('unknown_review', 'review was not found', { received: review_id });
-    }
-    return requireDocumentSource(review, expected_doc_hash);
-}
-/** Reject malformed ranges rather than normalizing them into a different anchor. */
-export function validateAnchorShape(anchor) {
-    const violations = [];
-    const hasStart = anchor.col_start !== undefined;
-    const hasEnd = anchor.col_end !== undefined;
-    if (!Number.isSafeInteger(anchor.line) || anchor.line < 1)
-        violations.push('line must be a positive integer');
-    if (!Number.isSafeInteger(anchor.end_line) || anchor.end_line < 1)
-        violations.push('end_line must be a positive integer');
-    if (Number.isSafeInteger(anchor.line) && Number.isSafeInteger(anchor.end_line) && anchor.end_line < anchor.line) {
-        violations.push('end_line must be greater than or equal to line');
-    }
-    if (hasStart !== hasEnd)
-        violations.push('col_start and col_end must be provided together');
-    if (hasStart && (!Number.isSafeInteger(anchor.col_start) || anchor.col_start < 0))
-        violations.push('col_start must be a non-negative integer');
-    if (hasEnd && (!Number.isSafeInteger(anchor.col_end) || anchor.col_end < 0))
-        violations.push('col_end must be a non-negative integer');
-    if (hasStart
-        && hasEnd
-        && anchor.line === anchor.end_line
-        && Number.isSafeInteger(anchor.col_start)
-        && Number.isSafeInteger(anchor.col_end)
-        && anchor.col_end <= anchor.col_start) {
-        violations.push('col_end must be greater than col_start for a single-line anchor');
-    }
-    if (anchor.quote !== undefined && typeof anchor.quote !== 'string')
-        violations.push('quote must be a string');
-    if (typeof anchor.quote === 'string' && Buffer.byteLength(anchor.quote, 'utf8') > COMMENT_QUOTE_MAX) {
-        violations.push(`quote exceeds ${COMMENT_QUOTE_MAX} bytes`);
-    }
-    if (violations.length > 0) {
-        throw new ReviewOperationError('invalid_anchor', 'comment anchor is invalid', { violations });
-    }
-}
-function isUtf8Boundary(bytes, offset) {
-    return offset === bytes.length || (bytes[offset] & 0xc0) !== 0x80;
-}
-/** Capture the full source range used to render an anchor, without silently clamping it. */
-export function captureAnchorText(lines, anchor) {
-    validateAnchorShape(anchor);
-    if (anchor.line > lines.length || anchor.end_line > lines.length) {
-        throw new ReviewOperationError('anchor_out_of_range', 'comment anchor exceeds the reviewed document', {
-            line_count: lines.length,
-            requested: { line: anchor.line, end_line: anchor.end_line },
-        });
-    }
-    if (anchor.col_start !== undefined && anchor.col_end !== undefined) {
-        const startLine = Buffer.from(lines[anchor.line - 1], 'utf8');
-        const endLine = Buffer.from(lines[anchor.end_line - 1], 'utf8');
-        if (anchor.col_start > startLine.length
-            || anchor.col_end > endLine.length
-            || !isUtf8Boundary(startLine, anchor.col_start)
-            || !isUtf8Boundary(endLine, anchor.col_end)) {
-            throw new ReviewOperationError('anchor_out_of_range', 'comment anchor columns exceed the reviewed document', {
-                line_count: lines.length,
-                requested: { line: anchor.line, end_line: anchor.end_line },
-            });
-        }
-    }
-    return lines.slice(anchor.line - 1, anchor.end_line).join('\n');
-}
+var f=Object.defineProperty;var i=(e,t)=>f(e,"name",{value:t,configurable:!0});import{createHash as m}from"node:crypto";import{readFileSync as g,statSync as b}from"node:fs";import{documentLines as p}from"../document-lines.js";import{COMMENT_QUOTE_MAX as l,ReviewOperationError as u}from"./types.js";const a=2*1024*1024;function o(e,t,n){throw new u("source_unreadable","review source is unreadable",{phase:t,reason:n,file:e})}i(o,"sourceUnreadable");function w(e,t="submit"){let n;try{const s=b(e);if(!s.isFile())return o(e,t,"unreadable");n=s.size}catch(s){return o(e,t,s.code==="ENOENT"?"missing":"unreadable")}if(n>a)return o(e,t,"too_large");let r;try{r=g(e)}catch(s){return o(e,t,s.code==="ENOENT"?"missing":"unreadable")}if(r.length>a)return o(e,t,"too_large");if(r.includes(0))return o(e,t,"binary");let d;try{d=new TextDecoder("utf-8",{fatal:!0}).decode(r)}catch{return o(e,t,"binary")}return{bytes:r,sha256:m("sha256").update(r).digest("hex"),lines:p(d)}}i(w,"readReviewSource");function _(e,t){const n=w(e.file,"comment");if(n.sha256!==t)throw new u("stale_document","review document changed since it was rendered",{reason:"document_changed",expected_sha256:t,actual_sha256:n.sha256});if(n.sha256!==e.anchor_base_sha256)throw new u("stale_document","review comment ranges have not been remapped to the current document",{reason:"anchor_base_stale",expected_sha256:e.anchor_base_sha256,actual_sha256:n.sha256});return n}i(_,"requireDocumentSource");function q(e,t){return _(e,t)}i(q,"requireCurrentDocument");function y(e,t,n){const r=e.prepare("SELECT file, anchor_base_sha256 FROM human_requests WHERE request_id = ? AND kind = 'review' AND state = 'open'").get(t);if(r===void 0)throw new u("unknown_review","review was not found",{received:t});return _(r,n)}i(y,"requireCurrentDocumentLocked");function h(e){const t=[],n=e.col_start!==void 0,r=e.col_end!==void 0;if((!Number.isSafeInteger(e.line)||e.line<1)&&t.push("line must be a positive integer"),(!Number.isSafeInteger(e.end_line)||e.end_line<1)&&t.push("end_line must be a positive integer"),Number.isSafeInteger(e.line)&&Number.isSafeInteger(e.end_line)&&e.end_line<e.line&&t.push("end_line must be greater than or equal to line"),n!==r&&t.push("col_start and col_end must be provided together"),n&&(!Number.isSafeInteger(e.col_start)||e.col_start<0)&&t.push("col_start must be a non-negative integer"),r&&(!Number.isSafeInteger(e.col_end)||e.col_end<0)&&t.push("col_end must be a non-negative integer"),n&&r&&e.line===e.end_line&&Number.isSafeInteger(e.col_start)&&Number.isSafeInteger(e.col_end)&&e.col_end<=e.col_start&&t.push("col_end must be greater than col_start for a single-line anchor"),e.quote!==void 0&&typeof e.quote!="string"&&t.push("quote must be a string"),typeof e.quote=="string"&&Buffer.byteLength(e.quote,"utf8")>l&&t.push(`quote exceeds ${l} bytes`),t.length>0)throw new u("invalid_anchor","comment anchor is invalid",{violations:t})}i(h,"validateAnchorShape");function c(e,t){return t===e.length||(e[t]&192)!==128}i(c,"isUtf8Boundary");function I(e,t){if(h(t),t.line>e.length||t.end_line>e.length)throw new u("anchor_out_of_range","comment anchor exceeds the reviewed document",{line_count:e.length,requested:{line:t.line,end_line:t.end_line}});if(t.col_start!==void 0&&t.col_end!==void 0){const n=Buffer.from(e[t.line-1],"utf8"),r=Buffer.from(e[t.end_line-1],"utf8");if(t.col_start>n.length||t.col_end>r.length||!c(n,t.col_start)||!c(r,t.col_end))throw new u("anchor_out_of_range","comment anchor columns exceed the reviewed document",{line_count:e.length,requested:{line:t.line,end_line:t.end_line}})}return e.slice(t.line-1,t.end_line).join(`
+`)}i(I,"captureAnchorText");export{I as captureAnchorText,w as readReviewSource,q as requireCurrentDocument,y as requireCurrentDocumentLocked,h as validateAnchorShape};

package/dist/core/review/project.js

@@ -1,36 +1 @@
-import { join } from 'node:path';
-import { atomicWriteJson } from '../human/convention.js';
-/** Build the immutable approval result from the review row and one comment snapshot. */
-export function buildReviewResult(review, snapshot, approvedSha256, at) {
-    return {
-        schema: 'humanloop.review-response/v1',
-        review_id: review.review_id,
-        file: review.file,
-        submitted: true,
-        approved: true,
-        changed: approvedSha256 !== review.baseline_sha256,
-        baseline_sha256: review.baseline_sha256,
-        approved_sha256: approvedSha256,
-        companion_node_id: review.companion_node_id,
-        comments_total: snapshot.total,
-        comments_unresolved: snapshot.unresolved,
-        comments: snapshot.comments,
-        submittedAt: at,
-        savedAt: at,
-    };
-}
-/** Project every terminal review from its SQL row; no ticket bridge participates. */
-export function projectReviewResult(review, result) {
-    const resultPath = join(review.review_dir, 'result.json');
-    atomicWriteJson(resultPath, result);
-    const output = review.output_path ?? join(review.review_dir, 'feedback.json');
-    if (output !== resultPath)
-        atomicWriteJson(output, result);
-    return { destinations: output === resultPath ? [resultPath] : [resultPath, output] };
-}
-/** Cancellation has no review approval payload but still receives a durable projection. */
-export function projectReviewCancellation(review, result) {
-    const resultPath = join(review.review_dir, 'result.json');
-    atomicWriteJson(resultPath, result);
-    return { destinations: [resultPath] };
-}
+var u=Object.defineProperty;var i=(e,o)=>u(e,"name",{value:o,configurable:!0});import{join as s}from"node:path";import{atomicWriteJson as r}from"../human/convention.js";function m(e,o,t,n){return{schema:"humanloop.review-response/v1",review_id:e.review_id,file:e.file,submitted:!0,approved:!0,changed:t!==e.baseline_sha256,baseline_sha256:e.baseline_sha256,approved_sha256:t,companion_node_id:e.companion_node_id,comments_total:o.total,comments_unresolved:o.unresolved,comments:o.comments,submittedAt:n,savedAt:n}}i(m,"buildReviewResult");function l(e,o){const t=s(e.review_dir,"result.json");r(t,o);const n=e.output_path??s(e.review_dir,"feedback.json");return n!==t&&r(n,o),{destinations:n===t?[t]:[t,n]}}i(l,"projectReviewResult");function _(e,o){const t=s(e.review_dir,"result.json");return r(t,o),{destinations:[t]}}i(_,"projectReviewCancellation");export{m as buildReviewResult,_ as projectReviewCancellation,l as projectReviewResult};

package/dist/core/review/realize.js

@@ -1,71 +1 @@
-import { existsSync } from 'node:fs';
-import { getNode } from '../canvas/index.js';
-import { openRequest, cancelBindingRequest, getRequest } from '../human/requests.js';
-import { NodeIdConflictError } from '../runtime/nodes.js';
-import { reviveNode } from '../runtime/revive.js';
-import { awaitCompanionBind, spawnReviewCompanion } from './companion.js';
-import { signalReviewActivity } from './signal.js';
-import { COMPANION_BIND_DEADLINE_MS, ReviewOperationError } from './types.js';
-function requireBinding(reviewId) {
-    const request = getRequest(reviewId);
-    if (request === null || request.kind !== 'review')
-        throw new ReviewOperationError('unknown_review', 'review was not found', { received: reviewId });
-    return request;
-}
-function specification(request) {
-    try {
-        const value = JSON.parse(request.specification_json);
-        if (typeof value !== 'object' || value === null)
-            throw new Error('not an object');
-        return value;
-    }
-    catch (error) {
-        throw new Error(`corrupt review specification: ${request.request_id}`, { cause: error });
-    }
-}
-/** Finish a committed binding without bridge creation or directory descriptor publication. */
-export async function realizeReview(reviewId) {
-    let request = requireBinding(reviewId);
-    if (request.state !== 'binding')
-        return request;
-    const spec = specification(request);
-    const branchFile = spec['branch_file'];
-    if (typeof branchFile !== 'string' || !existsSync(branchFile)) {
-        if (cancelBindingRequest(request.request_id, new Date().toISOString(), 'branch_lost'))
-            signalReviewActivity();
-        throw new ReviewOperationError('branch_lost', 'review branch was lost before realization', { review_id: request.request_id });
-    }
-    if (request.file === null || request.requester_node_id === null || request.companion_node_id === null) {
-        throw new Error(`binding review has incomplete identity: ${request.request_id}`);
-    }
-    const origin = getNode(request.requester_node_id);
-    if (origin === null)
-        throw new ReviewOperationError('review_origin_missing', 'review origin was not found', { origin_node_id: request.requester_node_id });
-    if (getNode(request.companion_node_id) === null) {
-        const modelSpec = spec['model_spec'];
-        if (typeof modelSpec !== 'string')
-            throw new Error(`binding review has no model spec: ${request.request_id}`);
-        try {
-            await spawnReviewCompanion({
-                nodeId: request.companion_node_id, reviewId: request.request_id, originNodeId: request.requester_node_id,
-                branchFile, targetFile: request.file, modelSpec, cwd: origin.cwd, profileId: origin.profile_id ?? null,
-            });
-        }
-        catch (error) {
-            if (!(error instanceof NodeIdConflictError))
-                throw error;
-        }
-    }
-    const companion = getNode(request.companion_node_id);
-    if (companion === null)
-        throw new ReviewOperationError('review_companion_unbound', 'review companion was not created', { review_id: request.request_id });
-    if (companion.pi_session_id === null)
-        await reviveNode(companion.node_id, { resume: false, capacity: 'refuse' });
-    if (!await awaitCompanionBind(request.companion_node_id, COMPANION_BIND_DEADLINE_MS)) {
-        throw new ReviewOperationError('review_companion_unbound', 'review companion did not bind in time', { review_id: request.request_id, waited_ms: COMPANION_BIND_DEADLINE_MS });
-    }
-    if (openRequest(request.request_id, new Date().toISOString()))
-        signalReviewActivity();
-    request = requireBinding(reviewId);
-    return request;
-}
+var l=Object.defineProperty;var o=(i,e)=>l(i,"name",{value:e,configurable:!0});import{existsSync as p}from"node:fs";import{getNode as s}from"../canvas/index.js";import{openRequest as m,cancelBindingRequest as v,getRequest as g}from"../human/requests.js";import{NodeIdConflictError as h}from"../runtime/nodes.js";import{reviveNode as q}from"../runtime/revive.js";import{awaitCompanionBind as b,spawnReviewCompanion as y}from"./companion.js";import{signalReviewActivity as _}from"./signal.js";import{COMPANION_BIND_DEADLINE_MS as u,ReviewOperationError as n}from"./types.js";function f(i){const e=g(i);if(e===null||e.kind!=="review")throw new n("unknown_review","review was not found",{received:i});return e}o(f,"requireBinding");function N(i){try{const e=JSON.parse(i.specification_json);if(typeof e!="object"||e===null)throw new Error("not an object");return e}catch(e){throw new Error(`corrupt review specification: ${i.request_id}`,{cause:e})}}o(N,"specification");async function A(i){let e=f(i);if(e.state!=="binding")return e;const a=N(e),r=a.branch_file;if(typeof r!="string"||!p(r))throw v(e.request_id,new Date().toISOString(),"branch_lost")&&_(),new n("branch_lost","review branch was lost before realization",{review_id:e.request_id});if(e.file===null||e.requester_node_id===null||e.companion_node_id===null)throw new Error(`binding review has incomplete identity: ${e.request_id}`);const t=s(e.requester_node_id);if(t===null)throw new n("review_origin_missing","review origin was not found",{origin_node_id:e.requester_node_id});if(s(e.companion_node_id)===null){const c=a.model_spec;if(typeof c!="string")throw new Error(`binding review has no model spec: ${e.request_id}`);try{await y({nodeId:e.companion_node_id,reviewId:e.request_id,originNodeId:e.requester_node_id,branchFile:r,targetFile:e.file,modelSpec:c,cwd:t.cwd,profileId:t.profile_id??null})}catch(w){if(!(w instanceof h))throw w}}const d=s(e.companion_node_id);if(d===null)throw new n("review_companion_unbound","review companion was not created",{review_id:e.request_id});if(d.pi_session_id===null&&await q(d.node_id,{resume:!1,capacity:"refuse"}),!await b(e.companion_node_id,u))throw new n("review_companion_unbound","review companion did not bind in time",{review_id:e.request_id,waited_ms:u});return m(e.request_id,new Date().toISOString())&&_(),e=f(i),e}o(A,"realizeReview");export{A as realizeReview};

package/dist/core/review/remap.js

@@ -1,156 +1,19 @@
-import { withCanvasWrite } from '../canvas/db.js';
-import { nowIso } from '../fs-utils.js';
-import { signalReviewActivity } from './signal.js';
-import { replaceAnchorBaseLocked } from './store.js';
-import { ReviewOperationError, } from './types.js';
-import { captureAnchorText, readReviewSource, validateAnchorShape } from './document.js';
-function anchorFromRow(row) {
-    return {
-        line: row.anchor_line,
-        end_line: row.anchor_end_line,
-        ...(row.anchor_col_start === null ? {} : { col_start: row.anchor_col_start }),
-        ...(row.anchor_col_end === null ? {} : { col_end: row.anchor_col_end }),
-        ...(row.anchor_quote === null ? {} : { quote: row.anchor_quote }),
-    };
-}
-function rangeBatchStale(reason, expected, actual, comment_id) {
-    throw new ReviewOperationError('range_batch_stale', 'review comment ranges are stale', {
-        reason,
-        expected,
-        actual,
-        ...(comment_id === undefined ? {} : { comment_id }),
-    });
-}
-function validateEntry(entry) {
-    if (typeof entry.comment_id !== 'string' || entry.comment_id === '') {
-        throw new ReviewOperationError('invalid_anchor', 'comment anchor is invalid', {
-            violations: ['comment_id must be a non-empty string'],
-        });
-    }
-    const hasAnchor = entry.anchor !== undefined;
-    const detached = entry.detached === true;
-    if (hasAnchor === detached) {
-        throw new ReviewOperationError('invalid_anchor', 'comment anchor is invalid', {
-            violations: ['each range batch entry must provide exactly one of anchor or detached'],
-        });
-    }
-    if (hasAnchor)
-        validateAnchorShape(entry.anchor);
-}
-function appendReanchorEvent(db, args) {
-    db.prepare(`
+var q=Object.defineProperty;var _=(e,o)=>q(e,"name",{value:o,configurable:!0});import{withCanvasWrite as A}from"../canvas/db.js";import{nowIso as S}from"../fs-utils.js";import{signalReviewActivity as k}from"./signal.js";import{replaceAnchorBaseLocked as T}from"./store.js";import{ReviewOperationError as h}from"./types.js";import{captureAnchorText as b,readReviewSource as O,validateAnchorShape as N}from"./document.js";function D(e){return{line:e.anchor_line,end_line:e.anchor_end_line,...e.anchor_col_start===null?{}:{col_start:e.anchor_col_start},...e.anchor_col_end===null?{}:{col_end:e.anchor_col_end},...e.anchor_quote===null?{}:{quote:e.anchor_quote}}}_(D,"anchorFromRow");function p(e,o,c,r){throw new h("range_batch_stale","review comment ranges are stale",{reason:e,expected:o,actual:c,...r===void 0?{}:{comment_id:r}})}_(p,"rangeBatchStale");function I(e){if(typeof e.comment_id!="string"||e.comment_id==="")throw new h("invalid_anchor","comment anchor is invalid",{violations:["comment_id must be a non-empty string"]});const o=e.anchor!==void 0,c=e.detached===!0;if(o===c)throw new h("invalid_anchor","comment anchor is invalid",{violations:["each range batch entry must provide exactly one of anchor or detached"]});o&&N(e.anchor)}_(I,"validateEntry");function L(e,o){e.prepare(`
     INSERT INTO review_comment_events
       (review_id, comment_id, kind, state_change, revision_after, actor_kind, actor_node_id, at, operation_id, payload)
     SELECT review_id, ?, 'reanchored', 1, ?, ?, ?, ?, ?, ?
     FROM review_comments WHERE comment_id = ?
-  `).run(args.row.comment_id, args.row.revision + 1, args.actor.kind, args.actor.node_id, args.at, args.operation_id, JSON.stringify({
-        previous_anchor: args.previous_anchor,
-        anchor: args.anchor,
-        previous_anchor_state: args.previous_anchor_state,
-        anchor_state: args.anchor_state,
-        previous_doc_hash: args.previous_doc_hash,
-        doc_hash: args.doc_hash,
-        line_text: args.line_text,
-    }), args.row.comment_id);
-}
-/** Atomically replace every live review coordinate with ranges expressed against the current source. */
-export function applyRangeBatch(args) {
-    const result = withCanvasWrite((db) => {
-        const review = db.prepare("SELECT file, anchor_base_sha256 FROM human_requests WHERE request_id = ? AND kind = 'review' AND state = 'open'").get(args.review_id);
-        if (review === undefined) {
-            throw new ReviewOperationError('unknown_review', 'review was not found', { received: args.review_id });
-        }
-        const source = readReviewSource(review.file, 'comment');
-        if (source.sha256 !== args.to_doc_hash)
-            rangeBatchStale('to_doc_hash', args.to_doc_hash, source.sha256);
-        if (review.anchor_base_sha256 !== args.from_doc_hash) {
-            rangeBatchStale('from_doc_hash', review.anchor_base_sha256, args.from_doc_hash);
-        }
-        const comments = db.prepare(`
+  `).run(o.row.comment_id,o.row.revision+1,o.actor.kind,o.actor.node_id,o.at,o.operation_id,JSON.stringify({previous_anchor:o.previous_anchor,anchor:o.anchor,previous_anchor_state:o.previous_anchor_state,anchor_state:o.anchor_state,previous_doc_hash:o.previous_doc_hash,doc_hash:o.doc_hash,line_text:o.line_text}),o.row.comment_id)}_(L,"appendReanchorEvent");function U(e){const o=A(c=>{const r=c.prepare("SELECT file, anchor_base_sha256 FROM human_requests WHERE request_id = ? AND kind = 'review' AND state = 'open'").get(e.review_id);if(r===void 0)throw new h("unknown_review","review was not found",{received:e.review_id});const a=O(r.file,"comment");a.sha256!==e.to_doc_hash&&p("to_doc_hash",e.to_doc_hash,a.sha256),r.anchor_base_sha256!==e.from_doc_hash&&p("from_doc_hash",r.anchor_base_sha256,e.from_doc_hash);const d=c.prepare(`
       SELECT comment_id, status, revision, text, anchor_line, anchor_end_line,
              anchor_col_start, anchor_col_end, anchor_quote, anchor_line_text,
              anchor_doc_hash, anchor_state
       FROM review_comments
       WHERE review_id = ? AND status != 'deleted'
       ORDER BY created_seq
-    `).all(args.review_id);
-        for (const comment of comments) {
-            if (comment.anchor_doc_hash !== args.from_doc_hash) {
-                rangeBatchStale('comment_hash', args.from_doc_hash, comment.anchor_doc_hash, comment.comment_id);
-            }
-        }
-        const entries = new Map();
-        const extra = [];
-        for (const entry of args.entries) {
-            validateEntry(entry);
-            if (entries.has(entry.comment_id)) {
-                extra.push(entry.comment_id);
-            }
-            else {
-                entries.set(entry.comment_id, entry);
-            }
-        }
-        const commentIds = new Set(comments.map((comment) => comment.comment_id));
-        for (const commentId of entries.keys()) {
-            if (!commentIds.has(commentId))
-                extra.push(commentId);
-        }
-        const missing = comments.filter((comment) => !entries.has(comment.comment_id)).map((comment) => comment.comment_id);
-        if (missing.length > 0 || extra.length > 0) {
-            throw new ReviewOperationError('range_batch_incomplete', 'range batch must cover every non-deleted comment exactly once', {
-                missing,
-                extra,
-            });
-        }
-        let updated = 0;
-        let detached = 0;
-        const at = nowIso();
-        for (const comment of comments) {
-            const entry = entries.get(comment.comment_id);
-            const previousAnchor = anchorFromRow(comment);
-            const nextAnchor = entry.detached === true ? previousAnchor : entry.anchor;
-            const nextState = entry.detached === true ? 'detached' : 'live';
-            const nextLineText = entry.detached === true
-                ? comment.anchor_line_text
-                : captureAnchorText(source.lines, nextAnchor);
-            const changed = comment.anchor_state !== nextState
-                || comment.anchor_doc_hash !== source.sha256
-                || comment.anchor_line !== nextAnchor.line
-                || comment.anchor_end_line !== nextAnchor.end_line
-                || comment.anchor_col_start !== (nextAnchor.col_start ?? null)
-                || comment.anchor_col_end !== (nextAnchor.col_end ?? null)
-                || comment.anchor_quote !== (nextAnchor.quote ?? null)
-                || comment.anchor_line_text !== nextLineText;
-            if (!changed)
-                continue;
-            const revision = comment.revision + 1;
-            db.prepare(`
+    `).all(e.review_id);for(const n of d)n.anchor_doc_hash!==e.from_doc_hash&&p("comment_hash",e.from_doc_hash,n.anchor_doc_hash,n.comment_id);const i=new Map,s=[];for(const n of e.entries)I(n),i.has(n.comment_id)?s.push(n.comment_id):i.set(n.comment_id,n);const R=new Set(d.map(n=>n.comment_id));for(const n of i.keys())R.has(n)||s.push(n);const v=d.filter(n=>!i.has(n.comment_id)).map(n=>n.comment_id);if(v.length>0||s.length>0)throw new h("range_batch_incomplete","range batch must cover every non-deleted comment exactly once",{missing:v,extra:s});let f=0,w=0;const E=S();for(const n of d){const m=i.get(n.comment_id),x=D(n),t=m.detached===!0?x:m.anchor,l=m.detached===!0?"detached":"live",u=m.detached===!0?n.anchor_line_text:b(a.lines,t);if(!(n.anchor_state!==l||n.anchor_doc_hash!==a.sha256||n.anchor_line!==t.line||n.anchor_end_line!==t.end_line||n.anchor_col_start!==(t.col_start??null)||n.anchor_col_end!==(t.col_end??null)||n.anchor_quote!==(t.quote??null)||n.anchor_line_text!==u))continue;const y=n.revision+1;c.prepare(`
         UPDATE review_comments
         SET revision = ?, anchor_line = ?, anchor_end_line = ?, anchor_col_start = ?, anchor_col_end = ?,
             anchor_quote = ?, anchor_line_text = ?, anchor_doc_hash = ?, anchor_state = ?,
             updated_by_kind = ?, updated_by_node_id = ?, updated_at = ?
         WHERE comment_id = ?
-      `).run(revision, nextAnchor.line, nextAnchor.end_line, nextAnchor.col_start ?? null, nextAnchor.col_end ?? null, nextAnchor.quote ?? null, nextLineText, source.sha256, nextState, args.actor.kind, args.actor.node_id, at, comment.comment_id);
-            appendReanchorEvent(db, {
-                row: comment,
-                actor: args.actor,
-                operation_id: args.operation_id,
-                at,
-                previous_anchor: previousAnchor,
-                anchor: nextAnchor,
-                previous_anchor_state: comment.anchor_state,
-                anchor_state: nextState,
-                previous_doc_hash: comment.anchor_doc_hash,
-                doc_hash: source.sha256,
-                line_text: nextLineText,
-            });
-            updated += 1;
-            if (nextState === 'detached')
-                detached += 1;
-        }
-        replaceAnchorBaseLocked(db, args.review_id, source.bytes, source.sha256);
-        return { updated, detached, anchor_base_sha256: source.sha256 };
-    });
-    signalReviewActivity();
-    return result;
-}
+      `).run(y,t.line,t.end_line,t.col_start??null,t.col_end??null,t.quote??null,u,a.sha256,l,e.actor.kind,e.actor.node_id,E,n.comment_id),L(c,{row:n,actor:e.actor,operation_id:e.operation_id,at:E,previous_anchor:x,anchor:t,previous_anchor_state:n.anchor_state,anchor_state:l,previous_doc_hash:n.anchor_doc_hash,doc_hash:a.sha256,line_text:u}),f+=1,l==="detached"&&(w+=1)}return T(c,e.review_id,a.bytes,a.sha256),{updated:f,detached:w,anchor_base_sha256:a.sha256}});return k(),o}_(U,"applyRangeBatch");export{U as applyRangeBatch};

package/dist/core/review/signal.js

@@ -1,12 +1 @@
-import { activityPath, signalActivity, watchActivity } from '../activity-signal.js';
-/** Signal one durable review mutation to every open viewer without watching every review directory. */
-export function reviewActivityPath() {
-    return activityPath('review-activity');
-}
-/** Signal one review mutation (comment create/edit/resolve/delete/reanchor/delivery) to every open review surface. */
-export function signalReviewActivity() {
-    signalActivity('review-activity');
-}
-export function watchReviewActivity(onChange) {
-    return watchActivity('review-activity', onChange);
-}
+var v=Object.defineProperty;var t=(i,e)=>v(i,"name",{value:e,configurable:!0});import{activityPath as c,signalActivity as r,watchActivity as a}from"../activity-signal.js";function o(){return c("review-activity")}t(o,"reviewActivityPath");function w(){r("review-activity")}t(w,"signalReviewActivity");function u(i){return a("review-activity",i)}t(u,"watchReviewActivity");export{o as reviewActivityPath,w as signalReviewActivity,u as watchReviewActivity};

package/dist/core/review/stage.js

@@ -1,184 +1 @@
-import { createHash } from 'node:crypto';
-import { existsSync, readFileSync, realpathSync, rmSync, statSync } from 'node:fs';
-import { isAbsolute } from 'node:path';
-import { getNode } from '../canvas/index.js';
-import { ensureReviewDir, reviewBranchPath, reviewDir } from '../canvas/paths.js';
-import { checkReviewableFile } from '../human/visible-paths.js';
-import { newNodeId } from '../runtime/nodes.js';
-import { captureOrigin, materializeReviewBranch } from './companion.js';
-import { getRequestByIdempotencyKey } from '../human/requests.js';
-import { ReviewOperationError, } from './types.js';
-function targetError(file, reason) {
-    return new ReviewOperationError('review_target_unreviewable', 'review target cannot be staged', {
-        phase: 'create',
-        file,
-        reason,
-    });
-}
-function validateReviewFile(file) {
-    if (!isAbsolute(file))
-        throw targetError(file, 'not_absolute');
-    if (!existsSync(file))
-        throw targetError(file, 'missing');
-    try {
-        if (!statSync(file).isFile())
-            throw targetError(file, 'not_a_file');
-    }
-    catch (error) {
-        if (error instanceof ReviewOperationError)
-            throw error;
-        throw targetError(file, 'missing');
-    }
-    const reviewable = checkReviewableFile(file);
-    if (!reviewable.ok) {
-        const reason = reviewable.reason === 'file is not UTF-8 text'
-            ? 'binary'
-            : reviewable.reason === 'file is too large to review (2 MB limit)'
-                ? 'too_large'
-                : reviewable.reason === 'file is too long to review (5,000 line limit)'
-                    ? 'too_many_lines'
-                    : 'not_a_file';
-        throw targetError(file, reason);
-    }
-    try {
-        return realpathSync(file);
-    }
-    catch {
-        throw targetError(file, 'missing');
-    }
-}
-function sessionIdentityChanged(error) {
-    // R6 materializes through materializeSessionBranchAt(), whose only identity-race
-    // signal is this precise error. Other branch errors must not trigger recapture.
-    return error instanceof Error && error.message === 'source session id does not match captured coordinates';
-}
-function branchUnavailable(originNodeId) {
-    return new ReviewOperationError('review_branch_unavailable', 'review branch could not be materialized', {
-        reason: 'materialize_failed',
-        origin_node_id: originNodeId,
-    });
-}
-async function captureStagedOrigin(originNodeId) {
-    try {
-        return await captureOrigin(originNodeId);
-    }
-    catch (error) {
-        if (error instanceof ReviewOperationError && error.code === 'review_origin_missing') {
-            throw new ReviewOperationError('review_origin_missing', 'review origin was not found', {
-                origin_node_id: originNodeId,
-                reason: 'origin_missing',
-            });
-        }
-        throw error;
-    }
-}
-function originUnstable(originNodeId) {
-    return new ReviewOperationError('review_origin_unstable', 'review origin changed sessions while staging', { origin_node_id: originNodeId });
-}
-function materializeWithOneRecapture(reviewId, originNodeId, capture, initialCapturedAt) {
-    return (async () => {
-        let current = capture;
-        let capturedAt = initialCapturedAt;
-        try {
-            materializeReviewBranch(reviewId, current);
-            return { capture: current, capturedAt };
-        }
-        catch (error) {
-            if (!sessionIdentityChanged(error))
-                throw branchUnavailable(originNodeId);
-        }
-        current = await captureStagedOrigin(originNodeId);
-        capturedAt = new Date().toISOString();
-        try {
-            materializeReviewBranch(reviewId, current);
-            return { capture: current, capturedAt };
-        }
-        catch (error) {
-            if (sessionIdentityChanged(error))
-                throw originUnstable(originNodeId);
-            throw branchUnavailable(originNodeId);
-        }
-    })();
-}
-/** Remove only review-directory state that was never committed to a review row. */
-export function discardStaging(reviewId) {
-    try {
-        rmSync(reviewDir(reviewId), { recursive: true, force: true });
-    }
-    catch {
-        // Startup collection handles residue that cannot be removed in this process.
-    }
-}
-/**
- * Capture an existing origin and write its immutable branch without creating a
- * canvas node or review row. The caller alone commits or discards this stage.
- */
-export async function stageReview(args) {
-    const existing = getRequestByIdempotencyKey(args.idempotency_key);
-    if (existing !== null)
-        return { existing };
-    if (getNode(args.origin_node_id) === null) {
-        throw new ReviewOperationError('review_origin_missing', 'review origin was not found', {
-            origin_node_id: args.origin_node_id,
-            reason: 'origin_missing',
-        });
-    }
-    const file = validateReviewFile(args.file);
-    const reviewId = newNodeId();
-    const companionNodeId = newNodeId();
-    try {
-        ensureReviewDir(reviewId);
-        const initialCapture = await captureStagedOrigin(args.origin_node_id);
-        const initialCapturedAt = new Date().toISOString();
-        const bytes = readFileSync(file);
-        const sha256 = createHash('sha256').update(bytes).digest('hex');
-        const { capture, capturedAt } = await materializeWithOneRecapture(reviewId, args.origin_node_id, initialCapture, initialCapturedAt);
-        const created = new Date().toISOString();
-        return {
-            row: {
-                review_id: reviewId,
-                idempotency_key: args.idempotency_key,
-                origin_kind: args.origin_kind,
-                origin_node_id: args.origin_node_id,
-                companion_node_id: companionNodeId,
-                review_dir: reviewDir(reviewId),
-                file,
-                origin_session_file: capture.sourceFile,
-                origin_session_id: capture.sourceSessionId,
-                origin_leaf_id: capture.leafId,
-                model_spec: capture.modelSpec,
-                branch_file: reviewBranchPath(reviewId),
-                baseline_sha256: sha256,
-                baseline_bytes: bytes.length,
-                anchor_base_sha256: sha256,
-                anchor_base_bytes: bytes,
-                state: 'binding',
-                created,
-                captured_at: capturedAt,
-                opened_at: null,
-                submit_requested_at: null,
-                output_path: args.output_path ?? null,
-                subtitle: args.subtitle ?? null,
-                approved_at: null,
-                approved_sha256: null,
-                changed: null,
-                comments_total: null,
-                comments_unresolved: null,
-                result_json: null,
-                canceled_at: null,
-                cancel_reason: null,
-                cancel_actor: null,
-                delivery_entry_id: null,
-                delivered_at: null,
-                delivery_error: null,
-                projected_at: null,
-                projection_error: null,
-            },
-            review_dir: reviewDir(reviewId),
-        };
-    }
-    catch (error) {
-        discardStaging(reviewId);
-        throw error;
-    }
-}
+var b=Object.defineProperty;var n=(e,i)=>b(e,"name",{value:i,configurable:!0});import{createHash as S}from"node:crypto";import{existsSync as R,readFileSync as k,realpathSync as O,rmSync as x,statSync as F}from"node:fs";import{isAbsolute as D}from"node:path";import{getNode as I}from"../canvas/index.js";import{ensureReviewDir as z,reviewBranchPath as A,reviewDir as d}from"../canvas/paths.js";import{checkReviewableFile as B}from"../human/visible-paths.js";import{newNodeId as m}from"../runtime/nodes.js";import{captureOrigin as j,materializeReviewBranch as h}from"./companion.js";import{getRequestByIdempotencyKey as C}from"../human/requests.js";import{ReviewOperationError as s}from"./types.js";function c(e,i){return new s("review_target_unreviewable","review target cannot be staged",{phase:"create",file:e,reason:i})}n(c,"targetError");function E(e){if(!D(e))throw c(e,"not_absolute");if(!R(e))throw c(e,"missing");try{if(!F(e).isFile())throw c(e,"not_a_file")}catch(t){throw t instanceof s?t:c(e,"missing")}const i=B(e);if(!i.ok){const t=i.reason==="file is not UTF-8 text"?"binary":i.reason==="file is too large to review (2 MB limit)"?"too_large":i.reason==="file is too long to review (5,000 line limit)"?"too_many_lines":"not_a_file";throw c(e,t)}try{return O(e)}catch{throw c(e,"missing")}}n(E,"validateReviewFile");function g(e){return e instanceof Error&&e.message==="source session id does not match captured coordinates"}n(g,"sessionIdentityChanged");function p(e){return new s("review_branch_unavailable","review branch could not be materialized",{reason:"materialize_failed",origin_node_id:e})}n(p,"branchUnavailable");async function f(e){try{return await j(e)}catch(i){throw i instanceof s&&i.code==="review_origin_missing"?new s("review_origin_missing","review origin was not found",{origin_node_id:e,reason:"origin_missing"}):i}}n(f,"captureStagedOrigin");function U(e){return new s("review_origin_unstable","review origin changed sessions while staging",{origin_node_id:e})}n(U,"originUnstable");function q(e,i,t,r){return(async()=>{let o=t,a=r;try{return h(e,o),{capture:o,capturedAt:a}}catch(l){if(!g(l))throw p(i)}o=await f(i),a=new Date().toISOString();try{return h(e,o),{capture:o,capturedAt:a}}catch(l){throw g(l)?U(i):p(i)}})()}n(q,"materializeWithOneRecapture");function H(e){try{x(d(e),{recursive:!0,force:!0})}catch{}}n(H,"discardStaging");async function Y(e){const i=C(e.idempotency_key);if(i!==null)return{existing:i};if(I(e.origin_node_id)===null)throw new s("review_origin_missing","review origin was not found",{origin_node_id:e.origin_node_id,reason:"origin_missing"});const t=E(e.file),r=m(),o=m();try{z(r);const a=await f(e.origin_node_id),l=new Date().toISOString(),u=k(t),w=S("sha256").update(u).digest("hex"),{capture:_,capturedAt:v}=await q(r,e.origin_node_id,a,l),y=new Date().toISOString();return{row:{review_id:r,idempotency_key:e.idempotency_key,origin_kind:e.origin_kind,origin_node_id:e.origin_node_id,companion_node_id:o,review_dir:d(r),file:t,origin_session_file:_.sourceFile,origin_session_id:_.sourceSessionId,origin_leaf_id:_.leafId,model_spec:_.modelSpec,branch_file:A(r),baseline_sha256:w,baseline_bytes:u.length,anchor_base_sha256:w,anchor_base_bytes:u,state:"binding",created:y,captured_at:v,opened_at:null,submit_requested_at:null,output_path:e.output_path??null,subtitle:e.subtitle??null,approved_at:null,approved_sha256:null,changed:null,comments_total:null,comments_unresolved:null,result_json:null,canceled_at:null,cancel_reason:null,cancel_actor:null,delivery_entry_id:null,delivered_at:null,delivery_error:null,projected_at:null,projection_error:null},review_dir:d(r)}}catch(a){throw H(r),a}}n(Y,"stageReview");export{H as discardStaging,Y as stageReview};