Skip to content

ci: add CodeRabbit AI code review configuration#48

Merged
FSM1 merged 2 commits into
mainfrom
claude/add-code-rabbit-ci-htJY2
Jan 24, 2026
Merged

ci: add CodeRabbit AI code review configuration#48
FSM1 merged 2 commits into
mainfrom
claude/add-code-rabbit-ci-htJY2

Conversation

@FSM1

@FSM1 FSM1 commented Jan 24, 2026

Copy link
Copy Markdown
Owner

Adds .coderabbit.yaml with project-specific review settings including:

  • Path-based instructions for api, web, and crypto packages
  • Appropriate filters to ignore generated/lock files
  • Security-focused review guidance for crypto code

Summary by CodeRabbit

  • Chores
    • Added automated PR review configuration with per-path guidance for quality, security, and performance. Review automation now generates high-level summaries, supports auto-replies, respects ignore filters for build artifacts, and flags review/status details. Review behavior and summary placeholders are configurable.

✏️ Tip: You can customize the high-level summary in your review settings.

Adds .coderabbit.yaml with project-specific review settings including:
- Path-based instructions for api, web, and crypto packages
- Appropriate filters to ignore generated/lock files
- Security-focused review guidance for crypto code
@coderabbitai

coderabbitai Bot commented Jan 24, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

Adds a .coderabbit.yaml configuration to enable and tune CodeRabbit automated PR reviews, including auto-review settings, per-path guidance for apps/api, apps/web, packages/crypto, ignore filters for build/dependency artifacts, and feature toggles for summaries and auto-replies.

Changes

Cohort / File(s) Summary
CodeRabbit Configuration
/.coderabbit.yaml
Adds a 79-line CodeRabbit config enabling automated PR reviews (auto_review enabled, drafts disabled, base: main), profile and workflow flags, high-level summary placeholders, per-path instructions for apps/api/**, apps/web/**, packages/crypto/**, **/*.test.ts, ignores for build/deps (node_modules, dist, .next, etc.), and chat/feature toggles (auto_reply, early_access)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A config file hops in with care,
CodeRabbit reviews code everywhere,
Path-wise guidance, filters so clear,
Automation enabled, no need to fear!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'ci: add CodeRabbit AI code review configuration' accurately and clearly summarizes the main change: adding a new CodeRabbit configuration file for automated code reviews.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.coderabbit.yaml:
- Around line 16-17: The YAML uses an invalid enum value for the review profile:
replace the profile key value "default" with one of the schema-approved values
(e.g., "chill" or "assertive") so the reviews.profile setting validates; update
the "profile" field in .coderabbit.yaml to a valid enum (for example change
profile: default to profile: chill).

Comment thread .coderabbit.yaml Outdated
@FSM1

FSM1 commented Jan 24, 2026

Copy link
Copy Markdown
Owner Author

@claude can you fix this? #48 (comment)

@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

Claude encountered an error —— View job


I'll analyze this and get back to you.

Change profile from 'default' to 'chill' as only 'chill' and
'assertive' are valid enum values in the CodeRabbit schema.
@FSM1 FSM1 merged commit 7453a1e into main Jan 24, 2026
9 of 10 checks passed
FSM1 added a commit that referenced this pull request Jan 25, 2026
* ci: add CodeRabbit AI code review configuration

Adds .coderabbit.yaml with project-specific review settings including:
- Path-based instructions for api, web, and crypto packages
- Appropriate filters to ignore generated/lock files
- Security-focused review guidance for crypto code

* fix(ci): use valid profile value in CodeRabbit config

Change profile from 'default' to 'chill' as only 'chill' and
'assertive' are valid enum values in the CodeRabbit schema.

---------

Co-authored-by: Claude <noreply@anthropic.com>
@coderabbitai coderabbitai Bot mentioned this pull request Jan 25, 2026
FSM1 added a commit that referenced this pull request Jan 25, 2026
* docs(06.2): create phase plan for Pencil restyle

Phase 6.2: Restyle App with Pencil Design
- 6 plans in 4 waves
- Wave 1: Design tokens foundation (01)
- Wave 2: File browser, dialogs, responsive/matrix (02, 03, 04) - parallel
- Wave 3: Component TSX updates (05)
- Wave 4: E2E updates and verification (06)

Terminal/hacker aesthetic: JetBrains Mono, green-on-black, [DIR]/[FILE] prefixes

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs(6.2): create phase plans with user decisions

Phase 6.2: Restyle App with Pencil Design - 6 plans in 4 waves

User decisions captured:
- Full matrix animation (not static fallback)
- Mobile 2-row stacked layout per Pencil design (ZVAUX)
  - Row 1: [DIR]/[FILE] icon + filename
  - Row 2: Date (left) | Size (right)

Plans:
- 01: Design tokens and typography foundation
- 02: File browser and breadcrumbs restyle
- 03: Dialogs, menus, modals, upload restyle
- 04: Responsive styles and matrix animation
- 05: Component TSX updates (text prefixes, labels)
- 06: E2E test updates and visual verification

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat(06.2-01): add JetBrains Mono font to index.html

