Skip to content

fix/dreamsync-token-limit - #499

Merged
coodos merged 3 commits into
mainfrom
feat/dreamsync-fix
Dec 8, 2025
Merged

coodos merged 3 commits into
mainfrom
feat/dreamsync-fix

Conversation

@coodos

@coodos coodos commented Dec 8, 2025 •

Copy link
Copy Markdown
Contributor

Description of change

Dreamsync API was having token context issues wherein, large wishlists had a tendency to be so long that we ran out of tokens to run AI analysis, this PR changes it so that now we use AI to pre-summarize wishlists to offers and wants and we can run matchmaking using that later.

Issue Number

Type of change

  • Fix (a change which fixes an issue)

How the change has been tested

Change checklist

  • I have ensured that the CI Checks pass locally
  • I have removed any unnecessary logic
  • My code is well documented
  • I have signed my commits
  • My code follows the pattern of the application
  • I have self reviewed my code

@coodos
coodos marked this pull request as ready for review December 8, 2025 09:51
@coderabbitai

coderabbitai Bot commented Dec 8, 2025 •

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@coodos has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 9 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 8282e3d and 3926c34.

📒 Files selected for processing (8)
  • platforms/dreamsync-api/src/database/entities/Wishlist.ts (1 hunks)
  • platforms/dreamsync-api/src/database/migrations/1765187375413-migration.ts (1 hunks)
  • platforms/dreamsync-api/src/index.ts (2 hunks)
  • platforms/dreamsync-api/src/services/AIMatchingService.ts (6 hunks)
  • platforms/dreamsync-api/src/services/MatchingService.ts (4 hunks)
  • platforms/dreamsync-api/src/services/SharedMatchingService.ts (1 hunks)
  • platforms/dreamsync-api/src/services/WishlistService.ts (3 hunks)
  • platforms/dreamsync-api/src/services/WishlistSummaryService.ts (1 hunks)
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/dreamsync-fix

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.

@coodos
coodos merged commit 923f111 into main Dec 8, 2025
4 checks passed
@coodos
coodos deleted the feat/dreamsync-fix branch December 8, 2025 10:00
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