build(deps): update latest

.github/workflows/challenges.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: 'ubuntu-latest'
     timeout-minutes: 30
     steps:
-      - uses: 'actions/checkout@v3.0.0'
+      - uses: 'actions/checkout@v3.3.0'
         with:
           fetch-depth: 0
 
@@ -21,7 +21,7 @@ jobs:
           SKIP_LOGIN: true
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v3.0.0'
+        uses: 'actions/setup-node@v3.6.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'

.github/workflows/cli.yml

@@ -10,10 +10,10 @@ jobs:
   lint:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v3.0.0'
+      - uses: 'actions/checkout@v3.3.0'
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v3.0.0'
+        uses: 'actions/setup-node@v3.6.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'
@@ -24,15 +24,15 @@ jobs:
       - run: 'npm run lint:commit -- --to "${{ github.sha }}"'
       - run: 'npm run lint:editorconfig'
       - run: 'npm run lint:markdown'
-      - run: 'npm run lint:typescript'
+      - run: 'npm run lint:eslint'
 
   build:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v3.0.0'
+      - uses: 'actions/checkout@v3.3.0'
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v3.0.0'
+        uses: 'actions/setup-node@v3.6.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'
@@ -43,10 +43,12 @@ jobs:
       - name: 'Build'
         run: 'npm run build'
 
+      - run: 'npm run build:typescript'
+
   test:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v3.0.0'
+      - uses: 'actions/checkout@v3.3.0'
 
       - name: 'Use Docker'
         uses: 'actions-hub/docker/cli@master'
@@ -54,7 +56,7 @@ jobs:
           SKIP_LOGIN: true
 
       - name: 'Use Node.js'
-        uses: 'actions/setup-node@v3.0.0'
+        uses: 'actions/setup-node@v3.6.0'
         with:
           node-version: 'lts/*'
           cache: 'npm'

.markdownlint-cli2.jsonc

@@ -1,11 +1,5 @@
 {
-  "config": {
-    "default": true,
-    "MD013": false,
-    "MD024": false,
-    "MD033": false,
-    "MD041": false
-  },
   "globs": ["**/*.{md,mdx}"],
-  "ignores": ["**/node_modules"]
+  "ignores": ["**/node_modules"],
+  "customRules": ["markdownlint-rule-relative-links"]
 }

.markdownlint.json

@@ -0,0 +1,8 @@
+{
+  "default": true,
+  "relative-links": true,
+  "extends": "markdownlint/style/prettier",
+  "MD024": false,
+  "MD033": false,
+  "MD041": false
+}

cli/commands/generate/challenge.ts

