gcf-board 0.2.19 → 0.3.0

package/README.md

@@ -5,7 +5,7 @@ A local web board for game projects on [GCF](../gcf-aif-ext): the items by statu
 ## Requirements
 
 - Node.js ≥ 22.18
-- [`ai-factory`](https://www.npmjs.com/package/ai-factory) installed globally, and in each project [`gcf-aif-ext`](../gcf-aif-ext) ≥ 0.6 (`gcf board --json` with `schema: 1`)
+- [`ai-factory`](https://www.npmjs.com/package/ai-factory) installed globally, and in each project [`gcf-aif-ext`](../gcf-aif-ext) ≥ 0.9 (`gcf board --json` with `schema: 2`: the columns and the item moves come from it as data)
 - for runs: [`gcf-harness-aif-ext`](../gcf-harness-aif-ext) ≥ 0.8 and Claude Code (`claude -p`)
 - gcf-board 0.2 needs `gcf-aif-ext` ≥ 0.7 (priority, questions, session marks, `run_mode`)
 

package/dist/server/app.js

@@ -13,7 +13,7 @@ import { briefError, runCli } from './cli.js';
 import { broadcast, planRev, subscribe, watchProject } from './events.js';
 import { followLog, readShown, shownShape } from './files.js';
 import { HttpError } from './http-error.js';
-import { moveArgs, runIds } from './moves.js';
+import { isCenterMove, moveArgs, runIds } from './moves.js';
 import { addProject, listProjects, projectOf, removeProject } from './registry.js';
 import { cancelResume, closeReport, forgetProject, resumeQueue, runsOf, startRun, unqueue, } from './runs.js';
 const send = (res, status, body) => {
@@ -102,12 +102,15 @@ export function createApp(o) {
                 watchers.set(p.name, watchProject(p));
     };
     const cli = (p, args, timeout) => runCli(o.entry, p.path, args, timeout);
+    // The last domain each project's board read gave: an item move is built from its form.
+    const domains = new Map();
     /** `gcf board --json` of a project, refused when its schema is not the one this board reads. */
     const boardOf = async (p) => {
         const board = jsonOf(await cli(p, ['gcf', 'board', '--json']), 'gcf board');
         if (board.schema !== BOARD_SCHEMA)
             throw new HttpError(502, `gcf board gives schema ${board.schema ?? 'none'}, this board reads ${BOARD_SCHEMA}: ` +
                 'update gcf-aif-ext and gcf-board together');
+        domains.set(p.name, board.domain);
         return board;
     };
     const routes = [
@@ -132,6 +135,7 @@ export function createApp(o) {
             async (_req, res, m) => {
                 await removeProject(m[1]);
                 forgetProject(m[1]);
+                domains.delete(m[1]);
                 watchers.get(m[1])?.();
                 watchers.delete(m[1]);
                 send(res, 200, { ok: true });
@@ -210,7 +214,11 @@ export function createApp(o) {
             /^\/api\/p\/([^/]+)\/move$/,
             async (req, res, m) => {
                 const p = await projectOf(m[1]);
-                const r = await cli(p, moveArgs(await bodyOf(req)));
+                const body = await bodyOf(req);
+                const domain = isCenterMove(body.cmd)
+                    ? undefined
+                    : (domains.get(p.name) ?? (await boardOf(p)).domain);
+                const r = await cli(p, moveArgs(body, domain?.moves));
                 broadcast({ type: 'board', project: p.name });
                 send(res, 200, { ok: r.ok, out: r.out.trim(), err: briefError(r.err) });
             },

package/dist/server/moves.js

@@ -1,6 +1,3 @@
-// The moves a board may run, each one `ai-factory gcf` command (Methodology §4.1, §11). Anything
-// else is refused: the board is a view, the commands own every rule. An operator's text goes
-// as one `--flag=value` argument, so no word of it reads as a flag.
 import { HttpError } from './http-error.js';
 const SLUG = /^[a-z0-9][a-z0-9-]*$/;
 function id(b, key = 'id') {
@@ -23,23 +20,17 @@ function required(b, key) {
         throw new HttpError(400, `${key} is required`);
     return v;
 }
-function oneOf(b, key, allowed) {
-    const v = b[key];
-    if (typeof v !== 'string' || !allowed.includes(v))
-        throw new HttpError(400, `${key} must be ${allowed.join(' | ')}`);
-    return v;
-}
 const opt = (b, key, flag = key) => {
     const v = text(b, key);
     return v === undefined ? [] : [`--${flag}=${v}`];
 };
-const many = (b, key) => {
+const many = (b, key, flag = key) => {
     const v = b[key];
     if (v === undefined)
         return [];
     if (!Array.isArray(v) || v.some((x) => typeof x !== 'string'))
         throw new HttpError(400, `${key} must be a list of text`);
-    return v.filter(Boolean).map((x) => `--${key}=${x}`);
+    return v.filter(Boolean).map((x) => `--${flag}=${x}`);
 };
 // The core limits a tag to 40 characters too.
 const TAG = /^[a-z0-9][a-z0-9:-]{0,39}$/;
@@ -55,7 +46,7 @@ const list = (b, key, re, what, flag = key) => {
 const idList = (b, key) => list(b, key, SLUG, 'ids');
 const tagList = (b, key, flag = key) => list(b, key, TAG, 'tags', flag);
 /** `--remove` / `--no-remove` when the operator chose; the command's default otherwise. */
-const remove = (b) => b.remove === true ? ['--remove'] : b.remove === false ? ['--no-remove'] : [];
+const remove = (b, key = 'remove', flag = key) => b[key] === true ? [`--${flag}`] : b[key] === false ? [`--no-${flag}`] : [];
 const NEW_ITEM_OPTIONS = [
     'gate',
     'title',
@@ -73,35 +64,8 @@ const NEW_ITEM_OPTIONS = [
     'effort',
 ];
 const MOVES = {
-    submit: (b) => ['submit', id(b), ...opt(b, 'unverified')],
-    close: (b) => ['close', id(b), ...many(b, 'constraint'), ...opt(b, 'unverified')],
-    verdict: (b) => [
-        'verdict',
-        id(b),
-        oneOf(b, 'verdict', ['keep', 'tune', 'kill']),
-        ...opt(b, 'note'),
-        ...opt(b, 'tags'),
-        ...many(b, 'constraint'),
-        ...remove(b),
-    ],
-    decide: (b) => [
-        'decide',
-        id(b),
-        oneOf(b, 'choice', ['go', 'no', 'wontfix', 'known', 'close', 'continue', 'kill']),
-        ...opt(b, 'reason'),
-        ...remove(b),
-    ],
-    block: (b) => ['block', id(b), `--reason=${required(b, 'reason')}`],
-    unblock: (b) => ['unblock', id(b), ...opt(b, 'note')],
     // A session's mark taken off: the item no longer shows running (Board-PRD §3.2, kind c).
     release: (b) => ['release', id(b)],
-    drop: (b) => [
-        'drop',
-        id(b),
-        `--reason=${required(b, 'reason')}`,
-        ...opt(b, 'tags'),
-        ...remove(b),
-    ],
     'item-new': (b) => [
         'item',
         'new',
@@ -156,12 +120,32 @@ const MOVES = {
         return ['permissions', ...args];
     },
 };
-/** The `ai-factory` arguments of a move, or a 400 naming what is wrong. */
-export function moveArgs(b) {
-    const cmd = typeof b.cmd === 'string' && Object.hasOwn(MOVES, b.cmd) ? b.cmd : null;
-    if (!cmd)
-        throw new HttpError(400, `unknown move: ${String(b.cmd)}`);
-    return ['gcf', ...MOVES[cmd](b)];
+/** An item's move by its domain form: the words with the item's id, then each field's flag. */
+function domainMoveArgs(spec, b) {
+    const words = spec.args.map((w) => (w === '{id}' ? id(b) : w));
+    const flags = spec.fields.flatMap((f) => {
+        const flag = f.flag ?? f.key;
+        if (f.kind === 'list')
+            return many(b, f.key, flag);
+        if (f.kind === 'remove')
+            return remove(b, f.key, flag);
+        return f.required ? [`--${flag}=${required(b, f.key)}`] : opt(b, f.key, flag);
+    });
+    return [...words, ...flags];
+}
+/** Whether a move is the center's own: its arguments need no domain. */
+export const isCenterMove = (cmd) => typeof cmd === 'string' && Object.hasOwn(MOVES, cmd);
+/**
+ * The `ai-factory` arguments of a move, or a 400 naming what is wrong: a center move, or one of
+ * `moves`, the domain's forms from the project's board read.
+ */
+export function moveArgs(b, moves = {}) {
+    const cmd = typeof b.cmd === 'string' ? b.cmd : '';
+    if (isCenterMove(cmd))
+        return ['gcf', ...MOVES[cmd](b)];
+    if (Object.hasOwn(moves, cmd))
+        return ['gcf', ...domainMoveArgs(moves[cmd], b)];
+    throw new HttpError(400, `unknown move: ${String(b.cmd)}`);
 }
 /** The ids of a run: one slug each, at least one. */
 export function runIds(b) {

package/dist/shared/board.js

@@ -1,11 +1,3 @@
 // The read model of `ai-factory gcf board --json` (Methodology §11): the board refuses a schema
 // it does not know. Types only, plus the version this board reads.
-export const BOARD_SCHEMA = 1;
-export const STATUSES = [
-    'ready',
-    'blocked',
-    'awaiting-verdict',
-    'awaiting-decision',
-    'unplayed',
-    'parked',
-];
+export const BOARD_SCHEMA = 2;