@feigi/fleet-ctl 3.100.3 → 3.102.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/commands/review-and-fix.md +1 -0
- package/package.json +1 -1
- package/scripts/closeout.mjs +2 -2
- package/scripts/dispatch-prompt.mjs +97 -9
- package/scripts/dispositions-check.mjs +74 -10
- package/scripts/fleet-event.mjs +2 -2
- package/scripts/fleet-tick.mjs +1 -21
- package/scripts/ledger-grammar.mjs +65 -0
- package/scripts/ledger.mjs +7 -21
- package/scripts/pull.mjs +2 -16
- package/scripts/ticket-router.mjs +1 -70
- package/scripts/ticket-text.mjs +94 -0
- package/scripts/tier-outcomes.mjs +305 -36
- package/skills/run-team/SKILL.md +3 -3
- package/skills/run-team/briefs/fix-applier.md +11 -0
- package/skills/run-team/references/close-out.md +1 -2
- package/skills/run-team/references/pull.md +6 -0
package/skills/run-team/SKILL.md
CHANGED
|
@@ -197,7 +197,7 @@ owes beyond its call:
|
|
|
197
197
|
then records `fleet-heartbeat.mjs --stop` last, because every reader takes
|
|
198
198
|
a recorded stop as the end of the run. A `NEED tier-outcomes append <pr>`
|
|
199
199
|
is a reviewed PR with no verdict row — for an implementer's, one that never
|
|
200
|
-
reached its finisher: append
|
|
200
|
+
reached its finisher: run `append <pr>` (**Guard: accumulate per PR**) and run `finish` again.
|
|
201
201
|
A PR closing none or several: a NEED if a settled `impl-<N>=PR#<pr>` token
|
|
202
202
|
names it, else a note. A step it fails is explained in
|
|
203
203
|
references/close-out.md. Exit 0 is the run closed.
|
|
@@ -281,8 +281,8 @@ digest asks of you:
|
|
|
281
281
|
|
|
282
282
|
**Guard: accumulate per PR, never conclude inside one run.** **Append one row to
|
|
283
283
|
`docs/metrics/tier-outcomes.tsv` when you rule each PR's review, before its finisher, with
|
|
284
|
-
`~/.fleet/bin/fleet-run tier-outcomes.mjs append <pr
|
|
285
|
-
|
|
284
|
+
`~/.fleet/bin/fleet-run tier-outcomes.mjs append <pr>`, never by hand** (that file's header carries the column meanings).
|
|
285
|
+
The script works both verdicts out from the Charges on the PR's first review. The script
|
|
286
286
|
fills `tier` itself; never type one in. `ledger.mjs dispatch` refuses the finisher until the row exists. That append is the whole duty. See references/pull.md.
|
|
287
287
|
|
|
288
288
|
**A verdict is a measurement only if it post-dates the run that produced it.** See references/reviewers.md.
|
|
@@ -11,6 +11,7 @@ arg violations violations
|
|
|
11
11
|
path worktree
|
|
12
12
|
path scratch
|
|
13
13
|
path mismatched-record
|
|
14
|
+
ticket ticket
|
|
14
15
|
literal your name
|
|
15
16
|
literal outcome
|
|
16
17
|
literal that path
|
|
@@ -64,6 +65,16 @@ claim, never your patch.
|
|
|
64
65
|
**Before you report, write every ruling to `<scratch>/dispositions-<your name>.json`**,
|
|
65
66
|
`<your name>` the `fix-pr-<pr>[-x]` you were dispatched as — the record the controller checks with
|
|
66
67
|
`dispositions-check.mjs` before it dispatches any finisher.
|
|
68
|
+
|
|
69
|
+
**Charge every standing entry of that record:** which entries stand, and the
|
|
70
|
+
`charge` and `chargeReason` fields, are in `review-and-fix.md`'s step 2. Read
|
|
71
|
+
the definitions of `closed_own_ticket` and `minted_false_claim` in the header
|
|
72
|
+
of `docs/metrics/tier-outcomes.tsv`, and charge each standing entry by them,
|
|
73
|
+
against the ticket below as its implementer was dispatched with it.
|
|
74
|
+
|
|
75
|
+
The ticket:
|
|
76
|
+
|
|
77
|
+
<ticket>
|
|
67
78
|
## kind: conflict
|
|
68
79
|
Bring `<branch>` current with `origin/main`, resolving every conflict by the
|
|
69
80
|
**No-undo audit** in `$(~/.fleet/bin/fleet-run --root)/commands/run-merge-bot.md`.
|
|
@@ -25,8 +25,7 @@
|
|
|
25
25
|
stop as the end of the run. A `NEED tier-outcomes append <pr>` is a
|
|
26
26
|
reviewed PR with no verdict row — for an implementer's PR, one that never
|
|
27
27
|
reached its finisher, since `ledger.mjs dispatch` refuses that finisher
|
|
28
|
-
until the row exists: append
|
|
29
|
-
you rule (**Guard: accumulate per PR**) and run `finish` again. A reviewed
|
|
28
|
+
until the row exists: run `append <pr>` (**Guard: accumulate per PR**) and run `finish` again. A reviewed
|
|
30
29
|
PR that closes no issue, or several, with no settled `impl-<N>=PR#<pr>`
|
|
31
30
|
token naming it takes no row: the check names it in a `note` line and
|
|
32
31
|
never asks for one. With such a token it is a NEED: `append` takes the
|
|
@@ -477,6 +477,12 @@ member-outcomes row (a `tier-unverifiable=` member has no ledger verdict, so
|
|
|
477
477
|
its one member-outcomes row decides), and BLANK with a WARNING naming why
|
|
478
478
|
when neither settles
|
|
479
479
|
it; never type a tier in over that.
|
|
480
|
+
It also works `closed_own_ticket` and `minted_false_claim` out from the Charges on
|
|
481
|
+
the PR's first review and composes `note` from them, so you type no verdict. A PR whose
|
|
482
|
+
record has no Charge on a standing finding, or whose review file or record is missing,
|
|
483
|
+
is refused, naming what is missing: check the record again, or dispatch the fix-applier
|
|
484
|
+
again as a dispositions retry. Verdicts given by hand, with `--note`, are for a merged
|
|
485
|
+
PR whose evidence is reclaimed, and `append` refuses them on any other.
|
|
480
486
|
|
|
481
487
|
**Why not decide inside one run.** A run holds 2-3 implementer PRs, and ticket
|
|
482
488
|
difficulty swamps the tier effect — a gojq parity harness and a two-statement
|