@platformatic/runtime 3.68.0 → 3.69.0

package/lib/runtime.js

@@ -2865,7 +2865,21 @@ export class Runtime extends EventEmitter {
     // Setup the interceptor
     // kInterceptorReadyPromise resolves when the worker
     // is ready to receive requests: after calling the replaceServer method
-    worker[kInterceptorReadyPromise] = this.#meshInterceptor.route(applicationId, worker)
+    //
+    // It is stored, not awaited: #startWorker awaits it later, and only if the
+    // worker is actually started. A worker that is discarded or torn down
+    // before that — a replacement abandoned because the runtime stopped
+    // mid-restart, for instance — leaves the promise with nobody to observe
+    // it. Closing the mesh interceptor rejects any routing still in flight
+    // ('The dispatcher has been closed.'), so that is a routine shutdown
+    // outcome, not an exotic one, and it surfaced as an unhandledRejection
+    // that could take the process down before graceful shutdown finished.
+    //
+    // Attaching a no-op handler marks the rejection observed WITHOUT
+    // swallowing it: the await in #startWorker still sees and reports it.
+    const interceptorReady = this.#meshInterceptor.route(applicationId, worker)
+    interceptorReady.catch(() => {})
+    worker[kInterceptorReadyPromise] = interceptorReady
 
     // Wait for initialization
     try {

package/lib/worker/controller.js

@@ -1,4 +1,5 @@
 import {
+  convertApplicationNameToPrefix,
   ensureLoggableError,
   FileWatcher,
   kHandledError,
@@ -28,12 +29,27 @@ import { ApplicationAlreadyStartedError, exitCodes, RuntimeNotStartedError } fro
 import { getApplicationUrl } from '../utils.js'
 import { markAsPlatformaticDispatcher, refreshGlobalDispatcher } from './interceptors.js'
 
-function fetchApplicationUrl (application, key) {
-  if (!key.endsWith('_URL') || !application.id) {
-    return null
+/**
+ * Resolves {PLT_<APPLICATION>_URL} placeholders that no environment variable defines, so that
+ * applications can reference each other without the user having to set a variable per pair.
+ *
+ * The mapping is built forwards, from the ids of the applications that actually exist in the runtime
+ * to the variable name each of them owns. Parsing the variable name instead would be ambiguous, both
+ * because the id to prefix conversion is not reversible ("with-logger" and "with_logger" share a
+ * prefix) and because plenty of _URL variables do not name an application at all: a connection string
+ * fragment like {VALKEY_URL} or a DSN like {PLT_DATABASE_URL} must keep resolving to nothing rather
+ * than silently becoming an HTTP URL.
+ */
+function buildApplicationUrlResolver (applications) {
+  const urls = new Map()
+
+  for (const { id } of applications ?? []) {
+    if (id) {
+      urls.set(`PLT_${convertApplicationNameToPrefix(id)}_URL`, getApplicationUrl(id))
+    }
   }
 
-  return getApplicationUrl(application.id)
+  return key => urls.get(key) ?? null
 }
 
 function handleUnhandled (app, event, listeners, timeout, err, ...args) {
@@ -78,6 +94,8 @@ export class Controller extends EventEmitter {
     this.capability = null
     this.#fileWatcher = null
 
+    const onMissingEnv = buildApplicationUrlResolver(runtimeConfig.applications)
+
     this.#context = {
       controller: this,
       runtimeConfig: this.runtimeConfig,
@@ -95,7 +113,10 @@ export class Controller extends EventEmitter {
       worker: workerData?.worker,
       resourceLimits: workerData?.resourceLimits,
       hasManagementApi: !!runtimeConfig.managementApi,
-      fetchApplicationUrl: fetchApplicationUrl.bind(null, applicationConfig),
+      fetchApplicationUrl: onMissingEnv,
+      // Capabilities spread the whole context into their loadConfiguration call, which is what makes
+      // the configuration they load resolve application references like the throwaway load below.
+      onMissingEnv,
       strictEnv: runtimeConfig.strictEnv
     }
   }
@@ -143,10 +164,11 @@ export class Controller extends EventEmitter {
         // Parse the configuration file the first time to obtain the schema. This load is thrown away:
         // the capability loads the configuration again below and that is the configuration the
         // application actually runs on. This one only has to yield $schema and module, so it must not
-        // diverge from the second load in any way the user can observe: it deliberately applies no
-        // onMissingEnv fallback and emits no strictEnv report, both of which are left to the load the
-        // application is built from.
+        // diverge from the second load in any way the user can observe: it resolves environment
+        // variables identically, and leaves the strictEnv report to the load the application is
+        // built from, which would otherwise be duplicated for every application.
         const unvalidatedConfig = await loadConfiguration(appConfig.config, null, {
+          onMissingEnv: this.#context.onMissingEnv,
           strictEnv: false
         })
         const pkg = await loadConfigurationModule(appConfig.path, unvalidatedConfig)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/runtime",
-  "version": "3.68.0",
+  "version": "3.69.0",
   "description": "Platformatic runtime",
   "main": "index.js",
   "type": "module",
@@ -35,18 +35,18 @@
     "pino-abstract-transport": "^2.0.0",
     "pino-sentry-transport": "^1.6.0",
     "split2": "^4.2.0",
-    "tstyche": "^6.2.0",
+    "tstyche": "^7.0.0",
     "typescript": "~6.0.3",
     "undici-oidc-interceptor": "^0.9.0",
     "why-is-node-running": "^2.2.2",
-    "@platformatic/composer": "3.68.0",
-    "@platformatic/db": "3.68.0",
-    "@platformatic/gateway": "3.68.0",
-    "@platformatic/node": "3.68.0",
-    "@platformatic/service": "3.68.0",
-    "@platformatic/sql-graphql": "3.68.0",
-    "@platformatic/sql-mapper": "3.68.0",
-    "@platformatic/wattpm-pprof-capture": "3.68.0"
+    "@platformatic/composer": "3.69.0",
+    "@platformatic/db": "3.69.0",
+    "@platformatic/gateway": "3.69.0",
+    "@platformatic/node": "3.69.0",
+    "@platformatic/service": "3.69.0",
+    "@platformatic/sql-graphql": "3.69.0",
+    "@platformatic/sql-mapper": "3.69.0",
+    "@platformatic/wattpm-pprof-capture": "3.69.0"
   },
   "dependencies": {
     "@fastify/accepts": "^5.0.0",
@@ -77,13 +77,13 @@
     "undici": "^7.27.2",
     "undici-thread-interceptor": "^1.3.1",
     "ws": "^8.16.0",
-    "@platformatic/generators": "3.68.0",
-    "@platformatic/globals": "3.68.0",
-    "@platformatic/itc": "3.68.0",
-    "@platformatic/metrics": "3.68.0",
-    "@platformatic/telemetry": "3.68.0",
-    "@platformatic/basic": "3.68.0",
-    "@platformatic/foundation": "3.68.0"
+    "@platformatic/basic": "3.69.0",
+    "@platformatic/foundation": "3.69.0",
+    "@platformatic/globals": "3.69.0",
+    "@platformatic/metrics": "3.69.0",
+    "@platformatic/generators": "3.69.0",
+    "@platformatic/telemetry": "3.69.0",
+    "@platformatic/itc": "3.69.0"
   },
   "engines": {
     "node": ">=22.19.0"

package/schema.json

@@ -1,5 +1,5 @@
 {
-  "$id": "https://schemas.platformatic.dev/@platformatic/runtime/3.68.0.json",
+  "$id": "https://schemas.platformatic.dev/@platformatic/runtime/3.69.0.json",
   "$schema": "http://json-schema.org/draft-07/schema#",
   "title": "Platformatic Runtime Config",
   "type": "object",