Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions .changeset/calm-otters-merge.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
"kitcn": patch
---

## Patches

- Improve generated procedure-name lookups with merge-local entries and stale-state warnings.
35 changes: 22 additions & 13 deletions docs/plans/2026-09-10-fix-crpc-client-error-guards.md
Original file line number Diff line number Diff line change
Expand Up @@ -109,26 +109,26 @@ Work Checklist:
- [x] Resolve the full-check blocker or record a maintainer disposition.

Completion Gates:
| Gate | Result | Evidence |
| --- | --- | --- |
| Focused regression | passed | 36 cRPC tests pass after rebuilding |
| Package build | passed | All four build groups |
| Typechecks | passed | Package and repository root |
| Lint | passed | `biome check && eslint` |
| Code review | recorded | Implementation review reported no actionable findings |
| Release artifact | present | `.changeset/crpc-guards-across-entrypoints.md` |
| Full repository check | passed before final review repair | Exact-head CI run `34914786162` passed in 6m28s; final repaired-head CI pending |
| PR ownership | recorded | #464 |
| PR plan reference | passed | Body names this plan and fetched head contains it with exact PR #464 |
| Gate | Applies | Result | Evidence |
| --- | --- | --- | --- |
| Focused regression | yes | passed | 36 cRPC tests pass after rebuilding |
| Package build | yes | passed | All four build groups |
| Typechecks | yes | passed | Package and repository root |
| Lint | yes | passed | `biome check && eslint` |
| Code review | yes | passed | Local P0/P1 autoreview clean at 0.94; automated final-head review clean |
| Release artifact | yes | present | `.changeset/crpc-guards-across-entrypoints.md` at final head |
| Full repository check | yes | passed | Exact-head CI run `34916336275` passed in 6m38s |
| PR ownership | yes | recorded | #464 |
| PR plan reference | yes | passed | Body names this plan and fetched head contains exact PR #464 |

Phase / pass table:
| Phase | Status | Evidence |
| --- | --- | --- |
| Reproduction | complete | Built React and Solid tests fail on base `c12407fc` |
| Implementation | complete | Validated shape check; shared guard delegation |
| Focused verification | complete | cRPC tests, build, typechecks and lint pass |
| Full repository verification | in progress | PR #467 removed the fixture blocker; repaired-head CI pending |
| PR delivery | in progress | #464; exact task evidence and updated-head approval verified |
| Full repository verification | complete | Local `bun check` and exact-head CI passed |
| PR delivery | complete | Receipt `5673217013`; merged as `ac73cc6a` |

Verification evidence:
Results recorded during implementation on 2026-09-14:
Expand Down Expand Up @@ -176,3 +176,12 @@ Open risks:
- Errors from another JavaScript realm fail `instanceof Error`.
The reproduced package-entrypoint failure occurs within one realm.
- An unsupported code supplied by bypassing TypeScript is rejected.

Reboot status:
| Question | Answer |
| --- | --- |
| Where am I? | Complete; PR #464 is merged |
| Where am I going? | Parent autoclosure goal completion |
| What is the goal? | Recognize cRPC client errors across built package entrypoints |
| What have I learned? | Release #468 consumed the earlier living changeset before final merge |
| What have I done? | Passed final proof, resolved feedback, posted receipt, and merged `ac73cc6a` |
Original file line number Diff line number Diff line change
Expand Up @@ -306,7 +306,7 @@ Phase / pass table:
| Implementation | complete | package overlay and runtime cleanup owners fixed with tests | verification |
| Verification | complete | focused, fixture, scenario, package, lint/typecheck, and final 362-second root check pass | delivery |
| Commit / PR / GitHub sync | complete | implementation `d4c24966`, plan binding `0f0da6c4`, PR #467, required body read-back | closeout |
| Closeout | in progress | prior Ubuntu CI passed; two final P2 review repairs require a new exact-head gate | rerun, push, receipt, merge |
| Closeout | complete | exact-head CI, terminal receipt, and merged state verified | none |

Findings:
- `eslint-plugin-react@7.37.5` declares ESLint support through `^9.7`, while
Expand Down Expand Up @@ -520,15 +520,14 @@ Timeline:
Reboot status:
| Question | Answer |
|----------|--------|
| Where am I? | Final review repair and verification |
| Where am I going? | Rerun the full gate, push the exact repair head, close review, merge #467, then unblock #464/#465 |
| Where am I? | Complete; PR #467 is merged |
| Where am I going? | Parent autoclosure goal completion |
| What is the goal? | Deterministically pin compatible ESLint 9 in generated Next scaffolds and ship the prerequisite PR |
| What have I learned? | The loose upstream range behaves differently on Ubuntu CI and violates the plugin peer range |
| What have I done? | Reproduced four CI failures, implemented the owner fixes, closed prior review cycles, and added green proof for the final two P2 edge cases |
| What have I done? | Reproduced the failures, repaired the owner, closed review, passed exact-head CI, and merged `5794ed38` |

Open risks:
- GitHub Ubuntu resolution may expose a second install-order issue after the
exact pin; the prerequisite PR CI is the authoritative final proof.
None for this prerequisite. Exact-head Ubuntu CI passed before merge.

Hard closeout guard:
- A local-only final response for verified code-changing work is invalid unless
Expand Down
Loading
Loading