@platformatic/watt-extra 1.5.2 → 1.5.3-alpha.1

package/.claude/settings.local.json

@@ -0,0 +1,11 @@
+{
+  "permissions": {
+    "allow": [
+      "Read(//work/workspaces/workspace-platformatic/platformatic/**)",
+      "Bash(npx borp:*)",
+      "Bash(timeout 30 npx borp -c 1 --timeout=20000 ./test/trigger-flamegraphs.test.js)"
+    ],
+    "deny": [],
+    "ask": []
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/watt-extra",
-  "version": "1.5.2",
+  "version": "1.5.3-alpha.1",
   "description": "The Platformatic runtime manager",
   "type": "module",
   "scripts": {
@@ -29,7 +29,7 @@
     "fastify": "^5.4.0",
     "fastify-plugin": "^5.0.1",
     "neostandard": "^0.12.0",
-    "next": "^15.3.4",
+    "next": "^16.0.0",
     "platformatic": "^3.14.0",
     "pprof-format": "^2.1.0",
     "why-is-node-running": "^2.3.0"

package/plugins/alerts.js

@@ -40,7 +40,8 @@ async function alerts (app, _opts) {
       }
 
       const timestamp = Date.now()
-      const healthWithTimestamp = { ...healthInfo, timestamp }
+      const serviceId = healthInfo.application
+      const healthWithTimestamp = { ...healthInfo, timestamp, service: serviceId }
       delete healthWithTimestamp.healthConfig // we don't need to store this
 
       healthCache.push(healthWithTimestamp)
@@ -75,7 +76,8 @@ async function alerts (app, _opts) {
       if (healthInfo.unhealthy) {
         const currentTime = Date.now()
 
-        const serviceId = healthInfo.id
+        const serviceId = healthInfo.application
+        healthInfo.service = serviceId // ICC expects "service" field
         const lastAlertTime = lastServicesAlertTime[serviceId]
 
         if (lastAlertTime && currentTime - lastAlertTime < alertRetentionWindow) {

package/test/alerts.test.js

@@ -63,7 +63,7 @@ test('should send alert when service becomes unhealthy', async (t) => {
   // Manually trigger health event with unhealthy state
   const healthInfo = {
     id: 'main:0',
-    service: 'main',
+    application: 'main',
     currentHealth: {
       elu: 0.95,
       heapUsed: 76798040,
@@ -88,8 +88,11 @@ test('should send alert when service becomes unhealthy', async (t) => {
   assert.ok(alertReceived, 'Alert should have been received')
   assert.strictEqual(alertReceived.applicationId, applicationId)
   assert.deepStrictEqual(alertReceived.alert, healthInfo)
+  assert.strictEqual(alertReceived.alert.application, 'main')
+  assert.strictEqual(alertReceived.alert.service, 'main')
   assert.ok(Array.isArray(alertReceived.healthHistory), 'Health history should be an array')
   assert.ok(alertReceived.healthHistory.length > 0, 'Health history should not be empty')
+  assert.strictEqual(alertReceived.healthHistory[0].application, 'main')
   assert.strictEqual(alertReceived.healthHistory[0].service, 'main')
 
   assert.ok(flamegraphReceived, 'Flamegraph should have been received')
@@ -98,7 +101,7 @@ test('should send alert when service becomes unhealthy', async (t) => {
   assert.ok(profile, 'Profile should be decoded')
 })
 
-test('should not send alert when service is healthy', async (t) => {
+test('should not send alert when application is healthy', async (t) => {
   const applicationName = 'test-app'
   const applicationId = randomUUID()
   const applicationPath = join(__dirname, 'fixtures', 'service-1')
@@ -137,7 +140,7 @@ test('should not send alert when service is healthy', async (t) => {
   // Manually trigger health event with healthy state
   const healthInfo = {
     id: 'service-1',
-    service: 'service-1',
+    application: 'service-1',
     currentHealth: {
       elu: 0.5,
       heapUsed: 76798040,
@@ -204,7 +207,7 @@ test('should cache health data and include it in alerts', async (t) => {
   for (let i = 0; i < 3; i++) {
     const healthyInfo = {
       id: 'service-1',
-      service: 'service-1',
+      application: 'service-1',
       currentHealth: {
         elu: 0.5 + (i * 0.1), // Different values to distinguish them
         heapUsed: 76798040,
@@ -229,7 +232,7 @@ test('should cache health data and include it in alerts', async (t) => {
   // Now send an unhealthy event to trigger alert
   const unhealthyInfo = {
     id: 'service-1',
-    service: 'service-1',
+    application: 'service-1',
     currentHealth: {
       elu: 0.95,
       heapUsed: 76798040,
@@ -254,7 +257,8 @@ test('should cache health data and include it in alerts', async (t) => {
   assert.strictEqual(alertReceived.applicationId, applicationId)
 
   assert.strictEqual(alertReceived.alert.id, unhealthyInfo.id)
-  assert.strictEqual(alertReceived.alert.service, unhealthyInfo.service)
+  assert.strictEqual(alertReceived.alert.application, 'service-1')
+  assert.strictEqual(alertReceived.alert.service, 'service-1')
   assert.strictEqual(alertReceived.alert.unhealthy, unhealthyInfo.unhealthy)
 
   assert.ok(Array.isArray(alertReceived.healthHistory), 'Health history should be an array')
@@ -264,7 +268,9 @@ test('should cache health data and include it in alerts', async (t) => {
     assert.ok('unhealthy' in entry, 'Entry should have unhealthy property')
     assert.ok('currentHealth' in entry, 'Entry should have currentHealth property')
     assert.ok('timestamp' in entry, 'Entry should have timestamp property')
-    assert.ok('service' in entry || 'id' in entry, 'Entry should have service or id property')
+    assert.ok('service' in entry, 'Entry should have service property')
+    assert.ok('application' in entry, 'Entry should have application property')
+    assert.strictEqual(entry.service, entry.application, 'service should match application')
     assert.ok(!('healthConfig' in entry), 'Entry should not have healthConfig property')
   }
 
@@ -363,9 +369,9 @@ test('should respect alert retention window', async (t) => {
   })
 
   // Create a health info template
-  const createHealthInfo = (serviceId, unhealthy = true) => ({
-    id: serviceId,
-    service: serviceId,
+  const createHealthInfo = (applicationId, unhealthy = true) => ({
+    id: applicationId,
+    application: applicationId,
     currentHealth: {
       elu: unhealthy ? 0.95 : 0.5,
       heapUsed: 76798040,
@@ -489,7 +495,7 @@ test('should send alert when flamegraphs are disabled', async (t) => {
   // Manually trigger health event with unhealthy state
   const healthInfo = {
     id: 'main:0',
-    service: 'main',
+    application: 'main',
     currentHealth: {
       elu: 0.95,
       heapUsed: 76798040,
@@ -514,8 +520,11 @@ test('should send alert when flamegraphs are disabled', async (t) => {
   assert.ok(alertReceived, 'Alert should have been received')
   assert.strictEqual(alertReceived.applicationId, applicationId)
   assert.deepStrictEqual(alertReceived.alert, healthInfo)
+  assert.strictEqual(alertReceived.alert.application, 'main')
+  assert.strictEqual(alertReceived.alert.service, 'main')
   assert.ok(Array.isArray(alertReceived.healthHistory), 'Health history should be an array')
   assert.ok(alertReceived.healthHistory.length > 0, 'Health history should not be empty')
+  assert.strictEqual(alertReceived.healthHistory[0].application, 'main')
   assert.strictEqual(alertReceived.healthHistory[0].service, 'main')
   assert.equal(alertReceived.flamegraph, null, 'Flamegraph should be null')
 })
@@ -566,7 +575,7 @@ test('should send alert when failed to send a flamegraph', async (t) => {
   // Manually trigger health event with unhealthy state
   const healthInfo = {
     id: 'main:0',
-    service: 'main',
+    application: 'main',
     currentHealth: {
       elu: 0.95,
       heapUsed: 76798040,
@@ -591,8 +600,11 @@ test('should send alert when failed to send a flamegraph', async (t) => {
   assert.ok(alertReceived, 'Alert should have been received')
   assert.strictEqual(alertReceived.applicationId, applicationId)
   assert.deepStrictEqual(alertReceived.alert, healthInfo)
+  assert.strictEqual(alertReceived.alert.application, 'main')
+  assert.strictEqual(alertReceived.alert.service, 'main')
   assert.ok(Array.isArray(alertReceived.healthHistory), 'Health history should be an array')
   assert.ok(alertReceived.healthHistory.length > 0, 'Health history should not be empty')
+  assert.strictEqual(alertReceived.healthHistory[0].application, 'main')
   assert.strictEqual(alertReceived.healthHistory[0].service, 'main')
   assert.equal(alertReceived.flamegraph, null, 'Flamegraph should be null')
 })