Skip to content

Conversation

@ManishMadan2882
Copy link
Collaborator

@ManishMadan2882 ManishMadan2882 commented Oct 28, 2025

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Bug Fixes & UI Improvements
  • Separated attachment ID for Redux operations to prevent state conflicts between UI removal and backend task tracking
  • Fixed popup components (Sources, Tools) and toast notifications to use proper z-index and positioning
  • Updated ShareConversationModal to use consistent button styling and layout
  • Renamed components for consistency: ConnectorTreeComponent → ConnectorTree, FileTreeComponent → FileTree
  • Removed unused CreateAPIKeyModal and SaveAPIKeyModal files and related models

Features & Enhancements

  • Implemented auto-clear functionality for invalid sources from localStorage on preference slice initialization

  • Enhanced preferenceApi with validation logic to detect and remove stale source references

  • Fixed "Delete All Conversations" modal close button behavior for better UX

  • Implemented "play one at a time" functionality for TTS to prevent multiple audio streams playing simultaneously

  • implemented LRU cache for the audio

  • Added key to the ConversationMesages and ConversationBubble to trigger an implicit remount(this fixes a bug, where the playing state of one message was displayed on another conversation after switching)

  • Cleaning the Text message for TTS (cleans markdown symbols and emojis, skips code blocks and mermaid flowcharts)

  • Why was this change needed? (You can also link to an open issue here)
    Enhancement, UX, Quality

@vercel
Copy link

vercel bot commented Oct 28, 2025

@ManishMadan2882 is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@codecov
Copy link

codecov bot commented Oct 28, 2025

Codecov Report

❌ Patch coverage is 3.44828% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.70%. Comparing base (9b2383b) to head (7b0d4fe).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
application/utils.py 3.84% 25 Missing ⚠️
application/api/user/attachments/routes.py 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2112      +/-   ##
==========================================
+ Coverage   42.28%   42.70%   +0.41%     
==========================================
  Files         136      137       +1     
  Lines        9315     9458     +143     
==========================================
+ Hits         3939     4039     +100     
- Misses       5376     5419      +43     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions bot added the application Application label Oct 28, 2025
@ManishMadan2882 ManishMadan2882 marked this pull request as ready for review October 28, 2025 14:42
@dartpain dartpain merged commit 6a4cb61 into arc53:main Oct 28, 2025
6 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants