Skip to content

Commit 9c361c3

Browse files
committed
docs(platform): record the platform-core-chat reconciliation
Note both merge commits, the embedded decisions for review and the verification each merge passed, including the Node 23 network tests that fail the same way before the merge.
1 parent 5f2fc54 commit 9c361c3

1 file changed

Lines changed: 17 additions & 0 deletions

File tree

‎apps/sim/platform/docs/consolidation-checkpoint.md‎

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,23 @@ Decisions in Merge B for the `embedded/` owner to review:
1919
- `host-bridge.module.css` lives in `packages/emcn-platform/src/styles/` and names the `EMCN Geist` family. An embedded surface paints its canvas only with `canvas`, through `[data-embedded]:not([data-canvas])`; the zone keeps painting by default.
2020
- `embedded/` may import only itself, `platform/shared` and the EMCN packages (`check:platform-design`, `check:boundaries` rule 7). `check:platform-css` reads the host bridge from its package.
2121

22+
Reconciliation evidence, run in a separate worktree on the merge commits:
23+
24+
| Check | Merge A | Merge B |
25+
| --- | --- | --- |
26+
| Lint | Touched files clean | 30/30 tasks |
27+
| Types | Platform and app projects | 32/32 workspaces, including every EMCN package |
28+
| Audits | Platform audits passed; the full suite's unused-export finding was fixed and rechecked | 68/68 |
29+
| Platform production build | Passed | Passed |
30+
| Main app production build | Not run | Passed with CI's environment: 1,708 pages |
31+
| Platform browser suite, fixtures on and off | 101/101 | 118/118 |
32+
| Electron renderer | Not run | Passed |
33+
| Script tests | 834 | 845 |
34+
| Platform and `app/o` unit tests | 385 | 383, plus `@sim/emcn` 2 and `@sim/emcn-legacy` 36 |
35+
| Full repository tests | Not run | 20 of 21 tasks. App: 40,122 passed, 22 skipped, 3 failed in `gateway.server`, `egress-end-to-end.server` and `processing-recovery-queue-transport`. Those three fail the same way on `b5a30e9744` under the local Node 23.10, which lacks `tls.setDefaultCACertificates`, and pass under Node 24.19 (21/21) |
36+
| Block registry | Not run | Passed against `origin/staging` |
37+
| Live suite (`checks/live`) | Not run: needs a running app and database | Not run |
38+
2239
## Canonical owners
2340

2441
| Owner | Responsibility |

0 commit comments

Comments
 (0)