@pmelab/gtd 49.0.0 → 50.0.0

package/dist/steering.mjs

@@ -1,2 +1,2 @@
-import { A as formatFinding, C as openThreadFindings, D as getParseCount, E as parseThreadsWithFindings, O as headingSectionBodies, S as THREAD_REPLY_ACTION_TITLE, T as parseThreads, _ as reviewRisks, a as unansweredQuestions, b as isAnswered, c as stripCodeThreads, d as ANSWER_FIX, f as ANSWER_LEAD, g as reviewNotes, h as reviewDecisions, i as steeringFormatOrFreeForm, k as headingSections, l as freeFormFormat, m as reviewChunks, n as openThreads, o as viewOf, p as ANSWER_WONT_FIX, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES, u as applySteeringEdits, v as unansweredNotes, w as parseFootnotes, x as FOOTNOTE_ACTION_TITLE, y as FREE_TEXT_PLACEHOLDER } from "./chunks/steering-BXwjmocB.mjs";
-export { ANSWER_FIX, ANSWER_LEAD, ANSWER_WONT_FIX, BUILT_IN_MODE_NAMES, FOOTNOTE_ACTION_TITLE, FREE_TEXT_PLACEHOLDER, THREAD_REPLY_ACTION_TITLE, applySteeringEdits, formatFinding, freeFormFormat, getParseCount, headingSectionBodies, headingSections, isAnswered, openThreadFindings, openThreads, parseCodeThreads, parseFootnotes, parseThreads, parseThreadsWithFindings, reviewChunks, reviewDecisions, reviewNotes, reviewRisks, steeringFormatFor, steeringFormatOrFreeForm, stripCodeThreads, unansweredNotes, unansweredQuestions, viewOf };
+import { A as headingSections, C as THREAD_REPLY_ACTION_TITLE, D as parseThreadsWithFindings, E as parseThreads, O as getParseCount, S as FOOTNOTE_ACTION_TITLE, T as parseFootnotes, _ as reviewRisks, a as unansweredQuestions, b as FREE_TEXT_PLACEHOLDER, c as stripCodeThreads, d as ANSWER_FIX, f as ANSWER_LEAD, g as reviewNotes, h as reviewDecisions, i as steeringFormatOrFreeForm, j as formatFinding, k as headingSectionBodies, l as freeFormFormat, m as reviewChunks, n as openThreads, o as viewOf, p as ANSWER_WONT_FIX, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES, u as applySteeringEdits, v as severityLine, w as openThreadFindings, x as isAnswered, y as unansweredNotes } from "./chunks/steering-DtUVUz9l.mjs";
+export { ANSWER_FIX, ANSWER_LEAD, ANSWER_WONT_FIX, BUILT_IN_MODE_NAMES, FOOTNOTE_ACTION_TITLE, FREE_TEXT_PLACEHOLDER, THREAD_REPLY_ACTION_TITLE, applySteeringEdits, formatFinding, freeFormFormat, getParseCount, headingSectionBodies, headingSections, isAnswered, openThreadFindings, openThreads, parseCodeThreads, parseFootnotes, parseThreads, parseThreadsWithFindings, reviewChunks, reviewDecisions, reviewNotes, reviewRisks, severityLine, steeringFormatFor, steeringFormatOrFreeForm, stripCodeThreads, unansweredNotes, unansweredQuestions, viewOf };

package/dist/testkit.mjs

@@ -1,6 +1,6 @@
 import { answered, applyScript, checkScript, configProblems, configure, installContext, isFactoryWorkflow, moveScript, numeric, protocolMismatch, removeScript, restoreScript, revertScript, showScript, t as flows_exports, writeScript } from "./flows.mjs";
 import { A as beatKindOf, B as ACCESS_REFUSAL, C as renderLandPlain, D as renderBeatJson, E as beatDocument, O as statusOf, S as renderLandJson, T as renderBeatPlain, U as renderFormat, _ as selectPath, a as contextLimitFault, g as resolveSession, h as globMatches, j as demandOf, l as parseContextWindow, m as slicesDocument, o as contextShapeFault, p as renderSlicesJson, v as judgeJson, x as noopText, y as landFields } from "./chunks/wire-BJuZWR3C.mjs";
