Skip to content

Fix/frontend compilation and syntax#262

Open
DebasmitaBose0 wants to merge 2 commits into
Akshay473:mainfrom
DebasmitaBose0:fix/frontend-compilation-and-syntax
Open

Fix/frontend compilation and syntax#262
DebasmitaBose0 wants to merge 2 commits into
Akshay473:mainfrom
DebasmitaBose0:fix/frontend-compilation-and-syntax

Conversation

@DebasmitaBose0
Copy link
Copy Markdown
Contributor

Closes #261

Summary

This PR addresses and fully resolves multiple fatal compilation and syntax bugs across the Next.js client workspace, restoring build success under the Next Turbopack compiler.

Changes Made

  • Dependency Swap: Replaced bcrypt with pure-JS bcryptjs and @types/bcryptjs in package.json to bypass native C++ compilation errors. Updated client/lib/auth.ts.
  • De-Duplication: Resolved duplicate exports, imports, and pages in client/app/login/page.tsx, preserving the premium theme secure portal login.
  • Route Re-write: Cleaned up scrambled syntax inside client/app/api/analyze/route.ts, establishing a clean, secure proxy pipeline between Next.js and the FastAPI downstream backend.
  • Dashboard Syntax Fix: Fixed duplicate async declarations of runAutomatedFlow, added proper fallback handling, and corrected mixed parentheses rendering blocks on line 1499 of client/app/dashboard/page.tsx.

Testing

  • Executed npm install and ran npm run build locally.
  • Result: ✓ Compiled successfully in 52s with all static pages optimized and zero compilation errors.

Impact

Vastly improves developer experience (DX) and repo health by proactively catching compilation bugs.

Checklist

  • Code follows project standards
  • Tested locally
  • No unrelated changes included
  • Responsive design verified
  • Accessibility considered

@vercel
Copy link
Copy Markdown

vercel Bot commented May 31, 2026

@DebasmitaBose0 is attempting to deploy a commit to the akshay473's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

fix: resolve duplicate LoginPage declarations, scrambled routing syntax, and missing bcryptjs dependency

1 participant