pepr 0.38.0-rc → 0.38.1

package/package.json

@@ -11,9 +11,11 @@
   },
   "files": [
     "/dist",
-    "/src"
+    "/src",
+    "!src/**/*.test.ts",
+    "!dist/**/*.test.d.ts*"
   ],
-  "version": "0.38.0-rc",
+  "version": "0.38.1",
   "main": "dist/lib.js",
   "types": "dist/lib.d.ts",
   "scripts": {
@@ -39,12 +41,12 @@
   },
   "dependencies": {
     "@types/ramda": "0.30.2",
-    "express": "4.21.0",
+    "express": "4.21.1",
     "fast-json-patch": "3.1.1",
     "json-pointer": "^0.6.2",
-    "kubernetes-fluent-client": "4.0.0-rc-http2-watch",
-    "pino": "9.4.0",
-    "pino-pretty": "11.2.2",
+    "kubernetes-fluent-client": "3.1.2",
+    "pino": "9.5.0",
+    "pino-pretty": "11.3.0",
     "prom-client": "15.1.3",
     "ramda": "0.30.1"
   },

package/src/fixtures/loader.ts

@@ -1,6 +1,6 @@
 import { kind } from "kubernetes-fluent-client";
 
-import { AdmissionRequest } from "../lib/k8s";
+import { AdmissionRequest } from "../lib/types";
 import createPod from "./data/create-pod.json";
 import deletePod from "./data/delete-pod.json";
 

package/src/lib/assets/pods.ts

@@ -128,11 +128,11 @@ export function watcher(assets: Assets, hash: string, buildTimestamp: string, im
               ],
               resources: {
                 requests: {
-                  memory: "64Mi",
-                  cpu: "100m",
+                  memory: "256Mi",
+                  cpu: "200m",
                 },
                 limits: {
-                  memory: "256Mi",
+                  memory: "512Mi",
                   cpu: "500m",
                 },
               },
@@ -267,11 +267,11 @@ export function deployment(
               ],
               resources: {
                 requests: {
-                  memory: "64Mi",
-                  cpu: "100m",
+                  memory: "256Mi",
+                  cpu: "200m",
                 },
                 limits: {
-                  memory: "256Mi",
+                  memory: "512Mi",
                   cpu: "500m",
                 },
               },

package/src/lib/assets/yaml.ts

@@ -63,11 +63,11 @@ export async function overridesFile({ hash, name, image, config, apiToken }: Ass
       },
       resources: {
         requests: {
-          memory: "64Mi",
-          cpu: "100m",
+          memory: "256Mi",
+          cpu: "200m",
         },
         limits: {
-          memory: "256Mi",
+          memory: "512Mi",
           cpu: "500m",
         },
       },
@@ -129,11 +129,11 @@ export async function overridesFile({ hash, name, image, config, apiToken }: Ass
       },
       resources: {
         requests: {
-          memory: "64Mi",
-          cpu: "100m",
+          memory: "256Mi",
+          cpu: "200m",
         },
         limits: {
-          memory: "256Mi",
+          memory: "512Mi",
           cpu: "500m",
         },
       },

package/src/lib/controller/index.ts

@@ -6,14 +6,14 @@ import fs from "fs";
 import https from "https";
 
 import { Capability } from "../capability";
-import { MutateResponse, AdmissionRequest, ValidateResponse } from "../k8s";
+import { MutateResponse, ValidateResponse } from "../k8s";
 import Log from "../logger";
 import { metricsCollector, MetricsCollector } from "../metrics";
 import { ModuleConfig, isWatchMode } from "../module";
 import { mutateProcessor } from "../mutate-processor";
 import { validateProcessor } from "../validate-processor";
 import { PeprControllerStore } from "./store";