-import { A as formatFinding, E as parseThreadsWithFindings, O as headingSectionBodies, a as unansweredQuestions, k as headingSections, n as openThreads, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES } from "./chunks/steering-BXwjmocB.mjs";
+import { A as headingSections, D as parseThreadsWithFindings, a as unansweredQuestions, j as formatFinding, k as headingSectionBodies, n as openThreads, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES } from "./chunks/steering-DtUVUz9l.mjs";
 import { $ as logWarning, $i as BaseProto, $n as map$5, $r as make$19, $t as provide$1, A as catchIf, Ai as add$1, An as scopeMake, Ar as flatMap$3, At as suspend$5, B as flatMap$4, Bi as getOrElse, Bn as get$3, Br as sync$1, Bt as unsafeMakeLatch, C as all$1, Ci as toArray, Cn as isInterrupted, Cr as exitFail, Ct as sandbox, D as async, Di as GenericTag, Dn as squash, Dr as exitZipRight, Dt as scopedWith$1, E as asVoid$1, Ei as unsafeHead, En as reduceWithContext, Er as exitVoid, Et as scoped$4, F as ensuring$2, Fi as fromIterable, Fn as join$3, Fr as map$3, Ft as tryMap, G as forkIn, Gi as match$4, Gn as TypeId$10, Gr as whenEffect, Gt as zipLeft, H as fnUntraced, Hi as isNone, Hn as modify$1, Hr as uninterruptible, Ht as when, I as exit, Ii as splitAt, In as sequential, Ir as onInterrupt, It as tryPromise, J as ignore, Ji as isLeft, Jn as all, Jr as DeferredTypeId$1, Jt as spaced$1, K as forkScoped, Ki as none, Kn as Class, Kr as withFiberRuntime, Kt as zipRight$3, L as fail$5, Li as filter$1, Ln as bind, Lr as scopeClose, Lt as try_, M as die$2, Mi as getOption, Mn as _await$2, Mr as interruptAsFiber, Mt as tap, N as dieMessage, Ni as omit, Nn as inheritAll$1, Nr as interruptFiber, Nt as tapErrorCause, O as catchAll$1, Oi as Reference, On as ensuring$3, Or as fiberId, Ot as serviceOption, P as either, Pi as empty$5, Pn as interruptAllAs$1, Pr as interruptible$1, Pt as timeout, Q as isEffect, Qi as pipeArguments, Qn as isFailure, Qr as ids, Qt as mergeAll$2, R as failCause$4, Ri as flatMap$5, Rn as bindTo, Rr as succeed$4, Rt as uninterruptible$1, S as addFinalizer$1, Si as takeRight, Sn as isDieType, Sr as exitAs, St as runtime, T as as, Ti as unsafeGet, Tn as map$6, Tr as exitSucceed, Tt as scopeWith, U as forEach$2, Ui as isSome, Un as set$1, Ur as uninterruptibleMask$1, Ut as withFiberRuntime$1, V as flatten$4, Vi as getOrUndefined, Vn as make$16, Vr as tap$1, Vt as unsafeMakeSemaphore$1, W as forkDaemon, Wi as map$8, Wn as update$1, Wr as void_$2, Wt as zip$1, X as interruptible, Xi as match$2, Xn as fail$4, Xr as combine, Xt as tagged, Y as interrupt$4, Yi as left, Yn as die, Yr as empty$6, Yt as TaggedError$1, Z as intoDeferred, Zi as right, Zn as failCause$3, Zr as combineAll, Zt as fromEffect$5, _ as decodeSync, _i as of, _n as fail$3, _r as deferredIsDone, _t as raceWith, a as Literal, aa as constFalse, ai as size$2, an as updateContext$1, ar as zipRight$2, at as matchCause, b as acquireRelease, bi as reverse, bn as interrupt$3, br as deferredUnsafeDone, bt as runPromise, c as String$, ca as constVoid, ci as append$1, cn as Scope, cr as asyncInterrupt, ct as negate, d as Union, da as pipe, di as empty$4, dn as close, dr as deferredAwait, dt as orElse, ea as NodeInspectSymbol, ei as none$1, en as provideMerge$1, er as mapBoth, et as makeSemaphore, f as between, fi as filter, fn as extend, fr as deferredComplete, ft as orElseSucceed, gi as map$7, gn as die$1, gr as deferredInterruptWith, gt as race, hi as isEmpty$2, hn as NoSuchElementException, hr as deferredInterrupt, ht as provideService, i as Defect, ia as hasProperty, ii as has, in as unsafeRunPromiseExit, ir as zip$2, it as match$5, j as catchTag, ji as get$5, jn as scopeTag, jr as interrupt$2, jt as sync$2, k as catchAllCause, ki as Tag, kn as forEachConcurrentDiscard, kr as fiberRefLocally, kt as succeed$6, l as Struct, la as dual, li as appendAll, ln as addFinalizer, lr as catchAll, lt as onError, m as optional$1, mi as isChunk, mn as make$18, mr as deferredFailCause, mt as provide$2, n as Boolean$, na as toJSON, ni as make$17, nn as succeed$3, nr as succeed$5, nt as mapBoth$1, o as Never, oa as constTrue, oi as decode, on as forkIn$1, or as as$1, ot as matchCauseEffect, p as decodeUnknown, pi as fromIterable$1, pn as fork, pr as deferredFail, pt as promise, q as gen, qi as some, qn as spanTag, qr as zipRight$1, qt as zipWith, r as Cause, ra as equals, ri as set$2, rn as unsafeFork, rr as void_$3, rt as mapError, s as Number$, sa as constUndefined, si as isFinite, sn as unsafeMakeSemaphore, sr as asVoid, st as matchEffect, t as Array$, ta as format, ti as get$4, tn as scoped$3, tr as match$3, tt as map$4, u as TaggedError, ua as identity, ui as drop, un as addFinalizerExit, ur as currentContext$1, ut as orDie, v as encodeSync, vi as prepend, vn as failureOrCause, vr as deferredMake, vt as repeat, w as annotateLogs, wi as unsafeFromArray, wn as isInterruptedOnly, wr as exitFailCause, wt as scope, x as acquireUseRelease$2, xi as take$2, xn as interruptors, xr as deferredUnsafeMake, xt as runPromiseExit$1, y as _void, yi as reduce, yn as flipCauseOption, yr as deferredSucceed, yt as retry, z as fiberIdWith, zi as fromNullable, zn as sleep, zr as suspend$4, zt as uninterruptibleMask } from "./chunks/Schema-C6fLlQzx.mjs";
 import { createRequire } from "node:module";
 import * as Crypto from "node:crypto";

package/dist/web/main.js

@@ -31565,6 +31565,8 @@ const qaDescriptor = {
 	unansweredQuestions,
 	openThreads: openThreadsOf
 };
