Skip to content

Commit 45ce966

Browse files
Repoint migration references ahead of retiring RAC-MIGRATION.md
AGENTS.md now points at the playbook's gotcha catalog (which is where the retired doc's transferable lessons moved); the fidelity harness comments cite the playbook; the density preset comment carries its revisit context inline.
1 parent 919f938 commit 45ce966

4 files changed

Lines changed: 18 additions & 11 deletions

File tree

AGENTS.md

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,15 @@
11
# AI agent notes
22

3-
## RAC/Panda migration
3+
## Styling/theming
44

5-
This app is migrating from Chakra UI to react-aria-components + Panda CSS
6-
via `@microbit/ui`. **Read `RAC-MIGRATION.md` at the repo root before any
7-
styling/theming/UI work** — it is the status log and app-specific plan; the
8-
method and gotcha catalog are in `../ui/docs/migration-playbook.md`. The
9-
private theme package is the sibling repo `../python-editor-v3-microbit`.
5+
This app uses react-aria-components + Panda CSS via `@microbit/ui`
6+
(migrated from Chakra UI, July 2026). **Read the gotcha catalog in
7+
`../ui/docs/migration-playbook.md` before styling/theming/UI work**
8+
Panda's static extraction rules and RAC behaviours catalogued there still
9+
apply to new code. The private theme package is the sibling repo
10+
`../python-editor-v3-microbit` (consumed via a manual `node_modules`
11+
symlink locally — re-create it after `npm install`). The visual fidelity
12+
harness is `npm run fidelity` (see `bin/fidelity.mjs`).
1013

1114
## Commands
1215

bin/fidelity.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@
2323
// npm run fidelity -- --compare-only
2424
//
2525
// The split flags support cross-boundary runs that need paired
26-
// sibling-package versions (see RAC-MIGRATION.md): rebuild the private
26+
// sibling-package versions (see ../ui/docs/migration-playbook.md §6,
27+
// "Fidelity, both sides"): rebuild the private
2728
// package at the old commit, record the baseline half, restore everything,
2829
// then run the compare half.
2930
//

src/deployment/default/panda-preset.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,11 @@ const toTokens = (values: Record<string, string | number>) =>
1616
// spacing/size scale is Chakra's 0.25rem grid × 0.88, and fontSizes from
1717
// `md` up are × 0.9 (xs/sm kept full-size so text never gets too small).
1818
// The adjacent common-sizes.ts/font-sizes.ts are the (only) home of the
19-
// values now Chakra is gone. The team may revisit whether
20-
// this density stays after the migration (see RAC-MIGRATION.md notes) —
21-
// for now the port replicates it.
19+
// values now Chakra is gone. The team may revisit whether this bespoke
20+
// density stays or aligns with the family scale (the editor is more
21+
// information-dense than the other apps, so there may be grounds to keep
22+
// it); if alignment wins, delete these overrides and re-run the visual
23+
// pass. For now the port replicates it.
2224
const shrunkenScale = toTokens(commonSizes);
2325
const shrunkenFontSizes = toTokens(fontSizes);
2426

src/e2e/fidelity.spec.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
* of the default e2e project; bin/fidelity.mjs runs it twice — once against
88
* a baseline git ref with --update-snapshots, then against the working tree
99
* — and the Playwright HTML report shows the diffs. Baselines are per-run
10-
* artefacts in .fidelity/, never committed. See RAC-MIGRATION.md.
10+
* artefacts in .fidelity/, never committed. Method and lessons:
11+
* ../ui/docs/migration-playbook.md, "Fidelity harness".
1112
*/
1213
import { expect, Locator, Page } from "@playwright/test";
1314
import { test } from "./app-test-fixtures.js";

0 commit comments

Comments
 (0)