@jabworks/condux 0.13.1 → 0.13.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jabworks/condux",
-  "version": "0.13.1",
+  "version": "0.13.3",
   "description": "OpenCode plugin for the condux workflow toolkit — bundles the condux skills and injects the specialist agents (coder, explorer, planner, researcher) plus an opt-in plan-review listener.",
   "type": "module",
   "publishConfig": {

package/skills/plan-review/references/annotate-server.js

@@ -172,6 +172,24 @@ function gitRoot() {
   }
 }
 
+// Write atomically. A plain writeFileSync is open(O_CREAT|O_TRUNC) then write()
+// — two syscalls — so any reader polling for the file can win the gap and see a
+// created, still-empty file. Writing a sibling temp and rename()-ing it into
+// place is atomic on POSIX: readers see either the old file or the complete new
+// one, never a truncated one. Same directory so the rename never crosses a
+// filesystem. If rename is unavailable (some Windows AV/lock cases), fall back
+// to the direct write rather than losing the feedback.
+function writeFileAtomic(file, data) {
+  const tmp = file + '.' + process.pid + '.tmp';
+  try {
+    fs.writeFileSync(tmp, data);
+    fs.renameSync(tmp, file);
+  } catch (e) {
+    try { fs.rmSync(tmp, { force: true }); } catch (e2) { /* nothing to clean */ }
+    fs.writeFileSync(file, data);
+  }
+}
+
 function feedbackMarkdown(decision, thread) {
   const lines = ['# Plan Review Feedback', '',
     '- **Decision:** ' + decision,
@@ -262,7 +280,7 @@ function resolveDecision(decision, thread) {
     // browser tab live-reloads when the agent revises the plan on disk. On a
     // terminal decision (Approve/Reject) the review is over, so we exit after the
     // response flushes; on "Request Revisions" we keep serving the next round.
-    fs.writeFileSync(feedbackFile, md);
+    writeFileAtomic(feedbackFile, md);
     const payload = { decision: decision, feedback: md, feedbackFile: feedbackFile };
     if (decisionWaiter) {
       try { decisionWaiter.writeHead(200, { 'Content-Type': 'application/json' }); decisionWaiter.end(JSON.stringify(payload)); } catch (e) { /* client gone */ }
@@ -276,7 +294,7 @@ function resolveDecision(decision, thread) {
       setTimeout(function () { process.exit(0); }, 400);
     }
   } else {
-    fs.writeFileSync(feedbackFile, md);
+    writeFileAtomic(feedbackFile, md);
     log('\n[PLAN-REVIEW FEEDBACK -> ' + feedbackFile + ']\n' + md + '\n');
   }
 }

package/skills/plan-review/references/plan-review-template.html

@@ -39,6 +39,16 @@
   --info: #70b8ff;
   --info-muted: #0d2847;
   --info-border: #104d87;
+  /* categorical — series identity; neither the accent nor the semantics */
+  --cat-1: #cf686e;
+  --cat-2: #897ed6;
+  --cat-3: #41a265;
+  --cat-4: #3694d4;
+  --cat-5: #94911b;
+  --cat-6: #ba6cae;
+  --cat-7: #c37827;
+  --cat-8: #00a0a4;
+  --cat-other: #686560;
   --outline: rgba(237, 237, 236, 0.08);
   --subtle: #686560;
   --shadow-sm: 0 1px 2px rgba(6, 6, 5, 0.5);
@@ -112,6 +122,15 @@
     --info: #3f6fa6;
     --info-muted: #e6f1fc;
     --info-border: #bed7ef;
+    --cat-1: #b04d54;
+    --cat-2: #6f63b8;
+    --cat-3: #1e874b;
+    --cat-4: #0a78b7;
+    --cat-5: #797600;
+    --cat-6: #9d5192;
+    --cat-7: #a45e00;
+    --cat-8: #008f93;
+    --cat-other: #a8a294;
     --outline: rgba(54, 48, 42, 0.08);
     --subtle: #a8a294;
     --shadow-sm: 0 1px 2px rgba(54, 48, 42, 0.1);
@@ -148,6 +167,15 @@
   --info: #3f6fa6;
   --info-muted: #e6f1fc;
   --info-border: #bed7ef;
+  --cat-1: #b04d54;
+  --cat-2: #6f63b8;
+  --cat-3: #1e874b;
+  --cat-4: #0a78b7;
+  --cat-5: #797600;
+  --cat-6: #9d5192;
+  --cat-7: #a45e00;
+  --cat-8: #008f93;
+  --cat-other: #a8a294;
   --outline: rgba(54, 48, 42, 0.08);
   --subtle: #a8a294;
   --shadow-sm: 0 1px 2px rgba(54, 48, 42, 0.1);