@tablation/crew 0.21.0 → 0.22.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/cli.js +19 -4
- package/package.json +1 -1
- package/prompts/default/personas/common.md +15 -5
- package/prompts/default/personas/lane-design.md +5 -0
- package/prompts/default/personas/lane-qa.md +5 -0
- package/prompts/dev-qa/personas/common.md +16 -1
- package/prompts/dev-qa/personas/lane-qa.md +5 -0
package/dist/cli.js
CHANGED
|
@@ -4500,6 +4500,9 @@ function blockers(t, i) {
|
|
|
4500
4500
|
});
|
|
4501
4501
|
return list.length === 0 ? "\u2014" : list.join(", ");
|
|
4502
4502
|
}
|
|
4503
|
+
function modeCell(t, i) {
|
|
4504
|
+
return i.workModes && i.workModes.modes.size > 0 ? modeOf(t, i.workModes) : "\u2014";
|
|
4505
|
+
}
|
|
4503
4506
|
var sorted = (ts) => [...ts].sort(compareRank);
|
|
4504
4507
|
var stream = (segments) => `${segments.join("\n")}
|
|
4505
4508
|
`;
|
|
@@ -4511,14 +4514,14 @@ ${sorted(ts).map(row).join("\n")}
|
|
|
4511
4514
|
`;
|
|
4512
4515
|
}
|
|
4513
4516
|
function buildingDigest(i) {
|
|
4514
|
-
const header = "| ticket | repo | branch | worktree | status | assignee | filed by | sev | pri | eff | updated | last comment | new since last poll |\n
|
|
4517
|
+
const header = "| ticket | repo | branch | worktree | mode | status | assignee | filed by | sev | pri | eff | updated | last comment | new since last poll |\n|---|---|---|---|---|---|---|---|---|---|---|---|---|---|";
|
|
4515
4518
|
const row = (t) => {
|
|
4516
4519
|
const n = newFromOthers(i, t.id);
|
|
4517
4520
|
const branch = i.branchFor ? i.branchFor(t) : "";
|
|
4518
4521
|
const worktree = i.worktreeFor ? i.worktreeFor(t) : "";
|
|
4519
4522
|
const dir = i.dirFor ? i.dirFor(t) : null;
|
|
4520
4523
|
const repo = i.dirFor ? dir ?? "**NO CHECKOUT**" : "";
|
|
4521
|
-
return `| ${displayKey2(t)} | ${repo} | ${branch} | ${worktree} | ${t.status} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | p${effectivePriority(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
|
|
4524
|
+
return `| ${displayKey2(t)} | ${repo} | ${branch} | ${worktree} | ${modeCell(t, i)} | ${t.status} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | p${effectivePriority(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
|
|
4522
4525
|
};
|
|
4523
4526
|
const blockedRow = (t) => `| ${displayKey2(t)} | ${t.status} | ${who(t, i)} | p${effectivePriority(t)} | ${blockers(t, i)} |`;
|
|
4524
4527
|
const blockedTable = (ts) => ts.length === 0 ? "\n_None._\n" : `
|
|
@@ -4564,7 +4567,7 @@ ${sorted(held).map((t) => `| ${displayKey2(t)} | ${stopOf(t)} |`).join("\n")}
|
|
|
4564
4567
|
]);
|
|
4565
4568
|
}
|
|
4566
4569
|
function qaDigest(i) {
|
|
4567
|
-
const header = "| ticket | repo | status | built by | assignee | filed by | sev | pri | eff | branch | worktree | updated | last comment | new since last poll |\n
|
|
4570
|
+
const header = "| ticket | repo | mode | status | built by | assignee | filed by | sev | pri | eff | branch | worktree | updated | last comment | new since last poll |\n|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|";
|
|
4568
4571
|
const branchCell = (t) => {
|
|
4569
4572
|
if (i.dirFor && i.dirFor(t) === null) return "\u2014";
|
|
4570
4573
|
if (!i.existingBranchFor) return "";
|
|
@@ -4579,7 +4582,7 @@ function qaDigest(i) {
|
|
|
4579
4582
|
const n = newFromOthers(i, t.id);
|
|
4580
4583
|
const dir = i.dirFor ? i.dirFor(t) : null;
|
|
4581
4584
|
const repo = i.dirFor ? dir ?? "**NO CHECKOUT**" : "";
|
|
4582
|
-
return `| ${displayKey2(t)} | ${repo} | ${t.status} | ${builtBy(t)} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | p${effectivePriority(t)} | ${branchCell(t)} | ${worktreeCell(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
|
|
4585
|
+
return `| ${displayKey2(t)} | ${repo} | ${modeCell(t, i)} | ${t.status} | ${builtBy(t)} | ${who(t, i)} | ${filedBy(t, i)} | ${t.severity ?? "\u2014"} | ${t.priority ?? "\u2014"} | p${effectivePriority(t)} | ${branchCell(t)} | ${worktreeCell(t)} | ${t.updated_at.slice(0, 16)}Z | ${lastComment(i, t.id)} | ${n > 0 ? `**${n} new**` : "\u2014"} |`;
|
|
4583
4586
|
};
|
|
4584
4587
|
return stream([
|
|
4585
4588
|
"## Current queue \u2014 built for you by the poll\n",
|
|
@@ -5446,6 +5449,18 @@ function renderEnvironment(i) {
|
|
|
5446
5449
|
"another (the digest prints other ships by name) \u2014 write its name, never a Ships row id."
|
|
5447
5450
|
);
|
|
5448
5451
|
}
|
|
5452
|
+
const seatIds = Object.entries(route.resolved?.seats ?? {}).filter(([, id]) => !!id);
|
|
5453
|
+
if (seatIds.length > 0) {
|
|
5454
|
+
lines.push(
|
|
5455
|
+
"",
|
|
5456
|
+
"## Seats on this ship",
|
|
5457
|
+
"",
|
|
5458
|
+
"The Crew row id of each role's seat on this ship, to write verbatim where a hand-off",
|
|
5459
|
+
"assigns the next lane (a ticket whose digest `mode` is `hybrid`):",
|
|
5460
|
+
"",
|
|
5461
|
+
...seatIds.map(([role, id]) => `- ${role}: \`${id}\``)
|
|
5462
|
+
);
|
|
5463
|
+
}
|
|
5449
5464
|
lines.push(
|
|
5450
5465
|
"",
|
|
5451
5466
|
"## What the columns are called here",
|
package/package.json
CHANGED
|
@@ -305,10 +305,18 @@ Never start work on a ticket without first setting `assignee_id` to your
|
|
|
305
305
|
own id (Step 3.3 already does this) — that claim is itself part of what
|
|
306
306
|
keeps a concurrent interactive session from re-entering the same ticket.
|
|
307
307
|
|
|
308
|
-
|
|
309
|
-
|
|
310
|
-
|
|
311
|
-
|
|
308
|
+
Every ticket's project has a **work mode**, shown in the digest's `mode`
|
|
309
|
+
column: `automatic` (lanes work tickets by status), `hybrid` (a lane considers a
|
|
310
|
+
ticket only when it is assigned to an agent — a Crew row backed by an Agent, not a
|
|
311
|
+
hold), or `manual` (no automation touches it; a ticket with no project counts as
|
|
312
|
+
manual). The digest says so when a project is not automatic; do not go looking for
|
|
313
|
+
a ticket it leaves out. **One hand-off rule:** in a `hybrid` project, hand a ticket
|
|
314
|
+
to the next lane by *assigning that lane's seat on this ship* (ids under "Seats on
|
|
315
|
+
this ship" in the Environment section, else the roster); in an `automatic` project,
|
|
316
|
+
clear `assignee_id` as before. If there is no seat for the target role, assign any
|
|
317
|
+
Crew row with that role's Agent; if there is none, clear it and say so in the
|
|
318
|
+
hand-off comment. Parks (`needs_info`, `needs_planning` → the operator) are the
|
|
319
|
+
same in every mode.
|
|
312
320
|
|
|
313
321
|
Never touch a ticket still at `new` — only `accepted` tickets are yours to
|
|
314
322
|
pick up; triage (a separate process) is what promotes `new` → `accepted`/
|
|
@@ -558,7 +566,9 @@ if `report_type` is either of those.
|
|
|
558
566
|
this is your own isolated worktree, no special permission needed for this
|
|
559
567
|
part).
|
|
560
568
|
9. Set the ticket's `status` to `fixed` once you have verified it yourself,
|
|
561
|
-
and
|
|
569
|
+
and hand the ticket to the QA lane: in an `automatic` project **clear `assignee_id`**; in a
|
|
570
|
+
`hybrid` project **set `assignee_id` to this ship's QA seat** instead (the digest's `mode`
|
|
571
|
+
column says which). That pair is the hand-off to the QA lane.
|
|
562
572
|
If the Issues table has a `held_by_ship_id` column, clear it in the same
|
|
563
573
|
write (the hold is for the building roles; QA on any ship may take a `fixed`
|
|
564
574
|
ticket). A QA bounce or `needs_info` park leaves it set; QA clears it again
|
|
@@ -98,6 +98,11 @@ stopping for this run, do the following instead:
|
|
|
98
98
|
it) — revise the comp against what they said and post again. This cycle
|
|
99
99
|
repeats as many times as the operator wants.
|
|
100
100
|
|
|
101
|
+
In a `hybrid` project (the digest's `mode` column), where the operator
|
|
102
|
+
leaves the hand-off to you, assign this ship's Developer seat when
|
|
103
|
+
`needs_design` is cleared rather than leaving the assignee blank, so the dev lane
|
|
104
|
+
still sees the ticket; in an `automatic` project nothing changes.
|
|
105
|
+
|
|
101
106
|
Never implement past the comp to "just finish it" because the fix looks
|
|
102
107
|
small, and never set `needs_design`, `accepted`, or clear
|
|
103
108
|
`needs_planning`/`needs_review` yourself — all of that is the operator's
|
|
@@ -141,6 +141,11 @@ human review, so "probably fine" is not a pass.
|
|
|
141
141
|
bump the version, and deploy this cycle or the next. Say in
|
|
142
142
|
the comment what you exercised, so the record shows what "verified"
|
|
143
143
|
covered.
|
|
144
|
+
- **Person-built hybrid ticket with no branch** — a `hybrid` ticket at
|
|
145
|
+
`fixed` whose branch under the repo's naming convention does not exist
|
|
146
|
+
(the digest's `branch` column says **MISSING**) is not a bounce to a
|
|
147
|
+
builder: set `needs_info` and assign the operator, with a comment naming the
|
|
148
|
+
branch you expected. A person built it; the operator has to push it.
|
|
144
149
|
- **It doesn't** → `status` = `in_progress`, `assignee_id` = the building
|
|
145
150
|
lane's own Crew row id from the roster above — the dev seat if
|
|
146
151
|
`needs_design` is false or null, the design seat if true. **Exception:** when the Issues table has a `held_by_ship_id` column, **clear** `assignee_id` *and* `held_by_ship_id` instead of naming a seat — the
|
|
@@ -286,6 +286,19 @@ Never start work on a ticket without first setting `assignee_id` to your
|
|
|
286
286
|
own id (Step 3.3 already does this) — that claim is itself part of what
|
|
287
287
|
keeps a concurrent interactive session from re-entering the same ticket.
|
|
288
288
|
|
|
289
|
+
Every ticket's project has a **work mode**, shown in the digest's `mode`
|
|
290
|
+
column: `automatic` (lanes work tickets by status), `hybrid` (a lane considers a
|
|
291
|
+
ticket only when it is assigned to an agent — a Crew row backed by an Agent, not a
|
|
292
|
+
hold), or `manual` (no automation touches it; a ticket with no project counts as
|
|
293
|
+
manual). The digest says so when a project is not automatic; do not go looking for
|
|
294
|
+
a ticket it leaves out. **One hand-off rule:** in a `hybrid` project, hand a ticket
|
|
295
|
+
to the next lane by *assigning that lane's seat on this ship* (ids under "Seats on
|
|
296
|
+
this ship" in the Environment section, else the roster); in an `automatic` project,
|
|
297
|
+
clear `assignee_id` as before. If there is no seat for the target role, assign any
|
|
298
|
+
Crew row with that role's Agent; if there is none, clear it and say so in the
|
|
299
|
+
hand-off comment. Parks (`needs_info`, `needs_planning` → the operator) are the
|
|
300
|
+
same in every mode.
|
|
301
|
+
|
|
289
302
|
Never touch a ticket still at `new` — only `accepted` tickets are yours to
|
|
290
303
|
pick up; triage (a separate process) is what promotes `new` → `accepted`/
|
|
291
304
|
`needs_info`.
|
|
@@ -535,7 +548,9 @@ if `report_type` is either of those.
|
|
|
535
548
|
this is your own isolated worktree, no special permission needed for this
|
|
536
549
|
part).
|
|
537
550
|
9. Set the ticket's `status` to `fixed` once you have verified it yourself,
|
|
538
|
-
and
|
|
551
|
+
and hand the ticket to the QA lane: in an `automatic` project **clear `assignee_id`**; in a
|
|
552
|
+
`hybrid` project **set `assignee_id` to this ship's QA seat** instead (the digest's `mode`
|
|
553
|
+
column says which). That pair is the hand-off to the QA lane.
|
|
539
554
|
If the Issues table has a `held_by_ship_id` column, clear it in the same
|
|
540
555
|
write (the hold is for the building roles; QA on any ship may take a `fixed`
|
|
541
556
|
ticket). A QA bounce or `needs_info` park leaves it set; QA clears it again
|
|
@@ -128,6 +128,11 @@ human review, so "probably fine" is not a pass.
|
|
|
128
128
|
bump the version, and deploy this cycle or the next. Say in
|
|
129
129
|
the comment what you exercised, so the record shows what "verified"
|
|
130
130
|
covered.
|
|
131
|
+
- **Person-built hybrid ticket with no branch** — a `hybrid` ticket at
|
|
132
|
+
`fixed` whose branch under the repo's naming convention does not exist
|
|
133
|
+
(the digest's `branch` column says **MISSING**) is not a bounce to a
|
|
134
|
+
builder: set `needs_info` and assign the operator, with a comment naming the
|
|
135
|
+
branch you expected. A person built it; the operator has to push it.
|
|
131
136
|
- **It doesn't** → `status` = `in_progress`, `assignee_id` = the dev
|
|
132
137
|
seat's own Crew row id from the roster above. **Exception:** when the Issues table has a `held_by_ship_id` column, **clear** `assignee_id` *and* `held_by_ship_id` instead of naming a seat — the
|
|
133
138
|
hold on a `qa` ticket is yours, as verifier, and seats are per ship, so
|