Repository navigation
Replace em-dashes and deposition framing in public docs - #46
Conversation
…framing Applies the two-part writing pass already used on jasoneplumb.com and the profile README to README.md and docs/results.md. Em-dashes become commas, colons, semicolons, or parentheses depending on the clause; the README "## Evidence" heading becomes "## Results". docs/results.md keeps punctuation-only changes since it is a cited source for the external claims register. docs/field-reverification.md had no em-dashes and needed no changes. Refs #45 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
| | --- | --- | | ||
| | **Contribution** | Sole author: decision kernel and equivalence contract, replay harness, phone↔MCU shadow-comparison protocol, trace format, aggregation and ablation tooling, MCU ports. AI-assisted implementation gated by the tests, replay, and hardware-in-the-loop checks described below. Platform SDKs and map data are third-party. | | ||
| | **Status** | Working demonstrator on private hardware. Not a product, not a safety device, no users but the author. | | ||
| | **Evidence** | 23/23 traces replay exactly; 11,301 steps logged, 11,300 compared, 0 divergences across 8 instrumented rides, over a 23-trace/88.1 km/57,783-step corpus recorded 2026-07-20 to 2026-08-08. Re-verified 2026-09-12 against a tree identical to `dd8a7db25522734de1dbec614f66113c69d80dcb`. [Field results](docs/results.md) · [re-verification](docs/field-reverification.md) · [case study](https://www.jasoneplumb.com/case-studies/cue-equivalence-contract.html) | |
There was a problem hiding this comment.
The section heading was renamed from ## Evidence to ## Results (line 33), but this table row label | **Evidence** | was left unchanged. The deposition-style framing the PR set out to retire persists in the most prominent data row. Consider renaming to | **Field data** | or similar to match the new heading.
| The same `kernel/cue_policy.c` runs in three places — live on the phone, as | ||
| the MCU actuator (Pico W, RFC 0006), and offline in the replay harness — and | ||
| The same `kernel/cue_policy.c` runs in three places: live on the phone, as | ||
| the MCU actuator (Pico W, RFC 0006), and offline in the replay harness, and |
There was a problem hiding this comment.
The em-dash pair that cleanly delimited the three-item list has been replaced with a colon, but the closing clause "and the contract is" now reads as a potential fourth list member rather than the start of a new independent clause. The original em-dash pair was doing real structural work here. One fix: use a semicolon before "and the contract is" to clearly mark the clause boundary.
| the MCU actuator (Pico W, RFC 0006), and offline in the replay harness, and | |
| The same `kernel/cue_policy.c` runs in three places: live on the phone, as | |
| the MCU actuator (Pico W, RFC 0006), and offline in the replay harness; and |
| | `too_late` | 7 | Drove the §13 `max_notice_s` 15 → 20 widening — which then failed and is on the record in `kernel/cue_policy.h` | | ||
| | `unrecognized` | 7 | A delivery/perceptibility outcome, not a policy one (see [grading-guide.md](grading-guide.md)) — moves no policy lever | | ||
| | `too_late` | 7 | Drove the §13 `max_notice_s` 15 → 20 widening, which then failed and is on the record in `kernel/cue_policy.h` | | ||
| | `unrecognized` | 7 | A delivery/perceptibility outcome, not a policy one (see [grading-guide.md](grading-guide.md)), moves no policy lever | |
There was a problem hiding this comment.
Replacing the em-dash with a comma here creates a comma splice: "not a policy one (see …), moves no policy lever" joins two independent predicates with only a comma. The em-dash was doing structural work — it marked the transition to the key conclusion. A semicolon or a period would fix it.
| | `unrecognized` | 7 | A delivery/perceptibility outcome, not a policy one (see [grading-guide.md](grading-guide.md)), moves no policy lever | | |
| | `unrecognized` | 7 | A delivery/perceptibility outcome, not a policy one (see [grading-guide.md](grading-guide.md)); moves no policy lever | |
|
🔍 Review verdict: 3 findings Documentation-only PR replacing em-dashes with plain punctuation and renaming the Findings
All three are prose-level issues; none affect correctness, kernel behaviour, or compliance with FR/NFR requirements. |
…aries Renames the README Evidence table row to Results to match the renamed heading, and clarifies two clause boundaries in docs/results.md flagged by review (semicolon before a following independent clause; semicolon in the unrecognized-grade row) without changing any wording or figures. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
🔍 Review verdict: 0 findings Incremental review covering changes since a6afd2b. The delta consists entirely of documentation edits:
No code, kernel, replay, or logic changes. No NFR-003/NFR-004/NFR-005 concerns. Nothing to report. |
Summary
max_notice_spolicy change, the uncompared tail step).Closes #45
Test plan
grep -c "—" README.md docs/results.md docs/field-reverification.md→ 0/0/0grep -rn "—"→ none🤖 Generated with Claude Code