-import { ResponseItem } from "../types";
+import { ResponseItem, AdmissionRequest } from "../types";
 
 if (!process.env.PEPR_NODE_WARNINGS) {
   process.removeAllListeners("warning");

package/src/lib/k8s.ts

@@ -1,15 +1,7 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: 2023-Present The Pepr Authors
 
-import { GenericKind, GroupVersionKind, KubernetesObject, RegisterKind } from "kubernetes-fluent-client";
-
-// DEPRECATED: Use Operation in types.ts instead
-export enum Operation {
-  CREATE = "CREATE",
-  UPDATE = "UPDATE",
-  DELETE = "DELETE",
-  CONNECT = "CONNECT",
-}
+import { GenericKind, RegisterKind } from "kubernetes-fluent-client";
 
 /**
  * PeprStore for internal use by Pepr. This is used to store arbitrary data in the cluster.
@@ -28,99 +20,6 @@ export const peprStoreGVK = {
 
 RegisterKind(PeprStore, peprStoreGVK);
 
-// DEPRECATED: Use Operation in types.ts instead
-/**
- * GroupVersionResource unambiguously identifies a resource. It doesn't anonymously include GroupVersion
- * to avoid automatic coercion. It doesn't use a GroupVersion to avoid custom marshalling
- */
-export interface GroupVersionResource {
-  readonly group: string;
-  readonly version: string;
-  readonly resource: string;
-}
-
-// DEPRECATED: Use Operation in types.ts instead
-
-/**
- * A Kubernetes admission request to be processed by a capability.
- */
-export interface AdmissionRequest<T = KubernetesObject> {
-  /** UID is an identifier for the individual request/response. */
-  readonly uid: string;
-
-  /** Kind is the fully-qualified type of object being submitted (for example, v1.Pod or autoscaling.v1.Scale) */
-  readonly kind: GroupVersionKind;
-
-  /** Resource is the fully-qualified resource being requested (for example, v1.pods) */
-  readonly resource: GroupVersionResource;
-
-  /** SubResource is the sub-resource being requested, if any (for example, "status" or "scale") */
-  readonly subResource?: string;
-
-  /** RequestKind is the fully-qualified type of the original API request (for example, v1.Pod or autoscaling.v1.Scale). */
-  readonly requestKind?: GroupVersionKind;
-
-  /** RequestResource is the fully-qualified resource of the original API request (for example, v1.pods). */
-  readonly requestResource?: GroupVersionResource;
-
-  /** RequestSubResource is the sub-resource of the original API request, if any (for example, "status" or "scale"). */
-  readonly requestSubResource?: string;
-
-  /**
-   * Name is the name of the object as presented in the request. On a CREATE operation, the client may omit name and
-   * rely on the server to generate the name. If that is the case, this method will return the empty string.
-   */
-  readonly name: string;
-
-  /** Namespace is the namespace associated with the request (if any). */
-  readonly namespace?: string;
-
-  /**
-   * Operation is the operation being performed. This may be different than the operation
-   * requested. e.g. a patch can result in either a CREATE or UPDATE Operation.
-   */
-  readonly operation: Operation;
-
-  /** UserInfo is information about the requesting user */
-  readonly userInfo: {
-    /** The name that uniquely identifies this user among all active users. */
-    username?: string;
-
-    /**
-     * A unique value that identifies this user across time. If this user is deleted
-     * and another user by the same name is added, they will have different UIDs.
-     */
-    uid?: string;
-
-    /** The names of groups this user is a part of. */
-    groups?: string[];
-
-    /** Any additional information provided by the authenticator. */
-    extra?: {
-      [key: string]: string[];
-    };
-  };
-
-  /** Object is the object from the incoming request prior to default values being applied */
-  readonly object: T;
-
-  /** OldObject is the existing object. Only populated for UPDATE or DELETE requests. */
-  readonly oldObject?: T;
-
-  /** DryRun indicates that modifications will definitely not be persisted for this request. Defaults to false. */
-  readonly dryRun?: boolean;
-
-  /**
-   * Options contains the options for the operation being performed.
-   * e.g. `meta.k8s.io/v1.DeleteOptions` or `meta.k8s.io/v1.CreateOptions`. This may be
-   * different than the options the caller provided. e.g. for a patch request the performed
-   * Operation might be a CREATE, in which case the Options will a
-   * `meta.k8s.io/v1.CreateOptions` even though the caller provided `meta.k8s.io/v1.PatchOptions`.
-   */
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  readonly options?: any;
-}
-
 export interface MutateResponse {
   /** UID is an identifier for the individual request/response. This must be copied over from the corresponding AdmissionRequest. */
   uid: string;

package/dist/cli/init/utils.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=utils.test.d.ts.map

package/dist/cli/init/utils.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"utils.test.d.ts","sourceRoot":"","sources":["../../../src/cli/init/utils.test.ts"],"names":[],"mappings":""}

package/dist/cli/init/walkthrough.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=walkthrough.test.d.ts.map

package/dist/cli/init/walkthrough.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"walkthrough.test.d.ts","sourceRoot":"","sources":["../../../src/cli/init/walkthrough.test.ts"],"names":[],"mappings":""}

