refactor: replaces commander with native parseArgs (#215)

## Description

- replaces commander with native parseArgs
- removes the cli main entry from the `bin` folder to `src` (and to
`dist` after running typestript)

## Motivation and Context

commander is a _great_ command line parser parser. It has been a good
fit for the state-machine-cat cli. `parseArgs`, which is part of
`node:utils`, is suitable for simple cli needs. And simple cli needas
are what state-machine-cat has. Using parseArgs also reduces
state-machine-cat('s cli's) dependencies, which means less to maintain &
worry about.

## How Has This Been Tested?

- [x] automated non-regression tests
- [x] green ci

## Types of changes

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Documentation only change
- [x] Refactor (non-breaking change which fixes an issue without
changing functionality)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)

## Checklist

- [x] 📖

  - My change doesn't require a documentation update, or ...
  - it _does_ and I have updated it

- [x] ⚖️
- The contribution will be subject to [The MIT license](../LICENSE), and
I'm OK with that.
  - The contribution is my own original work.
  - I am ok with the stuff in [**CONTRIBUTING.md**](./CONTRIBUTING.md).

Author: sverweij

.c8rc.json

@@ -11,14 +11,14 @@
     ".mocharc.js",
     "**/*.d.ts",
     "**/*.d.mts",
-    "bin/*",
     "coverage/**/*",
     "dist/**/*",
     "doc/**/*",
     "docs/**/*",
     "public/**/*",
     "src/**/*-parser.mjs",
-    "src/**/*.template.js"
+    "src/**/*.template.js",
+    "src/cli/main.mts"
   ],
   "reporter": ["text-summary", "lcov", "html", "json-summary"]
 }

.github/codeql/codeql-config.yml

@@ -1,21 +1,20 @@
-ame: 'CodeQL config'
+ame: "CodeQL config"
 
 # See https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
 
 paths:
   - src/
-  - bin/
   - tools/
   - docs/sitesrc/
   - docs/smcat-online-interpreter.js
   - docs/state-machine-cat-inpage.js
   - docs/index.html
   - docs/inpage.html
 paths-ignore:
-  - '**/*.template.js'
-  - '**/*-parser.mjs'
+  - "**/*.template.js"
+  - "**/*-parser.mjs"
 queries:
   - uses: security-and-quality
 query-filters:
   - exclude:
-      id: js/useless-assignment-to-local
+      id: js/useless-assignment-to-local

.github/workflows/codeql-analysis.yml

@@ -8,7 +8,6 @@ on:
       - main
     paths:
       - src/
-      - bin/
       - tools/
       - docs/sitesrc/
       - docs/smcat-online-interpreter.js
@@ -20,7 +19,6 @@ on:
       - main
     paths:
       - src/
-      - bin/
       - tools/
       - docs/sitesrc/
       - docs/smcat-online-interpreter.js

bin/smcat.mjs

@@ -6,10 +6,10 @@ export default {
   options: {
     ...baseConfig.options,
     cache: "node_modules/.cache/dependency-cruiser/graph",
-    includeOnly: "^(bin|src)/",
+    includeOnly: "^src/",
     reporterOptions: {
       archi: {
-        collapsePattern: "^(bin|src/(cli|transform|[^/]+/[^/]+))",
+        collapsePattern: "^(src/(cli|transform|[^/]+/[^/]+))",
       },
       dot: {
         theme: {

config/dependency-cruiser/graph.mjs

@@ -7,7 +7,7 @@ export default {
     // This configuration ensures dependency-cruiser will skip these
     // rules
     { name: "no-orphans", severity: "ignore" },
-    { name: "no-unreachable-from-bin", severity: "ignore" },
+    { name: "no-unreachable-from-cli", severity: "ignore" },
     { name: "no-unreachable-from-api", severity: "ignore" },
     { name: "no-uncovered-by-tests", severity: "ignore" },
   ],

config/dependency-cruiser/lint-staged.mjs

@@ -58,7 +58,6 @@ export default {
       from: {},
       to: {
         dependencyTypes: ["deprecated"],
-        pathNot: "node_modules/viz\\.js/viz\\.js$",
       },
     },
     {
@@ -67,7 +66,6 @@ export default {
       to: {
         moreThanOneDependencyType: true,
         dependencyTypesNot: ["type-only"],
-        pathNot: "node_modules/viz\\.js/viz\\.js$",
       },
     },
     {
@@ -106,7 +104,7 @@ export default {
         "or when you want to run on the web - which state-machine-cat actually does).",
       severity: "error",
       from: {
-        pathNot: "^(bin|src/cli|test|tools)",
+        pathNot: "^(src/cli|test|tools)",
       },
       to: {
         dependencyTypes: ["core"],
@@ -158,25 +156,25 @@ export default {
     {
       name: "no-deps-on-cli",
       comment:
-        "This module, that's neither in bin/ nor in cli/ nor is a test for either, depends on " +
-        "cli/ and/ or bin/ code. As bin and cli are there to call other modules and not the other " +
+        "This module, that's not in cli/ nor is a test for it, depends on " +
+        "cli/ code. As cli is there to call other modules and not the other " +
         "way 'round, you're likely putting this module in the wrong spot.",
       severity: "error",
       from: {
-        pathNot: "^src/cli|^bin|^test/cli",
+        pathNot: "^src/cli|^test/cli",
       },
       to: {
-        path: "^src/cli|^bin",
+        path: "^src/cli",
       },
     },
     {
-      name: "no-unreachable-from-bin",
+      name: "no-unreachable-from-cli",
       comment:
         "This module is not reachable from the the command line interface. This means it's likely " +
         "'dead wood'. Either remove it, or start using it.",
       severity: "error",
       from: {
-        path: "^bin/smcat$",
+        path: "^src/cli/main.mts$",
       },
       to: {
         path: "^src/",
@@ -221,7 +219,7 @@ export default {
       to: {
         path: "^src/",
         reachable: false,
-        pathNot: ["\\.d\\.(c|m)?ts$"],
+        pathNot: ["\\.d\\.(c|m)?ts$", "^src/cli/main.mts$"],
       },
     },
     {