Feature/tech-debt-bash-2

eslint.config.mjs

@@ -0,0 +1,19 @@
+import { defineConfig } from 'eslint/config'
+import nextCoreWebVitals from 'eslint-config-next/core-web-vitals'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const __filename = fileURLToPath(import.meta.url)
+const __dirname = path.dirname(__filename)
+
+export default defineConfig([
+  {
+    extends: [...nextCoreWebVitals],
+    rules: {
+      'react-hooks/set-state-in-effect': 'off',
+      'react-hooks/refs': 'off',
+      'react-hooks/preserve-manual-memoization': 'off',
+      'react-compiler/react-compiler': 'off',
+    },
+  },
+])

next.config.js

@@ -18,6 +18,14 @@ const nextConfig = {
 
     return config
   },
+  turbopack: {
+    rules: {
+      '*.svg': {
+        loaders: ['@svgr/webpack'],
+        as: '*.jsx',
+      },
+    },
+  },
 }
 
 module.exports = withSentryConfig(nextConfig, {

package.json

@@ -29,26 +29,27 @@
     "jsdom": "^25.0.1",
     "localforage": "^1.10.0",
     "marked": "^16.3.0",
-    "next": "14.2.35",
+    "next": "16.1.1",
     "nextjs-progressloader": "^1.2.0",
     "p-retry": "^6.2.0",
     "prisma": "^5.19.0",
-    "react": "^18",
+    "react": "19.2.3",
     "react-dnd": "^16.0.1",
     "react-dnd-html5-backend": "^16.0.1",
     "react-dnd-touch-backend": "^16.0.1",
-    "react-dom": "^18.3",
+    "react-dom": "19.2.3",
     "react-redux": "^9.1.0",
     "react-zoom-pan-pinch": "^3.7.0",
     "server-only": "^0.0.1",
     "supabase": "^2.1.1",
     "swr": "^2.2.5",
-    "tapwrite": "1.1.93",
+    "tapwrite": "1.1.94",
     "uuid": "^11.1.0",
     "zod": "^3.23.8"
   },
   "description": "A comprehensive task management app",
   "devDependencies": {
+    "@eslint/eslintrc": "^3.3.3",
     "@faker-js/faker": "^8.4.1",
     "@ngrok/ngrok": "^1.4.1",
     "@svgr/webpack": "^8.1.0",
@@ -57,18 +58,19 @@
     "@types/jest": "^29.5.12",
     "@types/jsdom": "^21.1.7",
     "@types/node": "^20",
-    "@types/react": "^18",
+    "@types/react": "19.2.7",
     "@types/react-custom-scrollbars": "^4.0.13",
-    "@types/react-dom": "^18",
+    "@types/react-dom": "19.2.3",
     "autoprefixer": "^10.0.1",
-    "eslint": "^8",
-    "eslint-config-next": "14.0.4",
+    "eslint": "^9",
+    "eslint-config-next": "16.1.1",
     "husky": "^8.0.0",
     "jest": "^29.7.0",
     "nodemon": "^3.1.9",
     "open": "^10.1.0",
     "prettier": "^3.1.1",
     "tailwindcss": "^3.3.0",
+    "text-table": "^0.2.0",
     "tsx": "^4.16.5",
     "typescript": "^5"
   },

sentry.client.config.ts

@@ -2,47 +2,55 @@
 // The config you add here will be used whenever a users loads a page in their browser.
 // https://docs.sentry.io/platforms/javascript/guides/nextjs/
 
-import * as Sentry from '@sentry/nextjs'
+import * as Sentry from "@sentry/nextjs";
 
-const dsn = process.env.NEXT_PUBLIC_SENTRY_DSN || process.env.SENTRY_DSN
-const vercelEnv = process.env.NEXT_PUBLIC_VERCEL_ENV
-const isProd = process.env.NEXT_PUBLIC_VERCEL_ENV === 'production'
+const dsn = process.env.NEXT_PUBLIC_SENTRY_DSN || process.env.SENTRY_DSN;
+const vercelEnv = process.env.NEXT_PUBLIC_VERCEL_ENV;
+const isProd = process.env.NEXT_PUBLIC_VERCEL_ENV === "production";
 
 if (dsn) {
-  Sentry.init({
-    dsn,
-
-    // Adjust this value in production, or use tracesSampler for greater control
-    tracesSampleRate: isProd ? 0.2 : 1,
-    profilesSampleRate: 0.1,
-    // NOTE: reducing sample only 10% of transactions in prod to get general trends instead of detailed and overfitted data
-
-    // Setting this option to true will print useful information to the console while you're setting up Sentry.
-    debug: false,
-
-    // You can remove this option if you're not planning to use the Sentry Session Replay feature:
-    // NOTE: Since session replay barely helps us anyways, getting rid of it to reduce some bundle size at least
-    // replaysOnErrorSampleRate: 1.0,
-    // replaysSessionSampleRate: 0,
-    integrations: [
-      Sentry.browserTracingIntegration(),
-      //   Sentry.replayIntegration({
-      // Additional Replay configuration goes in here, for example:
-      //     maskAllText: true,
-      //     blockAllMedia: true,
-      //   }),
-    ],
-
-    // ignoreErrors: [/fetch failed/i],
-    ignoreErrors: [/fetch failed/i],
-
-    beforeSend(event) {
-      event.tags = {
-        ...event.tags,
-        // Adding additional app_env tag for cross-checking
-        app_env: isProd ? 'production' : vercelEnv || 'development',
-      }
-      return event
-    },
-  })
+	Sentry.init({
+		dsn,
+
+		// Adjust this value in production, or use tracesSampler for greater control
+		tracesSampleRate: isProd ? 0.2 : 1,
+		profilesSampleRate: 0.1,
+		// NOTE: reducing sample only 10% of transactions in prod to get general trends instead of detailed and overfitted data
+
+		// Setting this option to true will print useful information to the console while you're setting up Sentry.
+		debug: false,
+
+		// You can remove this option if you're not planning to use the Sentry Session Replay feature:
+		// NOTE: Since session replay barely helps us anyways, getting rid of it to reduce some bundle size at least
+		// replaysOnErrorSampleRate: 1.0,
+		// replaysSessionSampleRate: 0,
+		integrations: [
+			Sentry.browserTracingIntegration({
+				beforeStartSpan: (e) => {
+					console.info("SentryBrowserTracingSpan", e.name);
+					return e;
+				},
+			}),
+			//   Sentry.replayIntegration({
+			// Additional Replay configuration goes in here, for example:
+			//     maskAllText: true,
+			//     blockAllMedia: true,
+			//   }),
+		],
+
+		// ignoreErrors: [/fetch failed/i],
+		ignoreErrors: [/fetch failed/i],
+
+		beforeSend(event) {
+			if (!isProd && event.type === undefined) {
+				return null;
+			}
+			event.tags = {
+				...event.tags,
+				// Adding additional app_env tag for cross-checking
+				app_env: isProd ? "production" : vercelEnv || "development",
+			};
+			return event;
+		},
+	});
 }

sentry.server.config.ts

@@ -2,22 +2,31 @@
 // The config you add here will be used whenever the server handles a request.
 // https://docs.sentry.io/platforms/javascript/guides/nextjs/
 
-import * as Sentry from '@sentry/nextjs'
+import * as Sentry from "@sentry/nextjs";
 
-const dsn = process.env.NEXT_PUBLIC_SENTRY_DSN || process.env.SENTRY_DSN
+const dsn = process.env.NEXT_PUBLIC_SENTRY_DSN || process.env.SENTRY_DSN;
+const vercelEnv = process.env.NEXT_PUBLIC_VERCEL_ENV;
+const isProd = process.env.NEXT_PUBLIC_VERCEL_ENV === "production";
 
 if (dsn) {
-  Sentry.init({
-    dsn,
+	Sentry.init({
+		dsn,
 
-    // Adjust this value in production, or use tracesSampler for greater control
-    tracesSampleRate: 1,
+		// Adjust this value in production, or use tracesSampler for greater control
+		tracesSampleRate: 1,
 
-    // Setting this option to true will print useful information to the console while you're setting up Sentry.
-    debug: false,
+		// Setting this option to true will print useful information to the console while you're setting up Sentry.
+		debug: false,
 
-    // Uncomment the line below to enable Spotlight (https://spotlightjs.com)
-    // spotlight: process.env.NODE_ENV === 'development',
-    ignoreErrors: [/fetch failed/i],
-  })
+		// Uncomment the line below to enable Spotlight (https://spotlightjs.com)
+		// spotlight: process.env.NODE_ENV === 'development',
+		ignoreErrors: [/fetch failed/i],
+
+		beforeSend(event) {
+			if (!isProd && event.type === undefined) {
+				return null;
+			}
+			return event;
+		},
+	});
 }

src/app/(home)/page.tsx

@@ -74,11 +74,10 @@ export async function getViewSettings(token: string): Promise<CreateViewSettings
   return resp
 }
 