@@ -5,9 +5,9 @@ import chalk from 'chalk'
 import { Challenge } from '../../services/Challenge.js'
 
 export class GenerateChallengeCommand extends Command {
-  static paths = [['generate', 'challenge']]
+  public static override paths = [['generate', 'challenge']]
 
-  static usage = {
+  public static override usage = {
     description: 'Create the basic files needed for a new challenge.'
   }
 
@@ -23,7 +23,7 @@ export class GenerateChallengeCommand extends Command {
     validator: typanion.isString()
   })
 
-  async execute(): Promise<number> {
+  public async execute(): Promise<number> {
     try {
       const challenge = await Challenge.generate({
         name: this.challenge,

cli/commands/generate/solution.ts

@@ -5,9 +5,9 @@ import chalk from 'chalk'
 import { Solution } from '../../services/Solution.js'
 
 export class GenerateSolutionCommand extends Command {
-  static paths = [['generate', 'solution']]
+  public static override paths = [['generate', 'solution']]
 
-  static usage = {
+  public static override usage = {
     description: 'Create the basic files needed for a new solution.'
   }
 
@@ -35,7 +35,7 @@ export class GenerateSolutionCommand extends Command {
     validator: typanion.isString()
   })
 
-  async execute(): Promise<number> {
+  public async execute(): Promise<number> {
     try {
       const solution = await Solution.generate({
         name: this.solutionName,

cli/commands/run/solution.ts

@@ -11,9 +11,9 @@ import { Solution } from '../../services/Solution.js'
 import { TemporaryFolder } from '../../services/TemporaryFolder.js'
 
 export class RunSolutionCommand extends Command {
-  static paths = [['run', 'solution']]
+  public static override paths = [['run', 'solution']]
 
-  static usage = {
+  public static override usage = {
     description: 'Run the solution with the given `input.txt` file.'
   }
 
@@ -45,7 +45,7 @@ export class RunSolutionCommand extends Command {
     description: 'Display the output of the solution.'
   })
 
-  async execute(): Promise<number> {
+  public async execute(): Promise<number> {
     console.log()
     try {
       await TemporaryFolder.cleanAll()

cli/commands/run/test.ts

@@ -10,9 +10,9 @@ import { SolutionTestsResult } from '../../services/SolutionTestsResult.js'
 import { TemporaryFolder } from '../../services/TemporaryFolder.js'
 
 export class RunTestCommand extends Command {
-  static paths = [['run', 'test']]
+  public static override paths = [['run', 'test']]
 
-  static usage = {
+  public static override usage = {
     description:
       'Test if the solution is correct and display where it succeeds and fails.'
   }
@@ -44,7 +44,7 @@ export class RunTestCommand extends Command {
     description: 'Base of the current branch (usually master).'
   })
 
-  async execute(): Promise<number> {
+  public async execute(): Promise<number> {
     console.log()
     try {
       await TemporaryFolder.cleanAll()

cli/commands/search/index.ts

@@ -9,9 +9,9 @@ import { template } from '../../services/Template.js'
 import { Challenge } from '../../services/Challenge.js'
 
 export class SearchCommand extends Command {
-  static paths = [['search']]
+  public static override paths = [['search']]
 
-  static usage = {
+  public static override usage = {
     description: 'Search challenges in the programming language specified.'
   }
 
@@ -26,7 +26,7 @@ export class SearchCommand extends Command {
     validator: typanion.isString()
   })
 
-  async execute(): Promise<number> {
+  public async execute(): Promise<number> {
     try {
       await template.verifySupportedProgrammingLanguage(
         this.programmingLanguage

cli/services/Challenge.ts

@@ -22,9 +22,7 @@ export class Challenge implements ChallengeOptions {
   constructor(options: ChallengeOptions) {
     const { name } = options
     this.name = name
-    this.path = fileURLToPath(
-      new URL(`./${name}`, Challenge.BASE_URL)
-    )
+    this.path = fileURLToPath(new URL(`./${name}`, Challenge.BASE_URL))
   }
 
   public static async getChallenges(): Promise<Challenge[]> {
@@ -34,7 +32,9 @@ export class Challenge implements ChallengeOptions {
     })
   }
 
-  public static async generate(options: GenerateChallengeOptions): Promise<Challenge> {
+  public static async generate(
+    options: GenerateChallengeOptions
+  ): Promise<Challenge> {
     const { name, githubUser } = options
     const challenge = new Challenge({ name })
     if (await isExistingPath(challenge.path)) {

cli/services/GitAffected.ts

@@ -70,6 +70,9 @@ export class GitAffected implements GitAffectedOptions {
       const [, , programmingLanguageName] = filePath
         .replaceAll('\\', '/')
         .split('/')
+      if (programmingLanguageName == null) {
+        throw new Error('programmingLanguageName is null')
+      }
       return programmingLanguageName
     })
     const affectedInputOutput = files.filter((filePath) => {
@@ -78,6 +81,9 @@ export class GitAffected implements GitAffectedOptions {
     const affectedChallengesFromInputOutput = affectedInputOutput.map(
       (filePath) => {
         const [, challengeName] = filePath.replaceAll('\\', '/').split('/')
+        if (challengeName == null) {
+          throw new Error('challengeName is null')
+        }
         return new Challenge({ name: challengeName })
       }
     )

cli/services/Solution.ts

@@ -63,9 +63,7 @@ export class Solution implements SolutionOptions {
     try {
       await docker.build(this.temporaryFolder.id)
     } catch (error: any) {
-      throw new Error(
-        `solution: ${this.path}\n${error.message as string}\n`
-      )
+      throw new Error(`solution: ${this.path}\n${error.message as string}\n`)
     }
   }
 
@@ -90,9 +88,7 @@ export class Solution implements SolutionOptions {
       }
     } catch (error: any) {
       loader.fail()
-      throw new Error(
-        `solution: ${this.path}\n${error.message as string}\n`
-      )
+      throw new Error(`solution: ${this.path}\n${error.message as string}\n`)
     }
   }
 
@@ -199,6 +195,15 @@ export class Solution implements SolutionOptions {
     return solutions.map((solution) => {
       const [, challengeName, , programmingLanguageName, solutionName] =
         solution.replaceAll('\\', '/').split('/')
+
+      if (
+        challengeName == null ||
+        programmingLanguageName == null ||
+        solutionName == null
+      ) {
+        throw new Error(`Invalid solution path: ${solution}`)
+      }
+
       return new Solution({
         challenge: new Challenge({
           name: challengeName

cli/services/SolutionTestsResult.ts

@@ -37,7 +37,8 @@ export class SolutionTestsResult implements SolutionTestsResultOptions {
   }
 
   public print(options: SolutionTestsResultPrintOptions = {}): void {
-    const { shouldPrintBenchmark = false, shouldPrintTableResult = false } = options
+    const { shouldPrintBenchmark = false, shouldPrintTableResult = false } =
+      options
     const name = `${this.solution.challenge.name}/${this.solution.programmingLanguageName}/${this.solution.name}`
     console.log(`${chalk.bold('Name:')} ${name}\n`)
     const tableResult = [

cli/services/Template.ts

@@ -109,7 +109,9 @@ class Template {
 
   public async getProgrammingLanguages(): Promise<string[]> {
     const languages = await fs.promises.readdir(TEMPLATE_SOLUTION_PATH)
-    return languages.filter((language) => {return language !== 'base'})
+    return languages.filter((language) => {
+      return language !== 'base'
+    })
   }
 
   public async verifySupportedProgrammingLanguage(

cli/services/TemporaryFolder.ts

@@ -23,7 +23,9 @@ export class TemporaryFolder {
 
   public static async cleanAll(): Promise<void> {
     try {
-      const temporaryPath = fileURLToPath(new URL('../../temp', import.meta.url))
+      const temporaryPath = fileURLToPath(
+        new URL('../../temp', import.meta.url)
+      )
       await fs.promises.rm(temporaryPath, { recursive: true, force: true })
       await docker.removeImages()
     } catch {}

cli/services/Test.ts

@@ -4,9 +4,7 @@ import { performance } from 'node:perf_hooks'
 
 import type { Solution } from './Solution.js'
 import { docker } from './Docker.js'
-import {
-  SolutionTestsResult
-} from './SolutionTestsResult.js'
+import { SolutionTestsResult } from './SolutionTestsResult.js'
 import { TemporaryFolder } from './TemporaryFolder.js'
 
 export interface InputOutput {
@@ -74,9 +72,7 @@ export class Test implements TestOptions {
     return { input, output }
   }
 
-  static async runManyWithSolutions(
-    solutions: Solution[]
-  ): Promise<number> {
+  static async runManyWithSolutions(solutions: Solution[]): Promise<number> {
     const solutionTestsResultsPromises: Array<Promise<SolutionTestsResult>> = []
     let isSolutionSuccess = true
     for (const solution of solutions) {

package.json

@@ -17,12 +17,13 @@
   "bin": "build/index.js",
   "scripts": {
     "start": "node build/index.js",
-    "build": "rimraf ./build && swc ./cli --out-dir ./build && tsc",
+    "build": "rimraf ./build && swc ./cli --out-dir ./build",
     "build:dev": "swc ./src --out-dir ./build --watch",
+    "build:typescript": "tsc",
     "lint:commit": "commitlint",
     "lint:editorconfig": "editorconfig-checker",
     "lint:markdown": "markdownlint-cli2",
-    "lint:typescript": "eslint \"**/*.{js,jsx,ts,tsx}\"",
+    "lint:eslint": "eslint \"**/*.{js,jsx,ts,tsx}\"",
     "test": "tap"
   },
   "dependencies": {
@@ -38,10 +39,11 @@
     "validate-npm-package-name": "5.0.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "17.3.0",
-    "@commitlint/config-conventional": "17.3.0",
-    "@swc/cli": "0.1.59",
-    "@swc/core": "1.3.24",
+    "@commitlint/cli": "17.4.1",
+    "@commitlint/config-conventional": "17.4.0",
+    "@swc/cli": "0.1.57",
+    "@swc/core": "1.3.25",
+    "@tsconfig/strictest": "1.0.2",
     "@types/date-and-time": "0.13.0",
     "@types/mock-fs": "4.13.1",
     "@types/ms": "0.7.31",
@@ -49,21 +51,22 @@
     "@types/sinon": "10.0.13",
     "@types/tap": "15.0.7",
     "@types/validate-npm-package-name": "4.0.0",
-    "@typescript-eslint/eslint-plugin": "5.47.1",
-    "@typescript-eslint/parser": "5.47.1",
+    "@typescript-eslint/eslint-plugin": "5.48.1",
+    "@typescript-eslint/parser": "5.48.1",
     "editorconfig-checker": "4.0.2",
-    "eslint": "8.30.0",
+    "eslint": "8.31.0",
     "eslint-config-conventions": "6.0.0",
     "eslint-plugin-import": "2.26.0",
     "eslint-plugin-promise": "6.1.1",
     "eslint-plugin-unicorn": "45.0.2",
     "get-stream": "6.0.1",
-    "markdownlint-cli2": "0.5.1",
+    "markdownlint-cli2": "0.6.0",
+    "markdownlint-rule-relative-links": "1.1.1",
     "mock-fs": "5.2.0",
     "ms": "2.1.3",
     "rimraf": "3.0.2",
     "sinon": "15.0.1",
-    "tap": "16.3.2",
+    "tap": "16.3.3",
     "typescript": "4.9.4"
   }
 }

templates/docker/typescript/Dockerfile

@@ -1,9 +1,9 @@
-FROM node:18.12.1 AS builder-dependencies
+FROM node:18.13.0 AS builder-dependencies
 WORKDIR /usr/src/application
 COPY ./package*.json ./
 RUN npm install
 
-FROM node:18.12.1 AS builder
+FROM node:18.13.0 AS builder
 WORKDIR /usr/src/application
 COPY --from=builder-dependencies /usr/src/application/node_modules ./node_modules
 COPY ./ ./

tsconfig.json

@@ -1,4 +1,5 @@
 {
+  "extends": "@tsconfig/strictest/tsconfig.json",
   "compilerOptions": {
     "target": "ESNext",
     "module": "ESNext",
@@ -7,9 +8,8 @@
     "outDir": "./build",
     "rootDir": "./cli",
     "noEmit": true,
-    "strict": true,
-    "skipLibCheck": true,
-    "esModuleInterop": true
+    "checkJs": false,
+    "exactOptionalPropertyTypes": false
   },
   "exclude": ["node_modules", "challenges", "templates", "temp", "tmp"]
 }