homeostat 0.1.3 → 0.1.5

package/README.md

@@ -6,7 +6,7 @@ Homeostat turns review feedback that led to a code change into short, scored rul
 
 ```sh
 # In a repo for your team's knowledge base:
-npx --yes homeostat@0.1.3 init
+npx --yes homeostat@0.1.5 init
 
 # In Claude Code:
 claude plugin marketplace add ysugarmen/homeostat

package/dist/src/daily.js

@@ -67,6 +67,14 @@ async function daily(opts, io) {
             settled++;
         }
     }
+    // Lessons still waiting in an open review PR: a new lesson that repeats one waits with it.
+    git(root, "fetch", "--quiet", "origin", "+refs/heads/homeostat/review-*:refs/remotes/origin/homeostat/review-*");
+    const inReview = [...decisions.values()].flatMap((d) => d.decided.flatMap((r) => {
+        if (r.decision !== "review" || !r.pr || !open.has(r.pr))
+            return [];
+        const entry = parseEntry(showFile(root, remote(r.pr), r.path) ?? "").entry;
+        return entry ? [entry] : [];
+    }));
     const lessons = [];
     const decidedHashes = (m) => new Set(decisions.get(m)?.decided.map((r) => r.hash));
     const heartbeats = {};
@@ -152,7 +160,12 @@ async function daily(opts, io) {
         const mine = lessons.find((l) => l !== doc && l.kind === "doc" && l.path === doc.path && l.member === doc.member);
         if (!mine)
             continue;
-        const merged = mergeText(onBase(doc.path) ?? "", mine.text, doc.text);
+        // Each side stamps the document's `updated:` line with its own day; take the later one, or
+        // that single line would make every pair of edits from different days conflict.
+        const day = (t) => /^updated: (.*)$/m.exec(t)?.[1] ?? "";
+        const latest = [day(mine.text), day(doc.text)].sort().at(-1) ?? "";
+        const stamp = (t) => (latest ? t.replace(/^updated: .*$/m, `updated: ${latest}`) : t);
+        const merged = mergeText(onBase(doc.path) ?? "", stamp(mine.text), stamp(doc.text));
         lessons.splice(lessons.indexOf(doc), 1);
         if (merged === undefined) {
             mine.reason = `document edits go to the owning team; a section patch from review (${doc.url ?? "a review thread"}) touched the same lines and was left out`;
@@ -177,6 +190,13 @@ async function daily(opts, io) {
     }
     for (const l of lessons)
         decide(l, files, config, inForce, groups.get(l)?.length ?? 1);
+    for (const { l, entry } of fresh) {
+        const twin = inReview.find((r) => r.frontmatter.id === entry.frontmatter.id || sameLesson(r, entry));
+        if (!twin || l.decision === "declined")
+            continue;
+        l.decision = "review";
+        l.reason = `the same lesson as ${twin.frontmatter.id}, which is waiting in review: decide them together`;
+    }
     for (const [l, group] of groups) {
         if (group.length < 2)
             continue;
@@ -258,8 +278,17 @@ async function daily(opts, io) {
             if (pr.pr) {
                 row.reviewPr = pr.pr.url;
                 const logins = reviewerLogins(config, review, member);
-                if (logins.length)
-                    await provider.requestReviewers?.(kbRepo, pr.pr.id, logins).catch(() => undefined);
+                if (logins.length && provider.requestReviewers) {
+                    try {
+                        await provider.requestReviewers(kbRepo, pr.pr.id, logins);
+                        io.stdout(`  asked ${logins.map((l) => `@${l}`).join(", ")} to review ${pr.pr.url}\n`);
+                    }
+                    catch (e) {
+                        // Usually: they are not collaborators on the KB repo. The PR still waits in the queue.
+                        row.reviewError = `could not request ${logins.map((l) => `@${l}`).join(", ")}: ${e.message}`;
+                        io.stdout(`  ${row.reviewError}; reviewers must be collaborators on ${sources.kbRepo}\n`);
+                    }
+                }
             }
             for (const l of review)
                 Object.assign(l, { pr: pr.branch });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homeostat",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Turn your team's code-review lessons into rules every coding agent follows: Claude Code, MCP, Cursor, Copilot, AGENTS.md.",
   "keywords": [
     "claude-code",
@@ -39,9 +39,9 @@
   },
   "dependencies": {
     "commander": "^15.0.0",
-    "@homeostat/core": "0.1.3",
-    "@homeostat/mcp": "0.1.3",
-    "@homeostat/dashboard": "0.1.3"
+    "@homeostat/mcp": "0.1.5",
+    "@homeostat/dashboard": "0.1.5",
+    "@homeostat/core": "0.1.5"
   },
   "publishConfig": {
     "access": "public",