| 1 |
Time/cost/dependency-size expectations |
Not in spec — owner decided it stays out |
No wall-clock/cost/GB figure anywhere (embedding model named twice with no size; Oracle cost qualitative only). Owner: cost figures are perishable, machine-dependent data — the spec-philosophy class we deliberately exclude. Instead: the recommended baseline IS the minimum-resource path, and the scaled test measures it (see Measurement below). |
| 2 |
Footprint/runtime stop rule for the build agent |
Dropped by owner decision |
No bound exists in §6/§7; none will be added. The baseline path is the structural resource bound. Note for fairness to the 5.6 GB report: most of that footprint is legitimate dependencies (torch + model), not runaway output. Replaced by heavy-option flags (next-pass item B2). |
| 3 |
Keyless path not framed as a legitimate finish |
Mechanism YES, framing NO |
The 19-offline/6-live split, RUN_LIVE=1 gating, and skip-with-reason reporting all exist and work (run-09 and gate-probe-2 both completed keyless). But no sentence blesses it — "5 skipped" reads as unfinished debt. Fix: the baseline-completion sentence — "the recommended baseline build is complete when the offline suite is green; live ACs are an optional keyed extension; skipping them without a key is the expected result." Also resolves item 7. → batch item A1. |
| 4 |
Stale model names / OpenAI pinning / parameter constraints |
Defaults unchanged; scaffolding exists; F12 built the right slot |
D4's invariant is already provider-agnostic; CTX-D declares gpt-5 names perishable; R4 falls back to 128k for unknown models. Missing: current-model Options examples, a CTX-D "perishable parameter constraints" category (the temperature rejection), §2 key wording. Provenance constraint: "(course default)" stays on the gpt-5 family — provider-neutrality must not overwrite provenance. Current-model advice belongs in the gate-time "(Recommended)" slot F12 defined, not baked into defaults. → batch items A3 (spec) + B3 (guide rule). |
| 5 |
Seed/fixture ingestion idempotency (P3: verify-only) |
Split |
Toolbox re-registration dedup is specified + AC-tested (R9/AC14); store init is create-if-missing/no-wipe (R1/AC1/AC2, restart-tested). Re-running seed ingestion into a populated store is undefined — can't crash in the current realization (generated ids), but silently duplicates. Proposed (not applied): idempotent-seed AC or an explicit "re-seeding duplicates; reset() first" note. → watchlist C3. |
| 6 |
arXiv 429 (P3: verify-only) |
Not in text; evidence now n=2 |
run-09's AC24 hit 429 twice, passed later — transient; the arxiv client retries internally; inside the agent loop a 429 is already non-fatal via R11 (failed tool-log row, turn continues). Proposed (not applied): document transient-429 as expected for live (net) ACs rather than mandate backoff. → watchlist C4. |
| 7 |
Skipped ACs read as failure (P3: verify-only) |
Mechanism YES, healthy-shape sentence NO |
§7's skip rule is right; nothing states what a healthy run looks like per mode. Folded into the baseline-completion sentence (item 3). → batch item A1. |
| 8a |
File density / no on-ramp (R16, R18 verbatim: "overwhelming at first due to its density"; "needed a better explanations and instructions in the beginning"; R16's ask: "a brief, high-level overview at the top summarizing what the system actually does") |
NOT addressed |
First ~100 lines are meta-blockquote → gate instructions → 12-row Ledger; §1 Objective (the first "what does this build" sentence) is buried after all of it. Fix: an orientation block at the top — what you're building + the baseline-completion sentence (no cost numbers, per item 1). Sits with the opening blockquote, before §0, so the gate's "first section" rule is untouched. → batch items A1 (spec) + B1 (guide template). |
| 8b |
Process ceremony (R17 verbatim: "the process wrapper (how to ask) was heavier than it needed to be"; "I'd lean on the default → checklist → build flow") |
Addressed by PR #23 |
R17's requested flow is literally the F12 express lane: one baseline question skips all per-row questions → deviation-marked checklist → build. Gate-probe-2 demonstrated the exact flow. Note: R16 praised the stop-and-ask constraint and R17 called the decision content "excellent" — the objection was front-loading, not the mechanism; no further weakening. → watchlist C1 (confirm with next testers). |
Triage of the pre-F12 tester feedback (4 new testers + the P1/P2/P3 list), verified item-by-item against the post-F12 state (the PR #23 branch), with the owner's design decisions and the routing we converged on. This issue is the candidate slate for the next batch; nothing here is applied yet.
Verified statuses
RUN_LIVE=1gating, and skip-with-reason reporting all exist and work (run-09 and gate-probe-2 both completed keyless). But no sentence blesses it — "5 skipped" reads as unfinished debt. Fix: the baseline-completion sentence — "the recommended baseline build is complete when the offline suite is green; live ACs are an optional keyed extension; skipping them without a key is the expected result." Also resolves item 7. → batch item A1.live (net)ACs rather than mandate backoff. → watchlist C4.Owner decisions locked in this triage
(warning:)label — F12's two-sanctioned-labels rule stays intact.The batch (executed together after the full issue sweep)
A. Spec revisions batch (Loop A, agent-memory spec.md):
B. Next promotion pass (guide; one Opus validation round covers all):
C. Scaled-test measurement watchlist (measure, don't write):
Remaining from earlier, for the full sweep
🤖 Generated with Claude Code