add strict TS checking + fix some initial errors

Author: Sayeem2004

package-lock.json

@@ -13,6 +13,7 @@
     "license": "ISC",
     "devDependencies": {
         "@11ty/eleventy": "^3.1.2",
+        "@types/js-yaml": "^4.0.9",
         "@types/node": "^25.2.3",
         "concurrently": "^9.2.1",
         "typescript": "^5.9.3"

package.json

@@ -33,7 +33,7 @@ into an array of single characters, each representing one bit.
 
 Returns null if the string isn’t 16 or 32 bits long.
 */
-function parseMatchBits(matchStr) {
+function parseMatchBits(matchStr: string) {
     if (matchStr.length !== 16 && matchStr.length !== 32) {
         throw new Error(`Invalid match string length ${matchStr.length}; expected 16 or 32`);
     }
@@ -160,7 +160,7 @@ function detectEncodingType(doc) {
     const vars = Object.fromEntries(
         doc.encoding.variables.map((v) => [v.name, String(v.location)]),
     );
-    const eq = (name, hi, lo) => vars[name] === hi + "-" + lo;
+    const eq = (name: string, hi: number, lo: number) => vars[name] === hi + "-" + lo;
     if (eq("xd", 11, 7) && eq("xs1", 19, 15) && vars["imm"] === "31-20") return "I";
     if (eq("xd", 11, 7) && eq("xs1", 19, 15) && eq("xs2", 24, 20)) return "R";
     if (eq("xs2", 24, 20) && eq("xs1", 19, 15) && eq("imm", 11, 7)) return "S";
@@ -279,7 +279,7 @@ export default function loadInstructions() {
             if (doc.encoding && doc.encoding.match) {
                 const m = parseMatchBits(doc.encoding.match);
                 if (m) {
-                    const sliceBits = (hi, lo) => m.slice(31 - hi, 32 - lo).join("");
+                    const sliceBits = (hi: number, lo: number) => m.slice(31 - hi, 32 - lo).join("");
                     opcode = sliceBits(6, 0);
                     funct3 = sliceBits(14, 12);
                     const f7 = sliceBits(31, 25);

src/lib/loadInstructions.ts

@@ -55,7 +55,7 @@ function applyFilters() {
         /* Check if the item matches an active extension filter
             - If no filters are active, match everything
             - Otherwise, show only if its extension is selected */
-        const matchesExtension = !filtersActive || activeExtensions.has(li.dataset.extension);
+        const matchesExtension = !filtersActive || activeExtensions.has(li.dataset.extension || "");
 
         // Show or hide this <li> depending on whether both match conditions are true
         li.style.display = matchesQuery && matchesExtension ? "" : "none";
@@ -77,13 +77,13 @@ input.addEventListener("keydown", (e) => {
     const exact = items.find((li) => {
         if (li.dataset.mnemonic !== q) return false;
         if (!filtersActive) return true;
-        return activeExtensions.has(li.dataset.extension);
+        return activeExtensions.has(li.dataset.extension || "");
     });
 
     // If an exact match is found, redirect to that instruction's detail page
     if (exact) {
-        const link = exact.querySelector("a").getAttribute("href");
-        location.href = link;
+        const link = exact.querySelector("a")?.getAttribute("href");
+        if (link) location.href = link;
     }
 });
 

src/scripts/search.ts

@@ -5,6 +5,7 @@
         "moduleResolution": "NodeNext",
         "outDir": "src",
         "rootDir": "src",
+        "strict": true,
         "allowJs": true,
         "esModuleInterop": true,
         "skipLibCheck": true,