@platformatic/service 2.0.0-alpha.2 → 2.0.0-alpha.21

package/lib/plugins/graphql.js

@@ -14,8 +14,8 @@ async function setupGraphQL (app, opts) {
   const graphqlOptions = deepmerge({
     graphiql: true,
     additionalRouteOptions: {
-      schema: { hide: true }
-    }
+      schema: { hide: true },
+    },
   }, opts)
 
   app.register(mercurius, graphqlOptions)

package/lib/plugins/health-check.js

@@ -10,7 +10,7 @@ async function setupClients (app, opts) {
     exposeStatusRoute: '/status',
     healthCheckInterval: healthCheck.interval !== undefined ? healthCheck.interval : 5000,
     ...healthCheck,
-    healthCheck: healthCheck.fn
+    healthCheck: healthCheck.fn,
   })
 }
 

package/lib/plugins/metrics.js

@@ -12,7 +12,6 @@ const metricsPlugin = fp(async function (app, opts = {}) {
   const register = new promClient.Registry()
 
   const defaultMetrics = opts.defaultMetrics ?? { enabled: true }
-  const prefix = opts.prefix ?? ''
 
   if (opts.labels) {
     const labels = opts.labels ?? {}
@@ -22,41 +21,42 @@ const metricsPlugin = fp(async function (app, opts = {}) {
   app.register(require('fastify-metrics'), {
     defaultMetrics: {
       ...defaultMetrics,
-      register
+      register,
     },
     endpoint: null,
     name: 'metrics',
     clearRegisterOnInit: false,
     promClient: {
       ...promClient,
-      register
+      register,
     },
     routeMetrics: {
       enabled: true,
       customLabels: {
-        telemetry_id: (req) => req.headers['x-telemetry-id'] ?? 'unknown'
+        // TODO: check if this is set in prom
+        telemetry_id: (req) => req.headers['x-plt-telemetry-id'] ?? 'unknown',
       },
       overrides: {
         histogram: {
-          name: prefix + 'http_request_duration_seconds',
-          registers: [register]
+          name: 'http_request_duration_seconds',
+          registers: [register],
         },
         summary: {
-          name: prefix + 'http_request_summary_seconds',
-          registers: [register]
-        }
-      }
-    }
+          name: 'http_request_summary_seconds',
+          registers: [register],
+        },
+      },
+    },
   })
 
   app.register(fp(async (app) => {
     const httpLatencyMetric = new app.metrics.client.Summary({
-      name: prefix + 'http_request_all_summary_seconds',
+      name: 'http_request_all_summary_seconds',
       help: 'request duration in seconds summary for all requests',
       collect: () => {
         process.nextTick(() => httpLatencyMetric.reset())
       },
-      registers: [register]
+      registers: [register],
     })
 
     const ignoredMethods = ['HEAD', 'OPTIONS', 'TRACE', 'CONNECT']
@@ -75,7 +75,7 @@ const metricsPlugin = fp(async function (app, opts = {}) {
       }
     })
   }, {
-    encapsulate: false
+    encapsulate: false,
   }))
 
   if (defaultMetrics.enabled) {
@@ -90,7 +90,7 @@ const metricsPlugin = fp(async function (app, opts = {}) {
           eluMetric.set(result)
           startELU = endELU
         },
-        registers: [register]
+        registers: [register],
       })
       app.metrics.client.register.registerMetric(eluMetric)
 
@@ -123,35 +123,27 @@ const metricsPlugin = fp(async function (app, opts = {}) {
           previousIdleTime = idleTime
           previousTotalTime = totalTime
         },
-        registers: [register]
+        registers: [register],
       })
       app.metrics.client.register.registerMetric(cpuMetric)
     })
   }
 
   function cleanMetrics () {
+    const httpMetrics = ['http_request_duration_seconds', 'http_request_summary_seconds', 'http_request_all_summary_seconds']
     const metrics = app.metrics.client.register._metrics
     for (const metricName in metrics) {
-      if (defaultMetrics.enabled || metricName.startsWith(prefix)) {
+      if (defaultMetrics.enabled || httpMetrics.includes(metricName)) {
         delete metrics[metricName]
       }
     }
   }
 
-  let isRestarting = false
-  app.addHook('onReady', async () => {
-    app.addPreRestartHook(async () => {
-      isRestarting = true
-      cleanMetrics()
-    })
-  })
   app.addHook('onClose', async () => {
-    if (!isRestarting) {
-      cleanMetrics()
-    }
+    cleanMetrics()
   })
 }, {
-  encapsulate: false
+  encapsulate: false,
 })
 
 // This is a global httpServer to match global
@@ -178,7 +170,7 @@ async function createMetricsServer (app, hostname, port) {
       httpServer.on('request', handler)
       return httpServer
     },
-    logger: app.log.child({ name: 'prometheus' })
+    loggerInstance: app.log.child({ name: 'prometheus' }),
   })
 
   app.addHook('onClose', async () => {
@@ -222,7 +214,7 @@ module.exports = fp(async function (app, opts) {
             return reply.code(401).send({ message: 'Unauthorized' })
           }
           return done()
-        }
+        },
       })
       onRequestHook = metricsServer.basicAuth
     }
@@ -242,7 +234,7 @@ module.exports = fp(async function (app, opts) {
         }
         reply.type('text/plain')
         return promRegistry.metrics()
-      }
+      },
     })
   }
 

package/lib/plugins/openapi.js