package/dist/lib/adjudicators.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=adjudicators.test.d.ts.map

package/dist/lib/adjudicators.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"adjudicators.test.d.ts","sourceRoot":"","sources":["../../src/lib/adjudicators.test.ts"],"names":[],"mappings":""}

package/dist/lib/assets/helm.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=helm.test.d.ts.map

package/dist/lib/assets/helm.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"helm.test.d.ts","sourceRoot":"","sources":["../../../src/lib/assets/helm.test.ts"],"names":[],"mappings":""}

package/dist/lib/assets/pods.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=pods.test.d.ts.map

package/dist/lib/assets/pods.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"pods.test.d.ts","sourceRoot":"","sources":["../../../src/lib/assets/pods.test.ts"],"names":[],"mappings":""}

package/dist/lib/capability.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=capability.test.d.ts.map

package/dist/lib/capability.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"capability.test.d.ts","sourceRoot":"","sources":["../../src/lib/capability.test.ts"],"names":[],"mappings":""}

package/dist/lib/controller/store.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=store.test.d.ts.map

package/dist/lib/controller/store.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"store.test.d.ts","sourceRoot":"","sources":["../../../src/lib/controller/store.test.ts"],"names":[],"mappings":""}

package/dist/lib/errors.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=errors.test.d.ts.map

package/dist/lib/errors.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"errors.test.d.ts","sourceRoot":"","sources":["../../src/lib/errors.test.ts"],"names":[],"mappings":""}

package/dist/lib/filter.test.d.ts

@@ -1,3 +0,0 @@
-export declare const callback: () => undefined;
-export declare const podKind: import("kubernetes-fluent-client").GroupVersionKind;
-//# sourceMappingURL=filter.test.d.ts.map

package/dist/lib/filter.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"filter.test.d.ts","sourceRoot":"","sources":["../../src/lib/filter.test.ts"],"names":[],"mappings":"AAUA,eAAO,MAAM,QAAQ,iBAAkB,CAAC;AAExC,eAAO,MAAM,OAAO,qDAAyC,CAAC"}

package/dist/lib/finalizer.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=finalizer.test.d.ts.map

package/dist/lib/finalizer.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"finalizer.test.d.ts","sourceRoot":"","sources":["../../src/lib/finalizer.test.ts"],"names":[],"mappings":""}

package/dist/lib/helpers.test.d.ts

@@ -1,2 +0,0 @@
-export declare const callback: () => undefined;
-//# sourceMappingURL=helpers.test.d.ts.map

package/dist/lib/helpers.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"helpers.test.d.ts","sourceRoot":"","sources":["../../src/lib/helpers.test.ts"],"names":[],"mappings":"AAkCA,eAAO,MAAM,QAAQ,iBAAkB,CAAC"}

package/dist/lib/included-files.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=included-files.test.d.ts.map

package/dist/lib/included-files.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"included-files.test.d.ts","sourceRoot":"","sources":["../../src/lib/included-files.test.ts"],"names":[],"mappings":""}

package/dist/lib/logger.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=logger.test.d.ts.map

package/dist/lib/logger.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"logger.test.d.ts","sourceRoot":"","sources":["../../src/lib/logger.test.ts"],"names":[],"mappings":""}

