pepr 0.46.3-nightly.9 → 0.47.0-nightly.0

package/package.json

@@ -16,7 +16,7 @@
     "!src/fixtures/**",
     "!dist/**/*.test.d.ts*"
   ],
-  "version": "0.46.3-nightly.9",
+  "version": "0.47.0-nightly.0",
   "main": "dist/lib.js",
   "types": "dist/lib.d.ts",
   "scripts": {
@@ -49,12 +49,12 @@
   },
   "dependencies": {
     "@types/ramda": "0.30.2",
-    "express": "4.21.2",
+    "express": "5.1.0",
     "fast-json-patch": "3.1.1",
     "heredoc": "^1.3.1",
     "http-status-codes": "^2.3.0",
     "json-pointer": "^0.6.2",
-    "kubernetes-fluent-client": "3.4.5",
+    "kubernetes-fluent-client": "3.4.6",
     "pino": "9.6.0",
     "pino-pretty": "13.0.0",
     "prom-client": "15.1.3",
@@ -78,7 +78,7 @@
     "jest": "29.7.0",
     "js-yaml": "^4.1.0",
     "shellcheck": "^3.0.0",
-    "ts-jest": "29.3.0",
+    "ts-jest": "29.3.1",
     "undici": "^7.0.1"
   },
   "overrides": {

package/src/cli/init/enums.ts

@@ -7,3 +7,5 @@ export enum OnError {
   IGNORE = "ignore",
   REJECT = "reject",
 }
+
+export const UUID_LENGTH_LIMIT = 36;

package/src/cli/init/index.ts

@@ -23,6 +23,7 @@ import {
 import { createDir, sanitizeName, write } from "./utils";
 import { confirm, PromptOptions, walkthrough } from "./walkthrough";
 import { ErrorList } from "../../lib/errors";
+import { UUID_LENGTH_LIMIT } from "./enums";
 
 export default function (program: RootCmd): void {
   let response = {} as PromptOptions;
@@ -37,11 +38,9 @@ export default function (program: RootCmd): void {
     .option(`--errorBehavior <${ErrorList.join("|")}>`, "Set an errorBehavior.")
     .option(
       "--uuid [string]",
-      "Unique identifier for your module with a max length of 32 characters.",
+      "Unique identifier for your module with a max length of 36 characters.",
       (uuid: string): string => {
-        const uuidLengthLimit = 36;
-        // length of generated uuid
-        if (uuid.length > uuidLengthLimit) {
+        if (uuid.length > UUID_LENGTH_LIMIT) {
           throw new Error("The UUID must be 36 characters or fewer.");
         }
         return uuid.toLocaleLowerCase();

package/src/cli/init/walkthrough.ts

@@ -6,7 +6,7 @@ import prompt, { Answers, PromptObject } from "prompts";
 
 import { eslint, gitignore, prettier, readme, tsConfig } from "./templates";
 import { sanitizeName } from "./utils";
-import { OnError } from "./enums";
+import { OnError, UUID_LENGTH_LIMIT } from "./enums";
 import { ErrorList } from "../../lib/errors";
 
 export type PromptOptions = {
@@ -33,9 +33,9 @@ async function setUUID(uuid?: string): Promise<Answers<string>> {
     name: "uuid",
     message: "Enter a unique identifier for the new Pepr module.\n",
     validate: (val: string) => {
-      const uuidLengthLimit = 36;
       return (
-        val.length <= uuidLengthLimit || `The UUID must be ${uuidLengthLimit} characters or fewer.`
+        val.length <= UUID_LENGTH_LIMIT ||
+        `The UUID must be ${UUID_LENGTH_LIMIT} characters or fewer.`
       );
     },
   };

package/src/lib/common-types.ts

@@ -95,6 +95,7 @@ export type ValidateActionResponse = {
   allowed: boolean;
   statusCode?: number;
   statusMessage?: string;
+  warnings?: string[];
 };
 
 // DeepPartial utility type for deep optional properties

package/src/lib/controller/index.util.ts

@@ -22,12 +22,21 @@ export function karForMutate(mr: MutateResponse): KubeAdmissionReview {
 export function karForValidate(ar: AdmissionRequest, vr: ValidateResponse[]): KubeAdmissionReview {
   const isAllowed = vr.filter(r => !r.allowed).length === 0;
 
+  // Collect all warnings from the ValidateResponse array
+  const warnings = vr.reduce<string[]>((acc, curr) => {
+    if (curr.warnings && curr.warnings.length > 0) {
+      return [...acc, ...curr.warnings];
+    }
+    return acc;
+  }, []);
+
   const resp: ValidateResponse =
     vr.length === 0
       ? {
           uid: ar.uid,
           allowed: true,
           status: { code: 200, message: "no in-scope validations -- allowed!" },
+          warnings: warnings.length > 0 ? warnings : undefined,
         }
       : {
           uid: vr[0].uid,
@@ -39,6 +48,7 @@ export function karForValidate(ar: AdmissionRequest, vr: ValidateResponse[]): Ku
               .map(curr => curr.status?.message)
               .join("; "),
           },
+          warnings: warnings.length > 0 ? warnings : undefined,
         };
   return {
     apiVersion: "admission.k8s.io/v1",

package/src/lib/processors/validate-processor.ts

@@ -41,6 +41,11 @@ export async function processRequest(
       };
     }
 
+    // Transfer any warnings from the callback response to the validation response
+    if (callbackResp.warnings && callbackResp.warnings.length > 0) {
+      valResp.warnings = callbackResp.warnings;
+    }
+
     Log.info(
       actionMetadata,
       `Validation action complete (${label}): ${callbackResp.allowed ? "allowed" : "denied"}`,

package/src/lib/validate-request.ts

@@ -6,8 +6,8 @@
 import { KubernetesObject } from "kubernetes-fluent-client";
 
 import { clone } from "ramda";
-import { Operation } from "./enums";
 import { AdmissionRequest, ValidateActionResponse } from "./common-types";
+import { Operation } from "./enums";
 
 /**
  * The RequestWrapper class provides methods to modify Kubernetes objects in the context
@@ -79,9 +79,10 @@ export class PeprValidateRequest<T extends KubernetesObject> {
    *
    * @returns The validation response.
    */
-  Approve = (): ValidateActionResponse => {
+  Approve = (warnings?: string[]): ValidateActionResponse => {
     return {
       allowed: true,
+      warnings,
     };
   };
 
@@ -92,11 +93,16 @@ export class PeprValidateRequest<T extends KubernetesObject> {
    * @param statusCode Optional status code to return to the user.
    * @returns The validation response.
    */
-  Deny = (statusMessage?: string, statusCode?: number): ValidateActionResponse => {
+  Deny = (
+    statusMessage?: string,
+    statusCode?: number,
+    warnings?: string[],
+  ): ValidateActionResponse => {
     return {
       allowed: false,
       statusCode,
       statusMessage,
+      warnings,
     };
   };
 }