@maka/maka-cli 5.64.0 → 5.65.1

package/bundle/typescript/src/commands/game/sideQuest/utilities/affordances.js

@@ -0,0 +1,36 @@
+export const AFFORDANCE_FLAVOR = {
+    lock: {
+        verb: 'pick',
+        verbs: ['pick', 'breach'],
+        resists: `It's a mechanism, not a system -- no console to intrude on, no weave to unpick, nobody to talk round. It gives to HANDS: "pick" it quietly, or "breach" it with your shoulder.`,
+        failureMessage: `Your pick snaps in the mechanism -- and the lock's alarm isn't silent anymore.`,
+        hint: `This looks forceable -- "pick" works it quietly (Agility + locksmith); "breach" forces it with raw muscle (Strength + Body, LOUD even when it works). A key opens it clean, if you can find one.`,
+    },
+    panel: {
+        verb: 'hack',
+        verbs: ['hack'],
+        resists: `No keyway, no seam, no tumblers to feel for -- it's a sealed SYSTEM, and it only argues with a deck. "hack" it.`,
+        failureMessage: `Your intrusion trips a countermeasure -- the system logs the attempt and pings whoever's watching.`,
+        hint: `This looks hackable -- try "hack" to skip finding the code (Logic + hacking). Fail, and the system pings whoever's watching.`,
+    },
+    ward: {
+        verb: 'cast',
+        verbs: ['cast', 'dispel'],
+        resists: `Your hands find nothing to work and your tools find nothing to bite -- the barrier isn't matter. It answers to the Art: "cast" at it, or "dispel" it.`,
+        failureMessage: `The weave slips through your fingers -- backlash crackles outward, and something out there felt it.`,
+        hint: `This could be unwoven magically -- try "dispel" (Magic + counterspelling) or "cast" (the casting pool). Fail, and the backlash BITES.`,
+    },
+    person: {
+        verb: 'bluff',
+        verbs: ['bluff', 'persuade', 'subdue'],
+        resists: `There's no lock here at all. There's a PERSON -- and people don't open to tools, they open to a story, an argument, or a shove. "bluff" them, "persuade" them, or "subdue" them.`,
+        failureMessage: `The story comes apart mid-sentence -- and now they KNOW someone's running a game on them.`,
+        hint: `This one is a person, not a mechanism: "bluff" past them (Charisma + con), "persuade" them (Charisma + negotiation), or put them down with "subdue".`,
+    },
+};
+/** The verbs that work a thing of this kind -- used wherever the game
+ *  has to tell a player exactly what they may type. */
+export function verbsFor(kind) {
+    return AFFORDANCE_FLAVOR[kind]?.verbs ?? [];
+}
+//# sourceMappingURL=affordances.js.map

package/bundle/typescript/src/commands/game/sideQuest/utilities/affordances.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"affordances.js","sourceRoot":"","sources":["../../../../../../../src/commands/game/sideQuest/utilities/affordances.ts"],"names":[],"mappings":"AAsDA,MAAM,CAAC,MAAM,iBAAiB,GAAwC;IACpE,IAAI,EAAE;QACJ,IAAI,EAAE,MAAM;QACZ,KAAK,EAAE,CAAC,MAAM,EAAE,QAAQ,CAAC;QACzB,OAAO,EAAE,gLAAgL;QACzL,cAAc,EAAE,gFAAgF;QAChG,IAAI,EAAE,kMAAkM;KACzM;IACD,KAAK,EAAE;QACL,IAAI,EAAE,MAAM;QACZ,KAAK,EAAE,CAAC,MAAM,CAAC;QACf,OAAO,EAAE,iHAAiH;QAC1H,cAAc,EAAE,oGAAoG;QACpH,IAAI,EAAE,8HAA8H;KACrI;IACD,IAAI,EAAE;QACJ,IAAI,EAAE,MAAM;QACZ,KAAK,EAAE,CAAC,MAAM,EAAE,QAAQ,CAAC;QACzB,OAAO,EAAE,uJAAuJ;QAChK,cAAc,EAAE,qGAAqG;QACrH,IAAI,EAAE,uIAAuI;KAC9I;IACD,MAAM,EAAE;QACN,IAAI,EAAE,OAAO;QACb,KAAK,EAAE,CAAC,OAAO,EAAE,UAAU,EAAE,QAAQ,CAAC;QACtC,OAAO,EAAE,kLAAkL;QAC3L,cAAc,EAAE,2FAA2F;QAC3G,IAAI,EAAE,sJAAsJ;KAC7J;CACF,CAAC;AAEF;uDACuD;AACvD,MAAM,UAAU,QAAQ,CAAC,IAAoB;IAC3C,OAAO,iBAAiB,CAAC,IAAI,CAAC,EAAE,KAAK,IAAI,EAAE,CAAC;AAC9C,CAAC"}

package/bundle/typescript/src/commands/game/sideQuest/utilities/backlog-review.d.ts