package/dist/lib/metrics.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=metrics.test.d.ts.map

package/dist/lib/metrics.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"metrics.test.d.ts","sourceRoot":"","sources":["../../src/lib/metrics.test.ts"],"names":[],"mappings":""}

package/dist/lib/module.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=module.test.d.ts.map

package/dist/lib/module.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"module.test.d.ts","sourceRoot":"","sources":["../../src/lib/module.test.ts"],"names":[],"mappings":""}

package/dist/lib/mutate-request.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=mutate-request.test.d.ts.map

package/dist/lib/mutate-request.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"mutate-request.test.d.ts","sourceRoot":"","sources":["../../src/lib/mutate-request.test.ts"],"names":[],"mappings":""}

package/dist/lib/queue.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=queue.test.d.ts.map

package/dist/lib/queue.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"queue.test.d.ts","sourceRoot":"","sources":["../../src/lib/queue.test.ts"],"names":[],"mappings":""}

package/dist/lib/schedule.test.d.ts

@@ -1,15 +0,0 @@
-import { Unsubscribe } from "./storage";
-export declare class MockStorage {
-    private storage;
-    subscription: string;
-    constructor();
-    getItem(key: string): string | null;
-    setItem(key: string, value: string): void;
-    setItemAndWait(key: string, value: string): Promise<void>;
-    removeItem(key: string): void;
-    removeItemAndWait(key: string): Promise<void>;
-    clear(): void;
-    subscribe(): Unsubscribe;
-    onReady(): void;
-}
-//# sourceMappingURL=schedule.test.d.ts.map

package/dist/lib/schedule.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"schedule.test.d.ts","sourceRoot":"","sources":["../../src/lib/schedule.test.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,WAAW,EAAE,MAAM,WAAW,CAAC;AAExC,qBAAa,WAAW;IACtB,OAAO,CAAC,OAAO,CAA8B;IAC7C,YAAY,EAAE,MAAM,CAAC;;IAKrB,OAAO,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAInC,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI;IAIzC,cAAc,CAAC,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAOzD,UAAU,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI;IAI7B,iBAAiB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAO7C,KAAK,IAAI,IAAI;IAIb,SAAS,IAAI,WAAW;IAMxB,OAAO,IAAI,IAAI;CAKhB"}

package/dist/lib/storage.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=storage.test.d.ts.map

package/dist/lib/storage.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"storage.test.d.ts","sourceRoot":"","sources":["../../src/lib/storage.test.ts"],"names":[],"mappings":""}

package/dist/lib/tls.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=tls.test.d.ts.map

package/dist/lib/tls.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"tls.test.d.ts","sourceRoot":"","sources":["../../src/lib/tls.test.ts"],"names":[],"mappings":""}

package/dist/lib/utils.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=utils.test.d.ts.map

package/dist/lib/utils.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"utils.test.d.ts","sourceRoot":"","sources":["../../src/lib/utils.test.ts"],"names":[],"mappings":""}

package/dist/lib/validate-request.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=validate-request.test.d.ts.map

package/dist/lib/validate-request.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"validate-request.test.d.ts","sourceRoot":"","sources":["../../src/lib/validate-request.test.ts"],"names":[],"mappings":""}

package/dist/lib/watch-processor.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=watch-processor.test.d.ts.map

package/dist/lib/watch-processor.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"watch-processor.test.d.ts","sourceRoot":"","sources":["../../src/lib/watch-processor.test.ts"],"names":[],"mappings":""}

package/dist/sdk/sdk.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=sdk.test.d.ts.map

package/dist/sdk/sdk.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"sdk.test.d.ts","sourceRoot":"","sources":["../../src/sdk/sdk.test.ts"],"names":[],"mappings":""}

package/src/cli/init/utils.test.ts

