diff --git a/docs/UI_UX_PLATFORM_CORE_MIGRATION_ROADMAP.md b/docs/UI_UX_PLATFORM_CORE_MIGRATION_ROADMAP.md index e300862..6594451 100644 --- a/docs/UI_UX_PLATFORM_CORE_MIGRATION_ROADMAP.md +++ b/docs/UI_UX_PLATFORM_CORE_MIGRATION_ROADMAP.md @@ -10,7 +10,7 @@ Live source of truth for migrating `learning_ai_invt_trdg` UI/UX and platform cl - Current common-platform usage: `@bytelyst/api-client`, `@bytelyst/design-tokens`, `@bytelyst/errors`, `@bytelyst/kill-switch-client`, `@bytelyst/react-auth`, `@bytelyst/react-native-platform-sdk`, `@bytelyst/telemetry-client`, `@bytelyst/ui` - UI adapter in code: `web/src/components/ui/Primitives.tsx` imports from `@bytelyst/ui`, re-exports common primitives, and owns product status-to-tone mapping. - Design token foundation in code: `web/src/index.css` imports `@bytelyst/design-tokens/generated/tokens.css` and defines product `--bl-*` aliases. -- Remaining UI duplicates to migrate: hardcoded color literals and local status/surface/list/table patterns where still product-local. +- Remaining UI duplicates to migrate: none on the closed baseline; future product-local UI should only remain where it encodes trading semantics by design. - Common UI target: `@bytelyst/ui` through the product adapter only; direct `@bytelyst/ui` imports outside the adapter remain disallowed. - Design token target: `@bytelyst/design-tokens` with product aliases to `--bl-*`; chart/domain colors may remain product-owned when they encode trading semantics. @@ -41,7 +41,7 @@ Live source of truth for migrating `learning_ai_invt_trdg` UI/UX and platform cl ## Code Sync Snapshot -Reviewed against current product code on 2026-05-06: +Reviewed against the closed product baseline on 2026-05-09: - `web/package.json` includes `@bytelyst/ui` and `@bytelyst/design-tokens`. - `web/src/components/ui/Primitives.tsx` is present and is the only direct `@bytelyst/ui` import found in `web/src`. @@ -90,7 +90,7 @@ Note: common platform had a pre-existing local modification in `packages/ui/src/ - **Migration status**: Complete. All roadmap phases are checked complete and the strict UI baseline is enforced in `pnpm verify`. - **Final verification on the closed baseline**: Passed `BYTELYST_PACKAGE_SOURCE=common-plat pnpm verify`, `BYTELYST_PACKAGE_SOURCE=common-plat pnpm run audit:ui:strict`, and `git diff --check`. -- **Source-linked common-platform note**: Because `web/vite.config.ts` aliases `@bytelyst/ui` to `/opt/bytelyst/learning_ai_common_plat/packages/ui/src/index.ts` when the local checkout is present, full verification requires the local common-platform checkout to have its own dependencies installed. +- **Source-linked common-platform note**: Because `web/vite.config.ts` aliases `@bytelyst/ui` to `/opt/bytelyst/learning_ai_common_plat/packages/ui/src/index.ts` when the local checkout is present, full verification requires the local common-platform checkout to have its own dependencies installed first via `pnpm -C /opt/bytelyst/learning_ai_common_plat install --no-frozen-lockfile`. - **Ongoing maintenance rule**: Future UI work should keep `pnpm run audit:ui`, `pnpm run audit:ui:strict`, and adapter-first usage green unless a deliberate, documented deferral is accepted. ## Running Notes