pepr 1.3.0 → 2.0.0

package/package.json

@@ -16,7 +16,7 @@
     "!src/fixtures/**",
     "!dist/**/*.test.d.ts*"
   ],
-  "version": "1.3.0",
+  "version": "2.0.0",
   "main": "dist/lib.js",
   "types": "dist/lib.d.ts",
   "scripts": {
@@ -50,28 +50,32 @@
   },
   "dependencies": {
     "@types/ramda": "0.32.0",
-    "@typescript-eslint/eslint-plugin": "8.63.0",
-    "@typescript-eslint/parser": "8.63.0",
-    "commander": "14.0.3",
-    "eslint": "9.39.4",
+    "@typescript-eslint/eslint-plugin": "8.66.0",
+    "@typescript-eslint/parser": "8.66.0",
+    "commander": "15.0.0",
+    "eslint": "^10.8.0",
     "express": "5.2.1",
     "fast-json-patch": "3.1.1",
+    "globals": "17.9.0",
     "http-status-codes": "^2.3.0",
     "json-pointer": "^0.6.2",
-    "kubernetes-fluent-client": "3.11.10",
+    "kubernetes-fluent-client": "3.11.11",
     "pino": "10.3.1",
     "pino-pretty": "13.1.3",
     "prom-client": "15.1.3",
-    "quicktype-core": "^23.2.6",
+    "quicktype-core": "^26.0.0",
     "ramda": "0.32.0"
   },
   "devDependencies": {
     "@commitlint/cli": "21.2.1",
     "@commitlint/config-conventional": "21.2.0",
+    "@eslint/eslintrc": "^3.3.6",
+    "@eslint/js": "^10.0.1",
     "@types/command-line-args": "^5.2.3",
     "@types/express": "5.0.6",
     "@types/json-pointer": "^1.0.34",
     "@types/json-schema": "^7.0.15",
+    "@types/ms": "^2.1.0",
     "@types/node": "^24.13.3",
     "@types/node-forge": "1.3.14",
     "@types/readable-stream": "^4.0.21",
@@ -79,26 +83,26 @@
     "@types/ws": "^8.18.1",
     "@vitest/coverage-v8": "^4.0.4",
     "fast-check": "^4.0.0",
-    "globals": "^17.0.0",
     "husky": "^9.1.6",
-    "js-yaml": "^4.1.0",
+    "js-yaml": "^5.2.1",
+    "ms": "^2.1.3",
+    "prettier": "^3.6.2",
     "selfsigned": "^5.5.0",
     "shellcheck": "^4.1.0",
     "tsx": "^4.20.3",
-    "undici": "8.7.0",
+    "undici": "8.10.0",
     "vitest": "^4.0.4"
   },
   "overrides": {
-    "brace-expansion": "^5.0.7",
-    "ip-address": "^10.1.1",
+    "brace-expansion": "^5.0.9",
+    "ip-address": "^10.5.0",
     "tar": "^7.5.16",
-    "fast-uri": "^3.1.4"
+    "fast-uri": "^3.1.5"
   },
   "peerDependencies": {
     "@types/prompts": "^2.4.9",
     "esbuild": "^0.28.0",
     "node-forge": "^1.4.0",
-    "prettier": "^3.6.2",
     "prompts": "^2.4.2",
     "typescript": "^5.8.3",
     "uuid": "^13.0.0"

package/src/cli/build/buildModule.ts

@@ -5,7 +5,6 @@ import { execFileSync } from "child_process";
 import { BuildOptions, analyzeMetafile } from "esbuild";
 import { basename, extname, resolve } from "path";
 import { dependencies } from "../init/templates";
-import { peprFormat } from "../format";
 import { watchForChanges } from "./build.helpers";
 import { PeprConfig, Reloader } from "../types";
 import { BuildContext } from "esbuild";
@@ -70,8 +69,6 @@ export async function buildModule(
     const { cfg, modulePath, path: loadedPath, uuid } = await loadModule(outputDir, entryPoint);
     const { format, path } = resolveFormatAndPath(embed, requestedFormat, loadedPath);
 
-    await checkFormat();
-
     // Resolve node_modules folder (in support of npm workspaces!) and run tsc
     const npmRoot = execFileSync("npm", ["root"]).toString().trim();
     execFileSync(`${npmRoot}/.bin/tsc`, [
@@ -187,14 +184,3 @@ function handleModuleBuildError(e: BuildModuleResult): void {
     });
   }
 }
-
-async function checkFormat(): Promise<void> {
-  const validFormat = await peprFormat(true);
-
-  if (!validFormat) {
-    console.info(
-      "\x1b[33m%s\x1b[0m",
-      "Formatting errors were found. The build will continue, but you may want to run `npx pepr format` to address any issues.",
-    );
-  }
-}

package/src/cli/build/index.ts

@@ -102,8 +102,8 @@ export default function (program: Command): void {
     )
     .addOption(
       new Option(
-        "-I, --registry-info <registry/username>",
-        "Provide the image registry and username for building and pushing a custom WASM container. Requires authentication. Conflicts with --custom-image and --registry. Builds and pushes `'<registry/username>/custom-pepr-controller:<current-version>'`.",
+        "-I, --registry-info <registry[/namespace]>",
+        "Provide the image registry (and optional namespace) for building and pushing a custom WASM container. Requires authentication. Conflicts with --custom-image and --registry. Builds and pushes `'<registry[/namespace]>/custom-pepr-controller:<current-version>'` (e.g. docker.io/myuser, localhost:5000).",
       ).conflicts(["customImage", "registry"]),
     )
     .option("-P, --with-pull-secret <name>", "Use image pull secret for controller Deployment.", "")

package/src/cli/init/templates.ts

@@ -43,7 +43,13 @@ export type peprPackageJSON = {
     scripts: { "k3d-setup": string };
     dependencies: { pepr: string; undici: string };
     devDependencies: {
+      "@eslint/eslintrc": string;
+      "@eslint/js": string;
+      "@typescript-eslint/eslint-plugin": string;
+      "@typescript-eslint/parser": string;
       "@types/node": string;
+      eslint: string;
+      globals: string;
       typescript: string;
     };
   };
@@ -99,7 +105,13 @@ export function genPkgJSON(opts: InitOptions): peprPackageJSON {
       undici: "^7.0.1",
     },
     devDependencies: {
+      "@eslint/eslintrc": devDependencies["@eslint/eslintrc"],
+      "@eslint/js": devDependencies["@eslint/js"],
+      "@typescript-eslint/eslint-plugin": dependencies["@typescript-eslint/eslint-plugin"],
+      "@typescript-eslint/parser": dependencies["@typescript-eslint/parser"],
       "@types/node": devDependencies["@types/node"],
+      eslint: dependencies.eslint,
+      globals: dependencies.globals,
       typescript,
     },
     overrides: {

package/src/cli/init/utils.ts

@@ -38,7 +38,7 @@ export async function createDir(dir: string): Promise<void> {
   } catch (err) {
     // The directory already exists
     if (err && (err as NodeJS.ErrnoException).code === "EEXIST") {
-      throw new Error(`Directory ${dir} already exists`);
+      throw new Error(`Directory ${dir} already exists`, { cause: err });
     } else {
       throw err;
     }

package/src/cli.ts

@@ -7,7 +7,6 @@ import { banner } from "./cli/banner";
 import build from "./cli/build";
 import deploy from "./cli/deploy";
 import dev from "./cli/dev";
-import format from "./cli/format";
 import monitor from "./cli/monitor";
 import init from "./cli/init/index";
 import uuid from "./cli/uuid";
@@ -58,7 +57,6 @@ build(program);
 deploy(program);
 dev(program);
 update(program);
-format(program);
 monitor(program);
 uuid(program);
 kfc(program);

package/src/lib/assets/assets.ts

@@ -320,7 +320,8 @@ export class Assets {
         );
       }
     } catch (err) {
-      throw new Error(`Error generating helm chart: ${err.message}`);
+      const message = err instanceof Error ? err.message : String(err);
+      throw new Error(`Error generating helm chart: ${message}`, { cause: err });
     }
   };
 }

