From 57df5f61ba84ed23628180388241a1765a523a9b Mon Sep 17 00:00:00 2001
From: Rudy Celekli <rudy@gradiahq.com>
Date: Sun, 25 Oct 2026 15:58:45 -0400
Subject: [PATCH] fix(fires): report a 00:45 UTC detection as 00:45, not "45"

get_active_fires joined FIRMS acq_date and the unpadded acq_time, so a
detection at 00:45 UTC came back as "2026-01-01 45". The row now uses
acquisitionMsUtc, which pads the time, and reports an ISO UTC timestamp
like the other tool times.
---
 changelog.d/1395-fires-tool-acquired-time.md |  4 ++++
 src/tools/queries/hazards.js                 |  4 ++--
 src/tools/queries/hazards.test.mjs           | 17 +++++++++++++++++
 src/tools/queries/queries.test.mjs           |  2 +-
 4 files changed, 24 insertions(+), 3 deletions(-)
 create mode 100644 changelog.d/1395-fires-tool-acquired-time.md
 create mode 100644 src/tools/queries/hazards.test.mjs

diff --git a/changelog.d/1395-fires-tool-acquired-time.md b/changelog.d/1395-fires-tool-acquired-time.md
new file mode 100644
index 000000000..ea3bddd1d
--- /dev/null
+++ b/changelog.d/1395-fires-tool-acquired-time.md
@@ -0,0 +1,4 @@
+- Report a fire detection's acquisition time as a UTC timestamp in
+  `get_active_fires`. The tool joined FIRMS's unpadded date and time, so a
+  00:45 UTC detection read "2026-01-01 45"; it now reads
+  `2026-01-01T00:45:00.000Z` (Rudy Celekli, @rudycelekli, #1395).
diff --git a/src/tools/queries/hazards.js b/src/tools/queries/hazards.js
index 475df97ba..f644be562 100644
--- a/src/tools/queries/hazards.js
+++ b/src/tools/queries/hazards.js
@@ -1,5 +1,6 @@
 /** Natural hazard queries: earthquakes and active fires. */
 
+import { acquisitionMsUtc } from '../../data/firmsCsv.js';
 import { suggestView } from '../views.js';
 import { defineTool, ToolError } from '../catalog.js';
 import { AREA_SCHEMA, areaContains, resolveArea } from '../area.js';
@@ -114,8 +115,7 @@ export const getActiveFires = defineTool({
           lon: fire.lon,
           frp_mw: fire.frp ?? null,
           confidence: fire.confidence ?? null,
-          acquired:
-            [fire.acqDate, fire.acqTime].filter(Boolean).join(' ') || null,
+          acquired: isoTime(acquisitionMsUtc(fire.acqDate, fire.acqTime)),
           satellite: fire.satellite ?? null,
           day_night: fire.daynight ?? null,
         })),
diff --git a/src/tools/queries/hazards.test.mjs b/src/tools/queries/hazards.test.mjs
new file mode 100644
index 000000000..fe2e6ac1f
--- /dev/null
+++ b/src/tools/queries/hazards.test.mjs
@@ -0,0 +1,17 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+import { composeCatalog } from '../catalog.js';
+import { getActiveFires } from './hazards.js';
+
+const catalogWith = (fires) =>
+  composeCatalog({
+    tools: [getActiveFires],
+    services: { fires: { getSnapshot: async () => ({ fires }) } },
+  });
+
+test('a fire acquired at 00:45 UTC reads as 00:45, not "45"', async () => {
+  const result = await catalogWith([
+    { lat: 24, lon: 121, frp: 5, acqDate: '2026-01-01', acqTime: '45' },
+  ]).call('get_active_fires', { area: { bbox: [120, 23, 122, 25] } });
+  assert.equal(result.data.rows[0].acquired, '2026-01-01T00:45:00.000Z');
+});
diff --git a/src/tools/queries/queries.test.mjs b/src/tools/queries/queries.test.mjs
index b64f5e009..5f53cf9e8 100644
--- a/src/tools/queries/queries.test.mjs
+++ b/src/tools/queries/queries.test.mjs
@@ -125,7 +125,7 @@ test('fires need an area and report a missing key as unavailable', async () => {
     result.data.rows.map((row) => row.frp_mw),
     [40, 12],
   );
-  assert.equal(result.data.rows[1].acquired, '2026-01-01 0130');
+  assert.equal(result.data.rows[1].acquired, '2026-01-01T01:30:00.000Z');
 
   const keyless = composeCatalog({
     tools: coreTools,