@platformatic/runtime 0.31.1 → 0.33.0

package/README.md

@@ -1,6 +1,6 @@
 # @platformatic/runtime
 
-Check out the full documentation for Platformatic Runtime on [our website](https://oss.platformatic.dev/docs/getting-started/quick-start-guide).
+Check out the full documentation for Platformatic Runtime on [our website](https://docs.platformatic.dev/docs/getting-started/quick-start-guide).
 
 ## Install
 

package/fixtures/configs/hotreload.json

@@ -0,0 +1,20 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.32.0/runtime",
+  "entrypoint": "serviceApp",
+  "allowCycles": true,
+  "hotReload": true,
+  "autoload": {
+    "path": "../monorepo",
+    "exclude": ["docs", "composerApp"],
+    "mappings": {
+      "serviceAppWithLogger": {
+        "id": "with-logger",
+        "config": "platformatic.service.json"
+      },
+      "serviceAppWithMultiplePlugins": {
+        "id": "multi-plugin-service",
+        "config": "platformatic.service.json"
+      }
+    }
+  }
+}

package/fixtures/leven/.env.sample

@@ -0,0 +1,4 @@
+PLT_SERVER_HOSTNAME=127.0.0.1
+PORT=3043
+PLT_SERVER_LOGGER_LEVEL=info
+PLT_EXAMPLE=world

package/fixtures/leven/platformatic.runtime.json

@@ -0,0 +1,12 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.31.0/runtime",
+  "entrypoint": "rainy-empire",
+  "allowCycles": false,
+  "hotReload": true,
+  "autoload": {
+    "path": "services",
+    "exclude": [
+      "docs"
+    ]
+  }
+}

package/fixtures/leven/services/deeply-spittle/platformatic.service.json

@@ -0,0 +1,25 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.31.0/service",
+  "server": {
+    "hostname": "{PLT_SERVER_HOSTNAME}",
+    "port": "{PORT}",
+    "logger": {
+      "level": "{PLT_SERVER_LOGGER_LEVEL}"
+    }
+  },
+  "service": {
+    "openapi": true
+  },
+  "plugins": {
+    "paths": [
+      {
+        "path": "./plugins",
+        "encapsulate": false,
+        "options": {
+          "example": "{PLT_EXAMPLE}"
+        }
+      },
+      "./routes"
+    ]
+  }
+}

package/fixtures/leven/services/deeply-spittle/plugins/example.js

@@ -0,0 +1,6 @@
+/// <reference types="@platformatic/service" />
+'use strict'
+/** @param {import('fastify').FastifyInstance} fastify */
+module.exports = async function (fastify, opts) {
+  fastify.decorate('example', 'foobar')
+}

package/fixtures/leven/services/deeply-spittle/routes/root.js

@@ -0,0 +1,8 @@
+/// <reference types="@platformatic/service" />
+'use strict'
+/** @param {import('fastify').FastifyInstance} fastify */
+module.exports = async function (fastify, opts) {
+  fastify.get('/', async (request, reply) => {
+    return { hello: fastify.example }
+  })
+}

package/fixtures/leven/services/rainy-empire/platformatic.composer.json

@@ -0,0 +1,21 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.31.0/composer",
+  "server": {
+    "hostname": "{PLT_SERVER_HOSTNAME}",
+    "port": "{PORT}",
+    "logger": {
+      "level": "{PLT_SERVER_LOGGER_LEVEL}"
+    }
+  },
+  "composer": {
+    "services": [
+      {
+        "id": "deeply-splitte",
+        "openapi": {
+          "url": "/documentation/json"
+        }
+      }
+    ]
+  },
+  "watch": false
+}

package/fixtures/monorepo/serviceAppWithLogger/platformatic.service.json

@@ -15,5 +15,6 @@
     "paths": [
       "plugin.js"
     ]
-  }
+  },
+  "watch": true
 }

package/fixtures/no-env.service.json

@@ -0,0 +1,25 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.31.0/service",
+  "server": {
+    "hostname": "{PLT_SERVER_HOSTNAME}",
+    "port": "{PORT}",
+    "logger": {
+      "level": "{PLT_SERVER_LOGGER_LEVEL}"
+    }
+  },
+  "service": {
+    "openapi": true
+  },
+  "plugins": {
+    "paths": [
+      {
+        "path": "./plugins",
+        "encapsulate": false,
+        "options": {
+          "example": "{PLT_EXAMPLE}"
+        }
+      },
+      "./routes"
+    ]
+  }
+}

package/fixtures/start-command-in-runtime.js

@@ -1,10 +1,10 @@
 'use strict'
 const assert = require('node:assert')
 const { request } = require('undici')
