Fix typescript errors lint and build

[Loukass23]

Pull Request

📝 Description

This PR addresses and resolves a significant number of TypeScript errors and linting warnings, enabling the project to build successfully. The changes aim to improve code quality, type safety, and adherence to best practices, while also making the ESLint configuration more practical for the current project state.

What does this PR do?

• Fixes TypeScript Errors:
  • Updated ZodError handling to use error.issues instead of error.errors.
  • Adjusted ServerActionResult types to correctly handle ZodIssue[].
  • Corrected numerous Prisma model field mismatches (e.g., content to contentRich, authorId to userId in DocumentComment, removing non-existent fields like summary from Project updates).
  • Addressed return type inconsistencies for handlePrismaError.
  • Resolved issues related to Next.js 15 params prop type.
  • Removed or prefixed unused variables and imports.
• Resolves Linting Warnings:
  • Restructured React components (SaveToolbarButton, SimpleSaveButton) to ensure hooks are called unconditionally, preventing "React Hooks are called conditionally" errors.
  • Updated ESLint configuration (eslint.config.mjs) to be more lenient for no-explicit-any, prefer-nullish-coalescing, and no-console to facilitate development.
• Adds Missing Components/Utilities:
  • Created placeholder components for ProjectSummaryEditor and components/error/index.ts to resolve module not found errors during build.
  • Added markdown-joiner-transform.ts utility.
• Build System:
  • Ensures the project can now successfully build after these fixes.

🔄 Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality) - Minimal placeholder components created to enable build
• 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
• 📚 Documentation update
• 🎨 Style/UI changes
• ♻️ Code refactoring (no functional changes)
• ⚡ Performance improvements
• 🧪 Test additions or updates
• 🔧 Configuration changes
• 🏗️ Build system changes

📸 Screenshots/Videos

🤔 Questions and Concerns

Known Issues or Limitations

• The ProjectSummaryEditor and markdown-joiner-transform are minimal implementations created to resolve build errors. Their full functionality and integration might require further development.
• The discussionDatabasePlugin was temporarily commented out/nullified due to missing imports. Its re-integration and proper implementation might be needed.
• ESLint rules were relaxed for no-explicit-any, prefer-nullish-coalescing, and no-console. This might warrant further discussion on desired strictness.

📚 Additional Context

This PR was initiated to address the task of "Fix typescript errors or update rules if needed. Lint code and build." The project initially had 359 TypeScript errors and numerous linting warnings, preventing a successful build. This PR systematically tackles these issues to bring the project to a buildable state.

🙏 Thank You!

Thank you for contributing to CODAC! Your effort helps make this platform better for the entire Code Academy Berlin community.

First-time contributor?

Welcome! 🎉 Don't worry if you receive feedback - it's all part of the learning process. We're here to help you succeed!

Need help?

• Check our Contributing Guide
• Ask questions in this PR - we're happy to help
• Join our community discussions for real-time support

Happy coding! 🚀

---

 

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}