core-services-sdk 1.3.77 → 1.3.78

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "core-services-sdk",
-  "version": "1.3.77",
+  "version": "1.3.78",
   "main": "src/index.js",
   "type": "module",
   "types": "types/index.d.ts",

package/src/env/env-validation.js

@@ -409,3 +409,82 @@ export function prepareEnvValidation(definition, values, options = {}) {
     report,
   }
 }
+
+/**
+ * Builds a console-ready table output for environment validation.
+ *
+ * Always includes all variables.
+ * NOTES column always has a value:
+ * - 'OK' for valid variables
+ * - error message(s) for invalid variables
+ *
+ * @param {{
+ *   params: Array<{
+ *     key: string,
+ *     displayValue: string,
+ *     valid: boolean,
+ *     errors?: string[]
+ *   }>
+ * }} report
+ *
+ * @returns {string}
+ */
+export function buildConsoleOutput(report) {
+  const headers = ['STATUS', 'KEY', 'VALUE', 'NOTES']
+
+  const rows = report.params.map((p) => {
+    const status = p.valid ? 'OK' : 'ERROR'
+    const notes = p.valid ? 'OK' : p.errors?.join('; ') || 'Invalid value'
+
+    return [status, p.key, p.displayValue, notes]
+  })
+
+  const allRows = [headers, ...rows]
+
+  const colWidths = headers.map((_, i) =>
+    Math.max(...allRows.map((row) => String(row[i]).length)),
+  )
+
+  const formatRow = (row) =>
+    row.map((cell, i) => String(cell).padEnd(colWidths[i])).join('  ')
+
+  return [
+    'Environment variables',
+    '',
+    ...allRows.map(formatRow),
+    '',
+    report.params.every((p) => p.valid)
+      ? 'All variables are valid.'
+      : 'Some variables are invalid.',
+  ].join('\n')
+}
+
+/**
+ * Validates environment variables and prepares
+ * a fully printable console output.
+ *
+ * @param {Record<string, Object>} definition
+ * @param {Record<string, any>} values
+ * @param {{
+ *   mask?: (value: any) => string
+ * }} [options]
+ *
+ * @returns {{
+ *   success: boolean,
+ *   output: string,
+ *   report: any
+ * }}
+ */
+export function validateEnvForConsole(definition, values, options = {}) {
+  const { mask } = options
+
+  const validation = validateEnv(definition, values)
+  const report = buildEnvReport(definition, values, validation, mask)
+  const output = buildConsoleOutput(report)
+
+  return {
+    output,
+    report,
+    success: validation.success,
+  }
+}

package/tests/env/prepare-env-validation.test.js