package/src/lib/assets/k8sObjects.ts

@@ -1,13 +1,28 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: 2023-Present The Pepr Authors
 
-import { KubernetesObject } from "@kubernetes/client-node";
+import { KubernetesObject, V1Probe } from "@kubernetes/client-node";
 import { kind } from "kubernetes-fluent-client";
 import { gzipSync } from "zlib";
 import { secretOverLimit } from "../helpers";
 import { Assets, isAdmission, isWatcher, norWatchOrAdmission } from "./assets";
 import { genEnv } from "./environment";
 
+function defaultProbe(): V1Probe {
+  return {
+    httpGet: {
+      path: "/healthz",
+      port: 3000,
+      scheme: "HTTPS",
+    },
+    initialDelaySeconds: 10,
+    periodSeconds: 10,
+    timeoutSeconds: 1,
+    successThreshold: 1,
+    failureThreshold: 3,
+  };
+}
+
 /** Generate the pepr-system namespace */
 export function getNamespace(namespaceLabels?: Record<string, string>): KubernetesObject {
   if (namespaceLabels) {
@@ -95,30 +110,9 @@ export function getWatcher(
               image,
               imagePullPolicy: "IfNotPresent",
               args: ["/app/node_modules/pepr/dist/controller.js", hash],
-              startupProbe: {
-                httpGet: {
-                  path: "/healthz",
-                  port: 3000,
-                  scheme: "HTTPS",
-                },
-                initialDelaySeconds: 10,
-              },
-              readinessProbe: {
-                httpGet: {
-                  path: "/healthz",
-                  port: 3000,
-                  scheme: "HTTPS",
-                },
-                initialDelaySeconds: 10,
-              },
-              livenessProbe: {
-                httpGet: {
-                  path: "/healthz",
-                  port: 3000,
-                  scheme: "HTTPS",
-                },
-                initialDelaySeconds: 10,
-              },
+              startupProbe: defaultProbe(),
+              readinessProbe: defaultProbe(),
+              livenessProbe: defaultProbe(),
               ports: [
                 {
                   containerPort: 3000,
@@ -246,30 +240,9 @@ export function getDeployment(
               image,
               imagePullPolicy: "IfNotPresent",
               args: ["/app/node_modules/pepr/dist/controller.js", hash],
-              startupProbe: {
-                httpGet: {
-                  path: "/healthz",
-                  port: 3000,
-                  scheme: "HTTPS",
-                },
-                initialDelaySeconds: 10,
-              },
-              readinessProbe: {
-                httpGet: {
-                  path: "/healthz",
-                  port: 3000,
-                  scheme: "HTTPS",
-                },
-                initialDelaySeconds: 10,
-              },
-              livenessProbe: {
-                httpGet: {
-                  path: "/healthz",
-                  port: 3000,
-                  scheme: "HTTPS",
-                },
-                initialDelaySeconds: 10,
-              },
+              startupProbe: defaultProbe(),
+              readinessProbe: defaultProbe(),
+              livenessProbe: defaultProbe(),
               ports: [
                 {
                   containerPort: 3000,

package/src/lib/assets/yaml/overridesFile.ts

@@ -27,19 +27,18 @@ export type ChartOverrides = {
   name: string;
   image: string;
 };
+type Probe = {
+  httpGet: { path: string; port: number; scheme: "HTTPS" };
+  initialDelaySeconds: number;
+  periodSeconds: number;
+  timeoutSeconds: number;
+  successThreshold: number;
+  failureThreshold: number;
+};
 type Probes = {
-  startupProbe: {
-    httpGet: { path: string; port: number; scheme: "HTTPS" };
-    initialDelaySeconds: number;
-  };
-  readinessProbe: {
-    httpGet: { path: string; port: number; scheme: "HTTPS" };
-    initialDelaySeconds: number;
-  };
-  livenessProbe: {
-    httpGet: { path: string; port: number; scheme: "HTTPS" };
-    initialDelaySeconds: number;
-  };
+  startupProbe: Probe;
+  readinessProbe: Probe;
+  livenessProbe: Probe;
 };
 type Resources = {
   requests: { memory: string; cpu: string };
@@ -207,18 +206,20 @@ function runIdsForImage(image: string): { uid: number; gid: number; fsGroup: num
 
 function commonProbes(): Probes {
   return {
-    startupProbe: {
-      httpGet: { path: "/healthz", port: 3000, scheme: "HTTPS" },
-      initialDelaySeconds: 10,
-    },
-    readinessProbe: {
-      httpGet: { path: "/healthz", port: 3000, scheme: "HTTPS" },
-      initialDelaySeconds: 10,
-    },
-    livenessProbe: {
-      httpGet: { path: "/healthz", port: 3000, scheme: "HTTPS" },
-      initialDelaySeconds: 10,
-    },
+    startupProbe: defaultProbe(),
+    readinessProbe: defaultProbe(),
+    livenessProbe: defaultProbe(),
+  };
+}
+
+function defaultProbe(): Probe {
+  return {
+    httpGet: { path: "/healthz", port: 3000, scheme: "HTTPS" },
+    initialDelaySeconds: 10,
+    periodSeconds: 10,
+    timeoutSeconds: 1,
+    successThreshold: 1,
+    failureThreshold: 3,
   };
 }
 

package/src/lib/controller/migrateStore.ts

@@ -51,6 +51,6 @@ export async function migrateAndSetupWatch(storeData: StoreMigration): Promise<v
       }
     }
   }
-  storeCache = await sendUpdatesAndFlushCache(storeCache, namespace, name);
+  await sendUpdatesAndFlushCache(storeCache, namespace, name);
   setupWatch();
 }

package/src/lib/helpers.ts

@@ -184,8 +184,7 @@ const matchRegexToCapababilityNamespace = (
     capabilityNamespaces.length > 0
   ) {
     for (const regexNamespace of bindingRegexNamespaces) {
-      let matches = false;
-      matches =
+      const matches =
         regexNamespace !== "" &&
         capabilityNamespaces.some(capabilityNamespace =>
           matchesRegex(regexNamespace, capabilityNamespace),

package/src/runtime/controller.ts

@@ -46,7 +46,7 @@ function runModule(expectedHash: string): void {
     // Run the module
     fork(jsPath);
   } catch (e) {
-    throw new Error(`Failed to decompress module: ${e}`);
+    throw new Error(`Failed to decompress module: ${e}`, { cause: e });
   }
 }
 

package/dist/cli/format/format.helpers.d.ts

@@ -1,3 +0,0 @@
-import { ESLint } from "eslint";
-export declare function formatWithPrettier(results: ESLint.LintResult[], validateOnly: boolean): Promise<boolean>;
-//# sourceMappingURL=format.helpers.d.ts.map

package/dist/cli/format/format.helpers.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"format.helpers.d.ts","sourceRoot":"","sources":["../../../src/cli/format/format.helpers.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAIhC,wBAAsB,kBAAkB,CACtC,OAAO,EAAE,MAAM,CAAC,UAAU,EAAE,EAC5B,YAAY,EAAE,OAAO,GACpB,OAAO,CAAC,OAAO,CAAC,CAgBlB"}

package/dist/cli/format/index.d.ts

@@ -1,9 +0,0 @@
-import { Command } from "commander";
-export default function (program: Command): void;
-/**
- * Perform linting and formatting on the module
- * @param validateOnly
- * @returns success
- */
-export declare function peprFormat(validateOnly: boolean): Promise<boolean>;
-//# sourceMappingURL=index.d.ts.map

package/dist/cli/format/index.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/cli/format/index.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAGpC,MAAM,CAAC,OAAO,WAAW,OAAO,EAAE,OAAO,GAAG,IAAI,CAmB/C;AAED;;;;GAIG;AACH,wBAAsB,UAAU,CAAC,YAAY,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAoCxE"}

package/src/cli/format/format.helpers.ts

@@ -1,27 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-// SPDX-FileCopyrightText: 2023-Present The Pepr Authors
-
-import { ESLint } from "eslint";
-import { promises as fs } from "fs";
-import { format, resolveConfig } from "prettier";
-
-export async function formatWithPrettier(
-  results: ESLint.LintResult[],
-  validateOnly: boolean,
-): Promise<boolean> {
-  let hasFailure = false;
-  for (const { filePath } of results) {
-    const content = await fs.readFile(filePath, "utf8");
-    const cfg = await resolveConfig(filePath);
-    const formatted = await format(content, { filepath: filePath, ...cfg });
-
-    // If in validate-only mode, check if the file is formatted correctly
-    if (validateOnly && formatted !== content) {
-      hasFailure = true;
-      console.error(`File ${filePath} is not formatted correctly`);
-    } else {
-      await fs.writeFile(filePath, formatted);
-    }
-  }
-  return hasFailure;
-}

package/src/cli/format/index.ts

@@ -1,72 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-// SPDX-FileCopyrightText: 2023-Present The Pepr Authors
-
-import { ESLint } from "eslint";
-import { formatWithPrettier } from "./format.helpers";
-
-import { Command } from "commander";
-import Log from "../../lib/telemetry/logger";
-
-export default function (program: Command): void {
-  program
-    .command("format")
-    .description("Lint and format this Pepr module")
-    .option("-v, --validate-only", "Do not modify files, only validate formatting.")
-    .action(async opts => {
-      Log.warn(
-        "DEPRECATION NOTICE: The pepr format command will be removed in summer 2026. " +
-          "Once removed, module authors must run a linter separately from the pepr CLI.",
-      );
-
-      const success = await peprFormat(opts.validateOnly);
-
-      if (success) {
-        Log.info("Module formatted");
-      } else {
-        process.exit(1);
-      }
-    });
-}
-
-/**
- * Perform linting and formatting on the module
- * @param validateOnly
- * @returns success
- */
-export async function peprFormat(validateOnly: boolean): Promise<boolean> {
-  {
-    try {
-      const eslint = new ESLint();
-      const results = await eslint.lintFiles(["./**/*.ts"]);
-
-      // Track if any files failed
-      let hasFailure = false;
-
-      results.forEach(async result => {
-        const errorCount = result.fatalErrorCount + result.errorCount;
-        if (errorCount > 0) {
-          hasFailure = true;
-        }
-      });
-
-      const formatter = await eslint.loadFormatter("stylish");
-      const resultText = await formatter.format(results, {} as ESLint.LintResultData);
-
-      if (resultText) {
-        Log.info(resultText);
-      }
-
-      // Write the fixes if not in validate-only mode
-      if (!validateOnly) {
-        await ESLint.outputFixes(results);
-      }
-
-      hasFailure = hasFailure || (await formatWithPrettier(results, validateOnly));
-
-      return !hasFailure;
-    } catch (error) {
-      Log.error(error, `Error formatting module:`);
-      return false;
-    }
-  }
-}