build/20260612 062318/0050 json helper consolidation - #158
Merged
cahenesy merged 17 commits intoJun 12, 2026
Conversation
cahenesy
commented
Jun 12, 2026
Owner
- test(failing): json.sh canonical escaper/array/field-reader unit eval (TDD 0050 step 1; Verification §1/§3/§6)
- step(1): create scripts/lib/json.sh — canonical C0-safe escaper, compact array builders, quote-aware field reader (TDD 0050 §1; FR-27, FR-69)
- test(failing): A11/A3 strict-parse + A10/A5 quote-roundtrip regressions against state.sh's escaper and primary reader (TDD 0050 step 2; Verification §2/§6)
- step(2): wire state.sh through json.sh — json_escape delegates (A11), 4 inline array builders -> tl_json_array, _read_fragment_field -> tl_json_field (A10/A5) (TDD 0050 §2; FR-27, FR-39)
- test(failing): markers.sh delegation observables — json.sh bound on source, compact _tl_csv_to_json_array (TDD 0050 step 3; Verification §4)
- step(3): wire markers.sh through json.sh — _tl_json_escape + _tl_csv_to_json_array delegate; compact-array pin updated in lifecycle-helpers eval (TDD 0050 §3)
- test(failing): learnings.sh delegation — json.sh bound on standalone source, _json_str_array via tl_json_array_ws, C0-safe A3 path (TDD 0050 step 4; Verification §4)
- step(4): wire learnings.sh through json.sh — _json_str_array delegates to tl_json_array_ws; candidate-learnings writes ride the C0-safe escaper (A3) (TDD 0050 §4; FR-72)
- test(failing): drafts.sh third escaper (_tl_json_escape_full) deleted; canonical tl_json_escape bound (TDD 0050 step 5; Verification §5)
- step(5): drafts.sh drops _tl_json_escape_full — callers use the canonical tl_json_escape; stale C0-gap comments synced (TDD 0050 §5; FR-46)
- test(failing): gates.sh binds json.sh on standalone source; _step_block_entry pattern_tags pinned through the canonical builder (TDD 0050 step 6; Verification §4)
- step(6): wire gates.sh through json.sh — _step_block_entry pattern_tags via tl_json_array; consumer owns its source line (TDD 0050 §6; FR-72)
- test(failing): §4 context matrix, §5 single-source grep, §W aggregator wire-in dogfood (TDD 0050 step 7; TDD 0038 §3 wire-in rule)
- step(7): register json-helper eval in the ci-checks aggregator (JSH AND-chain term); bump plugin 3.35.0 -> 3.36.0 (TDD 0050 §7; FR-69)
- tdd(0050): reconcile Expected diff size + Touched files to measured build churn (numstat adds+dels; replaced bodies count twice)
- mark 0050-json-helper-consolidation implemented (verified + reviewed)
… (TDD 0050 step 1; Verification §1/§3/§6) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…act array builders, quote-aware field reader (TDD 0050 §1; FR-27, FR-69) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ns against state.sh's escaper and primary reader (TDD 0050 step 2; Verification §2/§6) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… 4 inline array builders -> tl_json_array, _read_fragment_field -> tl_json_field (A10/A5) (TDD 0050 §2; FR-27, FR-39) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ource, compact _tl_csv_to_json_array (TDD 0050 step 3; Verification §4) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…to_json_array delegate; compact-array pin updated in lifecycle-helpers eval (TDD 0050 §3) The lifecycle-helpers grep pinned the old spaced array form (["a", "b"]); TDD 0050 Failure-modes sanctions updating byte-pins of the retired cosmetic format (parsed value unchanged). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…source, _json_str_array via tl_json_array_ws, C0-safe A3 path (TDD 0050 step 4; Verification §4) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…s to tl_json_array_ws; candidate-learnings writes ride the C0-safe escaper (A3) (TDD 0050 §4; FR-72) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…; canonical tl_json_escape bound (TDD 0050 step 5; Verification §5) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ical tl_json_escape; stale C0-gap comments synced (TDD 0050 §5; FR-46) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ck_entry pattern_tags pinned through the canonical builder (TDD 0050 step 6; Verification §4) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…gs via tl_json_array; consumer owns its source line (TDD 0050 §6; FR-72) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…r wire-in dogfood (TDD 0050 step 7; TDD 0038 §3 wire-in rule) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ND-chain term); bump plugin 3.35.0 -> 3.36.0 (TDD 0050 §7; FR-69) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…uild churn (numstat adds+dels; replaced bodies count twice) Per-file actuals from git diff --numstat a195c97..HEAD; the two swept test files (lifecycle-helpers pin, draft-persistence [R] comment) are the Failure-modes 'tests pinning the OLD output' sweep, now declared. Same in-band scope-reconciliation convergence as the 0054 run. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Owner
Author
|
Per-requirement coverage map (FR-78 — reported, advisory): see the |
cahenesy
deleted the
build/20260612-062318/0050-json-helper-consolidation
branch
June 12, 2026 12:28
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.