review-workspace 1.1.0 → 1.2.0

package/dist/export/report.js

@@ -1,23 +1,3 @@
-function formatTarget(target) {
-    switch (target.type) {
-        case 'file':
-            return target.path;
-        case 'hunk':
-            return `${target.path}#hunk${target.hunkIndex}`;
-        case 'binary':
-            return `${target.path} (binary)`;
-        case 'line':
-            return `${target.path}:${target.line} (${target.side})`;
-    }
-}
-export function buildConcernComment(concern) {
-    const prefix = concern.target ? `[${formatTarget(concern.target)}] ` : '';
-    return `${prefix}${concern.note}`;
-}
-/** One comment per Concern, plain text — pasteable individually into an external tool. */
-export function buildConcernComments(state) {
-    return state.concerns.map(buildConcernComment);
-}
 /**
  * A self-contained Markdown report — no server needed to view it afterward.
  * Read-only with respect to the bundle: never mutates state.json or questions.jsonl.
@@ -41,16 +21,6 @@ export function buildReport(document, state) {
         }
     }
     lines.push('');
-    lines.push('## Concerns');
-    if (state.concerns.length === 0) {
-        lines.push('_None raised._');
-    }
-    else {
-        for (const comment of buildConcernComments(state)) {
-            lines.push(`- ${comment}`);
-        }
-    }
-    lines.push('');
     lines.push('## Notes');
     if (state.notes.length === 0) {
         lines.push('_None._');

package/dist/questions/questions-log.js

@@ -1,6 +1,9 @@
 import { appendFileSync, existsSync, readFileSync } from 'node:fs';
 import { randomUUID } from 'node:crypto';
 import { join } from 'node:path';
+// File name predates the Comment/kind split (it only ever held Questions) — kept
+// as-is so bundles created before this change keep reading correctly; only the
+// in-memory shape gained `kind`, not the on-disk path.
 function logPath(bundlePath) {
     return join(bundlePath, 'questions.jsonl');
 }
@@ -17,15 +20,22 @@ function readLog(bundlePath) {
 function appendEntry(bundlePath, entry) {
     appendFileSync(logPath(bundlePath), JSON.stringify(entry) + '\n');
 }
-/** Raises a new, immutable Question. Never mutates or removes any existing log entry. */
+/** Raises a new, immutable question Comment. Never mutates or removes any existing log entry. */
 export function raiseQuestion(bundlePath, body, target) {
-    const entry = { type: 'raised', id: randomUUID(), createdAt: new Date().toISOString(), body, target };
+    const entry = {
+        type: 'raised',
+        id: randomUUID(),
+        createdAt: new Date().toISOString(),
+        body,
+        target,
+        kind: 'question',
+    };
     appendEntry(bundlePath, entry);
-    return { id: entry.id, createdAt: entry.createdAt, body: entry.body, target: entry.target, status: 'open' };
+    return { id: entry.id, createdAt: entry.createdAt, body: entry.body, target: entry.target, kind: 'question', status: 'open' };
 }
 /**
- * Withdraws an existing Question and raises its replacement in one call — a
- * correction is recorded as new log entries, never as an edit to the original.
+ * Withdraws an existing question Comment and raises its replacement in one call —
+ * a correction is recorded as new log entries, never as an edit to the original.
  */
 export function withdrawAndReplaceQuestion(bundlePath, questionId, replacementBody, replacementTarget) {
     const replacement = {
@@ -34,6 +44,7 @@ export function withdrawAndReplaceQuestion(bundlePath, questionId, replacementBo
         createdAt: new Date().toISOString(),
         body: replacementBody,
         target: replacementTarget,
+        kind: 'question',
     };
     const withdrawal = {
         type: 'withdrawn',
@@ -51,30 +62,32 @@ export function withdrawAndReplaceQuestion(bundlePath, questionId, replacementBo
             createdAt: replacement.createdAt,
             body: replacement.body,
             target: replacement.target,
+            kind: 'question',
             status: 'open',
         },
     };
 }
-/** Reduces the append-only log into the current set of Questions. */
+/** Reduces the append-only log into the current set of Comments. */
 export function readQuestions(bundlePath) {
-    const questions = new Map();
+    const comments = new Map();
     for (const entry of readLog(bundlePath)) {
         if (entry.type === 'raised') {
-            questions.set(entry.id, {
+            comments.set(entry.id, {
                 id: entry.id,
                 createdAt: entry.createdAt,
                 body: entry.body,
                 target: entry.target,
+                kind: entry.kind ?? 'question',
                 status: 'open',
             });
         }
         else {
-            const question = questions.get(entry.questionId);
-            if (question) {
-                question.status = 'withdrawn';
-                question.supersededBy = entry.replacementId;
+            const comment = comments.get(entry.questionId);
+            if (comment) {
+                comment.status = 'withdrawn';
+                comment.supersededBy = entry.replacementId;
             }
         }
     }
-    return [...questions.values()].sort((a, b) => a.createdAt.localeCompare(b.createdAt));
+    return [...comments.values()].sort((a, b) => a.createdAt.localeCompare(b.createdAt));
 }

package/dist/review-state/types.js

@@ -1,6 +1,5 @@
 export const EMPTY_REVIEW_STATE = {
     groups: {},
-    concerns: [],
     notes: [],
     decision: 'unset',
 };

package/dist/server/create-server.js

@@ -8,7 +8,7 @@ import { watchBundle } from "./watch-bundle.js";
 import { raiseQuestion, readQuestions, withdrawAndReplaceQuestion } from "../questions/questions-log.js";
 import { readReviewState, writeReviewState } from "../review-state/review-state.js";
 import { render } from "../renderer/render.js";
-import { buildConcernComments, buildReport } from "../export/report.js";
+import { buildReport } from "../export/report.js";
 import { resolveAssetPath } from "../security/asset-path.js";
 export function generateWriteToken() {
     return randomBytes(24).toString('hex');
@@ -118,10 +118,6 @@ export function startReviewServer(bundlePath, opts = {}) {
             res.end(buildReport(latest.document, readReviewState(bundlePath)));
             return;
         }
-        if (req.method === 'GET' && url.pathname === '/concerns') {
-            respond(200, buildConcernComments(readReviewState(bundlePath)));
-            return;
-        }
         if (req.method === 'GET' && url.pathname.startsWith('/assets/')) {
             const assetPath = decodeURIComponent(url.pathname.slice('/assets/'.length));
             const resolution = resolveAssetPath(bundlePath, assetPath);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "review-workspace",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "A local, offline decision surface for reviewing a code Comparison — evidence-linked map of intent, behavior, risk, and observable effects.",
   "license": "MIT",
   "author": "Salih Sayer",