Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Sep 7, 2025

This PR addresses several critical issues that were preventing the project from building and passing type checks:

🔧 Build Issues Fixed

Session Password Configuration: The build was failing with "Password string too short (min 32 characters required)" during prerendering. Updated the default session password in config.ts to meet the 32+ character requirement for iron-webcrypto.

🔍 TypeScript Issues Fixed

Zod Validation Types: Fixed complex type constraint issues in apollo/validation.ts by simplifying the approach:

  • Removed overly complex Properties<T> type mapping that was causing generic constraint errors
  • Replaced with direct Zod schema definitions for SignupInputSchema and LoginInputSchema
  • This eliminates 5+ TypeScript errors related to Zod type internals

Import Issues: Fixed missing AnyZodObject export by replacing it with the correct ZodObject type in server/utils/validation.ts.

Form Type Safety: Enhanced type safety in the login form by adding computed properties to properly cast form field values for naive-ui component compatibility.

🧹 Linter & Test Fixes

Code Formatting: Fixed Prettier formatting issues in modified files.

Test Snapshots: Updated test snapshots to match the new Zod error format:

  • Error objects now include origin field instead of exact and type fields
  • Updated validation messages to match current Zod version

Unused Imports: Removed unused ZodRawShape import that was flagged by oxlint.

✅ Verification

All critical checks now pass:

  • pnpm build - Builds successfully without errors
  • pnpm typecheck - No TypeScript errors
  • pnpm lint - No linting errors (warnings are acceptable)
  • pnpm test:unit - All unit tests pass (19/19)

The changes are minimal and surgical, focusing only on fixing the blocking issues without altering working functionality.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@Copilot Copilot AI changed the title [WIP] fix tests and linter, and any other build issues you find in this branch Fix critical TypeScript, build, and linter issues Sep 7, 2025
@Copilot Copilot AI requested a review from tobiasdiez September 7, 2025 12:43
Copilot finished work on behalf of tobiasdiez September 7, 2025 12:43
@renovate renovate bot force-pushed the renovate/major-nuxt branch 3 times, most recently from dd4161e to 8fa367e Compare September 12, 2025 23:55
@renovate renovate bot force-pushed the renovate/major-nuxt branch 2 times, most recently from 82f0d1b to 459e65c Compare September 20, 2025 17:37
@renovate renovate bot force-pushed the renovate/major-nuxt branch 4 times, most recently from 38ad16a to d95760e Compare October 2, 2025 13:51
@renovate renovate bot force-pushed the renovate/major-nuxt branch 8 times, most recently from cf13ba2 to 81d19e7 Compare October 3, 2025 21:48
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