@@ -0,0 +1,65 @@
+import { IReviewItem, BacklogOutcome } from './backlog.js';
+/** Replace the rendered list. Called ONLY when a fresh queue is shown
+ *  to the player -- that is what makes the numbers theirs. */
+export declare function rememberQueue(items: IReviewItem[]): void;
+/** Exactly what was last rendered, in the order it was rendered. */
+export declare function reviewSnapshot(): IReviewItem[];
+/** 1-based, because the list is printed 1..n to a human. Out of range
+ *  and non-integers return undefined; the caller names the range. */
+export declare function reviewItemAt(n: number): IReviewItem | undefined;
+export declare function markReviewAnswered(id: string): void;
+export declare function isReviewAnswered(id: string): boolean;
+/** Only for tests and a fresh session. */
+export declare function resetReviewQueue(): void;
+/** How many items were waiting when the session started, or -1 if the
+ *  probe never answered (offline, logged out, not a play-tester). */
+export declare function pendingReviewCount(): number;
+/**
+ * The login-time count, fetched in the background.
+ *
+ * Same shape as the play-tester probe deliberately -- fire and forget,
+ * never block a boot, never throw, and an unanswered probe simply
+ * leaves the notice unsaid. Reusing that pattern rather than inventing
+ * a second one for the same job.
+ *
+ * It also folds its outcome into the play-tester standing for free:
+ * this endpoint is role-gated exactly like the board, so a 200 proves
+ * the role and a 403 disproves it, and a session that runs this probe
+ * needs no second request to classify the account.
+ */
+export declare function probeReviewQueue(opts: {
+    anonymous: boolean;
+}): Promise<void>;
+/**
+ * THE NOTICE, one line. "Someone asked about your report" and "this is
+ * fixed, can you confirm it" are different requests and must not read
+ * identically, but the count alone cannot tell them apart -- so the
+ * notice stays neutral about WHICH, and the list itself distinguishes
+ * them per item. Empty string when there is nothing to say.
+ */
+export declare function reviewNotice(): string;
+/** Fetch a fresh queue for display, and adopt it as the numbering. */
+export declare function loadReviewQueue(): Promise<{
+    outcome: BacklogOutcome;
+    items?: IReviewItem[];
+}>;
+/**
+ * IS THIS ROW A "GO AND CONFIRM IT" REQUEST, or a question about the
+ * report?
+ *
+ * ONE definition, used by BOTH the renderer and the action, because the
+ * cost of them disagreeing is silent and lands on a person: the list
+ * says "CAN YOU CONFIRM THIS IN PLAY?" and the keystroke posts a
+ * comment that closes nothing. That is exactly what shipped before
+ * 2026-08-27 (item jyjMYRXrS4KojAL2W) -- not because either half was
+ * wrong, but because only one half existed.
+ *
+ * The SERVER's `kind` wins. The status fallback is only for a server too
+ * old to send one, and it is a guess: deriving the kind from status
+ * mislabels a clarifying question asked about an item that happens to be
+ * in-review.
+ */
+export declare function isValidationRow(item: {
+    kind?: string;
+    status?: string;
+}): boolean;

package/bundle/typescript/src/commands/game/sideQuest/utilities/backlog-review.js

@@ -0,0 +1,137 @@
+import { fetchReviewQueue } from './backlog.js';
+import { noteBacklogOutcome } from './playtester.js';
+/**
+ * THE REVIEW QUEUE, AND THE NUMBERS THAT POINT AT IT (user request
+ * 2026-08-27: "when they login, they should get a message about a
+ * backlog item for review... a simple list (1,2,3,4,etc.) in which they
+ * can 'backlog review 1 <test/comment/etc>'").
+ *
+ * THE ONE INVARIANT THIS MODULE EXISTS TO HOLD: a number means what the
+ * player was shown when they read it. The server returns the queue
+ * OLDEST FIRST so the ordering is stable, and this keeps the SNAPSHOT
+ * that was rendered, so resolving an index never re-sorts, re-filters
+ * or refetches. A queue that reshuffles between being read and being
+ * answered posts the player's words against the wrong item -- and they
+ * would never find out, because the answer lands somewhere plausible.
+ *
+ * Two consequences that look like fussiness and are not:
+ *  - Answering does NOT renumber. Item 1 leaving the queue must not
+ *    silently turn item 2 into item 1 while the list is still on
+ *    screen; the entry is marked answered and everything keeps its
+ *    number until the player asks for a fresh list.
+ *  - Resolving a number with no snapshot does NOT fetch one and guess.
+ *    A list nobody has read cannot have been the list they counted on,
+ *    so the caller shows the queue and asks them to pick again.
+ */
+let snapshot = [];
+let answeredIds = new Set();
+/** Count from the background login probe, for the notice. -1 = unknown. */
+let pendingAtLogin = -1;
+let probing;
+/** Replace the rendered list. Called ONLY when a fresh queue is shown
+ *  to the player -- that is what makes the numbers theirs. */
+export function rememberQueue(items) {
+    snapshot = [...items];
+    answeredIds = new Set();
+}
+/** Exactly what was last rendered, in the order it was rendered. */
+export function reviewSnapshot() {
+    return snapshot;
+}
+/** 1-based, because the list is printed 1..n to a human. Out of range
+ *  and non-integers return undefined; the caller names the range. */
+export function reviewItemAt(n) {
+    if (!Number.isInteger(n) || n < 1 || n > snapshot.length)
+        return undefined;
+    return snapshot[n - 1];
+}
+export function markReviewAnswered(id) {
+    answeredIds.add(id);
+}
+export function isReviewAnswered(id) {
+    return answeredIds.has(id);
+}
+/** Only for tests and a fresh session. */
+export function resetReviewQueue() {
+    snapshot = [];
+    answeredIds = new Set();
+    pendingAtLogin = -1;
+    probing = undefined;
+}
+/** How many items were waiting when the session started, or -1 if the
+ *  probe never answered (offline, logged out, not a play-tester). */
+export function pendingReviewCount() {
+    return pendingAtLogin;
+}
+/**
+ * The login-time count, fetched in the background.
+ *
+ * Same shape as the play-tester probe deliberately -- fire and forget,
+ * never block a boot, never throw, and an unanswered probe simply
+ * leaves the notice unsaid. Reusing that pattern rather than inventing
+ * a second one for the same job.
+ *
+ * It also folds its outcome into the play-tester standing for free:
+ * this endpoint is role-gated exactly like the board, so a 200 proves
+ * the role and a 403 disproves it, and a session that runs this probe
+ * needs no second request to classify the account.
+ */
+export function probeReviewQueue(opts) {
+    if (opts.anonymous || pendingAtLogin >= 0)
+        return Promise.resolve();
+    probing ??= fetchReviewQueue()
+        .then(res => {
+        noteBacklogOutcome(res.outcome);
+        if (res.outcome === 'ok')
+            pendingAtLogin = res.items?.length ?? 0;
+    })
+        .catch(() => undefined);
+    return probing;
+}
+/**
+ * THE NOTICE, one line. "Someone asked about your report" and "this is
+ * fixed, can you confirm it" are different requests and must not read
+ * identically, but the count alone cannot tell them apart -- so the
+ * notice stays neutral about WHICH, and the list itself distinguishes
+ * them per item. Empty string when there is nothing to say.
+ */
+export function reviewNotice() {
+    const n = pendingReviewCount();
+    if (n <= 0)
+        return '';
+    return n === 1
+        ? `{cyan-fg}📋 One backlog item is waiting on you -- somebody needs a word back. ("backlog review"){/cyan-fg}`
+        : `{cyan-fg}📋 ${n} backlog items are waiting on you -- somebody needs a word back on each. ("backlog review"){/cyan-fg}`;
+}
+/** Fetch a fresh queue for display, and adopt it as the numbering. */
+export async function loadReviewQueue() {
+    const res = await fetchReviewQueue();
+    noteBacklogOutcome(res.outcome);
+    if (res.outcome === 'ok') {
+        rememberQueue(res.items ?? []);
+        pendingAtLogin = res.items?.length ?? 0;
+    }
+    return res;
+}
+/**
+ * IS THIS ROW A "GO AND CONFIRM IT" REQUEST, or a question about the
+ * report?
+ *
+ * ONE definition, used by BOTH the renderer and the action, because the
+ * cost of them disagreeing is silent and lands on a person: the list
+ * says "CAN YOU CONFIRM THIS IN PLAY?" and the keystroke posts a
+ * comment that closes nothing. That is exactly what shipped before
+ * 2026-08-27 (item jyjMYRXrS4KojAL2W) -- not because either half was
+ * wrong, but because only one half existed.
+ *
+ * The SERVER's `kind` wins. The status fallback is only for a server too
+ * old to send one, and it is a guess: deriving the kind from status
+ * mislabels a clarifying question asked about an item that happens to be
+ * in-review.
+ */
+export function isValidationRow(item) {
+    if (item.kind)
+        return item.kind === 'validation';
+    return item.status === 'complete' || item.status === 'in-review';
+}
+//# sourceMappingURL=backlog-review.js.map

