@onepointfour-packs/flaredeck-agent 0.2.185 → 0.2.187

package/dist/serve.js

@@ -53,6 +53,7 @@ import { EventStream, describeFailure } from './stream.js';
 import { mergeableComments, combineComments, unparkReason, isBossInstruction, isTeamAsk } from './comment-merge.js';
 import { LogShipper } from './log-ship.js';
 import { isOwnMove } from './own-moves.js';
+import { baseBranchesFromComponent } from './workspace.js';
 import { isAfterReleaseMove } from './flaredeck-proxy.js';
 import { canPublish, canPushTo } from './githubauth.js';
 /**
@@ -1037,7 +1038,10 @@ export async function serve(opts) {
             log('event', 'At this pane? Type /local and questions for you are shown here instead of on your phone (back to /remote after ten minutes unanswered).');
             log('event', 'Merges and promotes go through release, which follows the board\'s rules for who may release where. Deletes and other limits are instructions for now (docs/spec-manager-role.md §14).');
         }
-        if ((isLead || isManager) && opts.deck && !releaseSet) {
+        // Workers get it too: the board's hotfix agent ships its own hotfixes (docs/spec-hotfix.md). The
+        // tool itself refuses whatever the board does not allow a worker; it is mounted on a worker's turn
+        // only for a hotfix card.
+        if (opts.deck && !releaseSet) {
             const tenant = tenantFromToken(await opts.auth());
             const deck = opts.deck;
             if (tenant) {
@@ -1073,6 +1077,18 @@ export async function serve(opts) {
                         c.on('error', () => res(''));
                     }),
                     log: (t) => log('tool', t),
+                    self: () => selfId ?? '',
+                    cardInfo: async (code) => {
+                        const c = await deck.card(code).catch(() => null);
+                        if (!c)
+                            return null;
+                        const since = c.release_since ? Date.parse(String(c.release_since)) : NaN;
+                        return {
+                            laneType: c.lane?.type ?? null,
+                            minutesInReleaseLane: String(c.lane?.type || '').toLowerCase() === 'release' && Number.isFinite(since) ? (Date.now() - since) / 60_000 : null,
+                            releases: (Array.isArray(c.releases) ? c.releases : []).map((r) => ({ environment: String(r.environment ?? ''), sha: String(r.sha ?? '') })),
+                        };
+                    },
                     record: async (card, text) => { await callFlaredeckTool(config.mcpUrl, await opts.auth(), 'create-card-comment', { card_id: card, comment: text }); },
                     recordReleased: async (o) => {
                         const res = await fetch(`${new URL(config.mcpUrl).origin}/api/v1/${encodeURIComponent(tenant)}/cards/released`, {
@@ -2655,13 +2671,18 @@ export async function serve(opts) {
     /** The person this would go to may be at this pane: flag it, and keep it off their phone. */
     hold, 
     /** It goes with a script request, which notifies by itself: list the card, ring nobody. */
-    quiet) => {
+    quiet, 
+    /** A wall the harness found itself, not the model asking: not judged for "could do it yourself". */
+    wall) => {
         const label = ref.code || ref.card || 'card';
         try {
             const out = await callFlaredeckTool(config.mcpUrl, await opts.auth(), 'set-needs-human', {
                 // The code when there is one — it resolves on its own. The board always goes with the key.
                 card_id: ref.code || ref.card,
                 ...(ref.board ? { board_id: ref.board } : {}),
+                // The harness found this wall itself; the model did not reach for help. Not judged for
+                // "could you do it yourself" — C3PO's "which repository?" was refused that way, silently.
+                ...(wall ? { harness_wall: true } : {}),
                 reason,
                 ...(kind ? { kind } : {}),
                 ...(ask ? { question: ask.question, options: ask.options } : {}),
@@ -2701,7 +2722,7 @@ export async function serve(opts) {
             card: event.cardKey ?? '',
             ...(event.boardKey ? { board: event.boardKey } : {}),
             ...(event.cardCode ? { code: event.cardCode } : {}),
-        }, reason, kind);
+        }, reason, kind, undefined, undefined, undefined, undefined, true);
     };
     async function handle(event) {
         const prefix = codePrefix(event.cardCode);
@@ -2831,6 +2852,13 @@ export async function serve(opts) {
         // it may work in — one repo, or every repo in that folder — and the card says which of them
         // this work belongs to. See workspace.ts: the folder is the only declaration, so there is no
         // config list to go stale.
+        // No repository details on the event (the harness's own pickup): the card names its repository.
+        if (event.cardKey && !(event.baseBranches ?? []).length && opts.deck) {
+            const here = await opts.deck.card(event.cardCode || event.cardKey).catch(() => null);
+            const fromCard = baseBranchesFromComponent(here?.component);
+            if (fromCard.length)
+                event = { ...event, baseBranches: fromCard };
+        }
         const where = await resolveCheckout(config.repo, event, scope, opts.agent ?? '');
         let chosen = where.kind === 'repo' ? where.checkout : null;
         let fromCard = where.kind === 'repo' && where.branch ? { branch: where.branch, role: where.role } : null;
@@ -2871,7 +2899,8 @@ export async function serve(opts) {
         const willChange = !!event.cardKey && event.event !== 'review' && event.event !== 'intake' && event.event !== 'ask' && event.forYou !== false && !isLeadDispatch(event, isLead);
         if (!chosen && willChange && (where.kind === 'ambiguous' || where.kind === 'unknown')) {
             const reason = askingForComponent(where);
-            await flagForHuman(event, reason);
+            // Which repository is a technical question about the board's setup, not a business one: COR-0311 (19 Oct) went to its reporter.
+            await flagForHuman(event, reason, 'technical');
             return;
         }
         // A FIX with no branch of its own is the one case worth stopping for.
@@ -3696,7 +3725,7 @@ export async function serve(opts) {
                 flaredeck,
                 runs: runsSet,
                 schedules: schedulesSet,
-                ...(releaseSet && (isLead || isManager) ? { release: releaseSet } : {}),
+                ...(releaseSet && (isLead || isManager || event.hotfix === true) ? { release: releaseSet } : {}),
             },
             guard: {
                 ...guard,