Added linter

Author: Benji377

.github/workflows/test.yml

@@ -7,10 +7,44 @@ on:
     branches: [main]
 
 jobs:
+  lint:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      security-events: write
+      actions: read
+    steps:
+      - uses: actions/checkout@v6
+
+      - uses: actions/setup-node@v6
+        with:
+          node-version: 20
+          cache: npm
+
+      - run: npm ci
+
+      - name: Install SARIF formatter
+        run: npm install --no-save @microsoft/eslint-formatter-sarif
+
+      - name: Run ESLint
+        run: |
+          npx eslint packages/*/src \
+            --format @microsoft/eslint-formatter-sarif \
+            --output-file eslint-results.sarif
+        continue-on-error: true
+
+      - name: Upload SARIF results
+        uses: github/codeql-action/upload-sarif@v3
+        with:
+          sarif_file: eslint-results.sarif
+          category: eslint
+
   test:
     runs-on: ubuntu-latest
     permissions:
       contents: read
+      checks: write
+      pull-requests: write
     strategy:
       matrix:
         node-version: [18, 20, 22]
@@ -25,3 +59,12 @@ jobs:
       - run: npm ci
       - run: npm run build
       - run: npm test
+
+      - name: Test Report
+        uses: dorny/test-reporter@v1
+        if: success() || failure()
+        with:
+          name: Test Results (Node ${{ matrix.node-version }})
+          path: packages/core/test-results.xml
+          reporter: java-junit
+          fail-on-error: false

.gitignore

@@ -22,6 +22,9 @@ lib-cov
 coverage
 *.lcov
 
+# Test results
+test-results.xml
+
 # nyc test coverage
 .nyc_output
 

eslint.config.mjs

@@ -0,0 +1,18 @@
+import eslint from "@eslint/js";
+import tseslint from "typescript-eslint";
+
+export default tseslint.config(
+  eslint.configs.recommended,
+  ...tseslint.configs.recommendedTypeChecked,
+  {
+    languageOptions: {
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+  },
+  {
+    ignores: ["**/dist/**", "**/node_modules/**", "**/*.config.*"],
+  }
+);

package.json

@@ -12,6 +12,10 @@
     "dev:web": "npm run dev --workspace=packages/web"
   },
   "devDependencies": {
-    "typescript": "^5.7.0"
+    "@eslint/js": "^9.17.0",
+    "@types/eslint__js": "^8.42.3",
+    "eslint": "^9.17.0",
+    "typescript": "^5.7.0",
+    "typescript-eslint": "^8.18.2"
   }
 }

packages/core/package.json

@@ -43,8 +43,8 @@
     "upng-js": "^2.1.0"
   },
   "devDependencies": {
+    "@types/pako": "^2.0.3",
     "tsup": "^8.0.0",
-    "vitest": "^4.0.0",
-    "@types/pako": "^2.0.3"
+    "vitest": "^4.0.0"
   }
 }

packages/core/vitest.config.ts

@@ -3,5 +3,9 @@ import { defineConfig } from "vitest/config";
 export default defineConfig({
   test: {
     include: ["tests/**/*.test.ts"],
+    reporters: ["default", "junit"],
+    outputFile: {
+      junit: "./test-results.xml",
+    },
   },
 });

packages/web/src/main.ts

@@ -138,27 +138,27 @@ function renderFileList() {
   for (const { file, mode } of queue) {
     const li = document.createElement("li");
     li.className = "file-item";
-    li.innerHTML = `
-      <span class="file-item-name" title="${escapeHtml(file.name)}">${escapeHtml(file.name)}</span>
-      <span class="file-item-size">${formatSize(file.size)}</span>
-      <span class="file-item-badge ${mode === "encode" ? "badge-encode" : "badge-decode"}">${mode.toUpperCase()}</span>
-    `;
+    
+    const nameSpan = document.createElement("span");
+    nameSpan.className = "file-item-name";
+    nameSpan.title = file.name;
+    nameSpan.textContent = file.name;
+    
+    const sizeSpan = document.createElement("span");
+    sizeSpan.className = "file-item-size";
+    sizeSpan.textContent = formatSize(file.size);
+    
+    const badge = document.createElement("span");
+    badge.className = `file-item-badge ${mode === "encode" ? "badge-encode" : "badge-decode"}`;
+    badge.textContent = mode.toUpperCase();
+    
+    li.appendChild(nameSpan);
+    li.appendChild(sizeSpan);
+    li.appendChild(badge);
     fileItems.appendChild(li);
   }
 }
 
-function escapeHtml(text: string): string {
-  const map: Record<string, string> = {
-    '&': '&amp;',
-    '<': '&lt;',
-    '>': '&gt;',
-    '"': '&quot;',
-    "'": '&#x27;',
-    '/': '&#x2F;'
-  };
-  return text.replace(/[&<>"'\/]/g, (char) => map[char]);
-}
-
 clearBtn.addEventListener("click", () => {
   queue = [];
   renderFileList();
@@ -257,12 +257,23 @@ function addResultItem(name: string, data: Uint8Array, previewUrl?: string) {
 function addResultError(name: string, message: string) {
   const li = document.createElement("li");
   li.className = "result-item";
-  li.innerHTML = `
-    <div class="result-item-info">
-      <span class="result-item-name" title="${escapeHtml(name)}">${escapeHtml(name)}</span>
-    </div>
-    <span class="result-item-error" title="${escapeHtml(message)}">Error</span>
-  `;
+  
+  const info = document.createElement("div");
+  info.className = "result-item-info";
+  
+  const nameSpan = document.createElement("span");
+  nameSpan.className = "result-item-name";
+  nameSpan.title = name;
+  nameSpan.textContent = name;
+  info.appendChild(nameSpan);
+  
+  const errorSpan = document.createElement("span");
+  errorSpan.className = "result-item-error";
+  errorSpan.title = message;
+  errorSpan.textContent = "Error";
+  
+  li.appendChild(info);
+  li.appendChild(errorSpan);
   resultItems.appendChild(li);
 }