Skip to content

Add Phase 3 streaming integration tests#50

Merged
ananthpai1998 merged 1 commit intomainfrom
claude/streaming-integration-tests-017LBfMhkAVBr5QgAXfmsBSp
Nov 18, 2025
Merged

Add Phase 3 streaming integration tests#50
ananthpai1998 merged 1 commit intomainfrom
claude/streaming-integration-tests-017LBfMhkAVBr5QgAXfmsBSp

Conversation

@ananthpai1998
Copy link
Copy Markdown
Collaborator

Implemented comprehensive streaming integration tests covering:

  • Document streaming (create, update, revert)
  • Python code streaming (create, update, fix, revert)
  • Mermaid diagram streaming (create, update, fix, revert)
  • Stream interruption handling for all artifact types
  • Stream resumption after interruption
  • Data integrity verification after resume

All 19 tests passing successfully.

Pull Request

Description

Type of Change

  • 🚀 Feature - New functionality
  • 🐛 Bug fix - Fixes an issue
  • 🗃️ Database migration - Schema changes
  • 📝 Documentation - Updates to docs
  • ⚡ Performance improvement
  • 🎨 UI/UX - Visual or interaction changes
  • ♻️ Refactor - Code restructuring without functional changes
  • 🧪 Test - Adding or updating tests
  • 🔧 Chore - Maintenance tasks

Database Changes

  • ✅ No database changes
  • 📊 Schema changes - Migration file: lib/db/migrations/XXXX_migration_name.sql
  • 🔄 Data migration required
  • 🔒 RLS policy changes
  • 📈 Index additions/modifications

Migration Details (if applicable)

Tables affected:

Migration file:

  • lib/db/migrations/XXXX_migration_name.sql

Backward compatible:

  • Yes
  • No (breaking change - requires coordination)

Rollback script included:

  • Yes
  • No
  • N/A

Impact on existing data:

Verification queries:

Testing

  • ✅ Tested locally
  • ✅ All existing tests pass
  • ✅ New tests added
  • ✅ Manual testing performed
  • ✅ Tested with production-like data

Test Coverage

Screenshots/Videos (if UI changes)

Breaking Changes

  • ⚠️ This PR includes breaking changes

If yes, describe:

Performance Impact

  • No performance impact expected
  • Performance improvements expected
  • Potential performance impact (explain below)

Details:

Security Considerations

  • No security impact
  • Security improvement
  • Requires security review

Details:

Dependencies

New dependencies:

Updated dependencies:

Checklist

  • 📖 Code follows project style guidelines (run pnpm lint)
  • 🔍 Self-reviewed the code
  • 💬 Commented complex/unclear code
  • 📚 Updated relevant documentation (README, API docs, etc.)
  • 🧪 Added/updated tests for new functionality
  • 🔒 Verified RLS policies work correctly (if DB changes)
  • ⚡ Added indexes for new foreign keys (if DB changes)
  • 🏷️ Updated TypeScript types/interfaces
  • ♿ Considered accessibility (if UI changes)
  • 📱 Tested on mobile/responsive (if UI changes)
  • 🌐 No hardcoded strings (using i18n if applicable)

Related Issues

Closes #
Related to #

Additional Context

Deployment Notes

  • Requires environment variable changes
  • Requires manual migration by Super Admin
  • Requires coordination with other PRs
  • Should be deployed during low-traffic hours

Special instructions:


For Reviewers

Focus areas:
1.
2.

Testing instructions:
1.
2.


Super Admin Checklist (for DB migrations)

  • Reviewed migration script syntax
  • Verified backward compatibility
  • Tested on staging/local Supabase
  • Verified rollback script works
  • No data loss risk
  • Performance impact assessed (indexes added?)
  • RLS policies correct and secure
  • Ready to apply to production

Implemented comprehensive streaming integration tests covering:
- Document streaming (create, update, revert)
- Python code streaming (create, update, fix, revert)
- Mermaid diagram streaming (create, update, fix, revert)
- Stream interruption handling for all artifact types
- Stream resumption after interruption
- Data integrity verification after resume

All 19 tests passing successfully.
@vercel
Copy link
Copy Markdown

vercel bot commented Nov 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
chat Building Building Preview Comment Nov 18, 2025 5:00am

@ananthpai1998 ananthpai1998 merged commit 64631ff into main Nov 18, 2025
1 of 5 checks passed
sidselothcodes added a commit to sidselothcodes/chat that referenced this pull request Mar 8, 2026
- Add lib/repos.ts: parses .gitmodules and .siterepos for repo names,
  falls back to a default list when files are unavailable
- Add GET /api/repos endpoint returning available repo names
- Add useRepos() client hook to fetch repo list
- Add RepoSelector component: DropdownMenu with checkboxes for
  multi-select repo filtering (empty selection = all repos)
- Add selectedRepos field to chat request Zod schema
- Update RAG pipeline to support limitToRepoNames with Pinecone
  in operator for multi-repo vector filtering
- Wire repo state through Chat to MultimodalInput to RepoSelector
  and include in chat API request body

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ModelEarth added a commit to ModelEarth/chat that referenced this pull request Mar 10, 2026
Add RAG repo selector with multi-repo filtering (issue Earthscape#50)
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