@warren-dean/code-kit 0.10.0 → 0.11.0

package/hooks/mod/register.mjs

@@ -48,6 +48,7 @@ import {
 import { leadPrompt, nudgeText, restartPrompt, stallDue, stepKey, stepLabel } from './loop.mjs';
 import { goalOf, nextText } from './mission.mjs';
 import { contextOf, decisionCards } from './decisions.mjs';
+import { DEFAULT_TEAM_UI, TEAM_ID, teamFrame } from './teamview.mjs';
 import { glowOf, mapStoryOf, touchOf } from './codemap.mjs';
 import { harnessSettings } from '../lib/harness.mjs';
 import {
@@ -67,7 +68,8 @@ let reviewTurn = null; // 'next' until the lead's review turn starts, then its i
 let approving = null; // the open Approve… confirmation: { request, reason, error }
 let result = null; // what the last merge reported
 let notice = null; // why the person's last act failed
-let teamSince = null; // TEAM-6: since you last looked, as read when the Team tab opened
+let teamSince = null; // TEAM-6: since you last looked, as read when the Team pane's tab opened
+let teamUi = { ...DEFAULT_TEAM_UI }; // TEAM-7: the Team pane's own state
 let boardSynced = -Infinity; // TEAM-12: when the board was last synced
 let projectRead = -Infinity; // MAP-5: when the whole project's map was last read
 const BOARD_EVERY_MS = 10 * 60000;
@@ -176,6 +178,14 @@ export function register(on) {
         immediate: true,
       })
       .catch(unavailable('harness'));
+    await $.command
+      .register({
+        name: 'team',
+        description:
+          "code-kit's team: who's at work on what, claims, intents, drift and the team's setup",
+        immediate: true,
+      })
+      .catch(unavailable('team'));
     rateLimits = (await $.session.usage().catch(() => null))?.rateLimits ?? [];
     const cli = `${$.plugin.root}/bin/code-kit.mjs`;
     const cwd = e.cwd ?? (await $.session.cwd());
@@ -267,8 +277,8 @@ export function register(on) {
           }
         }
         if (paneUi.tab === 'map') trailData = (await json('trail')) ?? trailData;
-        // TEAM-6: since you last looked, read once each time the Team tab opens, which marks it read.
-        if (paneUi.tab === 'team' && model.check?.team && !teamSince)
+        // TEAM-6: since you last looked, read once each time its tab opens, which marks it read.
+        if (teamUi.tab === 'activity' && model.check?.team && !teamSince)
           teamSince = (await json('team', 'since')) ?? { people: {} };
         // JOURNAL-1, DECIDE-2: the lineage, for the Journal tab and the decisions' cards.
         const deciding = held.size > 0 || (model.requests?.open ?? []).length > 0;
@@ -412,6 +422,8 @@ export function register(on) {
           // TRACE-3: the map follows the specs too.
           specs ? await listing(specs) : '',
           await stamp('.claude/state/requests.jsonl'),
+          // TEAM-7: a live provider (the hub's helper) rewrites the team's state as events arrive.
+          await stamp('.claude/state/team.json'),
           await stamp('.claude/state/merge-queue.json'),
           await listing('.claude/approvals'),
           await listing('.claude/state/stop-blocks'),
@@ -578,6 +590,22 @@ export function register(on) {
         await act.reload();
       },
       keepClaim: (line) => act.teamAct(['team', 'keep', line.claim.story], 'Keeping the claim'),
+      // TEAM-9: the tech lead hands a claim, with its lease, to someone else.
+      reassignTo: async (story, handle) => {
+        const to = handle.trim().replace(/^@/, '');
+        if (!to) return;
+        teamUi = { ...teamUi, reassign: null, handle: '' };
+        await act.teamAct(['team', 'reassign', story, to], `Reassigning ${story}`);
+      },
+      // TEAM-13: the team's setup is changed through its init, by the lead, with the person.
+      teamSetup: async () => {
+        await $.prompt
+          .submit({ text: "Run /code-kit:team-init: I want to change this project's team setup." })
+          .catch((err) => {
+            notice = `Couldn't ask the lead: ${err?.message ?? err}`;
+          });
+        $.ui.invalidate('ui.render');
+      },
       releaseClaim: (line) =>
         act.teamAct(['team', 'release', line.claim.story], 'Releasing the claim'),
       // TEAM-10: agreed, the lead records it where agents read it, through a pull request.
@@ -599,11 +627,43 @@ export function register(on) {
           ['intend', 'drop', String(line.intent.id), '--reason', 'dropped in the code-kit pane'],
           'Dropping the intent',
         ),
-      // TEAM-7: the Team tab, opened from the band.
-      team: async () => {
-        paneUi = { ...paneUi, tab: 'team', story: null };
-        if (!(await $.ui.panes()).some((p) => p.id === PANE_ID))
-          await $.ui.open({ id: PANE_ID, title: 'Lanes', focus: true, closeOnEscape: true });
+      // TEAM-7: the Team pane, opened from the band (on what the line is about) or /team (which closes
+      // it when it's open).
+      team: async (line, { toggle = false } = {}) => {
+        const open = (await $.ui.panes()).some((p) => p.id === TEAM_ID);
+        if (open && toggle) {
+          await $.ui.close({ id: TEAM_ID });
+          return {};
+        }
+        if (toggle || !model.state) await act.reload();
+        if (model.state?.kind === 'none') return { text: NOT_CODE_KIT };
+        if (!model.check?.team)
+          return {
+            text: "This project isn't set up as a team: one developer, no claims or owners. To share it with a team, run /code-kit:team-init.",
+          };
+        const tab =
+          line?.kind === 'teamIntent'
+            ? 'intents'
+            : line?.kind === 'teamDrift'
+              ? 'drift'
+              : ['teamStale', 'teamCollision'].includes(line?.kind)
+                ? 'claims'
+                : line
+                  ? 'people'
+                  : teamUi.tab;
+        teamUi = { ...teamUi, tab };
+        // Opening it reads since-you-last-looked again, which marks it read.
+        if (tab !== 'activity' || !open) teamSince = null;
+        if (!open) {
+          const placed = await $.ui.open({
+            id: TEAM_ID,
+            title: 'Team',
+            focus: true,
+            closeOnEscape: true,
+          });
+          if (!placed?.isPlaced && placed?.reason)
+            return { text: `The Team pane is waiting: ${placed.reason}` };
+        }
         await act.readMaps();
         $.ui.invalidate('ui.render');
         return {};
@@ -1089,7 +1149,7 @@ export function register(on) {
         await act.stalls();
         await act.animate();
         const now = await act.fingerprint();
-        const paneOpen = (await $.ui.panes()).some((p) => p.id === PANE_ID);
+        const paneOpen = (await $.ui.panes()).some((p) => [PANE_ID, TEAM_ID].includes(p.id));
         quiet += 1;
         if (now === seen && quiet < (paneOpen ? 5 : 30)) return;
         quiet = 0;
@@ -1113,6 +1173,9 @@ export function register(on) {
     text: `code-kit is still starting; try /${name} again in a moment.`,
   });
   on('command.run', { command: 'lanes' }, async ($, e) => (act ? act.lanes() : starting('lanes')));
+  on('command.run', { command: 'team' }, async ($, e) =>
+    act ? act.team(null, { toggle: true }) : starting('team'),
+  );
   on('command.run', { command: 'harness' }, async ($, e) =>
     act ? act.harness() : starting('harness'),
   );
@@ -1539,8 +1602,6 @@ export function register(on) {
         needs: linesNow(now).filter((l) => !l.mission),
         cards: cardsNow(now),
         journal: journalNow(),
-        team: model.team ?? null,
-        teamSince,
         now,
         live: liveNow(now),
         held: [...held.values()],
@@ -1567,7 +1628,6 @@ export function register(on) {
       $.ui.resolve(e),
       {
         onTab: (tab) => {
-          if (tab !== 'team') teamSince = null;
           paneUi = { ...paneUi, tab };
           $.ui.invalidate('ui.render');
           act?.readMaps().then(() => $.ui.invalidate('ui.render'));
@@ -1704,6 +1764,38 @@ export function register(on) {
     }),
   );
 
+  // TEAM-7: the Team pane.
+  on('ui.render', { component: 'Pane', requestId: TEAM_ID }, async ($, e) =>
+    teamFrame(
+      {
+        team: model.team ?? null,
+        since: teamSince,
+        config: model.check?.team ? { ...model.check.team, lanes: model.check.lanes ?? {} } : null,
+        ui: teamUi,
+      },
+      $.ui.resolve(e),
+      {
+        onTab: (tab) => {
+          if (tab !== 'activity') teamSince = null;
+          teamUi = { ...teamUi, tab, reassign: null, handle: '' };
+          $.ui.invalidate('ui.render');
+          act?.readMaps().then(() => $.ui.invalidate('ui.render'));
+        },
+        onNeed: (id, line) => act?.[id]?.(line),
+        onReassign: (story) => {
+          teamUi = { ...teamUi, reassign: story, handle: '' };
+          $.ui.invalidate('ui.render');
+        },
+        onHandle: (handle) => {
+          teamUi = { ...teamUi, handle };
+          $.ui.invalidate('ui.render');
+        },
+        onReassignTo: (story, handle) => act?.reassignTo(story, handle),
+        onSetup: () => act?.teamSetup(),
+      },
+    ),
+  );
+
   on('ui.render', { component: 'Pane', requestId: RESULT_ID }, async ($, e) =>
     resultPane(result, $.ui.resolve(e)),
   );

package/hooks/mod/register.test.ts

@@ -8,6 +8,7 @@ const PANE = 'code-kit-lanes';
 const APPROVE = 'code-kit-approve';
 const RESULT = 'code-kit-result';
 const SETTINGS = 'code-kit-settings';
+const TEAM = 'code-kit-team';
 
 /** A project as the stubs report it; tests change it to move the world on. */
 function project() {
@@ -200,7 +201,7 @@ function stub(on: any, w: World) {
       w.acts.push([...argv.slice(2)]);
       return ran(0, JSON.stringify(w.teamSince));
     }
-    if (sub === 'team' && ['keep', 'release', 'board'].includes(argv[3])) {
+    if (sub === 'team' && ['keep', 'release', 'board', 'reassign'].includes(argv[3])) {
       w.acts.push([...argv.slice(2)]);
       return ran(0, '');
     }
@@ -430,6 +431,7 @@ const bandUi = ($: any) =>
 const press = ($: any, key: string, requestId?: string) =>
   $.ui.press({ plugin: 'code-kit', key, ...(requestId ? { requestId } : {}) });
 const lanes = ($: any) => $.command.run({ command: 'lanes', args: '' });
+const teamCmd = ($: any) => $.command.run({ command: 'team', args: '' });
 const dayjs = {
   at: '2026-10-06T00:00:00.000Z',
   actor: 'web-engineer',
@@ -3017,7 +3019,32 @@ test("TEAM-8 when the shared repository can't be reached the band says so as a n
   await band.unmount();
 });
 
-test('TEAM-6 and TEAM-7 the Team tab: since you last looked, then each person with their lanes, claims, pulls and intents', async ($, on) => {
+test('TEAM-7 /team opens the Team pane on its people: each with their lanes, claims, pulls and intents; /team again closes it', async ($, on) => {
+  const w = project();
+  w.team = teamOf('sam');
+  (w.check as any).team = { techLead: 'alex' };
+  await start($, on, w);
+  await teamCmd($);
+  expect(w.opened).toEqual([TEAM]);
+  const ui = await pane($, TEAM);
+  expect((await ui.find({ key: 'team-header' }))?.text).toMatch(
+    /tech lead alex[\s\S]*you are sam · own web[\s\S]*2 claimed · 1 intent open · 1 pull request/,
+  );
+  expect((await ui.find({ key: 'team-person-alex' }))?.text).toMatch(/alex · tech lead/);
+  expect((await ui.find({ key: 'team-person-sam' }))?.text).toMatch(
+    /sam · you · owns web[\s\S]*ST-4 · stale since Tuesday/,
+  );
+  expect((await ui.find({ key: 'team-person-kim' }))?.text).toMatch(
+    /#14 Booking list \(web\/st-9\)[\s\S]*intends: Split the booking form/,
+  );
+  // GitHub's own provider has no drift checks, so no Drift tab.
+  expect(await ui.find({ key: 'team-tab-drift' })).toBeUndefined();
+  await ui.unmount();
+  await teamCmd($);
+  expect(w.closed).toEqual([TEAM]);
+});
+
+test('TEAM-6 since you last looked is read when its tab opens, which marks it read, and again each time it opens', async ($, on) => {
   const w = project();
   w.team = teamOf('sam');
   w.teamSince = {
@@ -3027,28 +3054,147 @@ test('TEAM-6 and TEAM-7 the Team tab: since you last looked, then each person wi
   };
   (w.check as any).team = { techLead: 'alex' };
   await start($, on, w);
-  await lanes($);
-  const ui = await pane($, PANE);
-  expect((await ui.find({ key: 'tab-team' }))?.props.hotkey).toBe('8');
-  await press($, 'tab-team', PANE);
+  await teamCmd($);
+  const ui = await pane($, TEAM);
+  const reads = () => w.acts.filter((a) => a[0] === 'team' && a[1] === 'since').length;
+  expect(reads()).toBe(0);
+  expect((await ui.find({ key: 'team-tab-activity' }))?.props.hotkey).toBe('2');
+  await press($, 'team-tab-activity', TEAM);
   expect(w.acts).toContainEqual(['team', 'since', '--json']);
   expect((await ui.find({ key: 'team-since' }))?.text).toMatch(
-    /kim: pushed 2 commits to web\/st-9/,
-  );
-  expect((await ui.find({ key: 'team-person-alex' }))?.text).toMatch(/alex · tech lead/);
-  expect((await ui.find({ key: 'team-person-sam' }))?.text).toMatch(
-    /sam · you · owns web[\s\S]*ST-4 · stale since Tuesday/,
+    /kim[\s\S]*pushed 2 commits to web\/st-9/,
   );
-  expect((await ui.find({ key: 'team-person-kim' }))?.text).toMatch(
-    /#14 Booking list \(web\/st-9\)[\s\S]*intends: Split the booking form/,
+  await press($, 'team-tab-people', TEAM);
+  await press($, 'team-tab-activity', TEAM);
+  expect(reads()).toBe(2);
+  await ui.unmount();
+});
+
+test("TEAM-9 the Claims tab: Keep and Release on yours, and nothing on others'", async ($, on) => {
+  const w = project();
+  w.team = teamOf('sam');
+  (w.check as any).team = { techLead: 'alex' };
+  await start($, on, w);
+  await teamCmd($);
+  const ui = await pane($, TEAM);
+  await press($, 'team-tab-claims', TEAM);
+  expect((await ui.find({ key: 'team-claims-ST-4' }))?.text).toMatch(
+    /ST-4[\s\S]*sam \(you\)[\s\S]*stale since Tuesday[\s\S]*leases apps\/web\/booking\/\*\*/,
   );
-  await press($, 'team-release-ST-4', PANE);
+  expect(await ui.find({ key: 'team-release-ST-9' })).toBeUndefined();
+  expect(await ui.find({ key: 'team-reassign-ST-4' })).toBeUndefined();
+  await press($, 'team-keep-ST-4', TEAM);
+  expect(w.acts).toContainEqual(['team', 'keep', 'ST-4']);
+  await press($, 'team-release-ST-4', TEAM);
   expect(w.acts).toContainEqual(['team', 'release', 'ST-4']);
   await ui.unmount();
 });
 
-test('TEAM-7 without a team there is no Team tab', async ($, on) => {
+test("TEAM-9 the tech lead releases or reassigns anyone's claim from the Claims tab", async ($, on) => {
   const w = project();
+  w.team = teamOf('alex');
+  (w.check as any).team = { techLead: 'alex' };
+  await start($, on, w);
+  await teamCmd($);
+  const ui = await pane($, TEAM);
+  await press($, 'team-tab-claims', TEAM);
+  await press($, 'team-release-ST-9', TEAM);
+  expect(w.acts).toContainEqual(['team', 'release', 'ST-9']);
+  expect(await ui.find({ key: 'team-reassign-to-ST-9' })).toBeUndefined();
+  await press($, 'team-reassign-ST-9', TEAM);
+  expect(await ui.find({ key: 'team-reassign-to-ST-9' })).toBeDefined();
+  await $.ui.input({ plugin: 'code-kit', key: 'team-reassign-to-ST-9', text: '@sam' });
+  expect(w.acts).toContainEqual(['team', 'reassign', 'ST-9', 'sam']);
+  expect(await ui.find({ key: 'team-reassign-to-ST-9' })).toBeUndefined();
+  await ui.unmount();
+});
+
+test('TEAM-5 the Intents tab lists the open intents, with Agree and Drop for the tech lead', async ($, on) => {
+  const w = project();
+  w.team = teamOf('alex');
+  (w.check as any).team = { techLead: 'alex' };
+  await start($, on, w);
+  await teamCmd($);
+  const ui = await pane($, TEAM);
+  await press($, 'team-tab-intents', TEAM);
+  expect((await ui.find({ key: 'team-intents-12' }))?.text).toMatch(
+    /#12[\s\S]*Split the booking form[\s\S]*kim[\s\S]*touches apps\/web\/\*\*/,
+  );
+  await press($, 'team-drop-12', TEAM);
+  expect(w.acts).toContainEqual([
+    'intend',
+    'drop',
+    '12',
+    '--reason',
+    'dropped in the code-kit pane',
+  ]);
+  await ui.unmount();
+});
+
+test('TEAM-5 a developer sees the intents without Agree or Drop', async ($, on) => {
+  const w = project();
+  w.team = teamOf('sam');
+  (w.check as any).team = { techLead: 'alex' };
+  await start($, on, w);
+  await teamCmd($);
+  const ui = await pane($, TEAM);
+  await press($, 'team-tab-intents', TEAM);
+  expect(await ui.find({ key: 'team-intents-12' })).toBeDefined();
+  expect(await ui.find({ key: 'team-agree-12' })).toBeUndefined();
+  await ui.unmount();
+});
+
+test("TEAM-13 the Setup tab shows the team's config and what's still missing, and asks the lead to change it", async ($, on) => {
+  const w = project();
+  w.team = {
+    ...teamOf('sam'),
+    provider: {
+      name: 'GitHub',
+      live: false,
+      detail: 'claims are refs on origin, intents are discussions in "Intents"',
+    },
+    setup: [
+      'Discussions are off for this repository. Turn them on in its Settings › Features, add a "Intents" category.',
+    ],
+  };
+  (w.check as any).team = {
+    techLead: 'alex',
+    staleDays: 3,
+    intents: 'Intents',
+    provider: 'github',
+    owners: { web: ['sam'] },
+  };
+  await start($, on, w);
+  await teamCmd($);
+  const ui = await pane($, TEAM);
+  expect((await ui.find({ key: 'team-provider' }))?.text).toBe(
+    'Through GitHub: claims are refs on origin, intents are discussions in "Intents"',
+  );
+  expect((await ui.find({ key: 'team-setup-count' }))?.text).toBe(
+    '1 thing to set up: see Setup (6).',
+  );
+  await press($, 'team-tab-setup', TEAM);
+  expect((await ui.find({ key: 'team-config' }))?.text).toMatch(
+    /Tech lead\s+alex[\s\S]*Provider\s+GitHub[\s\S]*Lane web\s+sam[\s\S]*Unowned lanes\s+api, core/,
+  );
+  expect((await ui.find({ key: 'team-todo' }))?.text).toMatch(/Discussions are off/);
+  await press($, 'team-setup', TEAM);
+  expect(w.prompts.at(-1)).toMatch(/\/code-kit:team-init/);
+  await ui.unmount();
+});
+
+test('TEAM-13 without a team, /team says how to set one up, and opens nothing', async ($, on) => {
+  const w = project();
+  await start($, on, w);
+  const said = await teamCmd($);
+  expect(said.text).toMatch(/isn't set up as a team[\s\S]*\/code-kit:team-init/);
+  expect(w.opened).toEqual([]);
+});
+
+test('TEAM-7 the Lanes pane has no Team tab: the team has its own pane', async ($, on) => {
+  const w = project();
+  w.team = teamOf('sam');
+  (w.check as any).team = { techLead: 'alex' };
   await start($, on, w);
   await lanes($);
   const ui = await pane($, PANE);
@@ -3056,6 +3202,100 @@ test('TEAM-7 without a team there is no Team tab', async ($, on) => {
   await ui.unmount();
 });
 
+test('a live provider says so in the Team pane, by its own name, and has a Drift tab', async ($, on) => {
+  const w = project();
+  w.team = {
+    ...teamOf('kim'),
+    provider: {
+      name: 'the code-kit teams hub',
+      live: true,
+      account: 'kim',
+      detail: 'claims, leases and intents as they happen',
+    },
+    findings: [],
+  };
+  (w.check as any).team = { techLead: 'alex' };
+  await start($, on, w);
+  await teamCmd($);
+  const ui = await pane($, TEAM);
+  expect((await ui.find({ key: 'team-provider' }))?.text).toBe(
+    'Through the code-kit teams hub · live · signed in as kim: claims, leases and intents as they happen',
+  );
+  await press($, 'team-tab-drift', TEAM);
+  expect(
+    await ui.find({ type: 'Text', text: 'No drift found on the branches at work.' }),
+  ).toBeDefined();
+  await ui.unmount();
+});
+
+test("with a live provider (the hub), the band says when it fell back to GitHub, and names drift on your work; the Team tab shows who's at work and the drift found", async ($, on) => {
+  const w = project();
+  const at = new Date().toISOString();
+  w.team = {
+    ...teamOf('kim'),
+    fallback: "The code-kit teams hub can't be reached.",
+    sessions: [
+      {
+        login: 'kim',
+        branch: 'web/st-9',
+        agents: [{ type: 'web-engineer', story: 'ST-9', tool: 'Edit' }],
+        seen: at,
+        away: false,
+      },
+      { login: 'sam', branch: 'api/st-4', agents: [], seen: at, away: true },
+    ],
+    findings: [
+      {
+        id: 1,
+        branch: 'web/st-9',
+        by: 'kim',
+        conflictsWith: 'spec docs/specs/02-booking.md',
+        severity: 'blocker',
+        text: 'BOOK-2 now holds prices in cents.',
+        at,
+      },
+      {
+        id: 2,
+        branch: 'api/st-4',
+        by: 'sam',
+        conflictsWith: 'branch web/st-9',
+        severity: 'note',
+        text: 'only a note',
+        at,
+      },
+    ],
+  };
+  (w.check as any).team = { techLead: 'alex' };
+  await start($, on, w);
+  const band = await bandUi($);
+  expect(
+    await band.find({
+      type: 'Text',
+      text: "team: the team's provider can't be reached, so claims are checked through GitHub until it's back",
+    }),
+  ).toBeDefined();
+  expect(
+    await band.find({ type: 'Text', text: 'drift on web/st-9: BOOK-2 now holds prices in cents.' }),
+  ).toBeDefined();
+  expect(await band.find({ type: 'Text', text: /only a note/ })).toBeUndefined();
+  // The drift line's Team button (the second: the collision's comes first) opens the Team pane on the drift.
+  await press($, 'band-team-2', 'above-prompt');
+  expect(w.opened).toEqual([TEAM]);
+  const ui = await pane($, TEAM);
+  expect((await ui.find({ key: 'team-provider' }))?.text).toMatch(
+    /can't be reached \(The code-kit teams hub can't be reached\.\): claims are checked through GitHub/,
+  );
+  expect((await ui.find({ key: 'team-drift-web/st-9' }))?.text).toMatch(
+    /web\/st-9 · kim[\s\S]*blocker · spec docs\/specs\/02-booking\.md: BOOK-2 now holds prices in cents\./,
+  );
+  await press($, 'team-tab-people', TEAM);
+  expect((await ui.find({ key: 'team-person-kim' }))?.text).toMatch(
+    /at work on web\/st-9: web-engineer on ST-9 \(Edit\)[\s\S]*drift on web\/st-9 \(blocker\): BOOK-2 now holds prices in cents\./,
+  );
+  expect((await ui.find({ key: 'team-person-sam' }))?.text).toMatch(/away \(last seen/);
+  await ui.unmount();
+});
+
 test('TEAM-12 with a board, the mod syncs it in the background, at most every 10 minutes', async ($, on) => {
   const w = project();
   w.team = teamOf('sam');