Skip to content

Conversation

@jbdevprimary
Copy link
Contributor

Summary

Migrates all state management to the @thumbcode/state package, removing the duplicate stores from src/stores.

This follows the established monorepo pattern where state should live in packages/ rather than src/.

Changes

Structure

  • Move tests from src/stores/__tests__ to packages/state/src/__tests__
  • Remove duplicate src/stores directory
  • Remove @/stores path alias from tsconfig

Imports

  • Update CredentialService to import from @thumbcode/state
  • Update agent-intelligence chat components to import from @thumbcode/state
  • Add @thumbcode/state dependency to agent-intelligence package

Configuration

  • Update jest.config.js to include packages/state tests
  • Add testPathIgnorePatterns for packages with custom jest configs (agent-intelligence)

Test plan

  • All 114 tests pass
  • TypeScript typecheck passes
  • Biome lint passes
  • CI/CD checks pass
  • CodeRabbit review

🤖 Generated with Claude Code

Migrates all state management to the @thumbcode/state package, removing
the duplicate stores from src/stores.

## Changes

### Structure
- Move tests from src/stores/__tests__ to packages/state/src/__tests__
- Remove duplicate src/stores directory
- Remove @/stores path alias from tsconfig

### Imports
- Update CredentialService to import from @thumbcode/state
- Update agent-intelligence chat components to import from @thumbcode/state
- Add @thumbcode/state dependency to agent-intelligence package

### Configuration
- Update jest.config.js to include packages/state tests
- Add testPathIgnorePatterns for packages with custom jest configs

This follows the monorepo pattern where state should live in packages/
rather than src/.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 18, 2026

Warning

Rate limit exceeded

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

⌛ 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.

📥 Commits

Reviewing files that changed from the base of the PR and between 2299d37 and 9b40b1a.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (16)
  • jest.config.js
  • packages/agent-intelligence/package.json
  • packages/agent-intelligence/src/components/chat/ChatBubble.tsx
  • packages/agent-intelligence/src/components/chat/ChatInput.tsx
  • packages/state/src/__tests__/agentStore.test.ts
  • packages/state/src/__tests__/credentialStore.test.ts
  • packages/state/src/__tests__/userStore.test.ts
  • src/services/credentials/CredentialService.ts
  • src/services/credentials/__tests__/CredentialService.test.ts
  • src/stores/agentStore.ts
  • src/stores/chatStore.ts
  • src/stores/credentialStore.ts
  • src/stores/index.ts
  • src/stores/projectStore.ts
  • src/stores/userStore.ts
  • tsconfig.json

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

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

@sonarqubecloud
Copy link

@jbdevprimary jbdevprimary merged commit 45b3410 into main Jan 18, 2026
12 checks passed
@jbdevprimary jbdevprimary deleted the fix/consolidate-state-stores branch January 18, 2026 20:37
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