@@ -2,6 +2,8 @@ import { describe, it, expect } from 'vitest'
 import { prepareEnvValidation } from '../../src/env/env-validation.js'
 
 describe('prepareEnvValidation', () => {
+  const mask = () => '****'
+
   const definition = {
     PORT: {
       type: 'number',
@@ -31,13 +33,8 @@ describe('prepareEnvValidation', () => {
     const result = prepareEnvValidation(definition, values, { mask })
 
     expect(result.success).toBe(true)
-    expect(result.validation.success).toBe(true)
-
     expect(result.table).toContain('Environment variables')
     expect(result.table).toContain('PORT')
-    expect(result.table).toContain('MODE')
-    expect(result.table).toContain('API_KEY')
-
     expect(result.table).toContain('OK')
     expect(result.table).toContain('****')
     expect(result.table).not.toContain('secret')

package/tests/env/validate-env-for-console.test.js

@@ -0,0 +1,122 @@
+import { describe, it, expect } from 'vitest'
+import { validateEnvForConsole } from '../../src/env/env-validation.js'
+
+describe('validateEnvForConsole', () => {
+  const definition = {
+    PORT: {
+      type: 'number',
+      required: true,
+      int: true,
+      min: 1,
+      max: 65535,
+    },
+    MODE: {
+      type: 'string',
+      enum: ['development', 'production'],
+      required: true,
+    },
+    API_KEY: {
+      type: 'string',
+      secret: true,
+    },
+  }
+
+  const mask = () => '****'
+
+  it('returns a printable table with all variables and OK notes when valid', () => {
+    const values = {
+      PORT: '3000',
+      MODE: 'development',
+      API_KEY: 'secret',
+    }
+
+    const result = validateEnvForConsole(definition, values, { mask })
+
+    expect(result.success).toBe(true)
+    expect(typeof result.output).toBe('string')
+
+    // header
+    expect(result.output).toContain('Environment variables')
+    expect(result.output).toContain('STATUS')
+    expect(result.output).toContain('KEY')
+    expect(result.output).toContain('VALUE')
+    expect(result.output).toContain('NOTES')
+
+    // rows
+    expect(result.output).toContain('PORT')
+    expect(result.output).toContain('MODE')
+    expect(result.output).toContain('API_KEY')
+
+    // notes always present
+    expect(result.output).toContain('OK      PORT')
+    expect(result.output).toContain('OK      MODE')
+    expect(result.output).toContain('OK      API_KEY')
+
+    // masking
+    expect(result.output).toContain('****')
+    expect(result.output).not.toContain('secret')
+  })
+
+  it('returns a printable table with ERROR notes when invalid', () => {
+    const values = {
+      PORT: '0',
+      MODE: 'prod',
+      API_KEY: 'secret',
+    }
+
+    const result = validateEnvForConsole(definition, values, { mask })
+
+    expect(result.success).toBe(false)
+    expect(typeof result.output).toBe('string')
+
+    // all variables still appear
+    expect(result.output).toContain('PORT')
+    expect(result.output).toContain('MODE')
+    expect(result.output).toContain('API_KEY')
+
+    // error rows
+    expect(result.output).toContain('ERROR   PORT')
+    expect(result.output).toContain('ERROR   MODE')
+
+    // notes are NOT empty
+    expect(result.output).toMatch(/PORT\s+0\s+.+/)
+    expect(result.output).toMatch(/MODE\s+prod\s+.+/)
+
+    // summary line
+    expect(result.output).toContain('Some variables are invalid.')
+  })
+
+  it('always sets NOTES to OK for valid variables even if others fail', () => {
+    const values = {
+      PORT: '3000',
+      MODE: 'prod', // invalid
+      API_KEY: 'secret',
+    }
+
+    const result = validateEnvForConsole(definition, values, { mask })
+
+    // PORT and API_KEY are valid → NOTES = OK
+    expect(result.output).toMatch(/OK\s+PORT\s+3000\s+OK/)
+    expect(result.output).toMatch(/OK\s+API_KEY\s+\*\*\*\*\s+OK/)
+
+    // MODE is invalid → NOTES = error message
+    expect(result.output).toMatch(/ERROR\s+MODE\s+prod\s+.+/)
+  })
+
+  it('always returns output even if multiple errors exist', () => {
+    const values = {
+      MODE: 'prod',
+    }
+
+    const result = validateEnvForConsole(definition, values)
+
+    expect(result.success).toBe(false)
+    expect(typeof result.output).toBe('string')
+    expect(result.output.length).toBeGreaterThan(0)
+
+    // all keys still listed
+    expect(result.output).toContain('PORT')
+    expect(result.output).toContain('MODE')
+    expect(result.output).toContain('API_KEY')
+  })
+})

package/types/env/env-validation.d.ts

@@ -279,3 +279,57 @@ export function prepareEnvValidation(
     }>
   }
 }
+/**
+ * Builds a console-ready table output for environment validation.
+ *
+ * Always includes all variables.
+ * NOTES column always has a value:
+ * - 'OK' for valid variables
+ * - error message(s) for invalid variables
+ *
+ * @param {{
+ *   params: Array<{
+ *     key: string,
+ *     displayValue: string,
+ *     valid: boolean,
+ *     errors?: string[]
+ *   }>
+ * }} report
+ *
+ * @returns {string}
+ */
+export function buildConsoleOutput(report: {
+  params: Array<{
+    key: string
+    displayValue: string
+    valid: boolean
+    errors?: string[]
+  }>
+}): string
+/**
+ * Validates environment variables and prepares
+ * a fully printable console output.
+ *
+ * @param {Record<string, Object>} definition
+ * @param {Record<string, any>} values
+ * @param {{
+ *   mask?: (value: any) => string
+ * }} [options]
+ *
+ * @returns {{
+ *   success: boolean,
+ *   output: string,
+ *   report: any
+ * }}
+ */
+export function validateEnvForConsole(
+  definition: Record<string, any>,
+  values: Record<string, any>,
+  options?: {
+    mask?: (value: any) => string
+  },
+): {
+  success: boolean
+  output: string
+  report: any
+}