Skip to content

Remove retired theme variants and memoize PR review grouping#439

Merged
BunsDev merged 1 commit into
mainfrom
okcode/remove-vapor-cathedral
Apr 13, 2026
Merged

Remove retired theme variants and memoize PR review grouping#439
BunsDev merged 1 commit into
mainfrom
okcode/remove-vapor-cathedral

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 13, 2026

Copy link
Copy Markdown
Member

Summary

  • Removed retired theme variants from the theme system, CSS definitions, docs, and selection-style coverage.
  • Updated the theme picker storage/typing to recognize only the remaining supported color themes.
  • Memoized PR review file grouping and related file filtering in PrWorkspace to avoid recomputing on every render.
  • Trimmed design docs to reflect the current three-theme lineup.

Testing

  • Not run (not requested).
  • Verify the app still loads with each remaining theme: default, iridescent-void, carbon, cotton-candy, and custom.
  • Verify PR review single-file filtering still works when switching between file views.

- Drop Solar Witch, Vapor, and Cathedral Circuit from theme definitions
- Update theme selector, CSS, tests, and docs to match the reduced set
@vercel

vercel Bot commented Apr 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 13, 2026 5:34pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 13, 2026 5:34pm

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Apr 13, 2026
@BunsDev BunsDev merged commit da56503 into main Apr 13, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant