@feigi/fleet-ctl 3.21.8 → 3.21.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feigi/fleet-ctl",
-  "version": "3.21.8",
+  "version": "3.21.10",
   "description": "Agent fleet: run-team controller, merge bot, PR reviewer, and the ticket pipeline they share",
   "license": "Apache-2.0",
   "repository": {

package/scripts/member-record.mjs

@@ -468,8 +468,10 @@ export function foldOmpTranscript(jsonlText, filePath) {
 // `review-pr-<n>` and `merge-bot-<n>` ARE the dispatch name — exactly why
 // `parseMemberName` runs against this same stem below. Those members are
 // dispatched under the generic default `task` definition (`folded.agent` is
-// `undefined`), so the definition-based fix above cannot reach them, and
-// they fall through to prose classification of `folded.task` alone.
+// the literal string `task`, or `null` where no `session_init` line records
+// one), and classifyRole's definition branches match neither, so the
+// definition-based fix above cannot reach them; without `memberName` they
+// fall through to prose classification of `folded.task` alone.
 // Measured against docs/metrics/member-outcomes.tsv: 478 omp rows
 // carry such an AgentId, 52 of them booked `other` for want of this signal —
 // 31 `fix-pr-*`, 9 `impl-*` and 12 `merge-bot-*` members whose dispatch

package/scripts/member-record.test.mjs

@@ -309,16 +309,17 @@ test("readOmpMember: role is `-`, never a guess off the bare AgentId, when neith
 
 test("readOmpMember: a canonically-named AgentId is itself a role signal under the default `task` definition (#1506)", () => {
   // `fix-pr-1380`, `merge-bot-5`, `impl-1049` and friends are dispatched
-  // under run-team's default `task` definition — `session_init.agent` is
-  // absent. That is a different gap from #1502's: #1502 fixed a forwarding
+  // under run-team's default `task` definition — `session_init.agent` is the
+  // literal string `task`, which no definition branch of classifyRole
+  // matches. That is a different gap from #1502's: #1502 fixed a forwarding
   // bug where an EXISTING `folded.agent` value wasn't reaching classifyRole;
-  // here `session_init.agent` was never recorded at all. But the AgentId
+  // here the recorded definition names no role at all. But the AgentId
   // itself IS the dispatch name (run-team's own naming convention), so it
   // must reach classifyRole as `memberName` and win a real classification
   // even when the dispatch prompt's own prose says nothing role-shaped.
   const noKeywords = [
     sessionEvt("/Users/chris/dev/fleet-plugin"), thinkingEvt("high"),
-    sessionInitEvt("Apply the requested patch set and open a PR.", "anthropic/claude-sonnet-5"),
+    sessionInitEvt("Apply the requested patch set and open a PR.", "anthropic/claude-sonnet-5", "task"),
     assistantEvt("claude-sonnet-5", { input: 1, output: 1, cacheRead: 0, cacheWrite: 10, totalTokens: 12 }),
   ].join("\n");
   assert.equal(readOmpMember(noKeywords, "/fake/path.jsonl", "fix-pr-1380", 0).role, "reviewer");