let there be lint

.eslintrc.cjs

@@ -0,0 +1,11 @@
+/* eslint-env node */
+module.exports = {
+  extends: ["eslint:recommended", "plugin:@typescript-eslint/recommended"],
+  parser: "@typescript-eslint/parser",
+  plugins: ["@typescript-eslint"],
+  root: true,
+  ignorePatterns: ["dist/"],
+  rules: {
+    "@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_" }],
+  },
+};

.github/workflows/validate.yml

@@ -19,10 +19,13 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: "18"
-      - name: TypeCheck and Test
-        shell: bash
-        run: |
-          npm install
-          npm run gen
-          npm run tsc
-          npm test
+      - name: npm install
+        run: npm install
+      - name: Generate client
+        run: npm run gen
+      - name: TypeCheck
+        run: npm run tsc
+      - name: Lint
+        run: npx eslint .
+      - name: Test
+        run: npm test

generator/client/base.ts

@@ -7,7 +7,7 @@
  */
 
 import { OpenAPIV3 } from "openapi-types";
-import { snakeToPascal, topologicalSort } from "../util";
+import { topologicalSort } from "../util";
 import { IO } from "../io";
 import { Schema } from "../schema/base";
 import { OpenAPIV3 as O } from "openapi-types";
@@ -76,7 +76,7 @@ export function iterPathConfig(paths: OpenAPIV3.Document["paths"]) {
           path,
           conf,
           method,
-          opId: conf?.operationId!,
+          opId: conf!.operationId!,
         };
       });
     })

generator/client/msw-handlers.ts

@@ -214,7 +214,7 @@ export function generateMSWHandlers(spec: OpenAPIV3.Document) {
       bodyTypeRef && (method === "post" || method === "put")
         ? `schema.${refToSchemaName(bodyTypeRef)}`
         : "null";
-    const paramSchema = !!conf.parameters?.length
+    const paramSchema = conf.parameters?.length
       ? `schema.${snakeToPascal(opId)}Params`
       : "null";
 

generator/util.ts

@@ -29,7 +29,7 @@ export const processParamName = (s: string) => snakeToCamel(renameParam(s));
 
 /** `{project_name}` -> `${projectName}`. if no brackets, leave it alone */
 const segmentToInterpolation = (s: string) =>
-  s.startsWith("{") ? `\$\{path.${processParamName(s.slice(1, -1))}\}` : s;
+  s.startsWith("{") ? `$\{path.${processParamName(s.slice(1, -1))}}` : s;
 
 export const pathToTemplateStr = (s: string) =>
   "`" + s.split("/").map(segmentToInterpolation).join("/") + "`";