-const { startCommandInRuntime } = require('../lib/unified-api')
+const { startCommand } = require('../lib/unified-api')
 
 async function main () {
-  const entrypoint = await startCommandInRuntime(['-c', process.argv[2]])
+  const entrypoint = await startCommand(['-c', process.argv[2]])
   const endpoint = process.argv[3] ?? '/'
   const res = await request(entrypoint + endpoint)
 

package/fixtures/telemetry/platformatic.runtime.json

@@ -0,0 +1,19 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.31.0/runtime",
+  "entrypoint": "echo",
+  "allowCycles": false,
+  "hotReload": true,
+  "autoload": {
+    "path": "services",
+    "exclude": [
+      "docs"
+    ]
+  },
+  "telemetry": {
+    "serviceName": "test-runtime",
+    "version": "1.0.0",
+    "exporter": {
+      "type": "memory"
+    }
+  }
+}

package/fixtures/telemetry/services/echo/platformatic.service.json

@@ -0,0 +1,19 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.28.1/service",
+  "server": {
+    "hostname": "127.0.0.1",
+    "port": "0",
+    "logger": {
+      "level": "info"
+    }
+  },
+  "service": {
+    "openapi": true
+  },
+  "plugins": {
+    "paths": [
+      "./routes"
+    ],
+    "typescript": false
+  }
+}

package/fixtures/telemetry/services/echo/routes/span.js

@@ -0,0 +1,8 @@
+'use strict'
+module.exports = async function (fastify, opts) {
+  // This returns the traceId set on the span by the service
+  fastify.get('/', async (request, reply) => {
+    const traceId = request.span.spanContext().traceId
+    return { traceId }
+  })
+}

package/lib/api.js

@@ -11,10 +11,12 @@ class RuntimeApi {
 
   constructor (config, logger, loaderPort) {
     this.#services = new Map()
+    const telemetryConfig = config.telemetry
 
     for (let i = 0; i < config.services.length; ++i) {
       const service = config.services[i]
-      const app = new PlatformaticApp(service, loaderPort, logger)
+      const serviceTelemetryConfig = telemetryConfig ? { ...telemetryConfig, serviceName: `${telemetryConfig.serviceName}-${service.id}` } : null
+      const app = new PlatformaticApp(service, loaderPort, logger, serviceTelemetryConfig)
 
       this.#services.set(service.id, app)
     }
@@ -47,9 +49,9 @@ class RuntimeApi {
   }
 
   async #handleProcessLevelEvent (message) {
-    for (const service of this.#services.values()) {
+    await Promise.allSettled(this.#services.values().map(async (service) => {
       await service.handleProcessLevelEvent(message)
-    }
+    }))
   }
 
   async #executeCommand (message) {

package/lib/app.js

@@ -1,7 +1,9 @@
 'use strict'
+
 const { once } = require('node:events')
 const { dirname, basename } = require('node:path')
 const { FileWatcher } = require('@platformatic/utils')