@@ -16,28 +16,28 @@ async function setupOpenAPI (app, opts) {
     info: {
       title: 'Platformatic',
       description: 'This is a service built on top of Platformatic',
-      version: '1.0.0'
-    }
+      version: '1.0.0',
+    },
   }, typeof openapi === 'object' ? openapi : {})
   app.log.trace({ openapi: openapiConfig })
   const swaggerOptions = {
     exposeRoute: openapiConfig.exposeRoute,
     openapi: {
-      ...openapiConfig
+      ...openapiConfig,
     },
     refResolver: {
       buildLocalReference (json, baseUri, fragment, i) {
         // TODO figure out if we need def-${i}
         /* istanbul ignore next */
         return json.$id || `def-${i}`
-      }
-    }
+      },
+    },
   }
 
   if (openapi.path) {
     swaggerOptions.mode = 'static'
     swaggerOptions.specification = {
-      path: openapi.path
+      path: openapi.path,
     }
   }
 
@@ -49,11 +49,11 @@ async function setupOpenAPI (app, opts) {
   /** Serve spec file in yaml and json */
   app.get(`${routePrefix}/json`, {
     schema: { hide: true },
-    logLevel: 'warn'
+    logLevel: 'warn',
   }, async () => app.swagger())
   app.get(`${routePrefix}/yaml`, {
     schema: { hide: true },
-    logLevel: 'warn'
+    logLevel: 'warn',
   }, async () => app.swagger({ yaml: true }))
 
   app.register(ScalarApiReference, {
@@ -61,8 +61,8 @@ async function setupOpenAPI (app, opts) {
     ...openapi,
     routePrefix,
     configuration: {
-      customCss: scalarTheme.theme
-    }
+      customCss: scalarTheme.theme,
+    },
   })
 }
 

package/lib/plugins/plugins.js

@@ -7,14 +7,14 @@ const wrapper = require('./sandbox-wrapper')
 
 const { getJSPluginPath, isFileAccessible } = require('../utils')
 
-async function loadPlugins (app) {
+async function loadPlugins (app, opts) {
   const configManager = app.platformatic.configManager
   const config = configManager.current
 
   let isOutDirAccessible = false
   let outDir = null
 
-  const workingDir = configManager.dirname
+  const workingDir = opts?.context?.directory ?? configManager.dirname
   const tsConfigPath = configManager.current.plugins.typescript?.tsConfig || join(workingDir, 'tsconfig.json')
 
   // If the tsconfig.json file exists, then we need to adjust the plugin paths
@@ -32,14 +32,21 @@ async function loadPlugins (app) {
 
   if (outDir) {
     isOutDirAccessible = await isFileAccessible(outDir)
+
+    if (opts.context?.isProduction && !isOutDirAccessible) {
+      throw new Error(
+        `Cannot access directory '${outDir}'. Please run the 'build' command before running in production mode.`
+      )
+    }
   }
 
   if (config.plugins.paths && isOutDirAccessible) {
-    config.plugins.paths = config.plugins.paths.map((plugin) => {
+    config.plugins.paths = config.plugins.paths.map(plugin => {
       /* c8 ignore next 3 */
-      const tmp = typeof plugin === 'string'
-        ? getJSPluginPath(workingDir, plugin, outDir)
-        : { ...plugin, path: getJSPluginPath(workingDir, plugin.path, outDir) }
+      const tmp =
+        typeof plugin === 'string'
+          ? getJSPluginPath(workingDir, plugin, outDir)
+          : { ...plugin, path: getJSPluginPath(workingDir, plugin.path, outDir) }
       return tmp
     })
   }

package/lib/plugins/sandbox-wrapper.js

@@ -36,7 +36,7 @@ module.exports = fp(async function (app, opts) {
         forceESM: plugin.forceESM,
         ignoreFilter: plugin.ignoreFilter,
         matchFilter: plugin.matchFilter,
-        ...patternOptions
+        ...patternOptions,
       })
     } else {
       let loaded = await import(pathToFileURL(plugin.path))
@@ -71,7 +71,7 @@ function patternOptionsFromPlugin (plugin) {
   const patternOptionKeys = [
     'ignorePattern',
     'indexPattern',
-    'autoHooksPattern'
+    'autoHooksPattern',
   ]
 
   for (const key of patternOptionKeys) {

package/lib/plugins/typescript.js

@@ -4,16 +4,16 @@ const fp = require('fastify-plugin')
 const compiler = require('@platformatic/ts-compiler')
 const { extractTypeScriptCompileOptionsFromConfig } = require('../compile')
 
-async function setupTsCompiler (app) {
+async function setupTsCompiler (app, opts) {
   const configManager = app.platformatic.configManager
   const config = configManager.current
-  const workingDir = configManager.dirname
+  const workingDir = opts?.context?.directory ?? configManager.dirname
 
   await compiler.compile({
     ...extractTypeScriptCompileOptionsFromConfig(config),
     cwd: workingDir,
     clean: false,
-    logger: app.log
+    logger: app.log,
   })
 }
 

package/lib/root-endpoint/index.js

@@ -6,7 +6,7 @@ const userAgentParser = require('my-ua-parser')
 
 module.exports = async (app, opts) => {
   app.register(fastifyStatic, {
-    root: path.join(__dirname, 'public')
+    root: path.join(__dirname, 'public'),
   })
 
   // root endpoint
@@ -23,6 +23,6 @@ module.exports = async (app, opts) => {
         }
       }
       return { message: 'Welcome to Platformatic! Please visit https://docs.platformatic.dev' }
-    }
+    },
   })
 }