refactor(web): extract design system stylesheet#5545
Open
code-yeongyu wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
packages/web/DESIGN.mdinto the implementation contract for the current rendered web UI.packages/web/app/styles/design-system.cssand keepsapp/globals.cssas a filesystem alias to that entry.This replaces the closed LazyCodex-side PR: code-yeongyu/lazycodex#70. The correct landing repo for this work is
code-yeongyu/oh-my-openagent.Fidelity / QA
Visual control comparison used a fresh
origin/devworktree on port 3109 and this branch on port 3108:/desktop/mobile: 0 diff after normalizing dynamic build-time hero stats for screenshot comparison./docsmobile: 0 diff; pre-existing horizontal overflow delta remains 538 and did not worsen./manifestodesktop/mobile: 0 diff./docsdesktop: 34,654 raw AE pixels on a 1280x151508 full-page capture, with no overflow/console/page errors; crops show no structural layout change, only tiny full-page raster variance.Manual behavior QA:
BASE_URL=http://127.0.0.1:3108 bun .omo/ultrawork/design-system-fidelity/scripts/behavior-scenarios.mjs-> 7 passed, 0 failed.Automated gates:
bun run format:check-> pass.bun run lint-> pass.bun run type-check-> pass.bun run build-> pass.bun --bun playwright test --config=playwright.3110.config.ts-> 66 passed.Note: default
bun run test:e2einitially reused127.0.0.1:3000, which was already occupied by an unrelated local Node server (Sionic AI/STORM Console). The verified Playwright run used a temporary in-package config on port 3110.Lighthouse against production server on 3108:
/: performance 0.95, accessibility 1.00, best practices 1.00, SEO 0.92./docs: performance 0.88, accessibility 0.96, best practices 1.00, SEO 0.91./manifesto: performance 0.94, accessibility 0.96, best practices 1.00, SEO 0.92.Evidence summary:
.omo/evidence/20260624-design-system-fidelity/summary.md.Summary by cubic
Extracted the web design system into a single stylesheet to centralize tokens, base styles, and docs prose while keeping the UI pixel-identical. No copy, layout, or behavior changes.
Refactors
packages/web/app/styles/design-system.css;packages/web/app/globals.cssnow aliases it.packages/web/DESIGN.mdto be the implementation contract for the current UI..omo/.Verification
/,/docs, and/manifestofor desktop and mobile; known/docsmobile overflow (538) unchanged.Written for commit 6589362. Summary will update on new commits.