Skip to content

Fix build errors in vitest.config.ts#54

Merged
ananthpai1998 merged 1 commit intomainfrom
claude/fix-build-errors-01YHJ2crU7GPeBbgXJrh56Ui
Nov 18, 2025
Merged

Fix build errors in vitest.config.ts#54
ananthpai1998 merged 1 commit intomainfrom
claude/fix-build-errors-01YHJ2crU7GPeBbgXJrh56Ui

Conversation

@ananthpai1998
Copy link
Copy Markdown
Collaborator

Updated vitest configuration to be compatible with latest vitest v8 coverage provider:

  • Removed deprecated 'all' property from coverage options
  • Moved coverage threshold properties (lines, functions, branches, statements) under 'thresholds' object
  • Removed deprecated 'poolOptions' configuration

These changes resolve TypeScript errors that were preventing the build from succeeding.

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

Updated vitest configuration to be compatible with latest vitest v8 coverage provider:
- Removed deprecated 'all' property from coverage options
- Moved coverage threshold properties (lines, functions, branches, statements) under 'thresholds' object
- Removed deprecated 'poolOptions' configuration

These changes resolve TypeScript errors that were preventing the build from succeeding.
@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 Ready Ready Preview Comment Nov 18, 2025 3:27pm

@ananthpai1998 ananthpai1998 merged commit 1c57ebe into main Nov 18, 2025
5 of 9 checks passed
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