feat: pomodoro counter + long break logic (Day 4)

[MinitJain]

Summary

• Cycle persists on refresh — focusCount now inits from session.pomos_done (DB-backed) instead of always starting at 0
• pomos_done written to DB on every focus completion (auto-break and manual skip paths) and reset to 0 on settings change
• #N today badge — fetches today's count from pomodoro_logs on mount; increments optimistically on each completion (auth users only)
• Simplified round label — always shows "N pomodoros completed", consistent across all modes (focus / short / long break)
• Long break logic untouched — still triggers after settings.rounds completions

Test plan

• Complete a focus session — pomos_done increments in DB, label updates, #N today increments
• Refresh page mid-session — cycle count preserved from DB
• Complete 4 focus rounds — long break triggers correctly
• Change settings — pomos_done resets to 0 in DB and UI
• Guest user — #N today badge hidden, rest works normally

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • Session tracking now displays a counter showing pomodoros completed today.

• Tests

  • Added comprehensive test suites for timer functionality and session management.

• Chores

  • Added testing framework and dependencies; updated security policies.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
pomodoro-jam	Ready	Preview, Comment	Apr 17, 2026 11:17am

[coderabbitai]

Warning

Rate limit exceeded

@MinitJain has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 28 minutes and 36 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 28 minutes and 36 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2c8e717c-720f-4c4b-9a85-68013a30a22c

📥 Commits

Reviewing files that changed from the base of the PR and between 65b397a and 9518d46.

📒 Files selected for processing (2)

• components/session/SessionProvider.tsx
• components/session/StatsTab.tsx

Walkthrough

This PR establishes a comprehensive testing infrastructure using Vitest, adds test suites for the useTimer hook and timer utilities, implements pomodoro session counting with daily completion tracking persisted to Supabase, updates CSP headers to allow blob image sources, and adds a test step to the CI workflow.

Changes

Cohort / File(s)	Summary
Test Infrastructure vitest.config.ts, vitest.setup.ts, package.json	Added Vitest configuration with React plugin, jsdom environment, global test APIs, and @testing-library/jest-dom setup. Updated package.json with test scripts (npm test, npm run test:watch) and testing dependencies.
useTimer Hook Tests __tests__/hooks/useTimer.test.tsx	Added 184-line test suite verifying hook state transitions (start, pause, reset, setMode, skipAndStart), expiration behavior, time computation, and external state application with fake timers.
Timer Utilities Tests __tests__/lib/timer.test.ts, __tests__/lib/writeQueue.test.ts	Added 195-line test suite for timer utilities (computeTimeLeft, isTimerExpired, formatTime, createTimerState, sessionToTimerState, computeProgress) and 56-line test for write queue serialization behavior.
Session State Management components/session/SessionProvider.tsx	Added pomodoro completion tracking: initialized focusCountRef and todayCount state from session data, added Supabase query to fetch today's completed pomodoros, incremented todayCount and persisted pomos_done on focus completion and skip, updated UI label from round-based to completion-based formatting.
Configuration & Build .github/workflows/ci.yml, next.config.js	Added Test job step to CI workflow after Lint and before Typecheck. Updated CSP header img-src directive from data: to blob:.
Import Ordering components/profile/AvatarCropModal.tsx	Reordered imports to place type-only Area import after CSS import.

Sequence Diagram

sequenceDiagram
    participant SessionProvider
    participant Supabase Query
    participant Supabase DB
    participant UI

    SessionProvider->>SessionProvider: Initialize focusCountRef from session.pomos_done
    SessionProvider->>Supabase Query: Fetch completed pomodoros since today start
    Supabase Query->>Supabase DB: Query pomodoro_logs with completed_at >= start-of-today
    Supabase DB-->>Supabase Query: Return count
    Supabase Query-->>SessionProvider: Set todayCount state
    SessionProvider->>UI: Render "#<todayCount> today" badge

    rect rgba(100, 150, 200, 0.5)
    Note over SessionProvider,Supabase DB: On focus completion or skip
    SessionProvider->>SessionProvider: Increment todayCount
    SessionProvider->>Supabase DB: Update sessions.pomos_done
    Supabase DB-->>SessionProvider: Confirmation
    SessionProvider->>UI: Update completion count label
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• Feat/watcher settings ambient #12: Modifies SessionProvider.tsx to add pomodoro counting and completion tracking with DB persistence—shares the same pomodoro tracking logic expansion.
• fix: allow blob: URLs in CSP for avatar crop modal #25: Applies identical CSP header change (blob: for img-src) and import reordering in AvatarCropModal.tsx—overlapping code-level modifications.
• fix: display name in session tooltip, streak calendar timezone, explo… #13: Updates SessionProvider.tsx with additional Supabase query/update patterns—related session state management changes.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 60.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'feat: pomodoro counter + long break logic (Day 4)' accurately reflects the main feature additions: pomodoro counter persistence via DB-backed initialization and simplified round labeling, with testing infrastructure also added.
Description check	✅ Passed	The PR description comprehensively covers the changes (DB persistence, today badge, simplified labels), provides a detailed test plan with specific scenarios, and includes all critical checklist items.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/pomodoro-counter

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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