Claude/fix all the things 016m gv g ahqs puhs4 f st xr p jt#20
Merged
ananthpai1998 merged 3 commits intomainfrom Nov 17, 2025
Merged
Conversation
- Add explicit boolean type annotations to Switch onCheckedChange handlers across agent config components - Install missing @radix-ui packages (avatar, checkbox, collapsible, dropdown-menu, scroll-area, separator, switch) - Install missing uuid package for activity logger Fixed type errors in: - components/admin/agents/chat-model/chat-model-agent-config.tsx - components/admin/agents/document/document-agent-config.tsx - components/admin/agents/git-mcp/git-mcp-agent-config.tsx - components/admin/agents/mermaid/mermaid-agent-config.tsx - components/admin/agents/provider-tools/provider-tools-agent-config.tsx - components/admin/agents/python/python-agent-config.tsx - components/admin/logging-dashboard.tsx Build now passes successfully.
Removed unnecessary files and dependencies that were not being used: Files removed: - playwright.config.ts - No tests exist in the project - test-tool-config.js - Empty/unused configuration file - instrumentation.ts - OpenTelemetry not enabled in Next.js config - package-lock.json - Project uses pnpm, not npm - tsconfig.tsbuildinfo - Build artifact, should not be committed Dependencies removed: - @playwright/test - No tests to run - @vercel/otel - Only used in removed instrumentation.ts - @opentelemetry/api - Only needed for instrumentation - @opentelemetry/api-logs - Only needed for instrumentation Scripts removed: - "test" script - No test files exist Files kept (they ARE being used): - drizzle.config.ts - Database migrations actively used in build - biome.jsonc - Linting configuration (used by ultracite) - middleware.ts - Next.js middleware - next.config.ts, tsconfig.json, postcss.config.mjs - Core configs Build verified successful after cleanup.
The Vercel deployment is failing due to stale build cache from recently removed dependencies (@playwright/test, @vercel/otel, etc.). Changes: - Add VERCEL_BUILD_FIX.md with instructions to clear Vercel build cache - Update .gitignore to include tsconfig.tsbuildinfo (prevent future commits) - Remove Playwright references from .gitignore (no longer used) Local builds pass successfully - this is purely a Vercel cache issue. To fix on Vercel: 1. Go to project Settings → General 2. Clear Build Cache 3. Redeploy The file can be deleted after the Vercel issue is resolved.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Description
Type of Change
Database Changes
lib/db/migrations/XXXX_migration_name.sqlMigration Details (if applicable)
Tables affected:
Migration file:
lib/db/migrations/XXXX_migration_name.sqlBackward compatible:
Rollback script included:
Impact on existing data:
Verification queries:
Testing
Test Coverage
Screenshots/Videos (if UI changes)
Breaking Changes
If yes, describe:
Performance Impact
Details:
Security Considerations
Details:
Dependencies
New dependencies:
Updated dependencies:
Checklist
pnpm lint)Related Issues
Closes #
Related to #
Additional Context
Deployment Notes
Special instructions:
For Reviewers
Focus areas:
1.
2.
Testing instructions:
1.
2.
Super Admin Checklist (for DB migrations)