+const debounce = require('debounce')
 const {
   buildServer,
   loadConfig
@@ -13,9 +15,11 @@ class PlatformaticApp {
   #restarting
   #started
   #originalWatch
+  #fileWatcher
   #logger
+  #telemetryConfig
 
-  constructor (appConfig, loaderPort, logger) {
+  constructor (appConfig, loaderPort, logger, telemetryConfig) {
     this.appConfig = appConfig
     this.config = null
     this.#hotReload = false
@@ -24,7 +28,11 @@ class PlatformaticApp {
     this.server = null
     this.#started = false
     this.#originalWatch = null
-    this.#logger = logger
+    this.#fileWatcher = null
+    this.#logger = logger.child({
+      name: this.appConfig.id
+    })
+    this.#telemetryConfig = telemetryConfig
   }
 
   getStatus () {
@@ -36,16 +44,11 @@ class PlatformaticApp {
   }
 
   async restart (force) {
-    if (this.#restarting) {
-      return
-    }
-
-    if (!this.#hotReload && !force) {
+    if (this.#restarting || !this.#started || (!this.#hotReload && !force)) {
       return
     }
 
     this.#restarting = true
-    await this.stop()
 
     /* c8 ignore next 4 - tests may not pass in a MessagePort. */
     if (this.#loaderPort) {
@@ -53,7 +56,13 @@ class PlatformaticApp {
       await once(this.#loaderPort, 'message')
     }
 
-    await this.start()
+    this.#setuplogger(this.config.configManager)
+    try {
+      await this.server.restart()
+    } catch (err) {
+      this.#logAndExit(err)
+    }
+
     this.#restarting = false
   }
 
@@ -62,38 +71,37 @@ class PlatformaticApp {
       throw new Error('application is already started')
     }
 
-    if (!this.#restarting) {
-      await this.#initializeConfig()
-    }
+    this.#started = true
+
+    await this.#initializeConfig()
+    this.#originalWatch = this.config.configManager.current.watch
+    this.config.configManager.current.watch = false
+
     const { configManager } = this.config
+    configManager.update({
+      ...configManager.current,
+      telemetry: this.#telemetryConfig
+    })
     const config = configManager.current
 
-    this.#originalWatch = config.watch
-    config.watch = false
     this.#setuplogger(configManager)
 
     try {
       // If this is a restart, have the fastify server restart itself. If this
       // is not a restart, then create a new server.
-      if (this.#restarting) {
-        await this.server.restart()
-      } else {
-        this.server = await buildServer({
-          ...config,
-          configManager
-        })
-      }
+      this.server = await buildServer({
+        ...config,
+        configManager
+      })
     } catch (err) {
       this.#logAndExit(err)
     }
 
-    if (config.plugins !== undefined && this.#originalWatch !== false) {
+    if (config.plugins !== undefined) {
       this.#startFileWatching()
     }
 
-    this.#started = true
-
-    if (this.appConfig.entrypoint && !this.#restarting) {
+    if (this.appConfig.entrypoint) {
       try {
         await this.server.start()
         /* c8 ignore next 5 */
@@ -109,11 +117,9 @@ class PlatformaticApp {
       throw new Error('application has not been started')
     }
 
-    if (!this.#restarting) {
-      await this.server.close()
-    }
-
     await this.#stopFileWatching()
+    await this.server.close()
+
     this.#started = false
   }
 
@@ -148,12 +154,19 @@ class PlatformaticApp {
   async #initializeConfig () {
     const appConfig = this.appConfig
 
-    this.config = await loadConfig({}, ['-c', appConfig.config], null, {
-      watch: true,
-      onMissingEnv (key) {
-        return appConfig.localServiceEnvVars.get(key)
-      }
-    })
+    let _config
+    try {
+      _config = await loadConfig({}, ['-c', appConfig.config], null, {
+        watch: true,
+        onMissingEnv (key) {
+          return appConfig.localServiceEnvVars.get(key)
+        }
+      })
+    } catch (err) {
+      this.#logAndExit(err)
+    }
+
+    this.config = _config
     const { configManager } = this.config
 
     function applyOverrides () {
@@ -190,11 +203,13 @@ class PlatformaticApp {
     this.#hotReload = this.appConfig.hotReload
 
     configManager.on('update', async (newConfig) => {
-      this.server.platformatic.config = newConfig
-      applyOverrides()
-      this.server.log.debug('config changed')
-      this.server.log.trace({ newConfig }, 'new config')
-      await this.restart()
+      if (this.server) { // when we setup telemetry on config, we don't have a server yet
+        this.server.platformatic.config = newConfig
+        applyOverrides()
+        this.server.log.debug('config changed')
+        this.server.log.trace({ newConfig }, 'new config')
+        await this.restart()
+      }
     })
 
     configManager.on('error', (err) => {
@@ -206,14 +221,15 @@ class PlatformaticApp {
   #setuplogger (configManager) {
     // Set the logger if not present (and the config supports it).
     if (configManager.current.server) {
-      const childLogger = this.#logger.child({
-        name: this.appConfig.id
-      }, { level: configManager.current.server.logger?.level || 'info' })
+      const childLogger = this.#logger.child({}, { level: configManager.current.server.logger?.level || 'info' })
       configManager.current.server.logger = childLogger
     }
   }
 
   #startFileWatching () {
+    if (this.#fileWatcher) {
+      return
+    }
     const server = this.server
     const { configManager } = server.platformatic
     // TODO FileWatcher and ConfigManager both watch the configuration file
@@ -226,27 +242,30 @@ class PlatformaticApp {
     })
 
     /* c8 ignore next 4 */
-    fileWatcher.on('update', async () => {
-      this.server.log.debug('files changed')
+    const restart = debounce(() => {
+      this.server.log.info('files changed')
       this.restart()
-    })
+    }, 100) // debounce restart for 100ms
+    fileWatcher.on('update', restart)
 
     fileWatcher.startWatching()
     server.log.debug('start watching files')
     server.platformatic.fileWatcher = fileWatcher
     server.platformatic.configManager.startWatching()
+    this.#fileWatcher = fileWatcher
   }
 
   async #stopFileWatching () {
-    const watcher = this.server.platformatic.fileWatcher
     // The configManager automatically watches for the config file changes
     // therefore we need to stop it all the times.
     await this.config.configManager.stopWatching()
 
+    const watcher = this.#fileWatcher
     if (watcher) {
       this.server.log.debug('stop watching files')
       await watcher.stopWatching()
       this.server.platformatic.fileWatcher = undefined
+      this.#fileWatcher = null
     }
   }
 

package/lib/config.js

@@ -1,6 +1,7 @@
 'use strict'
 const { readFile, readdir } = require('node:fs/promises')
 const { basename, join, resolve: pathResolve } = require('node:path')
+const { closest } = require('fastest-levenshtein')
 const Topo = require('@hapi/topo')
 const ConfigManager = require('@platformatic/config')
 const { schema } = require('./schema')
@@ -71,6 +72,15 @@ async function _transformConfig (configManager) {
   }
 }
 
+function missingDependencyErrorMessage (clientName, service, configManager) {
+  const closestName = closest(clientName, [...configManager.current.serviceMap.keys()])
+  let errorMsg = `service '${service.id}' has unknown dependency: '${clientName}'.`
+  if (closestName) {
+    errorMsg += ` Did you mean '${closestName}'?`
+  }
+  return errorMsg
+}
+
 async function parseClientsAndComposer (configManager) {
   for (let i = 0; i < configManager.current.services.length; ++i) {
     const service = configManager.current.services[i]
@@ -86,8 +96,7 @@ async function parseClientsAndComposer (configManager) {
         const dependency = configManager.current.serviceMap.get(clientName)
 
         if (dependency === undefined) {
-          /* c8 ignore next 2 */
-          throw new Error(`service '${service.id}' has unknown dependency: '${clientName}'`)
+          throw new Error(missingDependencyErrorMessage(clientName, service, configManager))
         }
 
         dependency.dependents.push(service.id)
@@ -170,7 +179,7 @@ async function parseClientsAndComposer (configManager) {
 
           /* c8 ignore next 4 */
           if (dependency === undefined) {
-            reject(new Error(`service '${service.id}' has unknown dependency: '${clientName}'`))
+            reject(new Error(missingDependencyErrorMessage(clientName, service, configManager)))
             return
           }
 

package/lib/schema.js

@@ -1,6 +1,7 @@
 #! /usr/bin/env node
 'use strict'
 
+const telemetry = require('@platformatic/telemetry').schema
 const pkg = require('../package.json')
 const version = 'v' + pkg.version
 const platformaticRuntimeSchema = {
@@ -42,6 +43,7 @@ const platformaticRuntimeSchema = {
         }
       }
     },
+    telemetry,
     services: {
       type: 'array',
       default: [],
@@ -67,7 +69,14 @@ const platformaticRuntimeSchema = {
       type: 'string'
     },
     hotReload: {
-      type: 'boolean'
+      anyOf: [
+        {
+          type: 'boolean'
+        },
+        {
+          type: 'string'
+        }
+      ]
     },
     allowCycles: {
       type: 'boolean'

package/lib/start.js

@@ -46,8 +46,12 @@ async function startWithConfig (configManager, env = process.env) {
     env
   })
 
+  let exited = null
   worker.on('exit', () => {
     configManager.fileWatcher?.stopWatching()
+    if (typeof exited === 'function') {
+      exited()
+    }
   })
 
   worker.on('error', () => {
@@ -65,8 +69,9 @@ async function startWithConfig (configManager, env = process.env) {
       worker.postMessage({ signal: 'SIGUSR2' })
     })
 
-    closeWithGrace((event) => {
+    closeWithGrace((event, cb) => {
       worker.postMessage(event)
+      exited = cb
     })
 
     /* c8 ignore next 3 */

package/lib/unified-api.js

@@ -154,14 +154,6 @@ async function _start (args) {
 }
 
 async function startCommand (args) {
-  try {
-    await _start(args)
-  } catch (err) {
-    logErrorAndExit(err)
-  }
-}
-
-async function startCommandInRuntime (args) {
   try {
     const configType = await getConfigType(args)
     const config = await _loadConfig({}, args, configType)
@@ -200,6 +192,5 @@ module.exports = {
   loadConfig: _loadConfig,
   start: _start,
   startCommand,
-  startCommandInRuntime,
   getApp
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/runtime",
-  "version": "0.31.1",
+  "version": "0.33.0",
   "description": "",
   "main": "index.js",
   "bin": {
@@ -24,15 +24,17 @@
     "standard": "^17.1.0",
     "tsd": "^0.28.1",
     "typescript": "^5.1.6",
-    "@platformatic/sql-mapper": "0.31.1",
-    "@platformatic/sql-graphql": "0.31.1"
+    "@platformatic/sql-graphql": "0.33.0",
+    "@platformatic/sql-mapper": "0.33.0"
   },
   "dependencies": {
     "@hapi/topo": "^6.0.2",
     "close-with-grace": "^1.2.0",
     "commist": "^3.2.0",
+    "debounce": "^1.2.1",
     "desm": "^1.3.0",
     "es-main": "^1.2.0",
+    "fastest-levenshtein": "^1.0.16",
     "fastify": "^4.18.0",
     "fastify-undici-dispatcher": "^0.4.1",
     "help-me": "^4.2.0",
@@ -40,11 +42,12 @@
     "pino": "^8.14.1",
     "pino-pretty": "^10.0.0",
     "undici": "^5.22.1",
-    "@platformatic/composer": "0.31.1",
-    "@platformatic/config": "0.31.1",
-    "@platformatic/db": "0.31.1",
-    "@platformatic/service": "0.31.1",
-    "@platformatic/utils": "0.31.1"
+    "@platformatic/composer": "0.33.0",
+    "@platformatic/config": "0.33.0",
+    "@platformatic/db": "0.33.0",
+    "@platformatic/service": "0.33.0",
+    "@platformatic/telemetry": "0.33.0",
+    "@platformatic/utils": "0.33.0"
   },
   "standard": {
     "ignore": [
@@ -53,7 +56,8 @@
     ]
   },
   "scripts": {
-    "test": "npm run lint && c8 -x fixtures -x test node --test && tsd",
+    "test": "npm run lint && node --test && tsd",
+    "coverage": "npm run lint && c8 -x fixtures -x test node --test && tsd",
     "lint": "standard | snazzy"
   }
 }

package/runtime.mjs

@@ -6,7 +6,7 @@ import { join } from 'desm'
 import isMain from 'es-main'
 import helpMe from 'help-me'
 import parseArgs from 'minimist'
-import { start } from './lib/start.js'
+import { startCommand } from './lib/unified-api.js'
 import { compile as compileCmd } from './lib/compile.js'
 
 export const compile = compileCmd
@@ -22,7 +22,7 @@ const program = commist({ maxDistance: 2 })
 program.register('help', help.toStdout)
 program.register('help start', help.toStdout.bind(null, ['start']))
 program.register('help compile', help.toStdout.bind(null, ['compile']))
-program.register('start', start)
+program.register('start', startCommand)
 program.register('compile', compile)
 
 export async function run (argv) {

package/test/app.test.js

@@ -90,7 +90,7 @@ test('errors when stopping an already stopped application', async (t) => {
 })
 
 test('does not restart while restarting', async (t) => {
-  const { logger } = getLoggerAndStream()
+  const { logger, stream } = getLoggerAndStream()
   const appPath = join(fixturesDir, 'monorepo', 'serviceApp')
   const configFile = join(appPath, 'platformatic.service.json')
   const config = {
@@ -105,17 +105,33 @@ test('does not restart while restarting', async (t) => {
   }
   const app = new PlatformaticApp(config, null, logger)
 
-  t.after(app.stop.bind(app))
+  t.after(async () => {
+    try {
+      await app.stop()
+    } catch {}
+  })
   await app.start()
-  t.mock.method(app, 'stop')
   await Promise.all([
     app.restart(),
     app.restart(),
     app.restart()
   ])
+  await app.stop()
+  stream.end()
+  const lines = []
+  for await (const line of stream) {
+    lines.push(line)
+  }
 
-  // stop() should have only been called once despite three restart() calls.
-  assert.strictEqual(app.stop.mock.calls.length, 1)
+  let count = 0
+  for (const line of lines) {
+    // every time we restart we log listening
+    if (line.msg.match(/listening/)) {
+      count++
+    }
+  }
+
+  assert.strictEqual(count, 2)
 })
 
 test('restarts on SIGUSR2', async (t) => {
@@ -308,3 +324,35 @@ test('restarts on config change without overriding the configManager', async (t)
   }
   assert.strictEqual(configManager, app.server.platformatic.configManager)
 })
+
+test('logs errors if an env variable is missing', async (t) => {
+  const { logger, stream } = getLoggerAndStream()
+  const configFile = join(fixturesDir, 'no-env.service.json')
+  const config = {
+    id: 'no-env',
+    config: configFile,
+    path: fixturesDir,
+    entrypoint: true,
+    hotReload: true
+  }
+  const app = new PlatformaticApp(config, null, logger)
+
+  t.mock.method(process, 'exit', () => {
+    throw new Error('exited')
+  })
+
+  await assert.rejects(async () => {
+    await app.start()
+  }, /exited/)
+  assert.strictEqual(process.exit.mock.calls.length, 1)
+  assert.strictEqual(process.exit.mock.calls[0].arguments[0], 1)
+
+  stream.end()
+  const lines = []
+  for await (const line of stream) {
+    lines.push(line)
+  }
+  const lastLine = lines[lines.length - 1]
+  assert.strictEqual(lastLine.name, 'no-env')
+  assert.strictEqual(lastLine.msg, 'Cannot parse config file. Cannot read properties of undefined (reading \'get\')')
+})

package/test/cli/compile-2.test.mjs

@@ -0,0 +1,69 @@
+import { test } from 'node:test'
+import assert from 'node:assert'
+import { join } from 'desm'
+import path from 'node:path'
+import { cliPath, delDir } from './helper.mjs'
+import { execa } from 'execa'
+import { mkdtemp, cp, mkdir } from 'node:fs/promises'
+
+const base = join(import.meta.url, '..', 'tmp')
+
+try {
+  await mkdir(base, { recursive: true })
+} catch {
+}
+
+test('compile with tsconfig custom flags', async (t) => {
+  const tmpDir = await mkdtemp(path.join(base, 'test-runtime-compile-'))
+  const prev = process.cwd()
+  process.chdir(tmpDir)
+  t.after(() => {
+    process.chdir(prev)
+  })
+
+  t.after(delDir(tmpDir))
+
+  const folder = join(import.meta.url, '..', '..', 'fixtures', 'typescript-custom-flags')
+  await cp(folder, tmpDir, { recursive: true })
+
+  const { stdout } = await execa(cliPath, ['compile'])
+
+  const lines = stdout.split('\n').map(JSON.parse)
+  const expected = [{
+    name: 'movies',
+    msg: 'Typescript compilation completed successfully.'
+  }, {
+    name: 'titles',
+    msg: 'Typescript compilation completed successfully.'
+  }]
+
+  for (let i = 0; i < expected.length; i++) {
+    assert.deepStrictEqual(lines[i].name, expected[i].name)
+    assert.deepStrictEqual(lines[i].msg, expected[i].msg)
+  }
+})
+
+test('compile single service', async (t) => {
+  const tmpDir = await mkdtemp(path.join(base, 'test-runtime-compile-'))
+  const prev = process.cwd()
+  process.chdir(tmpDir)
+  t.after(() => {
+    process.chdir(prev)
+  })
+
+  t.after(delDir(tmpDir))
+
+  const folder = join(import.meta.url, '..', '..', 'fixtures', 'typescript', 'services', 'movies')
+  await cp(folder, tmpDir, { recursive: true })
+
+  const { stdout } = await execa(cliPath, ['compile'])
+
+  const lines = stdout.split('\n').map(JSON.parse)
+  const expected = [{
+    msg: 'Typescript compilation completed successfully.'
+  }]
+
+  for (let i = 0; i < expected.length; i++) {
+    assert.deepStrictEqual(lines[i].msg, expected[i].msg)
+  }
+})

package/test/cli/compile.test.mjs

@@ -2,10 +2,9 @@ import { test } from 'node:test'
 import assert from 'node:assert'
 import { join } from 'desm'
 import path from 'node:path'
-import { cliPath } from './helper.mjs'
+import { cliPath, delDir } from './helper.mjs'
 import { execa } from 'execa'
-import { mkdtemp, rm, cp, mkdir } from 'node:fs/promises'
-import { setTimeout as sleep } from 'node:timers/promises'
+import { mkdtemp, cp, mkdir } from 'node:fs/promises'
 
 const base = join(import.meta.url, '..', 'tmp')
 
@@ -14,25 +13,6 @@ try {
 } catch {
 }
 
-function delDir (tmpDir) {
-  return async function () {
-    // We give up after 10s.
-    // This is because on Windows, it's very hard to delete files if the file
-    // system is not collaborating.
-    for (let i = 0; i < 10; i++) {
-      try {
-        await rm(tmpDir, { recursive: true, force: true })
-        break
-      } catch (err) {
-        if (err.code === 'EBUSY') {
-          await sleep(1000)
-          continue
-        }
-      }
-    }
-  }
-}
-
 test('compile without tsconfigs', async () => {
   const config = join(import.meta.url, '..', '..', 'fixtures', 'configs', 'monorepo.json')
   await execa(cliPath, ['compile', '-c', config])

package/test/cli/helper.mjs

@@ -3,6 +3,7 @@ import { Agent, setGlobalDispatcher } from 'undici'
 import { execa } from 'execa'
 import split from 'split2'
 import { join } from 'desm'
+import { rm } from 'node:fs/promises'
 
 setGlobalDispatcher(new Agent({
   keepAliveTimeout: 10,
@@ -34,13 +35,34 @@ export async function start (...args) {
 
   for await (const messages of on(output, 'data')) {
     for (const message of messages) {
-      const url = message.url ??
-        message.msg.match(/server listening at (.+)/i)?.[1]
+      if (message.msg) {
+        const url = message.url ??
+          message.msg.match(/server listening at (.+)/i)?.[1]
 
-      if (url !== undefined) {
-        clearTimeout(errorTimeout)
-        return { child, url, output }
+        if (url !== undefined) {
+          clearTimeout(errorTimeout)
+          return { child, url, output }
+        }
       }
     }
   }
 }
+
+export function delDir (tmpDir) {
+  return async function () {
+    console.time('delDir')
+    // We give up after 10s.
+    // This is because on Windows, it's very hard to delete files if the file
+    // system is not collaborating.
+    try {
+      await rm(tmpDir, { recursive: true, force: true })
+    } catch (err) {
+      if (err.code !== 'EBUSY') {
+        throw err
+      } else {
+        console.log('Could not delete directory, retrying', tmpDir)
+      }
+    }
+    console.timeEnd('delDir')
+  }
+}

package/test/cli/start.test.mjs

@@ -74,7 +74,7 @@ test('does not start if node inspector flags are provided', async (t) => {
     for (const message of messages) {
       stderr += message
 
-      if (/Error: The Node.js inspector flags are not supported/.test(stderr)) {
+      if (/The Node.js inspector flags are not supported/.test(stderr)) {
         found = true
         break
       }

package/test/cli/watch.test.mjs

@@ -47,9 +47,13 @@ function createEsmLoggingPlugin (text, reloaded) {
   `
 }
 
+function saferm (path) {
+  return rm(path, { recursive: true, force: true }).catch(() => {})
+}
+
 test('watches CommonJS files', async (t) => {
   const tmpDir = await mkdtemp(join(base, 'watch-'))
-  t.after(() => rm(tmpDir, { recursive: true, force: true }))
+  t.after(() => saferm(tmpDir))
   t.diagnostic(`using ${tmpDir}`)
   const configFileSrc = join(fixturesDir, 'configs', 'monorepo.json')
   const configFileDst = join(tmpDir, 'configs', 'monorepo.json')
@@ -65,8 +69,6 @@ test('watches CommonJS files', async (t) => {
   await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v1', false))
   const { child } = await start('-c', configFileDst)
   t.after(() => child.kill('SIGINT'))
-  child.stdout.pipe(process.stderr)
-  child.stderr.pipe(process.stderr)
 
   await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v2', true))
 
@@ -79,7 +81,7 @@ test('watches CommonJS files', async (t) => {
 
 test('watches ESM files', async (t) => {
   const tmpDir = await mkdtemp(join(base, 'watch-'))
-  t.after(() => rm(tmpDir, { recursive: true, force: true }))
+  t.after(() => saferm(tmpDir))
   t.diagnostic(`using ${tmpDir}`)
   const configFileSrc = join(fixturesDir, 'configs', 'monorepo.json')
   const configFileDst = join(tmpDir, 'configs', 'monorepo.json')
@@ -95,8 +97,6 @@ test('watches ESM files', async (t) => {
   await writeFile(esmPluginFilePath, createEsmLoggingPlugin('v1', false))
   const { child } = await start('-c', configFileDst)
   t.after(() => child.kill('SIGINT'))
-  child.stdout.pipe(process.stderr)
-  child.stderr.pipe(process.stderr)
   await writeFile(esmPluginFilePath, createEsmLoggingPlugin('v2', true))
 
   for await (const log of child.ndj) {
@@ -108,7 +108,7 @@ test('watches ESM files', async (t) => {
 
 test('should not hot reload files with `--hot-reload false', async (t) => {
   const tmpDir = await mkdtemp(join(base, 'watch-'))
-  t.after(() => rm(tmpDir, { recursive: true, force: true }))
+  t.after(() => saferm(tmpDir))
   t.diagnostic(`using ${tmpDir}`)
   const configFileSrc = join(fixturesDir, 'configs', 'monorepo.json')
   const configFileDst = join(tmpDir, 'configs', 'monorepo.json')
@@ -124,11 +124,84 @@ test('should not hot reload files with `--hot-reload false', async (t) => {
   await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v1', false))
   const { child, url } = await start('-c', configFileDst, '--hot-reload', 'false')
   t.after(() => child.kill('SIGINT'))
-  child.stdout.pipe(process.stderr)
-  child.stderr.pipe(process.stderr)
   await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v2', true))
   await sleep(5000)
   const res = await request(`${url}/version`)
   const version = await res.body.text()
   assert.strictEqual(version, 'v1')
 })
+
+test('watches CommonJS files with hotreload', { timeout: 30000, skip: process.env.CI }, async (t) => {
+  const tmpDir = await mkdtemp(join(base, 'watch-'))
+  t.after(() => saferm(tmpDir))
+  t.diagnostic(`using ${tmpDir}`)
+  const configFileSrc = join(fixturesDir, 'configs', 'hotreload.json')
+  const configFileDst = join(tmpDir, 'configs', 'monorepo.json')
+  const appSrc = join(fixturesDir, 'monorepo')
+  const appDst = join(tmpDir, 'monorepo')
+  const cjsPluginFilePath = join(appDst, 'serviceAppWithLogger', 'plugin.js')
+
+  await Promise.all([
+    cp(configFileSrc, configFileDst),
+    cp(appSrc, appDst, { recursive: true })
+  ])
+
+  await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v1', false))
+  const { child } = await start('-c', configFileDst)
+  t.after(() => child.kill('SIGINT'))
+
+  await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v2', true))
+
+  let restartedSecondTime = false
+  let restartedThirdTime = false
+
+  for await (const log of child.ndj) {
+    if (log.msg === 'RELOADED v2') {
+      restartedSecondTime = true
+    } else if (log.msg === 'RELOADED v3') {
+      restartedThirdTime = true
+      break
+    } else if (log.msg?.match(/watching/)) {
+      await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v3', true))
+    }
+  }
+
+  assert.ok(restartedSecondTime)
+  assert.ok(restartedThirdTime)
+})
+
+test('watches CommonJS files with hotreload on a single service', { timeout: 30000, skip: process.env.CI }, async (t) => {
+  const tmpDir = await mkdtemp(join(base, 'watch-'))
+  t.after(() => saferm(tmpDir))
+  t.diagnostic(`using ${tmpDir}`)
+  const appSrc = join(fixturesDir, 'monorepo', 'serviceAppWithLogger')
+  const appDst = join(tmpDir)
+  const cjsPluginFilePath = join(appDst, 'plugin.js')
+
+  await Promise.all([
+    cp(appSrc, appDst, { recursive: true })
+  ])
+
+  await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v1', false))
+  const { child } = await start('-c', join(appDst, 'platformatic.service.json'))
+  t.after(() => child.kill('SIGINT'))
+
+  await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v2', true))
+
+  let restartedSecondTime = false
+  let restartedThirdTime = false
+
+  for await (const log of child.ndj) {
+    if (log.msg === 'RELOADED v2') {
+      restartedSecondTime = true
+    } else if (log.msg === 'RELOADED v3') {
+      assert.ok(restartedSecondTime)
+      restartedThirdTime = true
+      break
+    } else if (log.msg?.match(/listening/)) {
+      await writeFile(cjsPluginFilePath, createCjsLoggingPlugin('v3', true))
+    }
+  }
+
+  assert.ok(restartedThirdTime)
+})

package/test/config.test.js

@@ -55,6 +55,14 @@ test('performs a topological sort on services depending on allowCycles', async (
       await loadConfig({}, ['-c', configFile], platformaticRuntime)
     })
   })
+
+  await t.test('throws by adding the most probable service ', async () => {
+    const configFile = join(fixturesDir, 'leven', 'platformatic.runtime.json')
+
+    await assert.rejects(async () => {
+      await loadConfig({}, ['-c', configFile], platformaticRuntime)
+    }, 'service \'rainy-empire\' has unordered dependency: \'deeply-splitte\'. Did you mean \'deeply-spittle\'?')
+  })
 })
 
 test('can resolve service id from client package.json if not provided', async () => {

package/test/start.test.js

@@ -68,7 +68,7 @@ test('composer', async (t) => {
     const res = await request(entryUrl)
 
     assert.strictEqual(res.statusCode, 200)
-    assert.deepStrictEqual(await res.body.json(), { message: 'Welcome to Platformatic! Please visit https://oss.platformatic.dev' })
+    assert.deepStrictEqual(await res.body.json(), { message: 'Welcome to Platformatic! Please visit https://docs.platformatic.dev' })
   }
 
   {

package/test/telemetry.test.js

@@ -0,0 +1,37 @@
+'use strict'
+
+const assert = require('node:assert')
+const { request } = require('undici')
+const { test } = require('node:test')
+const { join } = require('node:path')
+const { loadConfig } = require('@platformatic/service')
+const { platformaticRuntime } = require('..')
+const { startWithConfig } = require('../lib/start')
+const fixturesDir = join(__dirname, '..', 'fixtures')
+
+test('propagate the traceId correctly to runtime services', async (t) => {
+  const configFile = join(fixturesDir, 'telemetry', 'platformatic.runtime.json')
+  const config = await loadConfig({}, ['-c', configFile], platformaticRuntime)
+  const app = await startWithConfig(config.configManager)
+
+  t.after(async () => {
+    await app.close()
+  })
+
+  const entryUrl = await app.start()
+
+  const traceId = '5e994e8fb53b27c91dcd2fec22771d15'
+  const spanId = '166f3ab30f21800b'
+  const traceparent = `00-${traceId}-${spanId}-01`
+  const res = await request(entryUrl, {
+    method: 'GET',
+    path: '/',
+    headers: {
+      traceparent
+    }
+  })
+
+  assert.strictEqual(res.statusCode, 200)
+  const response = await res.body.json()
+  assert.strictEqual(response.traceId, traceId)
+})

package/test/unified-api.test.js

@@ -254,7 +254,6 @@ test('start()', async (t) => {
     const scriptFile = join(fixturesDir, 'starter.js')
     const configFile = join(fixturesDir, 'monorepo', 'serviceAppWithLogger', 'platformatic.service.json')
     const child = spawn(process.execPath, [scriptFile, configFile])
-    child.stdout.pipe(process.stdout)
     child.stderr.pipe(process.stderr)
     const [exitCode] = await once(child, 'exit')
 
@@ -318,14 +317,11 @@ test('startCommand()', async (t) => {
 
     assert.strictEqual(exitCode, 42)
   })
-})
 
-test('startCommandInRuntime()', async (t) => {
   await t.test('can start a non-runtime application', async (t) => {
     const scriptFile = join(fixturesDir, 'start-command-in-runtime.js')
     const configFile = join(fixturesDir, 'monorepo', 'serviceAppWithLogger', 'platformatic.service.json')
     const child = spawn(process.execPath, [scriptFile, configFile])
-    child.stdout.pipe(process.stdout)
     child.stderr.pipe(process.stderr)
     const [exitCode] = await once(child, 'exit')
 
@@ -336,7 +332,6 @@ test('startCommandInRuntime()', async (t) => {
     const scriptFile = join(fixturesDir, 'start-command-in-runtime.js')
     const configFile = join(fixturesDir, 'configs', 'monorepo.json')
     const child = spawn(process.execPath, [scriptFile, configFile])
-    child.stdout.pipe(process.stdout)
     child.stderr.pipe(process.stderr)
     const [exitCode] = await once(child, 'exit')