Skip to content

fix: prevent message duplication in analysis refinement#16

Open
xharauenea wants to merge 1 commit intomainfrom
fix/old-messages-deduplication
Open

fix: prevent message duplication in analysis refinement#16
xharauenea wants to merge 1 commit intomainfrom
fix/old-messages-deduplication

Conversation

@xharauenea
Copy link
Contributor

Updated the refineAnalysis function to return only new messages, eliminating duplication caused by the previous implementation that appended all messages to the state. This change ensures cleaner state management in the reducer.

Description

Please include a summary of the changes and related context.

Type of change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔄 Refactor (code change that improves maintainability)
  • 📚 Documentation update
  • 🔧 Configuration or build change
  • ⚡ Performance improvement
  • 🚨 Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context

Why is this change needed? What problem does it solve?

How has this been tested?

Please describe the tests that you ran to verify your changes.

Test Configuration

  • Node version:
  • npm version:
  • OS:

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have run npm run lint and fixed any issues
  • I have run npm run prettier to format my code

Breaking Changes

Does this PR introduce any breaking changes?

  • No
  • Yes (please describe):

Screenshots (if applicable)

If this PR includes UI changes, please add screenshots.

Additional context

Add any other context about the PR here.


Thank you for contributing to CodeWave! 🚀

Updated the refineAnalysis function to return only new messages, eliminating duplication caused by the previous implementation that appended all messages to the state. This change ensures cleaner state management in the reducer.
@xharauenea xharauenea requested a review from rqirici December 30, 2025 12:18
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.

1 participant