getgloss 0.12.3 → 0.13.0

package/dist/server/daemon.js

@@ -10,7 +10,7 @@ import path from "path";
 // package.json
 var package_default = {
   name: "getgloss",
-  version: "0.12.3",
+  version: "0.13.0",
   description: "Local browser-based diff review for coding-agent loops.",
   type: "module",
   packageManager: "pnpm@10.33.2",
@@ -67,8 +67,8 @@ var package_default = {
     tsup: "8.5.1",
     tsx: "4.22.3",
     typescript: "5.9.3",
-    vite: "6.4.2",
-    vitest: "3.2.4"
+    vite: "6.4.3",
+    vitest: "3.2.6"
   },
   keywords: [
     "diff",
@@ -365,7 +365,13 @@ function isDiffLine(value) {
   return isRecord(value) && isOneOf(value.type, DIFF_LINE_TYPES) && isNullableNumber(value.oldLine) && isNullableNumber(value.newLine) && isString(value.content);
 }
 function isComment(value) {
-  return isRecord(value) && isString(value.id) && isString(value.filePath) && isNumber(value.startLine) && isNumber(value.endLine) && isOneOf(value.side, SIDES) && isString(value.body) && isString(value.originalSnippet) && isString(value.createdAt);
+  if (!isRecord(value)) {
+    return false;
+  }
+  if (value.kind === "general") {
+    return isString(value.id) && isString(value.body) && isString(value.createdAt);
+  }
+  return isString(value.id) && (value.kind === void 0 || value.kind === "line") && isString(value.filePath) && isNumber(value.startLine) && isNumber(value.endLine) && isOneOf(value.side, SIDES) && isString(value.body) && isString(value.originalSnippet) && isString(value.createdAt);
 }
 function isResolvedComment(value) {
   return isRecord(value) && isString(value.commentId) && value.status === "resolved" && isOptionalString(value.summary) && isString(value.resolvedAt);
@@ -580,11 +586,28 @@ import { Hono } from "hono";
 import { streamSSE } from "hono/streaming";
 
 // src/shared/comments.ts
+function isLineComment(comment) {
+  return comment.kind === void 0 || comment.kind === "line";
+}
 function compareCommentsByLocation(a, b) {
+  const aIsLine = isLineComment(a);
+  const bIsLine = isLineComment(b);
+  if (!aIsLine || !bIsLine) {
+    if (aIsLine !== bIsLine) {
+      return aIsLine ? 1 : -1;
+    }
+    return a.createdAt.localeCompare(b.createdAt) || a.id.localeCompare(b.id);
+  }
   return a.filePath.localeCompare(b.filePath) || a.startLine - b.startLine || a.endLine - b.endLine || a.side.localeCompare(b.side);
 }
 function countCommentFiles(comments) {
-  return new Set(comments.map((comment) => comment.filePath)).size;
+  const filePaths = /* @__PURE__ */ new Set();
+  for (const comment of comments) {
+    if (isLineComment(comment)) {
+      filePaths.add(comment.filePath);
+    }
+  }
+  return filePaths.size;
 }
 function formatLineRange(range, options = {}) {
   const startLine = Math.min(range.startLine, range.endLine);
@@ -781,6 +804,17 @@ function summarizeDiffFiles(files) {
   );
 }
 
+// src/shared/file-order.ts
+function compareFilePaths(leftPath, rightPath, leftTieBreaker = "", rightTieBreaker = "") {
+  return leftPath.localeCompare(rightPath, void 0, { sensitivity: "base" }) || leftPath.localeCompare(rightPath) || leftTieBreaker.localeCompare(rightTieBreaker, void 0, { sensitivity: "base" }) || leftTieBreaker.localeCompare(rightTieBreaker);
+}
+function compareDiffFiles(left, right) {
+  return compareFilePaths(left.path, right.path, left.oldPath ?? "", right.oldPath ?? "");
+}
+function sortDiffFiles(files) {
+  return files.toSorted(compareDiffFiles);
+}
+
 // src/shared/git-diff.ts
 var DIFF_ARGS = ["diff", "--no-color", "--find-renames", "--find-copies"];
 async function git(args, cwd) {
@@ -789,7 +823,7 @@ async function git(args, cwd) {
 }
 async function captureCommitRangeDiff(fromSha, toSha, repoRoot) {
   const rawDiff = await git([...DIFF_ARGS, `${fromSha}^`, toSha, "--"], repoRoot);
-  const files = parseUnifiedDiff(rawDiff);
+  const files = sortDiffFiles(parseUnifiedDiff(rawDiff));
   return {
     stats: summarizeDiffFiles(files),
     rawDiff,
@@ -1905,9 +1939,11 @@ function languageForSnippet(filePath, snippet) {
 }
 function serializeFeedbackMarkdown(bundle) {
   const comments = bundle.comments.toSorted(compareCommentsByLocation);
+  const generalComments = comments.filter((comment) => !isLineComment(comment));
+  const lineComments = comments.filter(isLineComment);
   const commentsByFile = /* @__PURE__ */ new Map();
   const files = [];
-  for (const comment of comments) {
+  for (const comment of lineComments) {
     const fileComments = commentsByFile.get(comment.filePath);
     if (fileComments) {
       fileComments.push(comment);
@@ -1925,6 +1961,12 @@ function serializeFeedbackMarkdown(bundle) {
     `Files: ${files.length}   Comments: ${comments.length}`,
     ""
   ];
+  if (generalComments.length > 0) {
+    lines.push("## General comments", "");
+    for (const comment of generalComments) {
+      lines.push(`### ${comment.id}`, comment.body.trim(), "");
+    }
+  }
   for (const filePath of files) {
     lines.push(`## ${filePath}`, "");
     for (const comment of commentsByFile.get(filePath) ?? []) {