@platformatic/runtime 0.31.0 → 0.31.1

package/fixtures/dbAppWithMigrationError/migrations/001.do.sql

@@ -0,0 +1,4 @@
+CREATE TABLE IF NOT EXISTS movies (
+  id INTEGER PRIMARY KEY,
+  title TEXT NOT NULL fiddlesticks
+);

package/fixtures/dbAppWithMigrationError/migrations/001.undo.sql

@@ -0,0 +1 @@
+DROP TABLE movies;

package/fixtures/dbAppWithMigrationError/platformatic.db.json

@@ -0,0 +1,28 @@
+{
+  "$schema": "https://platformatic.dev/schemas/v0.30.0/db",
+  "server": {
+    "hostname": "127.0.0.1",
+    "port": 3042
+  },
+  "migrations": {
+    "autoApply": true,
+    "dir": "migrations",
+    "table": "versions"
+  },
+  "types": {
+    "autogenerate": false
+  },
+  "plugins": {
+    "paths": [
+      "plugin.js"
+    ]
+  },
+  "db": {
+    "connectionString": "sqlite://db.sqlite",
+    "graphql": true,
+    "ignore": {
+      "versions": true
+    },
+    "events": false
+  }
+}

package/fixtures/dbAppWithMigrationError/plugin.js

@@ -0,0 +1,5 @@
+/// <reference path="./global.d.ts" />
+'use strict'
+
+/** @param {import('fastify').FastifyInstance} app */
+module.exports = async function (app) {}

package/lib/message-port-writable.js

@@ -11,6 +11,14 @@ class MessagePortWritable extends Writable {
     this.metadata = opts.metadata
   }
 
+  _write (chunk, encoding, callback) {
+    this.port.postMessage({
+      metadata: this.metadata,
+      logs: [chunk.toString().trim()]
+    })
+    process.nextTick(callback)
+  }
+
   _writev (chunks, callback) {
     // Process the logs here before trying to send them across the thread
     // boundary. Sometimes the chunks have an undocumented method on them

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/runtime",
-  "version": "0.31.0",
+  "version": "0.31.1",
   "description": "",
   "main": "index.js",
   "bin": {
@@ -24,8 +24,8 @@
     "standard": "^17.1.0",
     "tsd": "^0.28.1",
     "typescript": "^5.1.6",
-    "@platformatic/sql-mapper": "0.31.0",
-    "@platformatic/sql-graphql": "0.31.0"
+    "@platformatic/sql-mapper": "0.31.1",
+    "@platformatic/sql-graphql": "0.31.1"
   },
   "dependencies": {
     "@hapi/topo": "^6.0.2",
@@ -40,11 +40,11 @@
     "pino": "^8.14.1",
     "pino-pretty": "^10.0.0",
     "undici": "^5.22.1",
-    "@platformatic/composer": "0.31.0",
-    "@platformatic/config": "0.31.0",
-    "@platformatic/db": "0.31.0",
-    "@platformatic/service": "0.31.0",
-    "@platformatic/utils": "0.31.0"
+    "@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"
   },
   "standard": {
     "ignore": [

package/test/start.test.js

@@ -8,6 +8,7 @@ const { MessageChannel } = require('node:worker_threads')
 const { request } = require('undici')
 const { loadConfig } = require('@platformatic/service')
 const { buildServer, platformaticRuntime } = require('..')
+const { wrapConfigInRuntimeConfig } = require('../lib/config')
 const { startWithConfig } = require('../lib/start')
 const fixturesDir = join(__dirname, '..', 'fixtures')
 
@@ -156,3 +157,30 @@ test('handles uncaught exceptions with db app', async (t) => {
 
   assert.strictEqual(exitCode, 42)
 })
+
+test('logs errors during db migrations', async (t) => {
+  const configFile = join(fixturesDir, 'dbAppWithMigrationError', 'platformatic.db.json')
+  const config = await loadConfig({}, ['-c', configFile], 'db')
+  const runtimeConfig = await wrapConfigInRuntimeConfig(config)
+  const { port1, port2 } = new MessageChannel()
+  runtimeConfig.current.loggingPort = port2
+  runtimeConfig.current.loggingMetadata = { foo: 1, bar: 2 }
+  const runtime = await startWithConfig(runtimeConfig)
+  const messages = []
+
+  port1.on('message', (msg) => {
+    messages.push(msg)
+  })
+
+  await assert.rejects(async () => {
+    await runtime.start()
+  }, /The runtime exited before the operation completed/)
+
+  assert.strictEqual(messages.length, 2)
+  assert.deepStrictEqual(messages[0].metadata, runtimeConfig.current.loggingMetadata)
+  assert.strictEqual(messages[0].logs.length, 1)
+  assert.match(messages[0].logs[0], /running 001.do.sql/)
+  assert.deepStrictEqual(messages[1].metadata, runtimeConfig.current.loggingMetadata)
+  assert.strictEqual(messages[1].logs.length, 1)
+  assert.match(messages[1].logs[0], /near \\"fiddlesticks\\": syntax error/)
+})