Skip to content

fix: resolve failing CI — backend TS errors and frontend ESLint errors#256

Merged
wumibals merged 7 commits into
tosirano:mainfrom
wumibals:fix/ci-lint-and-typescript-errors
Jun 8, 2026
Merged

fix: resolve failing CI — backend TS errors and frontend ESLint errors#256
wumibals merged 7 commits into
tosirano:mainfrom
wumibals:fix/ci-lint-and-typescript-errors

Conversation

@wumibals

@wumibals wumibals commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes all failing CI checks across the Backend (NestJS) and Frontend (Next.js) jobs.

Backend fixes

  • auth.module.ts — cast expiresIn to any to satisfy JwtModuleOptions type constraint
  • analytics, contact, instagram modules — rewrote to use correct local imports, removed stale references to non-existent auth.service/auth.dto/jwt-auth.guard paths that were placed in wrong modules
  • backend/src/common/ — added guards/jwt-auth.guard.ts, guards/roles.guard.ts, decorators/roles.decorator.ts, enums/role.enum.ts which are required by scrap-menu controllers
  • scrap-menu/dto/ — added all 8 missing DTO files (create-* and update-* for all four resources)
  • package.json — removed unused @nestjs/swagger entry that was causing npm ci to fail due to package-lock.json mismatch

Frontend fixes

Escaped unescaped apostrophes/quotes (react/no-unescaped-entities) in:

  • LoginClient.tsx
  • AboutSection.tsx
  • ContactForm.tsx
  • TestimonialsSection.tsx
  • TrendingSection.tsx
  • VisualMoodBoard.tsx
  • ChangePasswordForm.tsx

Replaced bare <a href="/"> with <Link href="/"> in globalError.tsx (@next/next/no-html-link-for-pages)

wumibals added 7 commits June 8, 2026 12:14
Backend (NestJS):
- Fix auth.module.ts JwtModule useFactory type by casting expiresIn to any
- Rewrite analytics, contact, instagram modules to use correct imports and
  remove non-existent module references (wrong auth.service/dto/guard paths)
- Add backend/src/common/guards, decorators, enums used by scrap-menu controllers
- Add missing scrap-menu/dto files (create/update DTOs for all four resources)
- Remove @nestjs/swagger from package.json (unused; would break npm ci)

Frontend (Next.js):
- Escape unescaped apostrophes and quotes in JSX (react/no-unescaped-entities)
  in LoginClient, AboutSection, ContactForm, TestimonialsSection,
  TrendingSection, VisualMoodBoard, ChangePasswordForm
- Replace <a href="/"> with <Link href="/"> in globalError.tsx
  (@next/next/no-html-link-for-pages)
… paths

- Fix AuditLogsTable.tsx: correct relative import from ../ui/Button to
  ../../ui/Button (was resolving to non-existent admin/ui/Button)
- Add MenuPageClient.tsx for app/(public)/menu/page.tsx
- Add re-export stubs in components/ for all ui, features, menu, and
  admin components that exist in app/components/ but were imported via
  the @/ alias pointing to the root components/ directory
- Add components/dashboard/ with DashboardLayout, StatsCards,
  ActivityFeed, QuickActions, AnalyticsChart, AdminOverview,
  AdminUserTable
- Add lib/apiClient.ts re-exporting from lib/api/client
- Add lib/react-query/hooks stubs for workspace-tracking and bookings
@wumibals wumibals merged commit 3c0e408 into tosirano:main Jun 8, 2026
2 of 3 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.

1 participant