npm - @mrciphersmith/keryx - Versions diffs - 0.3.21 → 0.3.22 - Mend

@mrciphersmith/keryx 0.3.21 → 0.3.22

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (4) hide show

package/dist/cli.js CHANGED Viewed

@@ -18118,7 +18118,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@mrciphersmith/keryx",
-    version: "0.3.21",
+    version: "0.3.22",
     description: "Version-controlled project context for AI coding agents: code graph, architecture wiki, project memory, relevant tests, quality signals, and task flows.",
     private: false,
     publishConfig: {
@@ -151131,6 +151131,15 @@ function reinsertMainQueueItem(items, at, item) {
   next.splice(insertAt, 0, item);
   return next;
 }
+function drainIdleMainQueue(items, options) {
+  if (!options.isIdle())
+    return false;
+  const next = options.takeForced() ?? items.shift();
+  if (next === undefined)
+    return false;
+  options.dispatch(next);
+  return true;
+}
 // src/tui/queue-nav.ts
 var QUEUE_NAV_ACTIONS = ["force", "edit", "delete"];
@@ -160027,6 +160036,14 @@ Staying in the current session.
             const id = `mq${mainQueueSeq++}`;
             mainQueue.push({ id, question: line, displayQuestion: displayLine });
             paintMainQueue();
+            drainIdleMainQueue(mainQueue, {
+              isIdle: () => !destroyed && !chrome.isBusy() && !foregroundOperation.isActive && !forceHandoff.isAwaitingSettlement && leaseView()?.held() !== true,
+              takeForced: () => forceHandoff.takeNext(),
+              dispatch: (next) => {
+                paintMainQueue();
+                runLine(next.question);
+              }
+            });
           } else {
             appendUserEcho(otui, r, transcript, {
               id: `side-q${uid++}`,

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@mrciphersmith/keryx",
-  "version": "0.3.21",
+  "version": "0.3.22",
   "description": "Version-controlled project context for AI coding agents: code graph, architecture wiki, project memory, relevant tests, quality signals, and task flows.",
   "private": false,
   "publishConfig": {

package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md CHANGED Viewed

@@ -1587,10 +1587,12 @@ Fix: <concrete>
 - <what was checked> — <how>
 ### How this review was run
-- **Run by:** @<gh-login> with `review-orchestrator`
+- **Workflow:** `review-orchestrator`
 - **Scope:** `<base>..<head>`, round N, PR #N
-- **Orchestrator:** `review-orchestrator`
-- **<actual model name>:** <reviewers on that model>
+- **Models:** <actual models and reviewers, or not recorded>
+- **Tools:** <tools actually invoked, or not recorded>
+- **Skills:** <skills actually used, or not recorded>
+- **Subagents:** <dispatched reviewers and models, or none when confirmed; otherwise not recorded>
 - **Not run:** <reviewer — reason>
 - **Verification:** <mode>; confirmed N, refuted N, unverifiable N, unverified N
 ```
@@ -1663,17 +1665,15 @@ Automation values, names unchanged:
 Default is do not publish. No resolvable PR number means skip and say so.
-The comment is the report rendered from `templates/review-report.md`, English, with the domain file chosen above (`templates/pr-comment-frontend.md` / `templates/pr-comment-backend.md`) — never a summary written freehand. It does not use a tool heading, a finding table, or a meta table. It does not carry a co-author line, a `Generated with` trailer, or any sentence that names a vendor or a product as the author. Say who ran the orchestrator and which reviewers ran; do not sign the comment as them.
+The comment is the report rendered from `templates/review-report.md`, English, with the domain file chosen above (`templates/pr-comment-frontend.md` / `templates/pr-comment-backend.md`) — never a summary written freehand. It does not use a tool heading, a finding table, or a meta table. It does not carry a co-author line, a `Generated with` trailer, or any sentence that names a vendor or a product as the author. Name the workflow and reviewers that actually ran, with models, tools, skills and subagents. Never identify the operator from `gh auth status`, git config or a credential; do not sign the comment as a person, reviewer or model.
 The follow-up file path and the metadata rules (real model names, Run vs Not run, no `adaptive` in the model slot) live in that same template. Write the body to a temp file and post with `gh pr comment <n> --body-file <file>`.
 **No GitHub write without an approved draft.** Before ANY write — the comment, a thread reply, `keryx review comments reply`, a review — show the user the rendered body and wait for explicit approval of that body.
 Picking A or B above chooses *whether* to publish, not *what*; a comment is public and cannot be unsent. With no user to answer (a dispatched run), do not write: hand the rendered body back to the caller.
-Re-read head and the thread immediately before posting. If head moved, re-check
-the findings against the new head and name the commits that were not reviewed.
-Do not post a finding another reviewer already filed on this head; if yours
-extends theirs, cite theirs.
+Re-read head and thread before posting. If head moved, re-check findings
+and name unchecked commits. Do not duplicate findings; cite extended ones.
 ## Job Context Awareness

package/src/gdskills/bundled/skills/review/review-orchestrator/templates/review-report.md CHANGED Viewed

@@ -95,14 +95,19 @@ Do not list a checklist item that was not checked. Do not praise.
 ## How this review was run
-A list, not a meta table. Keep every fact the old Meta table carried.
+A factual run inventory, not an author or account signature. List only steps
+confirmed by the run record; never infer an operator identity from `gh auth`,
+git config, a credential or a PR author. Use `not recorded` when evidence for
+a tool, skill or model is unavailable; use `none` only when the run proves none.
 ```markdown
 ### How this review was run
-- **Run by:** @<gh-login> with `review-orchestrator`. Severity and lanes per the repository review rules, else the canonical rubric in this skill.
+- **Workflow:** `review-orchestrator`; severity and lanes per the repository review rules, else the canonical rubric in this skill.
 - **Scope:** `<base>..<head>`, round N, PR #N. Unchecked commits: <list or none>.
-- **Orchestrator:** `review-orchestrator`
-- **<actual model name>:** <reviewers dispatched on it, comma-separated>
+- **Models:** <actual model names and the reviewers that ran on each; or not recorded>.
+- **Tools:** <tools actually invoked for this review and their roles; or not recorded>.
+- **Skills:** <skills actually loaded or executed and their roles; or not recorded>.
+- **Subagents:** <dispatched subagents, their reviewer roles and actual models; or none when confirmed, otherwise not recorded>.
 - **Fallback:** <reviewer> via `general-purpose` because <native agent type unavailable | no bundled agent>. Or `none`.
 - **Not run:** <reviewer> — <short reason>. Or `none`.
 - **Selection:** <auto-detected scope | explicit flags | optional groups the user picked>
@@ -118,8 +123,8 @@ Model rules, unchanged from the old metadata table:
 - Name the model that actually ran. Never write `adaptive`, `inherit`,
   `unsupported`, or `current-session` in the model slot.
 - Group reviewers under the model they ran on. One bullet per model.
-- `Model strategy` (`ask` or `adaptive`) goes on the Run by line or under
-  Selection, not in place of a model name.
+- `Model strategy` (`ask` or `adaptive`) goes under Selection, never in
+  place of an actual model name.
 - Keep **Run** complete. If Not run is long, group it with counts and notable
   names; the full list belongs in the follow-up file.