fix(backend): Fixes typecheck issues

.github/pull_request_template.md

@@ -43,10 +43,10 @@ Closes #
 
 ## Checklist
 
-- [ ] My code follows the project's coding style (`pnpm -r run lint` passes).
-- [ ] TypeScript compiles without errors (`pnpm -r run typecheck`).
+- [ ] My code follows the project's coding style (`npm run lint` passes).
+- [ ] TypeScript compiles without errors (`npm run typecheck --workspaces --if-present`).
 - [ ] I have added or updated tests for the changes I made.
-- [ ] All tests pass locally (`pnpm -r run test`).
+- [ ] All tests pass locally (`npm run test --workspaces --if-present`).
 - [ ] I have updated documentation where necessary.
 - [ ] No new `console.log` or debug statements left in the code.
 - [ ] Breaking changes are documented in this PR description.

.github/scripts/ciScript.js

@@ -1,3 +1,19 @@
+const isTestFile = (file) => /\.(test|spec)\.[jt]sx?$/.test(file);
+
+const deriveTestFiles = (files) => {
+  return files.map((file) => {
+    if (isTestFile(file)) return file;
+
+    const withoutExt = file.replace(/\.[jt]sx?$/, '');
+    const parts = withoutExt.split('/');
+    const baseName = parts[parts.length - 1];
+    const dir = parts.slice(0, -1).join('/');
+
+    return `${dir}/__tests__/${baseName}.test.ts`;
+  });
+};
+
+
 module.exports = async ({ github, context, core }) => {
   const owner = context.repo.owner;
   const repo = context.repo.repo;
@@ -6,9 +22,9 @@ module.exports = async ({ github, context, core }) => {
   const prState = pr.state;
 
   const backendFiles = [];
-  const backendTests = [];
   const mobileFiles = [];
   const webFiles = [];
+  const dbFiles = [];
 
   try {
     if (prState === 'closed') {
@@ -34,60 +50,31 @@ module.exports = async ({ github, context, core }) => {
 
       if (fileName.startsWith('apps/backend/')) {
         backendFiles.push(fileName);
-
-        const relative = fileName.replace('apps/backend/src/', '');
-        const baseName = relative
-          .split('/')
-          .pop()
-          ?.replace(/\.(ts|tsx|js|jsx)$/, '');
-
-        if (baseName) {
-          backendTests.push(`src/__tests__/${baseName}.test.ts`);
-        }
-
       } else if (fileName.startsWith('apps/mobile/')) {
         mobileFiles.push(fileName);
       } else if (fileName.startsWith('apps/web/')) {
         webFiles.push(fileName);
+      }else if(fileName.startsWith('apps/backend/prisma')){
+        dbFiles.push(fileName)
+      }else if(fileName.includes('schema.prisma') || fileName.includes('/migrations/')){
+        dbFiles.push(fileName)
       }
     });
 
-    console.log({
-      backendFiles,
-      backendTests,
-      mobileFiles,
-      webFiles, 
-    });
-
-    core.setOutput(
-      "backendFiles",
-      backendFiles
-        .map(file => file.replace("apps/backend/", ""))
-        .join(" ")
-    );
-
-    core.setOutput(
-      "backendTests",
-      [...new Set(backendTests)].join(" ")
-    );
-
-    core.setOutput(
-      "mobileFiles",
-      mobileFiles
-        .map(file => file.replace("apps/mobile/", ""))
-        .join(" ")
-    );
+    const strippedBackend = backendFiles.map(f => f.replace('apps/backend/', ''));
+    const strippedMobile  = mobileFiles.map(f => f.replace('apps/mobile/', ''));
 
-    core.setOutput(
-      "webFiles",
-      webFiles
-        .map(file => file.replace("apps/web/", ""))
-        .join(" ")
-    );
+    console.log({ backendFiles, mobileFiles, webFiles, dbFiles });
 
-    core.setOutput("backendChanged", backendFiles.length > 0);
-    core.setOutput("mobileChanged", mobileFiles.length > 0);
-    core.setOutput("webChanged", webFiles.length > 0);
+    core.setOutput('backendFiles',     strippedBackend.join(' '));
+    core.setOutput('mobileFiles',      strippedMobile.join(' '));
+    core.setOutput('dbFiles', dbFiles.join(' '));
+    core.setOutput('webFiles',         webFiles.map(f => f.replace('apps/web/', '')).join(' '));
+    core.setOutput('backendTestFiles', deriveTestFiles(strippedBackend).join(' '));
+    core.setOutput('mobileTestFiles',  deriveTestFiles(strippedMobile).join(' '));
+    core.setOutput('backendChanged',   backendFiles.length > 0);
+    core.setOutput('mobileChanged',    mobileFiles.length > 0);
+    core.setOutput('webChanged',       webFiles.length > 0);
 
   } catch (error) {
     console.error(error);

.github/scripts/commentResults.js

@@ -9,62 +9,59 @@ module.exports = async ({
   backendTypecheck,
   mobileLint,
   mobileTest,
-  webCheck,
-  webBuild
+  webBuild,
+  backendLintOutput,
+  mobileLintOutput,
 }) => {
   const owner = context.repo.owner;
   const repo = context.repo.repo;
   const prNumber = context.payload.pull_request.number;
 
-  const emoji = (status) => {
-    if (status === 'success') return '✅';
-    if (status === 'failure') return '❌';
-    if (status === 'skipped') return '⏭️';
-    return '⚪';
+  const status = (s) => {
+    if (s === 'success') return 'PASS';
+    if (s === 'failure') return 'FAIL';
+    if (s === 'skipped') return 'SKIP';
+    return '-';
   };
 
-  const label = (status) => {
-    if (!status) return '⚪ unknown';
-    return `${emoji(status)} ${status}`;
+  const lintDetails = (output) => {
+    if (!output || !output.trim()) return '';
+    return `\n<details>\n<summary>View lint errors</summary>\n\n\`\`\`\n${output.trim()}\n\`\`\`\n</details>`;
   };
 
-  const anyFailure = [
-    backend,
-    mobile,
-    web
-  ].includes('failure');
-
-  const title = anyFailure
-    ? '❌ Some checks failed'
-    : '✅ CI completed';
-
+  const anyFailure = [backend, mobile, web].includes('failure');
+  const title = anyFailure ? 'CI — Checks Failed' : 'CI — All Checks Passed';
   const timestamp = new Date().toUTCString();
 
-  const body = `## CI Results — ${title}
+  const body = `## ${title}
+
+### Backend — ${status(backend)}
 
-### 🖥️ Backend (${label(backend)})
-| Check | Status |
+| Check | Result |
 |---|---|
-| Lint | ${label(backendLint)} |
-| Test | ${label(backendTest)} |
-| Typecheck | ${label(backendTypecheck)} |
+| Lint | ${status(backendLint)} |
+| Test | ${status(backendTest)} |
+| Typecheck | ${status(backendTypecheck)} |
+${backendLint === 'failure' ? lintDetails(backendLintOutput) : ''}
 
-### 📱 Mobile (${label(mobile)})
-| Check | Status |
+### Mobile — ${status(mobile)}
+
+| Check | Result |
 |---|---|
-| Lint | ${label(mobileLint)} |
-| Test | ${label(mobileTest)} |
+| Lint | ${status(mobileLint)} |
+| Test | ${status(mobileTest)} |
+${mobileLint === 'failure' ? lintDetails(mobileLintOutput) : ''}
+
+### Web — ${status(web)}
 
-### 🌐 Web (${label(web)})
-| Check | Status |
+| Check | Result |
 |---|---|
-| Check | ${label(webCheck)} |
-| Build | ${label(webBuild)} |
+| Build | ${status(webBuild)} |
 
 ---
-🕐 Last updated: \`${timestamp}\``;
+Last updated: \`${timestamp}\``;
 
-  const COMMENT_MARKER = '## CI Results —';
+  const COMMENT_MARKER = '## CI —';
 
   try {
     const comments = await github.paginate(

.github/scripts/discordPinReminder.js

@@ -3,35 +3,36 @@ module.exports = async ({ github, context }) => {
   const ignoreUsers = [
     'ShantKhatri',
     'Harxhit',
-    'blankirigaya'
-  ]
+    'blankirigaya',
+  ];
+
   try {
-      // Only continue if merged
-      if (!pr || !pr.merged) {  
-        console.log('PR not merged.');
-        return;
-      }
-
-      const prNumber = pr.number;
-      const contributor = pr.user.login;
+    if (!pr || !pr.merged) {
+      console.log('PR not merged.');
+      return;
+    }
+
+    const prNumber = pr.number;
+    const contributor = pr.user.login;
+
+    if (ignoreUsers.includes(contributor)) {
+      console.log(`Ignoring PR #${prNumber} by ${contributor}`);
+      return;
+    }
+
+    await github.rest.issues.createComment({
+      owner: context.repo.owner,
+      repo: context.repo.repo,
+      issue_number: prNumber,
+      body: `Congratulations @${contributor} on getting PR #${prNumber} merged!
+
+Thank you for your contribution to the project.
 
-      if(ignoreUsers.includes(contributor)){
-        console.log(`Ignoring PR #${prNumber} by ${contributor}`);
-        return; 
-      }
-
-      await github.rest.issues.createComment({
-        owner: context.repo.owner,
-        repo: context.repo.repo,
-        issue_number: prNumber,
-        body: `Congratulations @${contributor} on getting PR #${prNumber} merged! 
+To receive the appropriate GSSoC labels and recognition, please mention @Harxhit in the **#get-labels** channel on our Discord server and share your merged PR link.`,
+    });
 
-        Thank you for your contribution. Please mention @Harxhit in our Discord server to receive the appropriate GSSoC labels and recognition.
-        `
-      });
-
-      console.log(`Comment added to PR #${prNumber}`);
+    console.log(`Comment added to PR #${prNumber}`);
   } catch (error) {
-    console.error(error)
+    console.error(error);
   }
-};
+};