package/bundle/typescript/src/commands/game/sideQuest/utilities/backlog-review.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"backlog-review.js","sourceRoot":"","sources":["../../../../../../../src/commands/game/sideQuest/utilities/backlog-review.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAA+B,MAAM,cAAc,CAAC;AAC7E,OAAO,EAAE,kBAAkB,EAAE,MAAM,iBAAiB,CAAC;AAErD;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAEH,IAAI,QAAQ,GAAkB,EAAE,CAAC;AACjC,IAAI,WAAW,GAAG,IAAI,GAAG,EAAU,CAAC;AACpC,2EAA2E;AAC3E,IAAI,cAAc,GAAG,CAAC,CAAC,CAAC;AACxB,IAAI,OAAkC,CAAC;AAEvC;8DAC8D;AAC9D,MAAM,UAAU,aAAa,CAAC,KAAoB;IAChD,QAAQ,GAAG,CAAC,GAAG,KAAK,CAAC,CAAC;IACtB,WAAW,GAAG,IAAI,GAAG,EAAE,CAAC;AAC1B,CAAC;AAED,oEAAoE;AACpE,MAAM,UAAU,cAAc;IAC5B,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED;qEACqE;AACrE,MAAM,UAAU,YAAY,CAAC,CAAS;IACpC,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,GAAG,QAAQ,CAAC,MAAM;QAAE,OAAO,SAAS,CAAC;IAC3E,OAAO,QAAQ,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;AACzB,CAAC;AAED,MAAM,UAAU,kBAAkB,CAAC,EAAU;IAC3C,WAAW,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;AACtB,CAAC;AAED,MAAM,UAAU,gBAAgB,CAAC,EAAU;IACzC,OAAO,WAAW,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;AAC7B,CAAC;AAED,0CAA0C;AAC1C,MAAM,UAAU,gBAAgB;IAC9B,QAAQ,GAAG,EAAE,CAAC;IACd,WAAW,GAAG,IAAI,GAAG,EAAE,CAAC;IACxB,cAAc,GAAG,CAAC,CAAC,CAAC;IACpB,OAAO,GAAG,SAAS,CAAC;AACtB,CAAC;AAED;qEACqE;AACrE,MAAM,UAAU,kBAAkB;IAChC,OAAO,cAAc,CAAC;AACxB,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,gBAAgB,CAAC,IAA4B;IAC3D,IAAI,IAAI,CAAC,SAAS,IAAI,cAAc,IAAI,CAAC;QAAE,OAAO,OAAO,CAAC,OAAO,EAAE,CAAC;IACpE,OAAO,KAAK,gBAAgB,EAAE;SAC3B,IAAI,CAAC,GAAG,CAAC,EAAE;QACV,kBAAkB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAChC,IAAI,GAAG,CAAC,OAAO,KAAK,IAAI;YAAE,cAAc,GAAG,GAAG,CAAC,KAAK,EAAE,MAAM,IAAI,CAAC,CAAC;IACpE,CAAC,CAAC;SACD,KAAK,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,CAAC;IAC1B,OAAO,OAAO,CAAC;AACjB,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,YAAY;IAC1B,MAAM,CAAC,GAAG,kBAAkB,EAAE,CAAC;IAC/B,IAAI,CAAC,IAAI,CAAC;QAAE,OAAO,EAAE,CAAC;IACtB,OAAO,CAAC,KAAK,CAAC;QACZ,CAAC,CAAC,4GAA4G;QAC9G,CAAC,CAAC,eAAe,CAAC,uGAAuG,CAAC;AAC9H,CAAC;AAED,sEAAsE;AACtE,MAAM,CAAC,KAAK,UAAU,eAAe;IACnC,MAAM,GAAG,GAAG,MAAM,gBAAgB,EAAE,CAAC;IACrC,kBAAkB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;IAChC,IAAI,GAAG,CAAC,OAAO,KAAK,IAAI,EAAE,CAAC;QACzB,aAAa,CAAC,GAAG,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;QAC/B,cAAc,GAAG,GAAG,CAAC,KAAK,EAAE,MAAM,IAAI,CAAC,CAAC;IAC1C,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,eAAe,CAAC,IAAwC;IACtE,IAAI,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC,IAAI,KAAK,YAAY,CAAC;IACjD,OAAO,IAAI,CAAC,MAAM,KAAK,UAAU,IAAI,IAAI,CAAC,MAAM,KAAK,WAAW,CAAC;AACnE,CAAC"}