@@ -1,19 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-// SPDX-FileCopyrightText: 2023-Present The Pepr Authors
-
-import { expect, it } from "@jest/globals";
-
-import { sanitizeName } from "./utils";
-
-it.each([
-  //Test sanitizeName() with ["$BAD_INPUT", "$SANITIZED_INPUT"]
-  ["My Test Module", "my-test-module"],
-  ["!! 123 @@ Module", "123-module"],
-  ["---Test-Module---", "test-module"],
-])("sanitizeName() sanitizes '%s' correctly", (input: string, expected: string) => {
-  expect(sanitizeName(input)).toBe(expected);
-});
-
-it("sanitizeName() should throw TypeError when given a non-string", () => {
-  expect(() => sanitizeName({ input: 0 } as unknown as string)).toThrow(TypeError);
-});

package/src/cli/init/walkthrough.test.ts

@@ -1,102 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-// SPDX-FileCopyrightText: 2023-Present The Pepr Authors
-
-import { describe, expect, it } from "@jest/globals";
-import prompts from "prompts";
-import {
-  walkthrough,
-  confirm,
-  PromptOptions,
-  PartialPromptOptions,
-  setName,
-  setErrorBehavior,
-} from "./walkthrough";
-
-describe("when processing input", () => {
-  describe("walkthough() returns expected results", () => {
-    it.each([
-      //Test flag combinations with [["$FLAG", ...]]
-      [["description", "errorBehavior"]],
-      [["description"]],
-      [["errorBehavior"]],
-      [["name", "description", "errorBehavior"]],
-      [["name", "description"]],
-      [["name", "errorBehavior"]],
-      [["name"]],
-      [undefined],
-    ])(`when the set flags are: %s`, async (flagInput: string[] | undefined) => {
-      const expected: PromptOptions = {
-        name: "My Test Module",
-        description: "A test module for Pepr",
-        errorBehavior: "reject",
-      };
-
-      // Set values for the flag(s) under test by making a subset of (expected)
-      type SupportedFlagNames = keyof typeof expected;
-      type PartialTestInput = { [key in SupportedFlagNames]?: string };
-      const setFlags =
-        flagInput?.reduce((acc: PartialTestInput, key: string) => {
-          if (key in expected) {
-            acc[key as SupportedFlagNames] = expected[key as SupportedFlagNames];
-          }
-          return acc;
-        }, {} as PartialTestInput) || {};
-
-      // Simulate user-input for unset flags by making a subset of (expected - setFlags)
-      const promptInput = flagInput
-        ? Object.entries(expected)
-            .filter(([key]) => !flagInput.includes(key))
-            .map(([, value]) => value)
-        : Object.values(expected);
-      prompts.inject(Object.values(promptInput));
-
-      const result = await walkthrough(setFlags as PartialPromptOptions);
-
-      expect(result).toEqual(expected);
-    });
-
-    it("should prompt for input when given invalid input", async () => {
-      const expected = { name: "aaa" };
-      prompts.inject(["aaa"]);
-      const result = await setName("aa");
-      expect(result).toStrictEqual(expected);
-    });
-
-    it("should prompt for errorBehavior when given invalid input", async () => {
-      const expected = { errorBehavior: "audit" };
-      prompts.inject(["audit"]);
-      const result = await setErrorBehavior("not-valid" as "reject"); // Type-Coercion forces invalid input
-      expect(result).toStrictEqual(expected);
-    });
-  });
-
-  describe("confirm() handles input", () => {
-    it.each([
-      ["n", false],
-      ["no", false],
-      ["y", true],
-      ["yes", true],
-    ])("when prompt input is '%s'", async (userInput: string, expected: boolean) => {
-      prompts.inject([userInput]);
-      const result = await confirm(
-        "some string",
-        { path: "some path", print: "some print" },
-        "some Pepr TS path",
-      );
-      expect(result).toBe(expected);
-    });
-
-    it.each([[true], [false]])(
-      "when flag '--confirm' is %s",
-      async (confirmFlag: boolean | undefined) => {
-        const result = await confirm(
-          "some string",
-          { path: "some path", print: "some print" },
-          "some Pepr TS path",
-          confirmFlag,
-        );
-        expect(result).toBe(confirmFlag);
-      },
-    );
-  });
-});