Merge pull request #15 from Teslemetry/fm/o10-e101-s7n-nodelib-ci

ci: add lint and package tarball checks

Author: Bre77

.github/workflows/ci.yml

@@ -24,4 +24,10 @@ jobs:
 
       - run: npx tsc --noEmit
 
+      - run: pnpm run lint
+
       - run: pnpm test
+
+      - run: npx publint
+
+      - run: npm pack --dry-run

eslint.config.js

@@ -0,0 +1,26 @@
+// @ts-check
+import js from "@eslint/js";
+import tseslint from "typescript-eslint";
+
+export default tseslint.config(
+    js.configs.recommended,
+    tseslint.configs.recommended,
+    {
+        ignores: ["dist/**"],
+    },
+    {
+        rules: {
+            // ts-proto message types and Fleet API JSON payloads are inherently untyped.
+            "@typescript-eslint/no-explicit-any": "off",
+            "@typescript-eslint/no-unused-vars": [
+                "error",
+                {argsIgnorePattern: "^_", varsIgnorePattern: "^_"},
+            ],
+            // `this.debug && console.debug(...)` is this codebase's guard-log idiom.
+            "@typescript-eslint/no-unused-expressions": [
+                "error",
+                {allowShortCircuit: true},
+            ],
+        },
+    },
+);

package.json

@@ -12,15 +12,21 @@
         "url": "https://teslemetry.com"
     },
     "devDependencies": {
+        "@eslint/js": "^10.0.1",
         "@types/node": "^24.13.3",
+        "eslint": "^10.8.0",
+        "publint": "^0.3.23",
         "typescript": "^5.9.2",
+        "typescript-eslint": "^8.66.0",
         "vitest": "^4.1.10"
     },
     "scripts": {
         "prepare": "tsc",
         "watch": "tsc -w",
         "test": "vitest run",
-        "test:watch": "vitest"
+        "test:watch": "vitest",
+        "lint": "eslint .",
+        "pack:check": "publint"
     },
     "files": [
         "dist",