R2: Reorganize Desktop shell for IM command center#271
Merged
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
activeRailViewstate coverageuseSidebarResizenow usesuseShallow, the mobile top bar hides desktop-only menu/status chrome under 767px, and the UI capture script now honors--theme lightApp.tsxlint errors from unused imports/state and render-time ref writesdev/delicious233commit98fda1ab, which fixes the Desktop/validate CI baseline for this PRValidation
corepack.cmd pnpm exec vitest run src/__tests__/uiStore.test.ts(4/4)corepack.cmd pnpm typecheckcorepack.cmd pnpm test:ciin the R2 worktree: 420 + 325 + 24 + 335 tests passedcorepack.cmd pnpm exec eslint src/App.tsx --max-warnings 999(0 errors; 4 existing warnings)desktop-1440x920-light-home.png,desktop-1280x800-light-home.png,desktop-390x844-light-home-fixed.pnggit diff --check origin/dev/delicious233..HEADCI status
The direct R2 blockers in
frontend-desktopandvalidatehave been fixed ondev/delicious233and merged into this branch. The new PR run hasvalidategreen andfrontend-desktopis expected to use the fixed baseline.Repo-wide checks may still be red outside the R2 shell diff:
Merge notes
Code is ready for review as the R2 shell foundation, but do not merge while required GitHub checks remain red. Final competition visuals still need R3/R4/R7 on top: real transcript blocks, right-side evidence column, and TeamRun artifact/run proof.