+/** A chunk's leading `_critical: security_` paragraph: the tier gtd ranked it with, and why. */
+const SEVERITY_LINE_RE = /^_((?:critical|should|could)(?::[^_\n]*)?)_$/;
 /**
 * The canonical sample. Every detail here is load-bearing for the formatter
 * round-trip: both footnote bodies exceed 80 characters so a reflow actually
@@ -31855,12 +31857,26 @@ const answerListOf = (content, body) => {
 		})
 	};
 };
+/** A leading severity line is the chunk's tier, never its prose. */
+const splitSeverity = (content, run) => {
+	const first = run[0];
+	const label = first?.type === "paragraph" ? SEVERITY_LINE_RE.exec(sourceText(content, first).trim())?.[1] : void 0;
+	if (label === void 0) return { leadingRun: run };
+	return {
+		severity: {
+			label,
+			line: toLspPosition(first.position.start).line
+		},
+		leadingRun: run.slice(1)
+	};
+};
 /** Splits one chunk's body nodes into its file pointers (hunk pointers are task items, collected recursively at ANY nesting depth via `taskItems` — a nested hunk is the same kind of hunk as a top-level one) and description prose. The description is the chunk's own leading run and never a node that contains a hunk pointer: it stops at the first node whose `taskItems` include a real pointer (at any depth — a blockquote wrapping a pointer list stops it just as a top-level list would). Every node kind in what's left before that point contributes — a `footnoteDefinition` is the one exception, excluded by `blockNodesOfRun` itself (it's a note, never document content). */
 const parseChunkBody = (content, lines, title, body) => {
 	const answer = answerListOf(content, body);
 	const isAnswer = (n) => answer !== void 0 && (n === answer.list || n === body[answer.lead]);
 	const firstPointerIndex = body.findIndex((n) => !isAnswer(n) && taskItems(n).some((item) => hasPointerToken(content, item)));
-	const leadingRun = (firstPointerIndex === -1 ? body : body.slice(0, firstPointerIndex)).filter((n) => !isAnswer(n));
+	const run = (firstPointerIndex === -1 ? body : body.slice(0, firstPointerIndex)).filter((n) => !isAnswer(n));
+	const { severity, leadingRun } = splitSeverity(content, run);
 	const descriptionNodes = blockNodesOfRun(content, leadingRun);
 	const descriptionInline = collapseImages(blockRunInline(content, leadingRun));
 	const description = leadingRun.filter((n) => n.type !== "footnoteDefinition").map((n) => sourceText(content, n)).join(" ").trim();
@@ -31879,6 +31895,7 @@ const parseChunkBody = (content, lines, title, body) => {
 		files,
 		errors,
 		...answer === void 0 ? {} : { answers: answer.options },
+		...severity === void 0 ? {} : { severity },
 		boxed: body.some((n) => !isAnswer(n) && taskItems(n).length > 0)
 	};
 };
