build: updated tsconfig, fixed related errors

Author: matteobruni

src/build/build-distfiles.ts

@@ -25,12 +25,14 @@ export async function buildDistFiles(basePath: string): Promise<boolean> {
             text = data.toString(),
             libObj = JSON.parse(text) as Record<string, unknown>;
 
-        libObj.version = pkgInfo.version;
+        libObj["version"] = pkgInfo.version;
 
         if (pkgInfo.dependencies) {
-            libObj.dependencies = JSON.parse(JSON.stringify(pkgInfo.dependencies).replaceAll("workspace:", ""));
+            libObj["dependencies"] = JSON.parse(JSON.stringify(pkgInfo.dependencies).replaceAll("workspace:", ""));
         } else if (pkgInfo.peerDependencies) {
-            libObj.peerDependencies = JSON.parse(JSON.stringify(pkgInfo.peerDependencies).replaceAll("workspace:", ""));
+            libObj["peerDependencies"] = JSON.parse(
+                JSON.stringify(pkgInfo.peerDependencies).replaceAll("workspace:", ""),
+            );
         }
 
         const jsonIndent = 2;

src/build/build-tsc.ts

@@ -22,6 +22,8 @@ async function readConfig(basePath: string, file: string): Promise<string | unde
 
         return data.toString();
     }
+
+    return undefined;
 }
 
 /**

src/build/build.ts

@@ -24,15 +24,17 @@ buildCommand.option("-t, --tsc", "Build the library using TypeScript", false);
 buildCommand.argument("[path]", `Path to the project root folder, default is "src"`, "src");
 buildCommand.action(async (argPath: string) => {
     const opts = buildCommand.opts(),
-        ci = !!opts.ci,
-        all = !!opts.all || (!opts.bundle && !opts.clean && !opts.dist && !opts.lint && !opts.prettify && !opts.tsc),
-        doBundle = all || !!opts.bundle,
-        circularDeps = all || !!opts.circularDeps,
-        clean = all || !!opts.clean,
-        distfiles = all || !!opts.dist,
-        doLint = all || !!opts.lint,
-        prettier = all || !!opts.prettify,
-        tsc = all || !!opts.tsc;
+        ci = !!opts["ci"],
+        all =
+            !!opts["all"] ||
+            (!opts["bundle"] && !opts["clean"] && !opts["dist"] && !opts["lint"] && !opts["prettify"] && !opts["tsc"]),
+        doBundle = all || !!opts["bundle"],
+        circularDeps = all || !!opts["circularDeps"],
+        clean = all || !!opts["clean"],
+        distfiles = all || !!opts["dist"],
+        doLint = all || !!opts["lint"],
+        prettier = all || !!opts["prettify"],
+        tsc = all || !!opts["tsc"];
 
     const basePath = process.cwd(),
         { getDistStats } = await import("./build-diststats.js"),

src/tsconfig.json

@@ -97,24 +97,24 @@
         /* Ensure that casing is correct in imports. */
 
         /* Type Checking */
-        "strict": true
+        "strict": true,
         /* Enable all strict type-checking options. */
-        // "noImplicitAny": true,                            /* Enable error reporting for expressions and declarations with an implied 'any' type. */
-        // "strictNullChecks": true,                         /* When type checking, take into account 'null' and 'undefined'. */
-        // "strictFunctionTypes": true,                      /* When assigning functions, check to ensure parameters and the return values are subtype-compatible. */
-        // "strictBindCallApply": true,                      /* Check that the arguments for 'bind', 'call', and 'apply' methods match the original function. */
-        // "strictPropertyInitialization": true,             /* Check for class properties that are declared but not set in the constructor. */
-        // "noImplicitThis": true,                           /* Enable error reporting when 'this' is given the type 'any'. */
-        // "useUnknownInCatchVariables": true,               /* Default catch clause variables as 'unknown' instead of 'any'. */
-        // "alwaysStrict": true,                             /* Ensure 'use strict' is always emitted. */
-        // "noUnusedLocals": true,                           /* Enable error reporting when local variables aren't read. */
-        // "noUnusedParameters": true,                       /* Raise an error when a function parameter isn't read. */
-        // "exactOptionalPropertyTypes": true,               /* Interpret optional property types as written, rather than adding 'undefined'. */
-        // "noImplicitReturns": true,                        /* Enable error reporting for codepaths that do not explicitly return in a function. */
-        // "noFallthroughCasesInSwitch": true,               /* Enable error reporting for fallthrough cases in switch statements. */
-        // "noUncheckedIndexedAccess": true,                 /* Add 'undefined' to a type when accessed using an index. */
-        // "noImplicitOverride": true,                       /* Ensure overriding members in derived classes are marked with an override modifier. */
-        // "noPropertyAccessFromIndexSignature": true,       /* Enforces using indexed accessors for keys declared using an indexed type. */
+        "noImplicitAny": true /* Enable error reporting for expressions and declarations with an implied 'any' type. */,
+        "strictNullChecks": true /* When type checking, take into account 'null' and 'undefined'. */,
+        "strictFunctionTypes": true /* When assigning functions, check to ensure parameters and the return values are subtype-compatible. */,
+        "strictBindCallApply": true /* Check that the arguments for 'bind', 'call', and 'apply' methods match the original function. */,
+        "strictPropertyInitialization": true /* Check for class properties that are declared but not set in the constructor. */,
+        "noImplicitThis": true /* Enable error reporting when 'this' is given the type 'any'. */,
+        "useUnknownInCatchVariables": true /* Default catch clause variables as 'unknown' instead of 'any'. */,
+        "alwaysStrict": true /* Ensure 'use strict' is always emitted. */,
+        "noUnusedLocals": true /* Enable error reporting when local variables aren't read. */,
+        "noUnusedParameters": true /* Raise an error when a function parameter isn't read. */,
+        "exactOptionalPropertyTypes": true /* Interpret optional property types as written, rather than adding 'undefined'. */,
+        "noImplicitReturns": true /* Enable error reporting for codepaths that do not explicitly return in a function. */,
+        "noFallthroughCasesInSwitch": true /* Enable error reporting for fallthrough cases in switch statements. */,
+        "noUncheckedIndexedAccess": true /* Add 'undefined' to a type when accessed using an index. */,
+        "noImplicitOverride": true /* Ensure overriding members in derived classes are marked with an override modifier. */,
+        "noPropertyAccessFromIndexSignature": true /* Enforces using indexed accessors for keys declared using an indexed type. */
         // "allowUnusedLabels": true,                        /* Disable error reporting for unused labels. */
         // "allowUnreachableCode": true,                     /* Disable error reporting for unreachable code. */
 

src/utils/string-utils.ts

@@ -34,7 +34,7 @@ export function camelize(str: string, ...splits: string[]): string {
  */
 export function dash(str: string): string {
     const index = 0,
-        dashed = str.replace(/([A-Z])/g, g => `-${g[index].toLowerCase()}`),
+        dashed = str.replace(/([A-Z])/g, g => `-${g[index]?.toLowerCase() ?? ""}`),
         startPos = 1;
 
     return dashed.startsWith("-") ? dashed.substring(startPos) : dashed;