@platformatic/foundation 3.65.0 → 3.67.0

package/index.d.ts

@@ -3,6 +3,7 @@ import { JSONSchemaType } from 'ajv'
 import { EventEmitter } from 'node:events'
 import { ParseArgsOptionsConfig } from 'node:util'
 import { LevelWithSilentOrString, Logger } from 'pino'
+import type debug from 'debug';
 
 // Symbols
 export declare const kCanceled: unique symbol

package/lib/configuration.js

@@ -513,6 +513,7 @@ export async function loadConfiguration (source, schema, options = {}) {
 
   if (shouldReplaceEnv) {
     const missingEnv = new Set()
+    const fallbackEnv = new Set()
 
     config = replaceEnv(
       config,
@@ -522,6 +523,11 @@ export async function loadConfiguration (source, schema, options = {}) {
 
         if (typeof value === 'undefined' || value === null) {
           missingEnv.add(key)
+        } else {
+          // The variable is not set: it only has a value because onMissingEnv provided a fallback.
+          // Track it separately so that strictEnv can still report it, as a fallback can silently
+          // mask a misconfiguration.
+          fallbackEnv.add(key)
         }
 
         return value
@@ -533,17 +539,31 @@ export async function loadConfiguration (source, schema, options = {}) {
     // (runtime configurations) or in the runtime property (capabilities configurations).
     const strictEnv = normalizeStrictEnv(strictEnvOption ?? config.strictEnv ?? config.runtime?.strictEnv)
 
+    function warn (message) {
+      if (logger) {
+        logger.warn(message)
+      } else {
+        process.emitWarning(message)
+      }
+    }
+
+    // Variables resolved by a fallback are always reported as a warning, never as an error: they did
+    // resolve to a value, so failing on them would change which configurations are able to boot.
+    // This is emitted before handling the missing ones so that a throw does not swallow it.
+    if (strictEnv && fallbackEnv.size > 0) {
+      const keys = Array.from(fallbackEnv).sort().join(', ')
+
+      warn(
+        'The configuration references the following environment variables which are not set ' +
+          `and have been replaced by a fallback value: ${keys}`
+      )
+    }
+
     if (strictEnv && missingEnv.size > 0) {
       const keys = Array.from(missingEnv).sort().join(', ')
 
       if (strictEnv === 'warn') {
-        const message = `The configuration references the following environment variables which are not set: ${keys}`
-
-        if (logger) {
-          logger.warn(message)
-        } else {
-          process.emitWarning(message)
-        }
+        warn(`The configuration references the following environment variables which are not set: ${keys}`)
       } else {
         throw new MissingEnvVariablesError(keys)
       }

package/lib/schema.js

@@ -111,6 +111,10 @@ const extension = {
         options: {
           type: 'object',
           additionalProperties: true
+        },
+        build: {
+          type: 'boolean',
+          default: false
         }
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/foundation",
-  "version": "3.65.0",
+  "version": "3.67.0",
   "description": "Platformatic Foundation",
   "main": "index.js",
   "types": "index.d.ts",
@@ -30,10 +30,11 @@
     "pino": "^9.9.0",
     "pino-pretty": "^13.0.0",
     "semver": "^7.6.3",
-    "undici": "7.28.0",
+    "undici": "^7.28.0",
     "yaml": "^2.4.1"
   },
   "devDependencies": {
+    "@types/debug": "^4.0.0",
     "c8": "^11.0.0",
     "cleaner-spec-reporter": "^0.5.0",
     "eslint": "9",
@@ -42,7 +43,7 @@
     "pino": "^9.9.0",
     "pino-test": "^1.0.1",
     "tstyche": "^6.2.0",
-    "typescript": "^5.9.3"
+    "typescript": "~6.0.3"
   },
   "engines": {
     "node": ">=22.19.0"
@@ -50,6 +51,7 @@
   "scripts": {
     "test": "npm run test:types && node --test --test-reporter=cleaner-spec-reporter --test-concurrency=1 --test-timeout=2000000 test/*.test.js test/**/*.test.js",
     "test:types": "tstyche",
-    "lint": "eslint"
+    "lint": "eslint",
+    "lint:fix": "eslint --fix"
   }
 }