@pagopa/dx-cli 0.22.0 → 0.22.2

package/dist/adapters/commander/__tests__/env.test.js

@@ -1,43 +1,45 @@
+import fc from "fast-check";
 /**
  * Tests for the CLI environment schema.
  * Validates that `cliEnvSchema` correctly parses `process.env`.
  */
 import { afterEach, describe, expect, it, vi } from "vitest";
+import { z } from "zod";
 import { cliEnvSchema } from "../env.js";
+// The accepted stringbool values mirror those recognised by z.stringbool().
+const CI_TRUTHY = ["true", "1", "yes", "y", "on"];
+const CI_FALSEY = ["false", "0", "no", "n", "off"];
+const CI_ACCEPTED = [...CI_TRUTHY, ...CI_FALSEY];
 describe("cliEnvSchema", () => {
     afterEach(() => {
         vi.unstubAllEnvs();
     });
     describe("CI field", () => {
-        it("is undefined when CI is not set", () => {
-            vi.stubEnv("CI", undefined);
-            const result = cliEnvSchema.safeParse(process.env);
-            expect(result.success).toBe(true);
-            expect(result.success && result.data.CI).toBeUndefined();
+        it("should be true for truthy string-bool values", () => {
+            fc.assert(fc.property(fc.constantFrom(...CI_TRUTHY), (value) => {
+                vi.stubEnv("CI", value);
+                const result = cliEnvSchema.safeParse(process.env);
+                expect(result.success).toBe(true);
+                expect(result.success && result.data.CI).toBe(true);
+            }));
         });
-        it("captures any non-empty CI value", () => {
-            vi.stubEnv("CI", "true");
-            const result = cliEnvSchema.safeParse(process.env);
-            expect(result.success).toBe(true);
-            expect(result.success && result.data.CI).toBe("true");
+        it("should be false for falsey string-bool values or when unset", () => {
+            fc.assert(fc.property(fc.constantFrom(...CI_FALSEY, undefined), (value) => {
+                vi.stubEnv("CI", value);
+                const result = cliEnvSchema.safeParse(process.env);
+                expect(result.success).toBe(true);
+                expect(result.success && result.data.CI).toBe(false);
+            }));
         });
-        it("captures CI=false as a string (presence is the signal, not the value)", () => {
-            vi.stubEnv("CI", "false");
-            const result = cliEnvSchema.safeParse(process.env);
-            expect(result.success).toBe(true);
-            expect(result.success && result.data.CI).toBe("false");
+        it("should not parse unrecognised values", () => {
+            fc.assert(fc.property(
+            // Every string not in the accepted list
+            fc.string().filter((s) => !CI_ACCEPTED.includes(s.toLowerCase())), (value) => {
+                vi.stubEnv("CI", value);
+                const result = cliEnvSchema.safeParse(process.env);
+                expect(result.success).toBe(false);
+                expect(result.error).toBeInstanceOf(z.ZodError);
+            }));
         });
-        it("captures CI=1 for numeric-style env vars", () => {
-            vi.stubEnv("CI", "1");
-            const result = cliEnvSchema.safeParse(process.env);
-            expect(result.success).toBe(true);
-            expect(result.success && result.data.CI).toBe("1");
-        });
-    });
-    it("passes through an object with unrelated env keys without failing", () => {
-        vi.stubEnv("CI", "true");
-        const result = cliEnvSchema.safeParse(process.env);
-        expect(result.success).toBe(true);
-        expect(result.success && result.data.CI).toBe("true");
     });
 });

package/dist/adapters/commander/env.d.ts

@@ -8,6 +8,6 @@
  */
 import { z } from "zod";
 export declare const cliEnvSchema: z.ZodObject<{
-    CI: z.ZodOptional<z.ZodString>;
+    CI: z.ZodDefault<z.ZodCodec<z.ZodString, z.ZodBoolean>>;
 }, z.core.$loose>;
 export type CliEnv = z.infer<typeof cliEnvSchema>;

package/dist/adapters/commander/env.js

@@ -9,9 +9,7 @@
 import { z } from "zod";
 export const cliEnvSchema = z
     .object({
-    // Standard CI marker. Presence (any string value) signals an automated
-    // pipeline where interactive prompts must not block. Follows the same
-    // convention used by `is-interactive` and `ora`.
-    CI: z.string().optional(),
+    // Use a truthy value to enable CI mode.
+    CI: z.stringbool().default(false),
 })
     .loose();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pagopa/dx-cli",
-  "version": "0.22.0",
+  "version": "0.22.2",
   "type": "module",
   "description": "A CLI useful to manage DX tools.",
   "repository": {
@@ -48,7 +48,7 @@
     "semver": "^7.7.4",
     "yaml": "^2.8.4",
     "zod": "^4.4.2",
-    "@pagopa/dx-savemoney": "^0.2.5"
+    "@pagopa/dx-savemoney": "^0.2.6"
   },
   "devDependencies": {
     "@tsconfig/node24": "24.0.4",
@@ -58,6 +58,7 @@
     "@types/semver": "^7.7.1",
     "@vitest/coverage-v8": "^3.2.4",
     "eslint": "^10.3.0",
+    "fast-check": "^4.8.0",
     "memfs": "^4.57.2",
     "plop": "^4.0.5",
     "prettier": "3.8.3",