- Add Google Fonts preconnect for performance
- Load JetBrains Mono with weights 400, 600, 700
- Positioned before title tag in head section
- Enables monospace typography for terminal aesthetic

* feat(06.2-01): create design tokens in index.css

- Replace Inter font with JetBrains Mono monospace
- Change background from #242424 to pure black #000000
- Add complete CSS custom properties system:
  - Color primitives (green primary #00D084, black, dim variants)
  - Semantic colors (error, warning)
  - Typography tokens (font sizes, weights)
  - Spacing scale (8px to 32px)
  - Effects (green glow, border thickness)
- Remove place-items: center from body layout
- Add accessibility focus styles and selection colors
- Foundation for terminal/hacker aesthetic

* feat(06.2-01): restyle login page in App.css

- Update login container with terminal aesthetic styling
- Add > CIPHERBOX prompt using ::before pseudo-element
- Convert login button to sharp corners (border-radius: 0)
- Change button from gradient to solid green (#00D084)
- Add uppercase text transform and letter spacing
- Update dashboard header to use design tokens
- Convert API status indicator to use CSS variables
- Use monospace font throughout
- All colors now reference CSS custom properties
- Terminal/hacker aesthetic with green-on-black theme

* docs(06.2-01): complete design foundation plan

Tasks completed: 3/3
- Task 1: Add JetBrains Mono font to index.html
- Task 2: Create design tokens in index.css
- Task 3: Restyle login page in App.css

SUMMARY: .planning/phases/06.2-restyle-app-with-pencil-design/06.2-01-SUMMARY.md

* style(06.2-03): restyle context menu with terminal aesthetic

- Remove all light mode CSS and prefers-color-scheme queries
- Use design token variables throughout
- Remove border-radius (sharp corners)
- Add green glow to box-shadow
- Monospace font for menu items
- Green hover states and focus indicators

* feat(06.2-04): update responsive.css with terminal aesthetic

- Replace all rgba colors with design token variables
- Remove border-radius values (set to 0 for terminal aesthetic)
- Use green color scheme for all hover/active states
- Update file list to use 2-row stacked layout on mobile
- Hide TYPE column on mobile
- Add terminal styling to sidebar overlay

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* style(06.2-02): restyle file browser with terminal aesthetic

- Replace all rgba() colors with CSS design tokens
- Add green borders (--color-border) on sidebar and file list
- Change to 4-column grid (NAME, SIZE, TYPE, MODIFIED)
- Remove border-radius for sharp terminal edges
- Add green hover/active states using --color-green-darker
- Add left border highlight for selected items
- Use monospace font family (--font-family-mono) throughout
- Apply terminal glow effect (--glow-green) on hover

* style(06.2-03): restyle dialogs with terminal aesthetic

- Remove all light mode CSS and prefers-color-scheme queries
- Use design token variables throughout
- Remove border-radius (sharp corners)
- Add green glow effects on focus/hover
- Monospace font for all dialog text
- Green primary button, red destructive button styling

* feat(06.2-04): create MatrixBackground component

- Canvas-based matrix rain animation
- Performance-optimized: 30fps throttled rendering
- Low opacity (0.25) for subtle effect
- Handles window resize with cleanup
- Uses JetBrains Mono font for consistency
- aria-hidden for accessibility
- Cleanup on unmount prevents memory leaks

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* style(06.2-02): restyle breadcrumbs with terminal aesthetic

- Add terminal-style path prefix (~/) using ::before pseudo-element
- Replace all rgba() colors with CSS design tokens
- Remove border-radius for sharp terminal edges
- Add green hover state using --color-green-darker
- Apply monospace font (--font-family-mono)
- Add green glow effect (--glow-green) on back button hover
- Use design token spacing and typography throughout

* feat(06.2-04): add MatrixBackground to Login page

- Import and render MatrixBackground component
- Update text to lowercase for terminal aesthetic
- Update tagline to match design ('zero-knowledge encrypted storage')
- Loading state shows 'initializing...' instead of 'Loading...'
- Matrix animation renders as first child (background layer)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* style(06.2-03): restyle modal and upload UI with terminal aesthetic

- Remove all light mode CSS and prefers-color-scheme queries
- Use design token variables throughout
- Remove all border-radius (sharp corners)
- Add green glow effects
- Monospace font for all text
- Green progress bars and buttons
- Terminal-styled upload zone with dashed border

* feat(06.2-02): add TYPE column to file list

- Add TYPE column header to FileList component
- Implement getItemType() helper for file type detection
- Update FileListItem to render type column in grid
- Match 4-column design (NAME, SIZE, TYPE, MODIFIED) from CSS

Deviation: Plan specified CSS-only changes but components needed
update to render the TYPE column per design specification (Rule 2)

* docs(06.2-04): complete Mobile Responsive & Matrix Background plan

Tasks completed: 3/3
- Update responsive.css with terminal aesthetic
- Create MatrixBackground component
- Add MatrixBackground to Login page

SUMMARY: .planning/phases/06.2-restyle-app-with-pencil-design/06.2-04-SUMMARY.md

* docs(06.2-02): complete Component Styling plan

Tasks completed: 2/2
- Task 1: Restyle file-browser.css with terminal aesthetic
- Task 2: Restyle breadcrumbs.css with terminal aesthetic

Deviation: Added TYPE column to React components (Rule 2)

SUMMARY: .planning/phases/06.2-restyle-app-with-pencil-design/06.2-02-SUMMARY.md

* docs(06.2-03): complete overlay components styling plan

Tasks completed: 3/3
- Task 1: Restyle context-menu.css (terminal aesthetic)
- Task 2: Restyle dialogs.css (green accents, monospace)
- Task 3: Restyle modal.css and upload.css (terminal styling)

SUMMARY: .planning/phases/06.2-restyle-app-with-pencil-design/06.2-03-SUMMARY.md

* feat(06.2-05): replace emoji icons with text prefixes in file list

- FileListItem: [DIR]/[FILE] text prefixes instead of emoji icons
- FolderTreeNode: [DIR] text prefix for folders
- Restructure FileListItem with row-top (icon+name) and row-bottom (date+size) for mobile 2-row layout
- TYPE column shows 'dir' for folders, file extension for files
- Simplify type detection logic for terminal aesthetic

* feat(06.2-05): update file list header and upload zone with terminal text

- FileList header: uppercase column names (NAME, SIZE, TYPE, MODIFIED)
- UploadZone: terminal-style button text (--upload) and lowercase states
- Match terminal/hacker aesthetic with proper text casing

* feat(06.2-05): update auth button and dashboard header with terminal branding

- AuthButton: [CONNECT] text with lowercase loading state
- LogoutButton: lowercase text (logout, logging out...)
- Dashboard header: > CIPHERBOX branding with app-title class
- Settings link and user email: lowercase for terminal aesthetic
- Remove unused auth method formatting logic and imports

* docs(06.2-05): complete component text updates plan

Tasks completed: 3/3
- Task 1: Update FileListItem and FolderTreeNode with text prefixes
- Task 2: Update FileList header and upload zone
- Task 3: Update AuthButton and Dashboard header

SUMMARY: .planning/phases/06.2-restyle-app-with-pencil-design/06.2-05-SUMMARY.md

* test(06.2-06): update E2E selectors for terminal styling

- Update login button selector: [CONNECT] button uses .auth-button class
- Update logout button selector: logout button uses .logout-button class
- Replace text-based selectors with stable class selectors
- Updated page objects: LoginPage, DashboardPage
- Updated test files: smoke.spec.ts, login.spec.ts, logout.spec.ts, session.spec.ts

Note: Folder creation tests still reference non-existent 'New Folder' button.
These tests will be addressed when folder toolbar buttons are implemented.

* fix(06.2-06): update login page text selectors for terminal styling

- Update h1 selector: CipherBox → CIPHERBOX (uppercase)
- Update tagline selector: Zero-knowledge encrypted cloud storage → zero-knowledge encrypted storage
- Matches actual component text from Login.tsx after Pencil restyle

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(MatrixBackground): replace canvas references with targetCanvas for consistency

* fix(06.2-06): align component classNames with CSS selectors

- AuthButton: auth-button → login-button (matches App.css)
- LogoutButton: logout-button → logout-link (matches App.css)
- E2E page objects: updated selectors to match

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(06.2-06): update web3auth-helpers selectors for terminal styling

- Login button: button:has-text("Sign In") → button.login-button
- Logout button: button:has-text("Logout") → button.logout-link

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(e2e): resolve upload test flakiness and selector mismatches

- Update file-list.page.ts selectors from .file-list-item-text to
  .file-list-item-name to match actual component class names
- Add wait for file list to stabilize after navigating to root in
  test 3.1 to prevent race condition during upload
- Update logout test selector to recognize [CONNECT] button from
  terminal-styled login page

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: update @types/node to version 22.19.7 in devDependencies

* ci: add CodeRabbit AI code review configuration (#48)

* ci: add CodeRabbit AI code review configuration

Adds .coderabbit.yaml with project-specific review settings including:
- Path-based instructions for api, web, and crypto packages
- Appropriate filters to ignore generated/lock files
- Security-focused review guidance for crypto code

* fix(ci): use valid profile value in CodeRabbit config

Change profile from 'default' to 'chill' as only 'chill' and
'assertive' are valid enum values in the CodeRabbit schema.

---------

Co-authored-by: Claude <noreply@anthropic.com>

* fix: address CodeRabbit review comments

- Fix MD036: Replace bold text with proper headings in SUMMARY files
- Fix MD040: Add language tag to fenced code block
- Fix a11y: Add role="gridcell" to file list grid cells
- Fix a11y: Add aria-hidden to decorative icon text

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: add Phase 6.3 UI Structure Refactor to roadmap

- Insert Phase 6.3 after 6.2 for structural UI redesign
- Design-first approach using Pencil MCP as source of truth
- Covers page layouts, component hierarchy, toolbars, navigation

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* remove project statusline config

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: Michael <michael@Michaels-Virtual-Machine.local>
@FSM1 FSM1 deleted the claude/add-code-rabbit-ci-htJY2 branch January 27, 2026 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants