Repository navigation
Two declared corrections: fabric_evidence_gating's deleted-script claim, and the FCI-1 runners' (c) class - #12522
Conversation
…#10270 and is live under the FCI-1 live instrument; the lane stays deleted Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…mmar lacks [[ ]], function definitions, substring expansion, $(<file) and a flat a&&b||c Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
briansrls
left a comment
There was a problem hiding this comment.
APPROVE-MERGE — content at 2f7c27c
No blocking content finding in the six-file diff. The three shell changes and the FCI-1 carrier change are comments only; the remaining delta is the correction in the rung-drop's authored prose and its matching generated documentation line. No command, workflow job, gate predicate, standing, or restoration trigger changes.
The correction distinguishes the script from the CI lane. The file-specific history includes #10390 at 2094e9c (2026-09-04) and #10270 at 9d86c98 (2026-09-05). The current FCI-1 script is real, hand-invoked shell, not proof of a restored required lane. The existing lane-restoration condition is not discharged here.
The class-(b) correction is appropriately at the grammar/serialization boundary. I inspected the bash target source and the live instrument's actual functions, local/return, double-bracket glob tests and file-read substitutions. Merely being able to emit an ordinary command does not establish representation of those control/expansion constructs. The mixed AND/OR limitation is a serializer semantic limitation, not a claim that individual AND and OR nodes are absent. No grammar expansion or broad migration is required to land this correction.
Evidence: exact-head PR workflow 36406989974 completed/success. I did not rerun bash -n, docs regeneration, or the FCI-1 live instrument; their reported local results retain that attribution. I do not independently certify every numeric entry in the per-file construct census.
Landing caveat: GitHub reports mergeable=false at this head. This is content approval, not a claim that it is currently queue-ready. Reconcile main and regenerate the projection as needed, then report/rebind the resulting SHA and use the normal queue gates. Do not restore the deleted lane as part of that reconciliation.
No merge, enqueue, live instrument execution or dispatch performed.
Correct
gunbc.rung_dropfabric_evidence_gating: the calibration script was restored, the lane was notThe row's 2026-09-04 amendment says that ruling deleted
tools/fabric_ci_evidence_calibration.sh. Git history shows two separate facts:fabric-evidencejob, the calibration step and its bounds.tools/fabric_ci_fci1_live_instrument.shinvokes it against the exact binary before any srv3 observation.So the row overstates, and the script is not dead. Deleting it would break the FCI-1 live instrument.
Declared correction. A
CORRECTED 2026-09-28paragraph now opens the row. It quotes the wrong clause instead of deleting it, which follows the existing correction pattern (e.g.floor_cut_heal). It then states:w_RED_the_deleted_lanes_do_not_returnstill guards that. The rung, the population and the restoration trigger stand.docs/design-rung-drops.mdis regenerated throughtools.docs_projection_gateregen, not edited by hand. The diff is that one line.Part of node adhoc-da321263-f7f. The (c) runner migrations follow in separate PRs.
🤖 Generated with Claude Code
Second correction: the FCI-1 hand-shell runners are class (b), not (c)
#12430 classed
tools/fabric_ci_evidence_calibration.sh,tools/fabric_ci_fci1_bounded_execution_controls.sh,tools/fabric_ci_fci1_live_instrument.shand thegunbc.fci1_bounded_execution_context_emitcarrier as (c): "route available today, migration unauthored". That is wrong. The bash grammar (v2.extdeps.languages.bashrows /bash_build) has no production for constructs these runners depend on. Counts per file (calibration / controls / live instrument):[[ ]]with glob matchinglocal${var: -1}substring expansion$(<file)reada && b || c(known serializer limit: it renders as a subshell)Each header and the carrier's
//annotation now name these missing capabilities as the trigger. They carry aCORRECTED 2026-09-28line recording the (c)→(b) change, so the edit is not silent. There is no behaviour change: comments only, andbash -nis clean on all three scripts. The carrier's annotation is semantically erased (§4c).