-export default async function Main({
-  searchParams,
-}: {
-  searchParams: { token: string; taskId?: string } & UrlActionParamsType
+export default async function Main(props: {
+  searchParams: Promise<{ token: string; taskId?: string } & UrlActionParamsType>
 }) {
+  const searchParams = await props.searchParams
   const token = searchParams.token
 
   const parsedToken = z.string().safeParse(searchParams.token)

src/app/_fetchers/OneTaskDataFetcher.tsx

@@ -15,7 +15,6 @@
 }
 
 export const OneTaskDataFetcher = ({ token, task_id, initialTask }: OneTaskDataFetcherProps & PropsWithToken) => {
-  const hasDispatchedRef = useRef(false)
   const buildQueryString = (token: string) => {
     const queryParams = new URLSearchParams({ token })
 
@@ -30,7 +29,7 @@
   })
 
   useEffect(() => {
-    if (!hasDispatchedRef.current && data?.task) {
+    if (data?.task) {
       //only invalidate cache on mount.
       const newTask = structuredClone(data.task)
       if (initialTask?.body && newTask.body === undefined) {
@@ -44,9 +43,8 @@
         }
       }
       store.dispatch(setActiveTask(newTask))
-      hasDispatchedRef.current = true
     }
   }, [data])
 
   return null
 }

src/app/api/activity-logs/[id]/route.ts

@@ -4,7 +4,11 @@ import { ActivityLogService } from '@api/activity-logs/services/activity-log.ser
 import { IdParams } from '@api/core/types/api'
 import { unstable_noStore as noStore } from 'next/cache'
 
-export const GET = async (req: NextRequest, { params: { id } }: IdParams) => {
+export const GET = async (req: NextRequest, props: IdParams) => {
+  const params = await props.params
+
+  const { id } = params
+
   noStore()
   const user = await authenticate(req)
 

src/app/api/attachments/attachments.controller.ts

@@ -40,7 +40,8 @@ export const getAttachments = async (req: NextRequest) => {
   return NextResponse.json({ attachments })
 }
 
-export const deleteAttachment = async (req: NextRequest, { params: { id } }: IdParams) => {
+export const deleteAttachment = async (req: NextRequest, { params }: IdParams) => {
+  const { id } = await params
   const user = await authenticate(req)
   const attachmentsService = new AttachmentsService(user)
   await attachmentsService.deleteAttachment(id)

src/app/api/comment/comment.controller.ts

@@ -17,7 +17,8 @@ export const createComment = async (req: NextRequest) => {
   return NextResponse.json({ comment }, { status: httpStatus.CREATED })
 }
 
-export const deleteComment = async (req: NextRequest, { params: { id } }: IdParams) => {
+export const deleteComment = async (req: NextRequest, { params }: IdParams) => {
+  const { id } = await params
   const user = await authenticate(req)
 
   const commentService = new CommentService(user)
@@ -27,7 +28,8 @@ export const deleteComment = async (req: NextRequest, { params: { id } }: IdPara
   return NextResponse.json({ message: 'Comment deleted!' })
 }
 
-export const updateComment = async (req: NextRequest, { params: { id } }: IdParams) => {
+export const updateComment = async (req: NextRequest, { params }: IdParams) => {
+  const { id } = await params
   const user = await authenticate(req)
 
   const data = UpdateCommentSchema.parse(await req.json())

src/app/api/core/types/api.ts

@@ -17,7 +17,5 @@ export enum Resource {
  * NextParam when uuid is being used as a dynamic param (slug) for accessing a resource
  */
 export type IdParams = {
-  params: {
-    id: string
-  }
+  params: Promise<{ id: string }>
 }

src/app/api/core/utils/withRetry.ts

@@ -2,6 +2,8 @@ import { StatusableError } from '@/types/CopilotApiError'
 import pRetry, { FailedAttemptError } from 'p-retry'
 import * as Sentry from '@sentry/nextjs'
 
+export const RETRY_404_ENABLED = process.env.RETRY_404 === 'true'
+
 export const withRetry = async <T>(fn: (...args: any[]) => Promise<T>, args: any[]): Promise<T> => {
   let isEventProcessorRegistered = false
 
@@ -41,7 +43,11 @@ export const withRetry = async <T>(fn: (...args: any[]) => Promise<T>, args: any
         // Typecasting because Copilot doesn't export an error class
         const err = error as StatusableError
         // Retry if statusCode is 429 (ratelimit), 408 (timeouts), or any server related (5xx) error
-        return [408, 429].includes(err.status) || (err.status >= 500 && err.status <= 511)
+        return (
+          [408, 429].includes(err.status) ||
+          (err.status >= 500 && err.status <= 511) ||
+          (RETRY_404_ENABLED && err.status === 404)
+        )
       },
     },
   )

src/app/api/tasks/[id]/activity-logs/activity.controller.ts

@@ -5,7 +5,8 @@ import authenticate from '@api/core/utils/authenticate'
 import httpStatus from 'http-status'
 import { NextRequest, NextResponse } from 'next/server'
 
-export const get = async (req: NextRequest, { params: { id } }: IdParams) => {
+export const get = async (req: NextRequest, { params }: IdParams) => {
+  const { id } = await params
   const user = await authenticate(req)
 
   const { expandComments } = getSearchParams(req.nextUrl.searchParams, ['expandComments'])