package/bundle/typescript/src/commands/game/sideQuest/utilities/backlog.d.ts

@@ -1,4 +1,13 @@
-export type BacklogStatus = 'open' | 'in-progress' | 'complete' | 'wont-fix';
+/**
+ * `in-review` was missing here until 2026-08-27, and the first
+ * populated review payload is what exposed it: the server has been able
+ * to park a fixed item in review -- awaiting the reporter's word in
+ * play -- while this list did not know the state existed. An unknown
+ * status fell to statusMark's default and rendered as "[open]", so the
+ * board showed an item awaiting confirmation as though nobody had
+ * touched it, and `backlog list in-review` was refused as a typo.
+ */
+export type BacklogStatus = 'open' | 'in-progress' | 'in-review' | 'complete' | 'wont-fix';
 export declare const BACKLOG_STATUSES: BacklogStatus[];
 export interface IBacklogItem {
     _id: string;
@@ -37,6 +46,86 @@ export interface IBacklogMeta {
     platform?: string;
 }
 export declare function submitBacklog(text: string, meta: IBacklogMeta): Promise<BacklogOutcome>;
+/**
+ * AN ITEM WAITING ON THE PERSON WHO FILED IT (user request 2026-08-27:
+ * "when they login, they should get a message about a backlog item for
+ * review... a clarifying question, or playtest validation").
+ *
+ * `asked` is the last question aimed at this play-tester with no later
+ * word from them -- either someone needs the report clarified, or a fix
+ * has landed and wants confirming in play. The server models both as a
+ * question awaiting the reporter, so there is one mechanism; the two
+ * read differently in prose because they are different requests.
+ */
+export interface IReviewItem {
+    id: string;
+    text: string;
+    status: BacklogStatus;
+    /** WHICH REQUEST THIS IS, from the server rather than guessed. A
+     *  clarifying question and "this is fixed, confirm it in play" are
+     *  different jobs; the client used to infer the difference from
+     *  `status`, which would mislabel a clarifying question asked about
+     *  an item that happens to be in-progress. Optional because the
+     *  client must not break on an older server; see reviewKind. */
+    kind?: 'question' | 'validation';
+    /** The commits that closed it -- present on a validation request,
+     *  which is the case where "what changed?" is the player's next
+     *  thought. */
+    commits?: string[];
+    /** OPTIONAL on the wire, and handled as such. The review queue only
+     *  returns items with a question outstanding, so it should always be
+     *  here -- but "should always" is how a render throws in front of a
+     *  player, and an item we cannot describe is better skipped than
+     *  fatal. */
+    asked?: {
+        by?: string;
+        text: string;
+        at?: string;
+    };
+}
+/**
+ * The review queue, OLDEST FIRST. That order is load-bearing and comes
+ * from the server: the player answers by NUMBER, so the list they read
+ * and the list a number resolves against must be the same list in the
+ * same order. Callers must not re-sort, re-filter, or refetch between
+ * showing it and resolving an index -- a queue that reshuffles in
+ * between posts the answer against the wrong item, and it lands
+ * somewhere plausible enough that nobody notices.
+ */
+export declare function fetchReviewQueue(): Promise<{
+    outcome: BacklogOutcome;
+    items?: IReviewItem[];
+}>;
+/** Answer, ask, or annotate one item. `kind` defaults to a plain note;
+ *  a play-tester replying to a question posts an 'answer'. */
+export declare function postBacklogComment(itemId: string, text: string, opts?: {
+    kind?: 'question' | 'answer' | 'note';
+    awaitingReporter?: boolean;
+    runnerName?: string;
+}): Promise<BacklogOutcome>;
+/**
+ * THE REPORTER CONFIRMS A FIX IN PLAY -- the only thing that closes a
+ * validation request.
+ *
+ * MISSING UNTIL 2026-08-27, and the omission is the whole of backlog
+ * item jyjMYRXrS4KojAL2W ("after confirming a review, it should be
+ * removed from the review list"). The server has had
+ * POST /item/:id/signoff since the sign-off rule landed; the CLIENT
+ * never called it. `backlog review <n> <answer>` posted a COMMENT for
+ * every row, including the ones asking "can you confirm this in play?"
+ *
+ * So confirming did exactly nothing to the row. The comment landed, the
+ * answer was on the record, `signoff` stayed unset, and the queue kept
+ * showing the item because `status === in-review && !signoff` was still
+ * true. From the play-tester's chair the confirmation was simply
+ * ignored -- and the board could never reach `complete`, because the
+ * server refuses that without a sign-off it was never sent.
+ *
+ * REPORTER-ONLY server-side, which is the point: an admin can stage a
+ * fix and attach commits, but only the person who reported it can say
+ * it is actually fixed.
+ */
+export declare function postBacklogSignoff(itemId: string, note: string, runnerName?: string): Promise<BacklogOutcome>;
 export interface IRoleHolder {
     email?: string;
     roles: string[];

package/bundle/typescript/src/commands/game/sideQuest/utilities/backlog.js

@@ -25,7 +25,7 @@ const HOSTNAME = DEV ? 'localhost' : 'www.maka-cli.com';
 const PORT = DEV ? Number(process.env.MAKA_DEV_PORT ?? 3000) : 443;
 const API_BASE = 'api/v1/game-backlog';
 const TIMEOUT_MS = 10000;
-export const BACKLOG_STATUSES = ['open', 'in-progress', 'complete', 'wont-fix'];
+export const BACKLOG_STATUSES = ['open', 'in-progress', 'in-review', 'complete', 'wont-fix'];
 function request(method, apiPath, token, body) {
     return new Promise((resolve, reject) => {
         const payload = body === undefined ? undefined : JSON.stringify(body);
@@ -85,6 +85,108 @@ export async function submitBacklog(text, meta) {
         return 'offline';
     }
 }
+/**
+ * The review queue, OLDEST FIRST. That order is load-bearing and comes
+ * from the server: the player answers by NUMBER, so the list they read
+ * and the list a number resolves against must be the same list in the
+ * same order. Callers must not re-sort, re-filter, or refetch between
+ * showing it and resolving an index -- a queue that reshuffles in
+ * between posts the answer against the wrong item, and it lands
+ * somewhere plausible enough that nobody notices.
+ */
+export async function fetchReviewQueue() {
+    const token = authToken();
+    if (!token)
+        return { outcome: 'no-login' };
+    try {
+        const res = await request('GET', `/${API_BASE}/review`, token);
+        if (res.statusCode === 401)
+            return { outcome: 'expired' };
+        if (res.statusCode === 403)
+            return { outcome: 'forbidden' };
+        if (res.statusCode !== 200)
+            return { outcome: 'offline' };
+        const parsed = parseData(res.data);
+        return { outcome: 'ok', items: parsed?.items ?? [] };
+    }
+    catch {
+        return { outcome: 'offline' };
+    }
+}
+/** Answer, ask, or annotate one item. `kind` defaults to a plain note;
+ *  a play-tester replying to a question posts an 'answer'. */
+export async function postBacklogComment(itemId, text, opts = {}) {
+    const token = authToken();
+    if (!token)
+        return 'no-login';
+    try {
+        const res = await request('POST', `/${API_BASE}/item/${encodeURIComponent(itemId)}/comment`, token, {
+            text,
+            kind: opts.kind,
+            awaitingReporter: opts.awaitingReporter,
+            runnerName: opts.runnerName,
+        });
+        if (res.statusCode === 200 || res.statusCode === 201)
+            return 'ok';
+        if (res.statusCode === 401)
+            return 'expired';
+        if (res.statusCode === 403)
+            return 'forbidden';
+        if (res.statusCode === 400 || res.statusCode === 404)
+            return 'rejected';
+        return 'offline';
+    }
+    catch {
+        return 'offline';
+    }
+}
+/**
+ * THE REPORTER CONFIRMS A FIX IN PLAY -- the only thing that closes a
+ * validation request.
+ *
+ * MISSING UNTIL 2026-08-27, and the omission is the whole of backlog
+ * item jyjMYRXrS4KojAL2W ("after confirming a review, it should be
+ * removed from the review list"). The server has had
+ * POST /item/:id/signoff since the sign-off rule landed; the CLIENT
+ * never called it. `backlog review <n> <answer>` posted a COMMENT for
+ * every row, including the ones asking "can you confirm this in play?"
+ *
+ * So confirming did exactly nothing to the row. The comment landed, the
+ * answer was on the record, `signoff` stayed unset, and the queue kept
+ * showing the item because `status === in-review && !signoff` was still
+ * true. From the play-tester's chair the confirmation was simply
+ * ignored -- and the board could never reach `complete`, because the
+ * server refuses that without a sign-off it was never sent.
+ *
+ * REPORTER-ONLY server-side, which is the point: an admin can stage a
+ * fix and attach commits, but only the person who reported it can say
+ * it is actually fixed.
+ */
+export async function postBacklogSignoff(itemId, note, runnerName) {
+    const token = authToken();
+    if (!token)
+        return 'no-login';
+    try {
+        const res = await request('POST', `/${API_BASE}/item/${encodeURIComponent(itemId)}/signoff`, token, {
+            note,
+            runnerName,
+        });
+        if (res.statusCode === 200 || res.statusCode === 201)
+            return 'ok';
+        if (res.statusCode === 401)
+            return 'expired';
+        // 403 is a REAL answer here, not a role problem: the server is
+        // saying this item is not yours to sign off.
+        if (res.statusCode === 403)
+            return 'forbidden';
+        if (res.statusCode === 400 || res.statusCode === 404)
+            return 'rejected';
+        return 'offline';
+    }
+    catch {
+        return 'offline';
+    }
+}
 /** Who holds what. Admin-only server-side; a non-admin gets 'forbidden'
  *  and the command turns that into a sentence. */
 export async function fetchRoles() {

package/bundle/typescript/src/commands/game/sideQuest/utilities/backlog.js.map

@@ -1 +1 @@
-{"version":3,"file":"backlog.js","sourceRoot":"","sources":["../../../../../../../src/commands/game/sideQuest/utilities/backlog.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAE7C;;;;;;;;;;;;;;;;;;GAkBG;AAEH,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,QAAQ,KAAK,MAAM,CAAC;AAC5C,MAAM,QAAQ,GAAG,GAAG,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,kBAAkB,CAAC;AACxD,MAAM,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,aAAa,IAAI,IAAI,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC;AACnE,MAAM,QAAQ,GAAG,qBAAqB,CAAC;AACvC,MAAM,UAAU,GAAG,KAAK,CAAC;AAGzB,MAAM,CAAC,MAAM,gBAAgB,GAAoB,CAAC,MAAM,EAAE,aAAa,EAAE,UAAU,EAAE,UAAU,CAAC,CAAC;AA0BjG,SAAS,OAAO,CAAC,MAAsB,EAAE,OAAe,EAAE,KAAa,EAAE,IAAc;IACrF,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACrC,MAAM,OAAO,GAAG,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACtE,MAAM,GAAG,GAAG,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,KAAK,CAAC;QAC/B,MAAM,GAAG,GAAG,GAAG,CAAC,OAAO,CAAC;YACtB,QAAQ,EAAE,QAAQ;YAClB,IAAI,EAAE,IAAI;YACV,IAAI,EAAE,OAAO;YACb,MAAM;YACN,OAAO,EAAE,UAAU;YACnB,OAAO,EAAE;gBACP,cAAc,EAAE,KAAK;gBACrB,GAAG,CAAC,OAAO,KAAK,SAAS;oBACvB,CAAC,CAAC,EAAE,cAAc,EAAE,kBAAkB,EAAE,gBAAgB,EAAE,MAAM,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE;oBACtF,CAAC,CAAC,EAAE,CAAC;aACR;SACF,EAAE,GAAG,CAAC,EAAE;YACP,IAAI,IAAI,GAAG,EAAE,CAAC;YACd,GAAG,CAAC,EAAE,CAAC,MAAM,EAAE,KAAK,CAAC,EAAE,GAAG,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;YAC5C,GAAG,CAAC,EAAE,CAAC,KAAK,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,EAAE,UAAU,EAAE,GAAG,CAAC,UAAU,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;QAC1E,CAAC,CAAC,CAAC;QACH,GAAG,CAAC,EAAE,CAAC,SAAS,EAAE,GAAG,EAAE,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,KAAK,CAAC,iBAAiB,CAAC,CAAC,CAAC,CAAC;QACnE,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QACxB,IAAI,OAAO,KAAK,SAAS;YAAE,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QAC9C,GAAG,CAAC,GAAG,EAAE,CAAC;IACZ,CAAC,CAAC,CAAC;AACL,CAAC;AAED;0DAC0D;AAC1D,SAAS,SAAS,CAAI,GAAW;IAC/B,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAqB,CAAC;QACnD,OAAO,CAAC,MAAM,EAAE,IAAI,IAAI,MAAM,CAAM,CAAC;IACvC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAeD,MAAM,CAAC,KAAK,UAAU,aAAa,CAAC,IAAY,EAAE,IAAkB;IAClE,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,UAAU,CAAC;IAC9B,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,IAAI,QAAQ,OAAO,EAAE,KAAK,EAAE,EAAE,IAAI,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC;QACjF,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,IAAI,CAAC;QACxC,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,SAAS,CAAC;QAC7C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,WAAW,CAAC;QAC/C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,UAAU,CAAC;QAC9C,OAAO,SAAS,CAAC;IACnB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAID;kDACkD;AAClD,MAAM,CAAC,KAAK,UAAU,UAAU;IAC9B,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;IAC3C,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,KAAK,EAAE,IAAI,QAAQ,QAAQ,EAAE,KAAK,CAAC,CAAC;QAC9D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;QAC5D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,MAAM,MAAM,GAAG,SAAS,CAAkD,GAAG,CAAC,IAAI,CAAC,CAAC;QACpF,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,SAAS,IAAI,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,IAAI,EAAE,EAAE,CAAC;IAC/F,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;AACH,CAAC;AAED;;oCAEoC;AACpC,MAAM,CAAC,KAAK,UAAU,OAAO,CAC3B,MAA0B,EAC1B,KAAa,EACb,IAAY;IAEZ,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;IAC3C,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,IAAI,QAAQ,IAAI,MAAM,EAAE,EAAE,KAAK,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACpF,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;QACrD,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;QAC5D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,EAAE,CAAC;YACrD,MAAM,MAAM,GAAG,SAAS,CAAuB,GAAG,CAAC,IAAI,CAAC,CAAC;YACzD,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;QAC3D,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;AACH,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,YAAY,CAAC,MAAsB;IACvD,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;IAC3C,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,MAAM,CAAC,CAAC,CAAC,WAAW,kBAAkB,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACpE,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,KAAK,EAAE,IAAI,QAAQ,QAAQ,KAAK,EAAE,EAAE,KAAK,CAAC,CAAC;QACrE,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;QAC5D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,MAAM,MAAM,GAAG,SAAS,CAA4B,GAAG,CAAC,IAAI,CAAC,CAAC;QAC9D,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,IAAI,EAAE,EAAE,CAAC;IACvD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;AACH,CAAC"}
+{"version":3,"file":"backlog.js","sourceRoot":"","sources":["../../../../../../../src/commands/game/sideQuest/utilities/backlog.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAE7C;;;;;;;;;;;;;;;;;;GAkBG;AAEH,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,QAAQ,KAAK,MAAM,CAAC;AAC5C,MAAM,QAAQ,GAAG,GAAG,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,kBAAkB,CAAC;AACxD,MAAM,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,aAAa,IAAI,IAAI,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC;AACnE,MAAM,QAAQ,GAAG,qBAAqB,CAAC;AACvC,MAAM,UAAU,GAAG,KAAK,CAAC;AAYzB,MAAM,CAAC,MAAM,gBAAgB,GAAoB,CAAC,MAAM,EAAE,aAAa,EAAE,WAAW,EAAE,UAAU,EAAE,UAAU,CAAC,CAAC;AA0B9G,SAAS,OAAO,CAAC,MAAsB,EAAE,OAAe,EAAE,KAAa,EAAE,IAAc;IACrF,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACrC,MAAM,OAAO,GAAG,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACtE,MAAM,GAAG,GAAG,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,KAAK,CAAC;QAC/B,MAAM,GAAG,GAAG,GAAG,CAAC,OAAO,CAAC;YACtB,QAAQ,EAAE,QAAQ;YAClB,IAAI,EAAE,IAAI;YACV,IAAI,EAAE,OAAO;YACb,MAAM;YACN,OAAO,EAAE,UAAU;YACnB,OAAO,EAAE;gBACP,cAAc,EAAE,KAAK;gBACrB,GAAG,CAAC,OAAO,KAAK,SAAS;oBACvB,CAAC,CAAC,EAAE,cAAc,EAAE,kBAAkB,EAAE,gBAAgB,EAAE,MAAM,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE;oBACtF,CAAC,CAAC,EAAE,CAAC;aACR;SACF,EAAE,GAAG,CAAC,EAAE;YACP,IAAI,IAAI,GAAG,EAAE,CAAC;YACd,GAAG,CAAC,EAAE,CAAC,MAAM,EAAE,KAAK,CAAC,EAAE,GAAG,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;YAC5C,GAAG,CAAC,EAAE,CAAC,KAAK,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,EAAE,UAAU,EAAE,GAAG,CAAC,UAAU,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;QAC1E,CAAC,CAAC,CAAC;QACH,GAAG,CAAC,EAAE,CAAC,SAAS,EAAE,GAAG,EAAE,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,KAAK,CAAC,iBAAiB,CAAC,CAAC,CAAC,CAAC;QACnE,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QACxB,IAAI,OAAO,KAAK,SAAS;YAAE,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QAC9C,GAAG,CAAC,GAAG,EAAE,CAAC;IACZ,CAAC,CAAC,CAAC;AACL,CAAC;AAED;0DAC0D;AAC1D,SAAS,SAAS,CAAI,GAAW;IAC/B,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAqB,CAAC;QACnD,OAAO,CAAC,MAAM,EAAE,IAAI,IAAI,MAAM,CAAM,CAAC;IACvC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAeD,MAAM,CAAC,KAAK,UAAU,aAAa,CAAC,IAAY,EAAE,IAAkB;IAClE,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,UAAU,CAAC;IAC9B,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,IAAI,QAAQ,OAAO,EAAE,KAAK,EAAE,EAAE,IAAI,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC;QACjF,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,IAAI,CAAC;QACxC,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,SAAS,CAAC;QAC7C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,WAAW,CAAC;QAC/C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,UAAU,CAAC;QAC9C,OAAO,SAAS,CAAC;IACnB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAoCD;;;;;;;;GAQG;AACH,MAAM,CAAC,KAAK,UAAU,gBAAgB;IACpC,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;IAC3C,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,KAAK,EAAE,IAAI,QAAQ,SAAS,EAAE,KAAK,CAAC,CAAC;QAC/D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;QAC5D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,MAAM,MAAM,GAAG,SAAS,CAA2B,GAAG,CAAC,IAAI,CAAC,CAAC;QAC7D,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,IAAI,EAAE,EAAE,CAAC;IACvD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;AACH,CAAC;AAED;8DAC8D;AAC9D,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,MAAc,EACd,IAAY,EACZ,OAAmG,EAAE;IAErG,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,UAAU,CAAC;IAC9B,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,IAAI,QAAQ,SAAS,kBAAkB,CAAC,MAAM,CAAC,UAAU,EAAE,KAAK,EAAE;YAClG,IAAI;YACJ,IAAI,EAAE,IAAI,CAAC,IAAI;YACf,gBAAgB,EAAE,IAAI,CAAC,gBAAgB;YACvC,UAAU,EAAE,IAAI,CAAC,UAAU;SAC5B,CAAC,CAAC;QACH,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,IAAI,CAAC;QAClE,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,SAAS,CAAC;QAC7C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,WAAW,CAAC;QAC/C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,UAAU,CAAC;QACxE,OAAO,SAAS,CAAC;IACnB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,MAAc,EACd,IAAY,EACZ,UAAmB;IAEnB,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,UAAU,CAAC;IAC9B,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,IAAI,QAAQ,SAAS,kBAAkB,CAAC,MAAM,CAAC,UAAU,EAAE,KAAK,EAAE;YAClG,IAAI;YACJ,UAAU;SACX,CAAC,CAAC;QACH,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,IAAI,CAAC;QAClE,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,SAAS,CAAC;QAC7C,+DAA+D;QAC/D,6CAA6C;QAC7C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,WAAW,CAAC;QAC/C,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,UAAU,CAAC;QACxE,OAAO,SAAS,CAAC;IACnB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAID;kDACkD;AAClD,MAAM,CAAC,KAAK,UAAU,UAAU;IAC9B,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;IAC3C,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,KAAK,EAAE,IAAI,QAAQ,QAAQ,EAAE,KAAK,CAAC,CAAC;QAC9D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;QAC5D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,MAAM,MAAM,GAAG,SAAS,CAAkD,GAAG,CAAC,IAAI,CAAC,CAAC;QACpF,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,SAAS,IAAI,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,IAAI,EAAE,EAAE,CAAC;IAC/F,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;AACH,CAAC;AAED;;oCAEoC;AACpC,MAAM,CAAC,KAAK,UAAU,OAAO,CAC3B,MAA0B,EAC1B,KAAa,EACb,IAAY;IAEZ,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;IAC3C,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,IAAI,QAAQ,IAAI,MAAM,EAAE,EAAE,KAAK,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACpF,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;QACrD,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;QAC5D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,EAAE,CAAC;YACrD,MAAM,MAAM,GAAG,SAAS,CAAuB,GAAG,CAAC,IAAI,CAAC,CAAC;YACzD,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;QAC3D,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;AACH,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,YAAY,CAAC,MAAsB;IACvD,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;IAC1B,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;IAC3C,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,MAAM,CAAC,CAAC,CAAC,WAAW,kBAAkB,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACpE,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,KAAK,EAAE,IAAI,QAAQ,QAAQ,KAAK,EAAE,EAAE,KAAK,CAAC,CAAC;QACrE,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC;QAC5D,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG;YAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;QAC1D,MAAM,MAAM,GAAG,SAAS,CAA4B,GAAG,CAAC,IAAI,CAAC,CAAC;QAC9D,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,IAAI,EAAE,EAAE,CAAC;IACvD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;AACH,CAAC"}

package/bundle/typescript/src/commands/game/sideQuest/utilities/combat-exchange.js

@@ -1,7 +1,8 @@
 import { NPC } from '../models/npc.js';
 import { rollPool, formatRoll } from './dice.js';
 import { leaveMatrix, leaveDrone } from './planes.js';
-import { spotOf, spotCell, seatingIn, actorCell, OPEN_FLOOR } from './spots.js';
+import { spotOf, spotCell, seatingIn, actorCell, OPEN_FLOOR, isInCover, coverAvailableFor, takeCoverHere } from './spots.js';
+import { heldBy } from './grapple.js';
 import { distanceMeters, coverBetween } from './room-grid.js';
 import { classifyWeapon, rangePenalty, rangeBracketName } from './range.js';
 /**
@@ -73,6 +74,20 @@ export class CombatExchange {
         this._combatants = [this.actor, target];
         this.actor.inExchange = true;
         target.inExchange = true;
+        // A COMBATANT USES THE COVER IT IS ALREADY STANDING IN. Cover now
+        // costs the Simple Action canon says it does (SR5e p.166/190, see
+        // commands/cover.ts), which would otherwise have quietly stripped
+        // +2/+4 from every NPC in the game -- their AI has no reason to
+        // type "cover", so a rule change meant as a fix would have landed
+        // as a balance nerf nobody asked for. Anyone already behind
+        // something when the shooting starts is assumed to spend their
+        // Simple Action on it, which is what a person does. The PLAYER's
+        // choice still matters everywhere it should: taking cover mid-fight
+        // after moving, or in a room whose cover they had to cross to.
+        for (const c of this._combatants) {
+            if (!isInCover(c) && coverAvailableFor(c))
+                takeCoverHere(c);
+        }
         // Attacking someone whose hands are UP is the kind of story that
         // travels (SR5e p.368: notoriety for callous behavior / killing
         // innocents). Player-only, once per victim -- checked before the
@@ -416,9 +431,27 @@ export class CombatExchange {
         const weaponClass = strike.weaponItem?.isFirearm() ? classifyWeapon(strike.weaponItem) : undefined;
         const range = weaponClass ? rangePenalty(spatial.meters, weaponClass) : 0;
         const rangeMod = range === 'out-of-range' ? -6 : range;
-        const coverDice = spatial.cover === 'good' ? 4 : spatial.cover === 'partial' ? 2 : 0;
-        const attack = rollPool(strike.pool + directed + rangeMod);
-        const defense = rollPool(defender.getDefensePool() + inspired + counterspell + coverDice);
+        // p.190 grants the bonus to a defender who "uses a Take Cover
+        // action" -- so the dice follow the ACTION, not the address.
+        const coverDice = isInCover(defender)
+            ? (spatial.cover === 'good' ? 4 : spatial.cover === 'partial' ? 2 : 0)
+            : 0;
+        // POSTURE (SR5e p.187/189, and p.195 for the subdued). Derived at
+        // the point of use rather than stored twice: a GRAPPLED actor
+        // counts as prone for attacks against them (p.195) without anyone
+        // having to set a flag on them that could then disagree with the
+        // hold.
+        const onTheGround = (who) => who.posture === 'prone' || !!heldBy(who, this.scene);
+        // p.187 "Attacker Prone": -1 to a prone attacker's MELEE, cancelled
+        // when the opponent is down too.
+        const attackerFloor = isMelee && onTheGround(attacker) && !onTheGround(defender) ? -1 : 0;
+        // p.187 "Opponent Prone": +1 to hit someone who is down.
+        const targetDown = onTheGround(defender) ? 1 : 0;
+        // p.189 "Defender Prone": -2 to defend -- but NOT against ranged
+        // fire unless the shooter is inside five metres.
+        const proneDefence = onTheGround(defender) && (isMelee || spatial.meters <= 5) ? -2 : 0;
+        const attack = rollPool(strike.pool + directed + rangeMod + attackerFloor + targetDown);
+        const defense = rollPool(defender.getDefensePool() + inspired + counterspell + coverDice + proneDefence);
         const header = `${attacker.name} attacks ${defender.name} ${strike.label}`;
         meta.push(`${header}:`);
         const rangeNote = weaponClass && spatial.meters > 0