@@ -31915,7 +31932,7 @@ const parseChangesets = (tree, content, lines) => {
 	const errors = [];
 	for (const { heading, headingLine, body } of splitChunks(tree)) {
 		const title = headingText$1(content, heading);
-		const { description, descriptionNodes, descriptionInline, files, answers, boxed, errors: bodyErrors } = parseChunkBody(content, lines, title, body);
+		const { description, descriptionNodes, descriptionInline, files, answers, severity, boxed, errors: bodyErrors } = parseChunkBody(content, lines, title, body);
 		errors.push(...bodyErrors);
 		if (files.length === 0 && boxed) errors.push({
 			message: `Chunk "${title}" has no file pointers`,
@@ -31929,7 +31946,8 @@ const parseChangesets = (tree, content, lines) => {
 			descriptionInline,
 			files,
 			headingLine,
-			...answers === void 0 ? {} : { answers }
+			...answers === void 0 ? {} : { answers },
+			...severity === void 0 ? {} : { severity }
 		});
 	}
 	if (changesets.length === 0) errors.push({ message: "REVIEW.md has no '##' chunks" });
@@ -32230,6 +32248,7 @@ const reviewView = (content) => {
 			detail: chunk.description,
 			detailInline: chunk.descriptionInline,
 			body: chunk.descriptionNodes,
+			...chunk.severity === void 0 ? {} : { status: chunk.severity.label },
 			anchor: {
 				kind: "chunk",
 				index: chunkIndex
@@ -34779,6 +34798,11 @@ const ChunkRow = ({ chunk, chunkIndex, state }) => {
 					disabled: hunks.length === 0,
 					className: "flex-1 text-left",
 					children: [
+						chunk.status !== void 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+							"data-testid": `chunk-severity-${chunkIndex}`,
+							className: cn("text-small font-semibold", chunk.status.startsWith("critical") ? "text-warning" : "text-muted"),
+							children: chunk.status
+						}),
 						/* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
 							className: "font-semibold",
 							children: chunk.title

package/dist/workflow-kit.mjs

@@ -1,5 +1,5 @@
 import { changes, changesSince, hasThreadFor, head, mark, read, refuse, requireReplies, requireThreadsClosed, scope } from "./flows.mjs";
-import { T as parseThreads, _ as reviewRisks, c as stripCodeThreads, g as reviewNotes, h as reviewDecisions, v as unansweredNotes } from "./chunks/steering-BXwjmocB.mjs";
+import { E as parseThreads, _ as reviewRisks, c as stripCodeThreads, g as reviewNotes, h as reviewDecisions, y as unansweredNotes } from "./chunks/steering-DtUVUz9l.mjs";
 //#region src/workflow-kit/health.ts
 /** Hand a red suite that repeated attempts could not fix to a person, until `limit` rounds are spent. */
 const escalation = async (count, steps) => {

package/dist/workflows/cleanup.mjs

@@ -1,6 +1,6 @@
 import { agent, changes, glob, read, refuse, scope, start, vars } from "../flows.mjs";
-import { Dt as envDefaults, Et as defaults, Ht as requireFlag, S as coder, Vt as registryEntry, i as skills, n as summary, o as flagShipped, r as access, t as steering, zt as flagReferences } from "../chunks/reserved-CYZ1bSBy.mjs";
-import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-C1Oi2YEy.mjs";
+import { At as defaults, Gt as registryEntry, Kt as requireFlag, S as coder, Ut as flagReferences, i as skills, jt as envDefaults, n as summary, o as flagShipped, r as access, t as steering } from "../chunks/reserved-CdOzKFVX.mjs";
+import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-DRM2eC1z.mjs";
 //#region src/workflows/cleanup/cleanup.ts
 const removingPrompt = (name, registry) => `Clean up the release flag "${name}", which is default-on: delete its entry in ${registry}, and remove every \`GTD_FLAG_${name.toUpperCase()}\` check from the shipped sources. Keep the flag-on path; delete code only the off path reached, including tests that pin the off path.`;
 const cleanupFlow = async () => {

package/dist/workflows/feature.mjs

@@ -1,8 +1,8 @@
 import { agent, answered, applyScript, changes, changesSince, check, codeChanges, env, hasThreadFor, head, human, judge, judgeDiffShape, kept, mark, matches, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireThreadsClosed, revertScript, run, scope, slice, start, vars, wrote } from "../flows.mjs";
 import { lensEntriesOf } from "../workflow-kit.mjs";
-import { $ as naivePassPrompt, B as coverageQuestion, C as collecting, Dt as envDefaults, Et as defaults, F as builderSystem, G as designTriagePrompt, H as criterionLine, I as clearCriteriaMessage, L as clearCriteriaQuestion, M as agentWithSkills, N as buildReviewCollectingPrompt, O as planner, Ot as testFileGlobs, Q as naiveApplyFailure, S as coder, St as unwindFailure, T as fixNits, U as designGateAnswerMessage, V as coveringPrompt, W as designSystem, X as implementBuildingPrompt, Y as idleMessage, Z as implementFixPrompt, _ as REQUIREMENTS, _t as slicingPrompt, a as flagModeOn, at as refactorDroppedRefusal, b as answerReviewQuestions, bt as uncoveredMessage, ct as refactorRules, d as healthy, et as normaliseCriteriaScript, f as FEEDBACK, g as REFACTOR_NOTE, gt as seamSteerMessage, h as REFACTOR, ht as seamByHandMessage, i as skills, it as refactorBuildingPrompt, l as gate, mt as reviewerSystem, n as summary, ot as refactorFixPrompt, p as HANDOFF, r as access, s as shipped, st as refactorNotePrompt, t as steering, tt as oversizedMessage, vt as startGateBlockedMessage, xt as unwindFailedMessage, y as SLICES, z as coverageMessage } from "../chunks/reserved-CYZ1bSBy.mjs";
-import { a as refusePendingHandOff, c as joinReports, o as tail } from "../chunks/tail-C1Oi2YEy.mjs";
-import { i as reUnwind, n as humanReview, r as isCode$1, t as buildTail } from "../chunks/review-BewyJflS.mjs";
+import { $ as naiveApplyFailure, At as defaults, B as coverageMessage, C as collecting, Ct as uncoveredMessage, Et as unwindFailure, F as builderSystem, G as designSystem, H as coveringPrompt, K as designTriagePrompt, L as clearCriteriaMessage, M as agentWithSkills, Mt as testFileGlobs, N as buildReviewCollectingPrompt, O as planner, Q as implementFixPrompt, R as clearCriteriaQuestion, S as coder, T as fixNits, Tt as unwindFailedMessage, U as criterionLine, V as coverageQuestion, W as designGateAnswerMessage, X as idleMessage, Z as implementBuildingPrompt, _ as REQUIREMENTS, _t as reviewerSystem, a as flagModeOn, b as answerReviewQuestions, bt as slicingPrompt, ct as refactorFixPrompt, d as healthy, et as naivePassPrompt, f as FEEDBACK, g as REFACTOR_NOTE, h as REFACTOR, i as skills, jt as envDefaults, l as gate, lt as refactorNotePrompt, n as summary, nt as oversizedMessage, ot as refactorBuildingPrompt, p as HANDOFF, r as access, s as shipped, st as refactorDroppedRefusal, t as steering, tt as normaliseCriteriaScript, ut as refactorRules, vt as seamByHandMessage, xt as startGateBlockedMessage, y as SLICES, yt as seamSteerMessage } from "../chunks/reserved-CdOzKFVX.mjs";
+import { a as refusePendingHandOff, c as joinReports, o as tail } from "../chunks/tail-DRM2eC1z.mjs";
+import { i as reUnwind, n as humanReview, r as isCode$1, t as buildTail } from "../chunks/review-BChXEl2Y.mjs";
 import { posix } from "node:path";
 //#region src/workflows/shared/fences.ts
 const markFenced = (text) => {

package/dist/workflows/fix.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, removeScript, run, scope, start } from "../flows.mjs";
-import { _ as reviewRisks, g as reviewNotes } from "../chunks/steering-BXwjmocB.mjs";
-import { Dt as envDefaults, Et as defaults, K as finisherSystem, M as agentWithSkills, S as coder, c as baseline, i as skills, n as summary, q as fixReviewNotesPrompt, r as access, s as shipped, t as steering, v as REVIEW } from "../chunks/reserved-CYZ1bSBy.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-C1Oi2YEy.mjs";
+import { _ as reviewRisks, g as reviewNotes } from "../chunks/steering-DtUVUz9l.mjs";
+import { At as defaults, J as fixReviewNotesPrompt, M as agentWithSkills, S as coder, c as baseline, i as skills, jt as envDefaults, n as summary, q as finisherSystem, r as access, s as shipped, t as steering, v as REVIEW } from "../chunks/reserved-CdOzKFVX.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-DRM2eC1z.mjs";
 //#region src/workflows/fix/fix.ts
 /**
 * Fix every note a human left in a handed-off `.gtd/REVIEW.md` and every

package/dist/workflows/review.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, removeScript, run, start } from "../flows.mjs";
-import { Dt as envDefaults, Et as defaults, K as finisherSystem, M as agentWithSkills, S as coder, _ as REQUIREMENTS, dt as reviewFeedbackFixPrompt, ft as reviewGateBlockedMessage, i as skills, l as gate, n as summary, r as access, s as shipped, t as steering } from "../chunks/reserved-CYZ1bSBy.mjs";
-import { a as refusePendingHandOff } from "../chunks/tail-C1Oi2YEy.mjs";
-import { i as reUnwind, t as buildTail } from "../chunks/review-BewyJflS.mjs";
+import { At as defaults, M as agentWithSkills, S as coder, _ as REQUIREMENTS, i as skills, jt as envDefaults, l as gate, mt as reviewGateBlockedMessage, n as summary, pt as reviewFeedbackFixPrompt, q as finisherSystem, r as access, s as shipped, t as steering } from "../chunks/reserved-CdOzKFVX.mjs";
+import { a as refusePendingHandOff } from "../chunks/tail-DRM2eC1z.mjs";
+import { i as reUnwind, t as buildTail } from "../chunks/review-BChXEl2Y.mjs";
 //#region src/workflows/review/review.ts
 const fixFeedback = async () => {
 	await agentWithSkills("feedback.fixing", reviewFeedbackFixPrompt(), {

package/dist/workflows/rollout.mjs

@@ -1,5 +1,5 @@
 import { agent, changesSince, glob, head, mark, read, refuse, removeScript, run, scope, vars } from "../flows.mjs";
-import { Bt as flipProblems, Dt as envDefaults, Et as defaults, Ht as requireFlag, Rt as flagMentions, S as coder, f as FEEDBACK, i as skills, l as gate, n as summary, o as flagShipped, r as access, t as steering } from "../chunks/reserved-CYZ1bSBy.mjs";
+import { At as defaults, Ht as flagMentions, Kt as requireFlag, S as coder, Wt as flipProblems, f as FEEDBACK, i as skills, jt as envDefaults, l as gate, n as summary, o as flagShipped, r as access, t as steering } from "../chunks/reserved-CdOzKFVX.mjs";
 //#region src/workflows/rollout/rollout.ts
 const flippingPrompt = (name, registry) => `Roll out the release flag "${name}": in ${registry}, set \`defaultOn: true\` on its entry. Change nothing else — no other entry, no other field, no other file, no comments.`;
 const suiteBlockedMessage = () => "The suite is red after the rollout. Fix it by hand — a rollout may only touch the flag's registry entry — then land.";

package/dist/workflows.mjs

@@ -15648,6 +15648,9 @@ const qaDescriptor = {
 const ANSWER_LEAD = "Tick exactly one:";
 const ANSWER_FIX = "fix as suggested";
 const ANSWER_WONT_FIX = "won't fix";
+/** A chunk's leading `_critical: security_` paragraph: the tier gtd ranked it with, and why. */
+const SEVERITY_LINE_RE = /^_((?:critical|should|could)(?::[^_\n]*)?)_$/;
+const severityLine = (label) => `_${label}_`;
 /**
 * The canonical sample. Every detail here is load-bearing for the formatter
 * round-trip: both footnote bodies exceed 80 characters so a reflow actually
@@ -15938,12 +15941,26 @@ const answerListOf = (content, body) => {
 		})
 	};
 };
+/** A leading severity line is the chunk's tier, never its prose. */
+const splitSeverity = (content, run) => {
+	const first = run[0];
+	const label = first?.type === "paragraph" ? SEVERITY_LINE_RE.exec(sourceText(content, first).trim())?.[1] : void 0;
+	if (label === void 0) return { leadingRun: run };
+	return {
+		severity: {
+			label,
+			line: toLspPosition(first.position.start).line
+		},
+		leadingRun: run.slice(1)
+	};
+};
 /** Splits one chunk's body nodes into its file pointers (hunk pointers are task items, collected recursively at ANY nesting depth via `taskItems` — a nested hunk is the same kind of hunk as a top-level one) and description prose. The description is the chunk's own leading run and never a node that contains a hunk pointer: it stops at the first node whose `taskItems` include a real pointer (at any depth — a blockquote wrapping a pointer list stops it just as a top-level list would). Every node kind in what's left before that point contributes — a `footnoteDefinition` is the one exception, excluded by `blockNodesOfRun` itself (it's a note, never document content). */
 const parseChunkBody = (content, lines, title, body) => {
 	const answer = answerListOf(content, body);
 	const isAnswer = (n) => answer !== void 0 && (n === answer.list || n === body[answer.lead]);
 	const firstPointerIndex = body.findIndex((n) => !isAnswer(n) && taskItems(n).some((item) => hasPointerToken(content, item)));
-	const leadingRun = (firstPointerIndex === -1 ? body : body.slice(0, firstPointerIndex)).filter((n) => !isAnswer(n));
+	const run = (firstPointerIndex === -1 ? body : body.slice(0, firstPointerIndex)).filter((n) => !isAnswer(n));
+	const { severity, leadingRun } = splitSeverity(content, run);
 	const descriptionNodes = blockNodesOfRun(content, leadingRun);
 	const descriptionInline = collapseImages(blockRunInline(content, leadingRun));
 	const description = leadingRun.filter((n) => n.type !== "footnoteDefinition").map((n) => sourceText(content, n)).join(" ").trim();
@@ -15962,6 +15979,7 @@ const parseChunkBody = (content, lines, title, body) => {
 		files,
 		errors,
 		...answer === void 0 ? {} : { answers: answer.options },
+		...severity === void 0 ? {} : { severity },
 		boxed: body.some((n) => !isAnswer(n) && taskItems(n).length > 0)
 	};
 };
@@ -15998,7 +16016,7 @@ const parseChangesets = (tree, content, lines) => {
 	const errors = [];
 	for (const { heading, headingLine, body } of splitChunks(tree)) {
 		const title = headingText$1(content, heading);
-		const { description, descriptionNodes, descriptionInline, files, answers, boxed, errors: bodyErrors } = parseChunkBody(content, lines, title, body);
+		const { description, descriptionNodes, descriptionInline, files, answers, severity, boxed, errors: bodyErrors } = parseChunkBody(content, lines, title, body);
 		errors.push(...bodyErrors);
 		if (files.length === 0 && boxed) errors.push({
 			message: `Chunk "${title}" has no file pointers`,
@@ -16012,7 +16030,8 @@ const parseChangesets = (tree, content, lines) => {
 			descriptionInline,
 			files,
 			headingLine,
-			...answers === void 0 ? {} : { answers }
+			...answers === void 0 ? {} : { answers },
+			...severity === void 0 ? {} : { severity }
 		});
 	}
 	if (changesets.length === 0) errors.push({ message: "REVIEW.md has no '##' chunks" });
@@ -16313,6 +16332,7 @@ const reviewView = (content) => {
 			detail: chunk.description,
 			detailInline: chunk.descriptionInline,
 			body: chunk.descriptionNodes,
+			...chunk.severity === void 0 ? {} : { status: chunk.severity.label },
 			anchor: {
 				kind: "chunk",
 				index: chunkIndex
@@ -16460,15 +16480,19 @@ const reviewDescriptor = {
 	openThreads: openThreadsOf
 };
 const pointerLabel = (f) => `${f.path}${f.line === void 0 ? "" : `#${f.line}${f.rangeEnd === void 0 ? "" : `-${f.rangeEnd}`}`}`;
-const chunkNote = (chunk, prior) => chunk.description === (prior?.description ?? "") || prior === void 0 && chunk.answers !== void 0 ? [] : [{
-	line: chunk.headingLine,
-	note: {
-		kind: "chunk",
-		anchor: chunk.title,
-		before: prior?.description ?? "",
-		text: chunk.description
-	}
-}];
+const chunkProse = (chunk, retiered) => [chunk?.description ?? "", retiered && chunk?.severity !== void 0 ? severityLine(chunk.severity.label) : ""].filter((p) => p !== "").join("\n\n");
+const chunkNote = (chunk, prior) => {
+	const retiered = chunk.severity !== void 0 && chunk.severity.label !== prior?.severity?.label;
+	return chunkProse(chunk, retiered) === chunkProse(prior, retiered) || prior === void 0 && chunk.answers !== void 0 ? [] : [{
+		line: chunk.headingLine,
+		note: {
+			kind: "chunk",
+			anchor: chunk.title,
+			before: chunkProse(prior, retiered),
+			text: chunkProse(chunk, retiered)
+		}
+	}];
+};
 const pointerNotes = (chunk, prior) => chunk.files.flatMap((file) => {
 	const label = pointerLabel(file);
 	const was = prior?.files.find((f) => pointerLabel(f) === label);
@@ -16582,7 +16606,13 @@ const reviewDecisions = (before, after) => {
 const reviewChunks = (content) => parseReviewDoc(content).changesets.map((c) => ({
 	title: c.title,
 	line: c.headingLine,
-	pointers: c.files.length
+	pointers: c.files.length,
+	...c.severity === void 0 ? {} : { severityLine: c.severity.line },
+	files: c.files.map((f) => ({
+		path: f.path,
+		...f.line === void 0 ? {} : { line: f.line },
+		...f.rangeEnd === void 0 ? {} : { rangeEnd: f.rangeEnd }
+	}))
 }));
 //#endregion
 //#region src/steering/CodeThreads.ts
@@ -17609,6 +17639,56 @@ const withWalkthrough = (review, report) => {
 	const body = lines.join("\n").trimEnd();
 	return summary.length === 0 ? `${body}\n` : `${body}\n\n${summary.join("\n\n")}\n`;
 };
+const chunkBlocks = (review) => {
+	const lines = review.split("\n");
+	const chunks = reviewChunks(review);
+	return {
+		head: lines.slice(0, chunks[0]?.line ?? lines.length),
+		blocks: chunks.map((c, i) => ({
+			title: c.title,
+			lines: lines.slice(c.line, chunks[i + 1]?.line ?? lines.length)
+		}))
+	};
+};
+const isBlank = (line) => line !== void 0 && line.trim() === "";
+/** \`review\` with every chunk's severity line removed, so a re-review is ranked afresh. */
+const unranked = (review) => {
+	const drop = new Set(reviewChunks(review).flatMap((c) => c.severityLine ?? []));
+	if (drop.size === 0) return review;
+	const lines = review.split("\n");
+	return lines.filter((_, i) => !drop.has(i) && !(drop.has(i - 1) && isBlank(lines[i]))).join("\n");
+};
+const chunkTexts = (review) => new Map(chunkBlocks(review).blocks.map((b) => [b.title, b.lines.join("\n").trim()]));
+const trimBlankLines = (lines) => {
+	let start = 0;
+	let end = lines.length;
+	while (start < end && isBlank(lines[start])) start++;
+	while (end > start && isBlank(lines[end - 1])) end--;
+	return lines.slice(start, end);
+};
+/**
+* \`review\` with the \`ranked\` chunks first, in their order, each led by its
+* severity line; every other chunk follows in place.
+*/
+const rankedReview = (review, ranked) => {
+	const { head, blocks } = chunkBlocks(review);
+	const place = new Map(ranked.map((r, k) => [r.title, k]));
+	const labels = new Map(ranked.map((r) => [r.title, r.label]));
+	const order = (i) => place.get(blocks[i].title) ?? ranked.length;
+	const chunk = (i) => {
+		const { title, lines } = blocks[i];
+		const label = labels.get(title);
+		const heading = /^ {0,3}-+\s*$/.test(lines[1] ?? "") ? 2 : 1;
+		const body = trimBlankLines(lines.slice(heading)).join("\n");
+		return [
+			lines.slice(0, heading).join("\n"),
+			...label === void 0 ? [] : [severityLine(label)],
+			...body === "" ? [] : [body]
+		].join("\n\n");
+	};
+	const sorted = blocks.map((_, i) => i).sort((x, y) => order(x) - order(y) || x - y);
+	return `${[head.join("\n").trim(), ...sorted.map(chunk)].filter((p) => p !== "").join("\n\n")}\n`;
+};
 /**
 * \`.gtd/REVIEW.md\` as gtd writes it: one chunk per note, each ending in its
 * answer boxes, then the fixed and noise lists. Chunk titles are made unique —
@@ -18163,6 +18243,13 @@ const wrapItem = (item) => {
 	} else line = line === "" ? word : `${line} ${word}`;
 	return [...out, line];
 };
+const reviewRankingMessage = () => `Ranking each chunk of the review before the human reads it. Run
+\`gtd judge answer\` and pipe, per chunk \`C<n>\`: \`C<n>.security\`
+(yes/no), \`C<n>.reversibility\` (one-way/two-way),
+\`C<n>.blastRadius\` (local/module/contract) and \`C<n>.newConcept\`
+(yes/no) — or land without a verdict, which lists every chunk as
+critical.
+`;
 const buildReviewTriageMessage = () => `Judging each note the human added to \`.gtd/REVIEW.md\` (a line note, a
 footnote, or prose under a chunk) with one verdict: \`edit\` (a change
 request), \`question\`, \`nit\` (a small fix needing no re-plan) or
@@ -39052,7 +39139,8 @@ const severityOf = (note, judgment, minP) => {
 	if (critical.length > 0) return {
 		tier: "critical",
 		reason: `critical: ${critical.join(", ")}`,
-		rank: rank()
+		rank: rank(),
+		...a("security") === "yes" ? { security: true } : {}
 	};
 	const should = [...a("blastRadius") === "contract" ? ["contract"] : [], ...a("newConcept") === "yes" ? ["new concept"] : []];
 	if (should.length > 0) return {
@@ -39066,29 +39154,30 @@ const severityOf = (note, judgment, minP) => {
 		rank: 2
 	};
 };
-const severityQuestions = (note) => [
+/** `asks` names the change in full, `change` refers back to it. */
+const severityQuestions = (id, asks, change) => [
 	{
-		id: `${note.id}.security`,
+		id: `${id}.security`,
 		primitive: "noul",
-		instructions: `Does the change the human asks for in note "${note.id}" (on "${note.anchor}") touch security — authentication, authorisation, secrets, untrusted input or data exposure?`,
+		instructions: `Does ${asks} touch security — authentication, authorisation, secrets, untrusted input or data exposure?`,
 		criteria: "yes: it changes or could weaken a security property. no: it does not."
 	},
 	{
-		id: `${note.id}.reversibility`,
+		id: `${id}.reversibility`,
 		primitive: "choice",
-		instructions: `Once the change in note "${note.id}" ships, can it be undone without breaking anyone who already depends on it?`,
+		instructions: `Once ${change} ships, can it be undone without breaking anyone who already depends on it?`,
 		criteria: "one-way: a published API, file format, persisted data, protocol or other contract that cannot be taken back. two-way: can be reverted freely."
 	},
 	{
-		id: `${note.id}.blastRadius`,
+		id: `${id}.blastRadius`,
 		primitive: "choice",
-		instructions: `How far does the change in note "${note.id}" reach?`,
+		instructions: `How far does ${change} reach?`,
 		criteria: "local: one function or file. module: a whole module or several files. contract: a public interface other code or users depend on."
 	},
 	{
-		id: `${note.id}.newConcept`,
+		id: `${id}.newConcept`,
 		primitive: "noul",
-		instructions: `Does the change in note "${note.id}" introduce a concept new to this codebase — a new abstraction, entity or term? Judge from the evidence, never a glossary; reusing or extending an existing concept is not new.`,
+		instructions: `Does ${change} introduce a concept new to this codebase — a new abstraction, entity or term? Judge from the evidence, never a glossary; reusing or extending an existing concept is not new.`,
 		criteria: "yes: a genuinely new concept. no: reuses or extends an existing one."
 	}
 ];
@@ -39105,7 +39194,7 @@ const triage$1 = (severities) => async (notes) => {
 		answers: {},
 		truncated: []
 	} : await judge("review.triage", {
-		questions: asked.flatMap((n) => [verdictQuestion(n), ...severityQuestions(n)]),
+		questions: asked.flatMap((n) => [verdictQuestion(n), ...severityQuestions(n.id, `the change the human asks for in note "${n.id}" (on "${n.anchor}")`, `the change in note "${n.id}"`)]),
 		evidence: Object.fromEntries(asked.map((n) => [n.id, `Anchor: ${n.anchor}\nReviewer's text: ${n.before}\nHuman's text: ${n.text}`])),
 		message: buildReviewTriageMessage(),
 		label: "Judging each note"
@@ -39121,6 +39210,67 @@ const triage$1 = (severities) => async (notes) => {
 		return [n.id, "edit"];
 	}));
 };
+/** Review order: security, one-way doors, the judge unsure, then should, then could. */
+const chunkKey = (s) => s.tier === "could" ? 4 : s.tier === "should" ? 3 : s.security === true ? 0 : s.rank === 0 ? 1 : 2;
+/** A chunk's text, then the code each of its pointers covers: a bare path the whole file. */
+const chunkEvidence = (text, files) => [text, ...files.map(({ path, line, rangeEnd }) => {
+	const where = `${path}${line === void 0 ? "" : `#${line}${rangeEnd === void 0 ? "" : `-${rangeEnd}`}`}`;
+	const lines = read(path.replace(/^\.\//, ""))?.split("\n");
+	if (lines === void 0) return `${where}: no longer exists`;
+	return `${where}:\n${(line === void 0 ? lines : lines.slice(line - 1, rangeEnd ?? line)).join("\n")}`;
+})].join("\n\n");
+const bytes = (text) => new TextEncoder().encode(text).length;
+/** Fills each batch while the budget's even share still holds its widest evidence, so nothing is cut that fits alone. */
+const batched = (items, budget) => {
+	const batches = [];
+	let batch = [];
+	let widest = 0;
+	for (const item of items) {
+		const size = bytes(item.evidence);
+		if (batch.length > 0 && Math.floor(budget / (batch.length + 1)) < Math.max(widest, size)) {
+			batches.push(batch);
+			batch = [];
+			widest = 0;
+		}
+		batch.push(item);
+		widest = Math.max(widest, size);
+	}
+	return batch.length === 0 ? batches : [...batches, batch];
+};
+/**
+* Rates every chunk the reviewer wrote — a walkthrough chunk or a carried
+* finding — and sorts \`full\` by it, each chunk led by its tier and reason.
+* gtd's summary chunks are not rated and stay last.
+*/
+const ranked = async (review, full) => {
+	const texts = chunkTexts(full);
+	const chunks = reviewChunks(review).map((c, i) => ({
+		id: `C${i + 1}`,
+		title: c.title,
+		evidence: chunkEvidence(texts.get(c.title) ?? "", c.files)
+	}));
+	if (chunks.length === 0) return full;
+	const minP = numeric(vars.reviewSeverityMinP, Infinity);
+	const batches = batched(chunks, numeric(vars.judgeBudgetBytes, 32768));
+	const severities = /* @__PURE__ */ new Map();
+	for (const [i, batch] of batches.entries()) {
+		const judgment = await judge("review.ranking", {
+			questions: batch.flatMap((c) => severityQuestions(c.id, `the change review chunk "${c.id}" ("${c.title}") walks through`, `the change in review chunk "${c.id}"`)),
+			evidence: Object.fromEntries(batch.map((c) => [c.id, c.evidence])),
+			message: reviewRankingMessage(),
+			label: batches.length === 1 ? "Ranking the review" : `Ranking the review (${i + 1}/${batches.length})`
+		});
+		for (const c of batch) severities.set(c.id, severityOf(c, judgment, minP));
+	}
+	return rankedReview(full, chunks.map((c, i) => ({
+		c,
+		i,
+		s: severities.get(c.id)
+	})).sort((x, y) => chunkKey(x.s) - chunkKey(y.s) || x.i - y.i).map(({ c, s }) => ({
+		title: c.title,
+		label: s.reason
+	})));
+};
 const isCode$1 = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 const composedOf = (base, report, nitFix, answered) => ({
 	...report,
@@ -39172,12 +39322,13 @@ const requirePointers = (review, notes, codeChanged) => {
 const walkthrough = async (base, report, lap, carry) => {
 	const notes = titledNotes(report);
 	await reviewing(base, carry, lap, walkthroughBlock(notes, report.nitFix));
-	const review = read(REVIEW);
-	if (review === void 0) return;
-	requirePointers(review, notes, changesSince(base).some((c) => isCode$1(c.path)));
-	const full = withWalkthrough(review, report);
-	if (full !== review) await run("review.carrying", writeScript(REVIEW, full), {
-		label: "Adding the quality lap's findings",
+	const written = read(REVIEW);
+	if (written === void 0) return;
+	const review = unranked(written);
+	requirePointers(written, notes, changesSince(base).some((c) => isCode$1(c.path)));
+	const full = await ranked(review, withWalkthrough(review, report));
+	if (full !== written) await run("review.carrying", writeScript(REVIEW, full), {
+		label: "Ordering the review and adding the quality lap's findings",
 		file: REVIEW,
 		base
 	});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "49.0.0",
+  "version": "50.0.0",
   "private": false,
   "description": "Git-aware CLI that emits the next prompt for an autonomous coding agent based on the current repository state",
   "bin": {

package/src/steering/index.ts

@@ -16,6 +16,7 @@ export {
   reviewDecisions,
   reviewNotes,
   reviewRisks,
+  severityLine,
   unansweredNotes,
   type ReviewChoice,
   type ReviewDecision,

package/src/steering/review.test.ts

@@ -1276,6 +1276,33 @@ describe("review.view", () => {
     expect(view.nodes[1]!.detail).toBe("")
   })
 
+  it("shows a chunk's leading severity line as its status, not its prose", () => {
+    const view = review.view(
+      [
+        "# Review: abc1234",
+        "",
+        "## Ranked",
+        "",
+        "_critical: security, one-way door_",
+        "",
+        "The new export.",
+        "",
+        "- [ ] ./a.ts#1-1",
+        "",
+        "## Unranked",
+        "",
+        "_emphasis, not a tier_",
+        "",
+        "- [ ] ./a.ts#2-2",
+        "",
+      ].join("\n"),
+    )
+    expect(view.nodes.map((c) => [c.status, c.detail])).toEqual([
+      ["critical: security, one-way door", "The new export."],
+      [undefined, "_emphasis, not a tier_"],
+    ])
+  })
+
   it("is built from one parse of the document, not one per element", () => {
     const uniqueContent = [
       "# Review: def4567",
@@ -1706,6 +1733,29 @@ describe("reviewNotes", async () => {
     expect(reviewNotes(base, base)).toEqual([])
   })
 
+  it("an edited tier line is a note", () => {
+    const was = doc("_could_\n\nProse.")
+    const after = doc("_critical: this leaks the token_\n\nProse.")
+    expect(reviewNotes(was, after)).toMatchObject([
+      {
+        kind: "chunk",
+        before: "Prose.\n\n_could_",
+        text: "Prose.\n\n_critical: this leaks the token_",
+      },
+    ])
+  })
+
+  it("prose added under a tier is the human's alone, and a leading ! still forces", () => {
+    const was = doc("_could_\n\nProse.")
+    expect(reviewNotes(was, doc("_could_\n\n! Not here.\n\nProse."))).toMatchObject([
+      { kind: "chunk", text: "! Not here. Prose.", forced: true },
+    ])
+  })
+
+  it("a deleted tier line asks nothing", () => {
+    expect(reviewNotes(doc("_could_\n\nProse."), base)).toEqual([])
+  })
+
   it("a pointer note added", () => {
     const after = doc("Prose.").replace("#1-3", "#1-3 rename this")
     expect(reviewNotes(base, after)).toEqual([
@@ -1863,8 +1913,16 @@ describe("reviewChunks", async () => {
       "- [ ] _your answer_",
     ].join("\n")
     expect(reviewChunks(content)).toEqual([
-      { title: "Walkthrough", line: 4, pointers: 2 },
-      { title: "Fixed automatically", line: 9, pointers: 0 },
+      {
+        title: "Walkthrough",
+        line: 4,
+        pointers: 2,
+        files: [
+          { path: "./src/a.ts", line: 1, rangeEnd: 3 },
+          { path: "./src/b.ts", line: 4, rangeEnd: 6 },
+        ],
+      },
+      { title: "Fixed automatically", line: 9, pointers: 0, files: [] },
     ])
   })
 })