Dev#2495
Conversation
The design palette was branded after the TV show "Severance" (Lumon/MDR). Strip the brand from everything shipped or user-facing while keeping the exact colors: - packages/genie-tokens: rename internal token/palette identifiers (severed->muted, outieWarm->warm, lumonBeige->beige; innieGrey->mutedGrey, outieAmber->warmAmber) and their doc comments. Package is private and the hex values are unchanged, so the TUI/app/tmux render identically. - scrub comments/descriptions in src/tui, packages/genie-app, scripts/tmux, test/visual, .github/workflows/ci.yml, and the CHANGELOG entry. - README: drop "inspired by Severance" from the Design section. Left intact: the `design-system-severance` wish slug, which migration 050 matches on to archive legacy agent rows — renaming it would break that cleanup. Historical .genie/ planning docs are untouched too. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
`genie` with no args launches the TUI; `/wizard` is a skill invoked inside Claude Code / Codex, not a command typed after `genie`. The prior install snippet wrongly presented it as a shell/TUI command in a bash fence. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
docs: remove Severance theme refs + fix /wizard usage (follow-up to #2492)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughRename palette properties and token aliases to neutral petrol/mint names, update tests and comments to match, append new metrics-updater records for 2026-06-07, and bump manifest/plugin/package versions. ChangesPalette and tokens theming refactor
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request rebrands the design system palette from 'Severance Lumon-MDR' to 'petrol/mint' across the entire codebase, including documentation, configuration files, tests, and source code. As part of this change, several semantic tokens and palette keys have been renamed (e.g., severed to muted, outieWarm to warm, and lumonBeige to beige). Additionally, the README has been updated to guide users on running the onboarding wizard in AI coding agents. There are no review comments, so we have no further feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
packages/genie-tokens/__tests__/palette.test.ts (1)
53-85: 🛠️ Refactor suggestion | 🟠 Major | ⚡ Quick winAssert that the legacy aliases are actually removed.
These checks only prove the new names resolve. If
paletteortokensaccidentally keepinnieGrey/outieAmberorsevered/outieWarm/lumonBeige, this suite still passes even though the public rename regressed.Suggested test addition
describe('semantic tokens', () => { + test('legacy palette and token aliases are removed', () => { + expect('innieGrey' in palette).toBe(false); + expect('outieAmber' in palette).toBe(false); + expect('severed' in tokens).toBe(false); + expect('outieWarm' in tokens).toBe(false); + expect('lumonBeige' in tokens).toBe(false); + }); + const requiredAliases = [ 'accent', 'surface',As per coding guidelines, "Test edge cases and boundaries of every command, flag, and plugin contract"; this PR's stated contract change is removal of the old aliases, not just addition of the new ones.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/genie-tokens/__tests__/palette.test.ts` around lines 53 - 85, Add explicit negative assertions to ensure legacy aliases were removed: in the palette.test.ts tests that validate tokens and palette mappings, check that tokens and palette do NOT expose the old names (e.g., tokens.innieGrey, tokens.outieAmber, palette.severed, palette.outieWarm, palette.lumonBeige) by asserting they are undefined or not present; update the test suite around the requiredAliases and the "surface/danger/attention" test to include these negative checks so any regression that keeps legacy aliases will fail.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@packages/genie-tokens/__tests__/palette.test.ts`:
- Around line 53-85: Add explicit negative assertions to ensure legacy aliases
were removed: in the palette.test.ts tests that validate tokens and palette
mappings, check that tokens and palette do NOT expose the old names (e.g.,
tokens.innieGrey, tokens.outieAmber, palette.severed, palette.outieWarm,
palette.lumonBeige) by asserting they are undefined or not present; update the
test suite around the requiredAliases and the "surface/danger/attention" test to
include these negative checks so any regression that keeps legacy aliases will
fail.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 04bed2f3-5da1-46c5-a3bb-ac07c9f4db3e
⛔ Files ignored due to path filters (6)
.genie/assets/commits-30d.svgis excluded by!**/*.svg.genie/assets/loc-30d.svgis excluded by!**/*.svg.genie/assets/releases-30d.svgis excluded by!**/*.svgCHANGELOG.mdis excluded by!*.mdREADME.mdis excluded by!*.mdVELOCITY.mdis excluded by!*.md
📒 Files selected for processing (15)
.genie/agents/metrics-updater/daily-stats.jsonl.genie/agents/metrics-updater/runs.jsonl.genie/agents/metrics-updater/state.json.github/workflows/ci.ymlpackages/genie-app/lib/theme.tspackages/genie-tokens/__tests__/palette.test.tspackages/genie-tokens/package.jsonpackages/genie-tokens/palette.tspackages/genie-tokens/tokens.tsscripts/tmux/generate-theme.tsscripts/tmux/genie.tmux.confsrc/tui/components/TreeNode.tsxsrc/tui/theme.tssrc/tui/widgets/xterm-cell-paint.tstest/visual/tui-snapshot.test.tsx
3 participants
Summary by CodeRabbit
Updates
Chores