@oneuptime/common 11.3.9 → 11.3.10

package/build/dist/Server/Services/IncidentService.js

@@ -41,10 +41,11 @@ import IncidentStateTimeline from "../../Models/DatabaseModels/IncidentStateTime
 import Monitor from "../../Models/DatabaseModels/Monitor";
 import MonitorStatusTimeline from "../../Models/DatabaseModels/MonitorStatusTimeline";
 import { IsBillingEnabled } from "../EnvironmentConfig";
-import MetricService from "./MetricService";
+import MutableMetricService from "./MutableMetricService";
 import GlobalConfigService from "./GlobalConfigService";
 import IncidentMetricType from "../../Types/Incident/IncidentMetricType";
-import Metric, { MetricPointType } from "../../Models/AnalyticsModels/Metric";
+import MutableMetric from "../../Models/AnalyticsModels/MutableMetric";
+import { MetricPointType } from "../../Models/AnalyticsModels/Metric";
 import ServiceType from "../../Types/Telemetry/ServiceType";
 import OneUptimeDate from "../../Types/Date";
 import TelemetryUtil from "../Utils/Telemetry/Telemetry";
@@ -1319,12 +1320,14 @@ ${incident.remediationNotes || "No remediation notes provided."}
                         });
                         // only emit if the incident has been resolved
                         if (resolvedTimeline && resolvedTimeline.startsAt) {
-                            const postmortemMetric = new Metric();
+                            const postmortemMetric = new MutableMetric();
                             postmortemMetric.projectId = projectId;
                             postmortemMetric.primaryEntityId = incidentId;
                             postmortemMetric.primaryEntityType = ServiceType.Incident;
                             postmortemMetric.name =
                                 IncidentMetricType.PostmortemCompletionTime;
+                            postmortemMetric.metricPointId =
+                                IncidentMetricType.PostmortemCompletionTime;
                             postmortemMetric.value = OneUptimeDate.getDifferenceInSeconds(postmortemPostedAt, resolvedTimeline.startsAt);
                             postmortemMetric.attributes = {
                                 incidentId: incidentId.toString(),
@@ -1336,11 +1339,8 @@ ${incident.remediationNotes || "No remediation notes provided."}
                             postmortemMetric.metricPointType = MetricPointType.Sum;
                             const postmortemRetentionDays = await this.getMetricRetentionDays();
                             postmortemMetric.retentionDate = OneUptimeDate.addRemoveDays(OneUptimeDate.getCurrentDate(), postmortemRetentionDays);
-                            await MetricService.create({
-                                data: postmortemMetric,
-                                props: {
-                                    isRoot: true,
-                                },
+                            await MutableMetricService.createMutableMetrics({
+                                metrics: [postmortemMetric],
                             });
                             const postmortemMetricType = new MetricType();
                             postmortemMetricType.name =
@@ -1500,11 +1500,12 @@ ${incidentSeverity.name}
                         }
                         // emit severity change metric
                         try {
-                            const severityChangeMetric = new Metric();
+                            const severityChangeMetric = new MutableMetric();
                             severityChangeMetric.projectId = projectId;
                             severityChangeMetric.primaryEntityId = incidentId;
                             severityChangeMetric.primaryEntityType = ServiceType.Incident;
                             severityChangeMetric.name = IncidentMetricType.SeverityChange;
+                            severityChangeMetric.metricPointId = `severity-change:${ObjectID.generate().toString()}`;
                             severityChangeMetric.value = 1;
                             severityChangeMetric.attributes = {
                                 incidentId: incidentId.toString(),
@@ -1519,11 +1520,8 @@ ${incidentSeverity.name}
                             severityChangeMetric.metricPointType = MetricPointType.Sum;
                             const severityRetentionDays = await this.getMetricRetentionDays();
                             severityChangeMetric.retentionDate = OneUptimeDate.addRemoveDays(OneUptimeDate.getCurrentDate(), severityRetentionDays);
-                            await MetricService.create({
-                                data: severityChangeMetric,
-                                props: {
-                                    isRoot: true,
-                                },
+                            await MutableMetricService.createMutableMetrics({
+                                metrics: [severityChangeMetric],
                             });
                             const severityChangeMetricType = new MetricType();
                             severityChangeMetricType.name = IncidentMetricType.SeverityChange;
@@ -1847,14 +1845,13 @@ ${incidentSeverity.name}
                     await this.markMonitorsActiveForMonitoring(incident.projectId, incident.monitors);
                 }
                 if (incident.projectId && incident.id) {
-                    await MetricService.deleteBy({
-                        query: {
-                            projectId: incident.projectId,
-                            primaryEntityId: incident.id,
-                        },
-                        props: {
-                            isRoot: true,
-                        },
+                    const metricRetentionDays = await this.getMetricRetentionDays();
+                    await MutableMetricService.tombstoneEntityMetrics({
+                        projectId: incident.projectId,
+                        primaryEntityId: incident.id,
+                        primaryEntityType: ServiceType.Incident,
+                        metricNames: Object.values(IncidentMetricType),
+                        retentionDate: OneUptimeDate.addRemoveDays(OneUptimeDate.getCurrentDate(), metricRetentionDays),
                     });
                 }
             }
@@ -1941,12 +1938,13 @@ ${incidentSeverity.name}
         return Service.DEFAULT_METRIC_RETENTION_DAYS;
     }
     async refreshIncidentMetrics(data) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u, _v;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u;
         const incident = await this.findOneById({
             id: data.incidentId,
             select: {
                 projectId: true,
                 declaredAt: true,
+                postmortemPostedAt: true,
                 monitors: {
                     _id: true,
                     name: true,
@@ -2039,6 +2037,7 @@ ${incidentSeverity.name}
                 incidentId: data.incidentId,
             },
             select: {
+                _id: true,
                 projectId: true,
                 incidentStateId: true,
                 incidentState: {
@@ -2061,18 +2060,10 @@ ${incidentSeverity.name}
         });
         const firstIncidentStateTimeline = incidentStateTimelines[0];
         /*
-         * Serialize concurrent refreshes for this incident across pods. The
-         * write-once guard below is read-then-insert (findBy existing, then a
-         * conditional createMany). Both call sites fire refresh fire-and-forget
-         * after releasing the per-incident state-timeline mutex, so two
-         * close-together transitions (e.g. auto-acknowledge then auto-resolve)
-         * could interleave: both read a snapshot missing a metric and both
-         * insert it. The Metric table is a plain MergeTree with no row collapse,
-         * so those duplicates would be permanent and re-inflate the very Sum/Avg
-         * widgets this change fixes. A best-effort distributed lock keyed on the
-         * incident makes the read+insert atomic; if the lock can't be taken we
-         * still proceed (the existence check alone covers the common sequential
-         * case).
+         * Serialize concurrent refreshes for this incident across pods. Mutable
+         * metrics are versioned inserts, but the replace operation also tombstones
+         * stale metric-point identities. The lock keeps that read+insert cycle
+         * ordered for each incident.
          */
         let metricRefreshMutex = null;
         try {
@@ -2089,65 +2080,6 @@ ${incidentSeverity.name}
             });
         }
         try {
-            /*
-             * Write-once metrics — no deletes.
-             *
-             * The raw Metric table (`MetricItemV3`) feeds an AggregatingMergeTree
-             * materialized view (`MetricItemAggMV1m_mv`) that accumulates
-             * `sumState`/`countState` per (projectId, name, primaryEntityId,
-             * minute). The MV trigger fires only on INSERT — an `ALTER ... DELETE`
-             * mutation on the source rolls back nothing in the MV. So the old
-             * "delete every metric for this incident, then re-insert" refresh both
-             * (a) inflated those rollups (the Sum-of-IncidentCount widget read high)
-             * and (b) issued one heavy `ALTER ... DELETE` mutation per state
-             * transition per incident — a mutation storm.
-             *
-             * Every incident metric has a single eventually-final value:
-             * IncidentCount is constant `value = 1`; MTTA/MTTR/duration/time-in-state
-             * are each fixed once the relevant state transition has happened. So we
-             * load what we've already emitted for this incident and insert each
-             * metric exactly once, when it becomes final. Repeat refreshes that find
-             * nothing new are pure no-ops (no insert, no mutation).
-             */
-            const existingMetrics = await MetricService.findBy({
-                query: {
-                    projectId: incident.projectId,
-                    primaryEntityId: data.incidentId,
-                },
-                select: {
-                    name: true,
-                    time: true,
-                    attributes: true,
-                },
-                skip: 0,
-                limit: LIMIT_PER_PROJECT,
-                props: {
-                    isRoot: true,
-                },
-            });
-            const existingMetricNames = new Set(existingMetrics
-                .map((metric) => {
-                return metric.name;
-            })
-                .filter(Boolean));
-            /*
-             * Identity of a TimeInState row: the state plus the instant that state
-             * began (a state can be entered more than once over a re-opened
-             * incident). Second granularity is robust against DateTime64 round-trip
-             * jitter and two distinct entries into the same state within one second
-             * never happen.
-             */
-            const buildTimeInStateKey = (stateId, startsAt) => {
-                return `${stateId || ""}|${startsAt ? OneUptimeDate.toUnixTimestamp(startsAt) : ""}`;
-            };
-            const existingTimeInStateKeys = new Set(existingMetrics
-                .filter((metric) => {
-                return metric.name === IncidentMetricType.TimeInState;
-            })
-                .map((metric) => {
-                var _a;
-                return buildTimeInStateKey((_a = metric.attributes) === null || _a === void 0 ? void 0 : _a["incidentStateId"], metric.time);
-            }));
             const itemsToSave = [];
             const metricRetentionDays = await this.getMetricRetentionDays();
             const incidentMetricRetentionDate = OneUptimeDate.addRemoveDays(OneUptimeDate.getCurrentDate(), metricRetentionDays);
@@ -2180,29 +2112,20 @@ ${incidentSeverity.name}
                 ownerTeamIds: ownerTeamIds.join(", "),
                 ownerTeamNames: ownerTeamNames.join(", "),
             };
-            /*
-             * Only emit IncidentCount on the very first refresh (i.e. when no
-             * existing IncidentCount row is present for this primaryEntityId). See
-             * the write-once note above — emitting it on every refresh would
-             * accumulate phantom `sumState` entries in the MV. By keeping the
-             * original row alive and never re-emitting, the dashboard Sum stays
-             * equal to the true count of distinct incidents.
-             */
-            if (!existingMetricNames.has(IncidentMetricType.IncidentCount)) {
-                const incidentCountMetric = new Metric();
-                incidentCountMetric.projectId = incident.projectId;
-                incidentCountMetric.primaryEntityId = incident.id;
-                incidentCountMetric.primaryEntityType = ServiceType.Incident;
-                incidentCountMetric.name = IncidentMetricType.IncidentCount;
-                incidentCountMetric.value = 1;
-                incidentCountMetric.attributes = Object.assign({}, baseMetricAttributes);
-                incidentCountMetric.attributeKeys = TelemetryUtil.getAttributeKeys(incidentCountMetric.attributes);
-                incidentCountMetric.time = incidentStartsAt;
-                incidentCountMetric.timeUnixNano = OneUptimeDate.toUnixNano(incidentCountMetric.time);
-                incidentCountMetric.metricPointType = MetricPointType.Sum;
-                incidentCountMetric.retentionDate = incidentMetricRetentionDate;
-                itemsToSave.push(incidentCountMetric);
-            }
+            const incidentCountMetric = new MutableMetric();
+            incidentCountMetric.projectId = incident.projectId;
+            incidentCountMetric.primaryEntityId = incident.id;
+            incidentCountMetric.primaryEntityType = ServiceType.Incident;
+            incidentCountMetric.name = IncidentMetricType.IncidentCount;
+            incidentCountMetric.metricPointId = IncidentMetricType.IncidentCount;
+            incidentCountMetric.value = 1;
+            incidentCountMetric.attributes = Object.assign({}, baseMetricAttributes);
+            incidentCountMetric.attributeKeys = TelemetryUtil.getAttributeKeys(incidentCountMetric.attributes);
+            incidentCountMetric.time = incidentStartsAt;
+            incidentCountMetric.timeUnixNano = OneUptimeDate.toUnixNano(incidentCountMetric.time);
+            incidentCountMetric.metricPointType = MetricPointType.Sum;
+            incidentCountMetric.retentionDate = incidentMetricRetentionDate;
+            itemsToSave.push(incidentCountMetric);
             // Always register the metric type so it shows up in the type catalog.
             const metricType = new MetricType();
             metricType.name = IncidentMetricType.IncidentCount;
@@ -2227,29 +2150,27 @@ ${incidentSeverity.name}
                     metricType.description = "Time taken to acknowledge the incident";
                     metricType.unit = "seconds";
                     metricTypesMap[IncidentMetricType.TimeToAcknowledge] = metricType;
-                    // write-once: MTTA is fixed once the incident is first acknowledged.
-                    if (!existingMetricNames.has(IncidentMetricType.TimeToAcknowledge)) {
-                        const timeToAcknowledgeMetric = new Metric();
-                        timeToAcknowledgeMetric.projectId = incident.projectId;
-                        timeToAcknowledgeMetric.primaryEntityId = incident.id;
-                        timeToAcknowledgeMetric.primaryEntityType = ServiceType.Incident;
-                        timeToAcknowledgeMetric.name = IncidentMetricType.TimeToAcknowledge;
-                        timeToAcknowledgeMetric.value =
-                            OneUptimeDate.getDifferenceInSeconds((ackIncidentStateTimeline === null || ackIncidentStateTimeline === void 0 ? void 0 : ackIncidentStateTimeline.startsAt) ||
-                                OneUptimeDate.getCurrentDate(), incidentStartsAt);
-                        timeToAcknowledgeMetric.attributes = Object.assign({}, baseMetricAttributes);
-                        timeToAcknowledgeMetric.attributeKeys =
-                            TelemetryUtil.getAttributeKeys(timeToAcknowledgeMetric.attributes);
-                        timeToAcknowledgeMetric.time =
-                            (ackIncidentStateTimeline === null || ackIncidentStateTimeline === void 0 ? void 0 : ackIncidentStateTimeline.startsAt) ||
-                                incident.declaredAt ||
-                                incident.createdAt ||
-                                OneUptimeDate.getCurrentDate();
-                        timeToAcknowledgeMetric.timeUnixNano = OneUptimeDate.toUnixNano(timeToAcknowledgeMetric.time);
-                        timeToAcknowledgeMetric.metricPointType = MetricPointType.Sum;
-                        timeToAcknowledgeMetric.retentionDate = incidentMetricRetentionDate;
-                        itemsToSave.push(timeToAcknowledgeMetric);
-                    }
+                    const timeToAcknowledgeMetric = new MutableMetric();
+                    timeToAcknowledgeMetric.projectId = incident.projectId;
+                    timeToAcknowledgeMetric.primaryEntityId = incident.id;
+                    timeToAcknowledgeMetric.primaryEntityType = ServiceType.Incident;
+                    timeToAcknowledgeMetric.name = IncidentMetricType.TimeToAcknowledge;
+                    timeToAcknowledgeMetric.metricPointId =
+                        IncidentMetricType.TimeToAcknowledge;
+                    timeToAcknowledgeMetric.value = OneUptimeDate.getDifferenceInSeconds((ackIncidentStateTimeline === null || ackIncidentStateTimeline === void 0 ? void 0 : ackIncidentStateTimeline.startsAt) ||
+                        OneUptimeDate.getCurrentDate(), incidentStartsAt);
+                    timeToAcknowledgeMetric.attributes = Object.assign({}, baseMetricAttributes);
+                    timeToAcknowledgeMetric.attributeKeys =
+                        TelemetryUtil.getAttributeKeys(timeToAcknowledgeMetric.attributes);
+                    timeToAcknowledgeMetric.time =
+                        (ackIncidentStateTimeline === null || ackIncidentStateTimeline === void 0 ? void 0 : ackIncidentStateTimeline.startsAt) ||
+                            incident.declaredAt ||
+                            incident.createdAt ||
+                            OneUptimeDate.getCurrentDate();
+                    timeToAcknowledgeMetric.timeUnixNano = OneUptimeDate.toUnixNano(timeToAcknowledgeMetric.time);
+                    timeToAcknowledgeMetric.metricPointType = MetricPointType.Sum;
+                    timeToAcknowledgeMetric.retentionDate = incidentMetricRetentionDate;
+                    itemsToSave.push(timeToAcknowledgeMetric);
                 }
             }
             // time to resolve
@@ -2268,39 +2189,26 @@ ${incidentSeverity.name}
                 metricType.description = "Time taken to resolve the incident";
                 metricType.unit = "seconds";
                 metricTypesMap[IncidentMetricType.TimeToResolve] = metricType;
-                // write-once: MTTR is fixed once the incident is first resolved.
-                if (!existingMetricNames.has(IncidentMetricType.TimeToResolve)) {
-                    const timeToResolveMetric = new Metric();
-                    timeToResolveMetric.projectId = incident.projectId;
-                    timeToResolveMetric.primaryEntityId = incident.id;
-                    timeToResolveMetric.primaryEntityType = ServiceType.Incident;
-                    timeToResolveMetric.name = IncidentMetricType.TimeToResolve;
-                    timeToResolveMetric.value = OneUptimeDate.getDifferenceInSeconds((resolvedIncidentStateTimeline === null || resolvedIncidentStateTimeline === void 0 ? void 0 : resolvedIncidentStateTimeline.startsAt) ||
-                        OneUptimeDate.getCurrentDate(), incidentStartsAt);
-                    timeToResolveMetric.attributes = Object.assign({}, baseMetricAttributes);
-                    timeToResolveMetric.attributeKeys = TelemetryUtil.getAttributeKeys(timeToResolveMetric.attributes);
-                    timeToResolveMetric.time =
-                        (resolvedIncidentStateTimeline === null || resolvedIncidentStateTimeline === void 0 ? void 0 : resolvedIncidentStateTimeline.startsAt) ||
-                            incident.declaredAt ||
-                            incident.createdAt ||
-                            OneUptimeDate.getCurrentDate();
-                    timeToResolveMetric.timeUnixNano = OneUptimeDate.toUnixNano(timeToResolveMetric.time);
-                    timeToResolveMetric.metricPointType = MetricPointType.Sum;
-                    timeToResolveMetric.retentionDate = incidentMetricRetentionDate;
-                    itemsToSave.push(timeToResolveMetric);
-                }
+                const timeToResolveMetric = new MutableMetric();
+                timeToResolveMetric.projectId = incident.projectId;
+                timeToResolveMetric.primaryEntityId = incident.id;
+                timeToResolveMetric.primaryEntityType = ServiceType.Incident;
+                timeToResolveMetric.name = IncidentMetricType.TimeToResolve;
+                timeToResolveMetric.metricPointId = IncidentMetricType.TimeToResolve;
+                timeToResolveMetric.value = OneUptimeDate.getDifferenceInSeconds((resolvedIncidentStateTimeline === null || resolvedIncidentStateTimeline === void 0 ? void 0 : resolvedIncidentStateTimeline.startsAt) ||
+                    OneUptimeDate.getCurrentDate(), incidentStartsAt);
+                timeToResolveMetric.attributes = Object.assign({}, baseMetricAttributes);
+                timeToResolveMetric.attributeKeys = TelemetryUtil.getAttributeKeys(timeToResolveMetric.attributes);
+                timeToResolveMetric.time =
+                    (resolvedIncidentStateTimeline === null || resolvedIncidentStateTimeline === void 0 ? void 0 : resolvedIncidentStateTimeline.startsAt) ||
+                        incident.declaredAt ||
+                        incident.createdAt ||
+                        OneUptimeDate.getCurrentDate();
+                timeToResolveMetric.timeUnixNano = OneUptimeDate.toUnixNano(timeToResolveMetric.time);
+                timeToResolveMetric.metricPointType = MetricPointType.Sum;
+                timeToResolveMetric.retentionDate = incidentMetricRetentionDate;
+                itemsToSave.push(timeToResolveMetric);
             }
-            /*
-             * Incident duration — write-once, finalized at resolution.
-             *
-             * The previous implementation recomputed duration as
-             * (latest state transition − start) on every refresh, so an open
-             * incident's duration grew and had to be deleted + re-inserted each
-             * time. With no deletes we emit a single final duration once the
-             * incident reaches a resolved state: (first resolution − start). An
-             * incident that is never resolved has no duration row (its lifetime is
-             * not yet final), which keeps the "Avg Duration" widget meaningful.
-             */
             if (isIncidentResolved && resolvedIncidentStateTimeline) {
                 // register the metric type so the catalog stays complete across refreshes.
                 const metricType = new MetricType();
@@ -2308,22 +2216,47 @@ ${incidentSeverity.name}
                 metricType.description = "Duration of the incident";
                 metricType.unit = "seconds";
                 metricTypesMap[IncidentMetricType.IncidentDuration] = metricType;
-                if (!existingMetricNames.has(IncidentMetricType.IncidentDuration)) {
-                    const incidentEndsAt = resolvedIncidentStateTimeline.startsAt ||
-                        OneUptimeDate.getCurrentDate();
-                    const incidentDurationMetric = new Metric();
-                    incidentDurationMetric.projectId = incident.projectId;
-                    incidentDurationMetric.primaryEntityId = incident.id;
-                    incidentDurationMetric.primaryEntityType = ServiceType.Incident;
-                    incidentDurationMetric.name = IncidentMetricType.IncidentDuration;
-                    incidentDurationMetric.value = OneUptimeDate.getDifferenceInSeconds(incidentEndsAt, incidentStartsAt);
-                    incidentDurationMetric.attributes = Object.assign({}, baseMetricAttributes);
-                    incidentDurationMetric.attributeKeys = TelemetryUtil.getAttributeKeys(incidentDurationMetric.attributes);
-                    incidentDurationMetric.time = incidentEndsAt;
-                    incidentDurationMetric.timeUnixNano = OneUptimeDate.toUnixNano(incidentDurationMetric.time);
-                    incidentDurationMetric.metricPointType = MetricPointType.Sum;
-                    incidentDurationMetric.retentionDate = incidentMetricRetentionDate;
-                    itemsToSave.push(incidentDurationMetric);
+                const incidentEndsAt = resolvedIncidentStateTimeline.startsAt ||
+                    OneUptimeDate.getCurrentDate();
+                const incidentDurationMetric = new MutableMetric();
+                incidentDurationMetric.projectId = incident.projectId;
+                incidentDurationMetric.primaryEntityId = incident.id;
+                incidentDurationMetric.primaryEntityType = ServiceType.Incident;
+                incidentDurationMetric.name = IncidentMetricType.IncidentDuration;
+                incidentDurationMetric.metricPointId =
+                    IncidentMetricType.IncidentDuration;
+                incidentDurationMetric.value = OneUptimeDate.getDifferenceInSeconds(incidentEndsAt, incidentStartsAt);
+                incidentDurationMetric.attributes = Object.assign({}, baseMetricAttributes);
+                incidentDurationMetric.attributeKeys = TelemetryUtil.getAttributeKeys(incidentDurationMetric.attributes);
+                incidentDurationMetric.time = incidentEndsAt;
+                incidentDurationMetric.timeUnixNano = OneUptimeDate.toUnixNano(incidentDurationMetric.time);
+                incidentDurationMetric.metricPointType = MetricPointType.Sum;
+                incidentDurationMetric.retentionDate = incidentMetricRetentionDate;
+                itemsToSave.push(incidentDurationMetric);
+                if (incident.postmortemPostedAt) {
+                    const postmortemMetricType = new MetricType();
+                    postmortemMetricType.name =
+                        IncidentMetricType.PostmortemCompletionTime;
+                    postmortemMetricType.description =
+                        "Time from incident resolution to postmortem publication";
+                    postmortemMetricType.unit = "seconds";
+                    metricTypesMap[IncidentMetricType.PostmortemCompletionTime] =
+                        postmortemMetricType;
+                    const postmortemMetric = new MutableMetric();
+                    postmortemMetric.projectId = incident.projectId;
+                    postmortemMetric.primaryEntityId = incident.id;
+                    postmortemMetric.primaryEntityType = ServiceType.Incident;
+                    postmortemMetric.name = IncidentMetricType.PostmortemCompletionTime;
+                    postmortemMetric.metricPointId =
+                        IncidentMetricType.PostmortemCompletionTime;
+                    postmortemMetric.value = OneUptimeDate.getDifferenceInSeconds(incident.postmortemPostedAt, incidentEndsAt);
+                    postmortemMetric.attributes = Object.assign({}, baseMetricAttributes);
+                    postmortemMetric.attributeKeys = TelemetryUtil.getAttributeKeys(postmortemMetric.attributes);
+                    postmortemMetric.time = incident.postmortemPostedAt;
+                    postmortemMetric.timeUnixNano = OneUptimeDate.toUnixNano(postmortemMetric.time);
+                    postmortemMetric.metricPointType = MetricPointType.Sum;
+                    postmortemMetric.retentionDate = incidentMetricRetentionDate;
+                    itemsToSave.push(postmortemMetric);
                 }
             }
             // time-in-state metrics — emit one metric per state transition that has a completed duration
@@ -2331,19 +2264,15 @@ ${incidentSeverity.name}
                 if (!timeline.startsAt || !timeline.endsAt) {
                     continue;
                 }
-                // write-once: each completed state transition is emitted exactly once.
-                const timeInStateKey = buildTimeInStateKey((_j = timeline.incidentStateId) === null || _j === void 0 ? void 0 : _j.toString(), timeline.startsAt);
-                if (existingTimeInStateKeys.has(timeInStateKey)) {
-                    continue;
-                }
-                const stateName = ((_l = (_k = timeline.incidentState) === null || _k === void 0 ? void 0 : _k.name) === null || _l === void 0 ? void 0 : _l.toString()) || "Unknown";
-                const timeInStateMetric = new Metric();
+                const stateName = ((_k = (_j = timeline.incidentState) === null || _j === void 0 ? void 0 : _j.name) === null || _k === void 0 ? void 0 : _k.toString()) || "Unknown";
+                const timeInStateMetric = new MutableMetric();
                 timeInStateMetric.projectId = incident.projectId;
                 timeInStateMetric.primaryEntityId = incident.id;
                 timeInStateMetric.primaryEntityType = ServiceType.Incident;
                 timeInStateMetric.name = IncidentMetricType.TimeInState;
+                timeInStateMetric.metricPointId = `time-in-state:${timeline.id.toString()}`;
                 timeInStateMetric.value = OneUptimeDate.getDifferenceInSeconds(timeline.endsAt, timeline.startsAt);
-                timeInStateMetric.attributes = Object.assign(Object.assign({}, baseMetricAttributes), { incidentStateName: stateName, incidentStateId: (_m = timeline.incidentStateId) === null || _m === void 0 ? void 0 : _m.toString(), isCreatedState: ((_p = (_o = timeline.incidentState) === null || _o === void 0 ? void 0 : _o.isCreatedState) === null || _p === void 0 ? void 0 : _p.toString()) || "false", isAcknowledgedState: ((_r = (_q = timeline.incidentState) === null || _q === void 0 ? void 0 : _q.isAcknowledgedState) === null || _r === void 0 ? void 0 : _r.toString()) || "false", isResolvedState: ((_t = (_s = timeline.incidentState) === null || _s === void 0 ? void 0 : _s.isResolvedState) === null || _t === void 0 ? void 0 : _t.toString()) || "false" });
+                timeInStateMetric.attributes = Object.assign(Object.assign({}, baseMetricAttributes), { incidentStateName: stateName, incidentStateId: (_l = timeline.incidentStateId) === null || _l === void 0 ? void 0 : _l.toString(), isCreatedState: ((_o = (_m = timeline.incidentState) === null || _m === void 0 ? void 0 : _m.isCreatedState) === null || _o === void 0 ? void 0 : _o.toString()) || "false", isAcknowledgedState: ((_q = (_p = timeline.incidentState) === null || _p === void 0 ? void 0 : _p.isAcknowledgedState) === null || _q === void 0 ? void 0 : _q.toString()) || "false", isResolvedState: ((_s = (_r = timeline.incidentState) === null || _r === void 0 ? void 0 : _r.isResolvedState) === null || _s === void 0 ? void 0 : _s.toString()) || "false" });
                 timeInStateMetric.attributeKeys = TelemetryUtil.getAttributeKeys(timeInStateMetric.attributes);
                 timeInStateMetric.time = timeline.startsAt;
                 timeInStateMetric.timeUnixNano = OneUptimeDate.toUnixNano(timeInStateMetric.time);
@@ -2362,15 +2291,21 @@ ${incidentSeverity.name}
                 timeInStateMetricType.unit = "seconds";
                 metricTypesMap[timeInStateMetricType.name] = timeInStateMetricType;
             }
-            // write-once: a refresh that finds nothing new inserts nothing.
-            if (itemsToSave.length > 0) {
-                await MetricService.createMany({
-                    items: itemsToSave,
-                    props: {
-                        isRoot: true,
-                    },
-                });
-            }
+            await MutableMetricService.replaceEntityMetrics({
+                projectId: incident.projectId,
+                primaryEntityId: incident.id,
+                primaryEntityType: ServiceType.Incident,
+                metricNames: [
+                    IncidentMetricType.IncidentCount,
+                    IncidentMetricType.TimeToAcknowledge,
+                    IncidentMetricType.TimeToResolve,
+                    IncidentMetricType.IncidentDuration,
+                    IncidentMetricType.TimeInState,
+                    IncidentMetricType.PostmortemCompletionTime,
+                ],
+                metrics: itemsToSave,
+                retentionDate: incidentMetricRetentionDate,
+            });
             TelemetryUtil.indexMetricNameServiceNameMap({
                 metricNameServiceNameMap: metricTypesMap,
                 projectId: incident.projectId,
@@ -2389,8 +2324,8 @@ ${incidentSeverity.name}
                 }
                 catch (err) {
                     logger.error(err, {
-                        projectId: (_u = incident.projectId) === null || _u === void 0 ? void 0 : _u.toString(),
-                        incidentId: (_v = incident.id) === null || _v === void 0 ? void 0 : _v.toString(),
+                        projectId: (_t = incident.projectId) === null || _t === void 0 ? void 0 : _t.toString(),
+                        incidentId: (_u = incident.id) === null || _u === void 0 ? void 0 : _u.toString(),
                     });
                 }
             }