iobroker.poolcontrol 1.3.15 → 1.3.18

package/lib/helpers/chemistryPhHelper.js

@@ -1,6 +1,8 @@
 'use strict';
 
 const { I18n } = require('@iobroker/adapter-core');
+const MAX_HISTORY_AGE_MS = 30 * 24 * 60 * 60 * 1000;
+const MIN_SAMPLE_INTERVAL_MS = 15 * 60 * 1000;
 
 /**
  * chemistryPhHelper
@@ -107,6 +109,11 @@ const chemistryPhHelper = {
                 return;
             }
 
+            if (id.endsWith('chemistry.ph.input.manual_value') && state.ack === false) {
+                await this._processValue('manual', state.val, 'manual_value');
+                return;
+            }
+
             if (this._isRelevantOwnState(id)) {
                 this._scheduleEvaluation(`state_change:${id}`, 250);
             }
@@ -208,7 +215,8 @@ const chemistryPhHelper = {
                 await this._setBool('chemistry.ph.measurement.allowed', false);
                 await this._setString('chemistry.ph.measurement.ignored_reason', 'disabled');
                 await this._setString('chemistry.ph.debug.last_reason', reason || 'disabled');
-                await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(new Date()));
+                const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+                await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX
                 return;
             }
 
@@ -227,7 +235,8 @@ const chemistryPhHelper = {
                 await this._setBool('chemistry.ph.measurement.allowed', false);
                 await this._setString('chemistry.ph.measurement.ignored_reason', 'source_disabled');
                 await this._setString('chemistry.ph.debug.last_reason', reason || 'source_disabled');
-                await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(new Date()));
+                const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+                await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX
                 return;
             }
 
@@ -256,7 +265,8 @@ const chemistryPhHelper = {
                     await this._setBool('chemistry.ph.measurement.allowed', false);
                     await this._setString('chemistry.ph.measurement.ignored_reason', 'missing_source_state');
                     await this._setString('chemistry.ph.debug.last_reason', reason || 'missing_source_state');
-                    await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(new Date()));
+                    const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+                    await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX
                     return;
                 }
 
@@ -280,7 +290,8 @@ const chemistryPhHelper = {
                     await this._setBool('chemistry.ph.measurement.allowed', false);
                     await this._setString('chemistry.ph.measurement.ignored_reason', 'source_read_error');
                     await this._setString('chemistry.ph.debug.last_reason', `source_read_error: ${err.message}`);
-                    await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(new Date()));
+                    const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+                    await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX
                     return;
                 }
 
@@ -300,7 +311,8 @@ const chemistryPhHelper = {
                     await this._setBool('chemistry.ph.measurement.allowed', false);
                     await this._setString('chemistry.ph.measurement.ignored_reason', 'source_not_found');
                     await this._setString('chemistry.ph.debug.last_reason', reason || 'source_not_found');
-                    await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(new Date()));
+                    const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+                    await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX
                     return;
                 }
 
@@ -313,7 +325,8 @@ const chemistryPhHelper = {
             await this._setBool('chemistry.ph.evaluation.action_required', false);
             await this._setString('chemistry.ph.measurement.ignored_reason', 'unknown_source_mode');
             await this._setString('chemistry.ph.debug.last_reason', reason || 'unknown_source_mode');
-            await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(new Date()));
+            const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+            await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX
         } catch (err) {
             this.adapter.log.warn(`[chemistryPhHelper] Evaluation failed: ${err.message}`);
         }
@@ -323,7 +336,7 @@ const chemistryPhHelper = {
         const now = new Date();
         const value = Number(rawValue);
 
-        await this._setString('chemistry.ph.input.last_value_at', this._formatDateTime(now));
+        await this._setNumber('chemistry.ph.input.last_value_at', now.getTime()); // FIX: value.time uses numeric ms timestamp.
 
         if (source === 'manual') {
             await this._setBool('chemistry.ph.input.source_valid', true);
@@ -348,7 +361,7 @@ const chemistryPhHelper = {
             await this._setBool('chemistry.ph.measurement.allowed', false);
             await this._setString('chemistry.ph.measurement.ignored_reason', 'invalid_value');
             await this._setString('chemistry.ph.debug.last_reason', reason || 'invalid_value');
-            await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(now));
+            await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX: value.time uses numeric ms timestamp.
             return;
         }
 
@@ -361,18 +374,24 @@ const chemistryPhHelper = {
             await this._setBool('chemistry.ph.measurement.allowed', false);
             await this._setString('chemistry.ph.measurement.ignored_reason', measurementAllowed.reason);
             await this._setString('chemistry.ph.debug.last_reason', reason || measurementAllowed.reason);
-            await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(now));
+            await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX: value.time uses numeric ms timestamp.
             return;
         }
 
         await this._setBool('chemistry.ph.measurement.allowed', true);
         await this._setString('chemistry.ph.measurement.ignored_reason', '');
 
-        await this._updateHistory(value, now);
-        await this._evaluateValue(value);
+        await this._updateLastValues(value, now);
+
+        const history = await this._updateHistory(value, now, reason === 'external_state' || reason === 'manual_value');
+        const trend = await this._calculateTrend(value, now, history);
+        const evaluation = await this._evaluateValue(value);
+
+        await this._writeTrend(trend);
+        await this._writeOutputs(value, trend, evaluation);
 
         await this._setString('chemistry.ph.debug.last_reason', reason || 'value_processed');
-        await this._setString('chemistry.ph.debug.last_update', this._formatDateTime(now));
+        await this._setNumber('chemistry.ph.debug.last_update', now.getTime()); // FIX: value.time uses numeric ms timestamp.
     },
 
     async _checkMeasurementAllowed(now) {
@@ -427,24 +446,224 @@ const chemistryPhHelper = {
         return { allowed: true, reason: '', status: 'ok', recommendation: '' };
     },
 
-    async _updateHistory(value, now) {
+    async _updateLastValues(value, now) {
         const lastValid = await this._readNumberOrNull('chemistry.ph.input.last_valid_value');
-        const lastValidAt = await this._readString('chemistry.ph.input.last_valid_value_at');
+        const lastValidAt = await this._readTimestampOrNull('chemistry.ph.input.last_valid_value_at'); // FIX: accept numeric ms timestamps and legacy German strings.
 
         if (lastValid !== null && lastValidAt) {
             await this._setNumber('chemistry.ph.input.previous_value', lastValid);
-            await this._setString('chemistry.ph.input.previous_value_at', lastValidAt);
+            await this._setNumber('chemistry.ph.input.previous_value_at', lastValidAt); // FIX: pass stored timestamp through unchanged.
+
+            const minutes = Math.max(0, Math.round((now.getTime() - lastValidAt) / 60000)); // FIX: calculate from numeric ms timestamp.
+            await this._setNumber('chemistry.ph.input.minutes_since_previous_value', minutes);
+        }
 
-            const previousDate = this._parseGermanDateTime(lastValidAt);
+        await this._setNumber('chemistry.ph.input.last_valid_value', value);
+        await this._setNumber('chemistry.ph.input.last_valid_value_at', now.getTime()); // FIX: value.time uses numeric ms timestamp.
+    },
 
-            if (previousDate) {
-                const minutes = Math.max(0, Math.round((now.getTime() - previousDate.getTime()) / 60000));
-                await this._setNumber('chemistry.ph.input.minutes_since_previous_value', minutes);
+    async _updateHistory(value, now, forceSample) {
+        let samples = await this._readJsonArray('chemistry.ph.history.samples_json');
+        const nowTs = now.getTime();
+        const minTs = nowTs - MAX_HISTORY_AGE_MS;
+
+        samples = samples.filter(
+            sample => sample && Number(sample.ts) >= minTs && Number.isFinite(Number(sample.value)),
+        );
+
+        const newest = samples.length ? samples[samples.length - 1] : null;
+        const newestTs = newest ? Number(newest.ts) : 0;
+        const shouldStore = forceSample || !newest || nowTs - newestTs >= MIN_SAMPLE_INTERVAL_MS;
+
+        if (shouldStore) {
+            samples.push({
+                ts: nowTs,
+                time: this._formatDateTime(now),
+                value,
+            });
+        }
+
+        samples = samples.filter(sample => sample && Number(sample.ts) >= minTs);
+
+        await this._setString('chemistry.ph.history.samples_json', JSON.stringify(samples));
+        await this._setNumber('chemistry.ph.history.samples_count', samples.length);
+
+        if (samples.length) {
+            await this._setNumber(
+                // FIX: history value.time states store sample timestamps, not readable strings.
+                'chemistry.ph.history.oldest_sample_at',
+                Number(samples[0].ts),
+            );
+            await this._setNumber(
+                // FIX: history value.time states store sample timestamps, not readable strings.
+                'chemistry.ph.history.newest_sample_at',
+                Number(samples[samples.length - 1].ts),
+            );
+        }
+
+        return samples;
+    },
+
+    async _calculateTrend(currentValue, now, samples) {
+        const nowTs = now.getTime();
+
+        const ref24h = this._findReferenceSample(samples, nowTs - 24 * 60 * 60 * 1000);
+        const ref7d = this._findReferenceSample(samples, nowTs - 7 * 24 * 60 * 60 * 1000);
+        const ref30d = this._findReferenceSample(samples, nowTs - 30 * 24 * 60 * 60 * 1000);
+
+        const delta24h = ref24h ? currentValue - ref24h.value : 0;
+        const delta7d = ref7d ? currentValue - ref7d.value : 0;
+        const delta30d = ref30d ? currentValue - ref30d.value : 0;
+
+        const direction = this._getOverallDirection(ref24h, ref7d, ref30d, delta24h, delta7d, delta30d);
+        const status = this._getTrendStatus(delta24h, delta7d, delta30d, ref24h, ref7d, ref30d);
+
+        return {
+            ref24h,
+            ref7d,
+            ref30d,
+            delta24h,
+            delta7d,
+            delta30d,
+            direction,
+            status,
+        };
+    },
+
+    _findReferenceSample(samples, targetTs) {
+        if (!samples.length) {
+            return null;
+        }
+
+        let best = null;
+        let bestDistance = Number.MAX_SAFE_INTEGER;
+
+        for (const sample of samples) {
+            const ts = Number(sample.ts);
+            const value = Number(sample.value);
+
+            if (!Number.isFinite(ts) || !Number.isFinite(value)) {
+                continue;
+            }
+
+            const distance = Math.abs(ts - targetTs);
+
+            if (distance < bestDistance) {
+                bestDistance = distance;
+                best = {
+                    ts,
+                    value,
+                    time: sample.time || this._formatDateTime(new Date(ts)),
+                };
             }
         }
 
-        await this._setNumber('chemistry.ph.input.last_valid_value', value);
-        await this._setString('chemistry.ph.input.last_valid_value_at', this._formatDateTime(now));
+        return best;
+    },
+
+    _getOverallDirection(ref24h, ref7d, ref30d, delta24h, delta7d, delta30d) {
+        const available = [ref24h ? delta24h : null, ref7d ? delta7d : null, ref30d ? delta30d : null].filter(
+            value => value !== null,
+        );
+
+        if (!available.length) {
+            return 'not_enough_data';
+        }
+
+        const strongest = available.reduce((prev, current) => (Math.abs(current) > Math.abs(prev) ? current : prev), 0);
+
+        if (Math.abs(strongest) < 0.05) {
+            return 'stable';
+        }
+
+        return strongest > 0 ? 'rising' : 'falling';
+    },
+
+    _getTrendStatus(delta24h, delta7d, delta30d, ref24h, ref7d, ref30d) {
+        if (!ref24h && !ref7d && !ref30d) {
+            return 'not_enough_data';
+        }
+
+        if (delta24h > 0.2 || delta7d > 0.4 || delta30d > 0.6) {
+            return 'rising_fast';
+        }
+
+        if (delta24h > 0.1 || delta7d > 0.25 || delta30d > 0.4) {
+            return 'rising_noticeable';
+        }
+
+        if (delta24h > 0.05 || delta7d > 0.15 || delta30d > 0.25) {
+            return 'rising_slowly';
+        }
+
+        if (delta24h < -0.05 || delta7d < -0.15 || delta30d < -0.25) {
+            return 'falling';
+        }
+
+        return 'stable';
+    },
+
+    async _writeTrend(trend) {
+        await this._setNumber('chemistry.ph.trend.reference_24h_value', trend.ref24h ? trend.ref24h.value : 0);
+        await this._setNumber('chemistry.ph.trend.reference_24h_at', trend.ref24h ? trend.ref24h.ts : 0); // FIX
+        await this._setNumber('chemistry.ph.trend.delta_24h', trend.ref24h ? trend.delta24h : 0);
+
+        await this._setNumber('chemistry.ph.trend.reference_7d_value', trend.ref7d ? trend.ref7d.value : 0);
+        await this._setNumber('chemistry.ph.trend.reference_7d_at', trend.ref7d ? trend.ref7d.ts : 0); // FIX
+        await this._setNumber('chemistry.ph.trend.delta_7d', trend.ref7d ? trend.delta7d : 0);
+
+        await this._setNumber('chemistry.ph.trend.reference_30d_value', trend.ref30d ? trend.ref30d.value : 0);
+        await this._setNumber('chemistry.ph.trend.reference_30d_at', trend.ref30d ? trend.ref30d.ts : 0); // FIX
+        await this._setNumber('chemistry.ph.trend.delta_30d', trend.ref30d ? trend.delta30d : 0);
+
+        await this._setString('chemistry.ph.trend.direction', trend.direction);
+        await this._setString('chemistry.ph.trend.status', trend.status);
+    },
+
+    async _writeOutputs(value, trend, evaluation) {
+        const text =
+            `${I18n.translate('Current pH value')}: ${value.toFixed(2)}. ` +
+            `24h: ${trend.ref24h ? this._formatDelta(trend.delta24h) : I18n.translate('not enough data')}, ` +
+            `7d: ${trend.ref7d ? this._formatDelta(trend.delta7d) : I18n.translate('not enough data')}, ` +
+            `30d: ${trend.ref30d ? this._formatDelta(trend.delta30d) : I18n.translate('not enough data')}. ` +
+            `${evaluation.recommendation}`;
+
+        const html =
+            `<div>` +
+            `<b>${I18n.translate('Current pH value')}:</b> ${value.toFixed(2)}<br>` +
+            `<b>24h:</b> ${trend.ref24h ? `${trend.ref24h.value.toFixed(2)} / ${this._formatDelta(trend.delta24h)}` : I18n.translate('not enough data')}<br>` +
+            `<b>7d:</b> ${trend.ref7d ? `${trend.ref7d.value.toFixed(2)} / ${this._formatDelta(trend.delta7d)}` : I18n.translate('not enough data')}<br>` +
+            `<b>30d:</b> ${trend.ref30d ? `${trend.ref30d.value.toFixed(2)} / ${this._formatDelta(trend.delta30d)}` : I18n.translate('not enough data')}<br>` +
+            `<b>${I18n.translate('Trend status')}:</b> ${trend.status}<br>` +
+            `<b>${I18n.translate('Status')}:</b> ${evaluation.status}<br>` +
+            `<b>${I18n.translate('Recommendation')}:</b> ${this._escapeHtml(evaluation.recommendation)}` +
+            `</div>`;
+
+        const json = {
+            current: Number(value.toFixed(2)),
+            unit: 'pH',
+            trend_24h: {
+                reference: trend.ref24h ? Number(trend.ref24h.value.toFixed(2)) : null,
+                delta: trend.ref24h ? Number(trend.delta24h.toFixed(2)) : null,
+            },
+            trend_7d: {
+                reference: trend.ref7d ? Number(trend.ref7d.value.toFixed(2)) : null,
+                delta: trend.ref7d ? Number(trend.delta7d.toFixed(2)) : null,
+            },
+            trend_30d: {
+                reference: trend.ref30d ? Number(trend.ref30d.value.toFixed(2)) : null,
+                delta: trend.ref30d ? Number(trend.delta30d.toFixed(2)) : null,
+            },
+            direction: trend.direction,
+            trend_status: trend.status,
+            status: evaluation.status,
+            action_required: evaluation.actionRequired,
+            recommendation: evaluation.recommendation,
+        };
+
+        await this._setString('chemistry.ph.outputs.summary_text', text);
+        await this._setString('chemistry.ph.outputs.summary_html', html);
+        await this._setString('chemistry.ph.outputs.summary_json', JSON.stringify(json));
     },
 
     async _evaluateValue(value) {
@@ -452,35 +671,48 @@ const chemistryPhHelper = {
         const max = await this._readNumber('chemistry.ph.evaluation.target_max');
 
         if (value < min) {
-            await this._setString('chemistry.ph.evaluation.status', 'low');
-            await this._setString(
-                'chemistry.ph.evaluation.recommendation',
-                I18n.translate(
-                    'pH value is too low. Check whether pH plus should be added according to the product instructions. Then circulate and measure again.',
-                ),
+            const recommendation = I18n.translate(
+                'pH value is too low. Check whether pH plus should be added according to the product instructions. Then circulate and measure again.',
             );
+
+            await this._setString('chemistry.ph.evaluation.status', 'low');
+            await this._setString('chemistry.ph.evaluation.recommendation', recommendation);
             await this._setBool('chemistry.ph.evaluation.action_required', true);
-            return;
+
+            return {
+                status: 'low',
+                actionRequired: true,
+                recommendation,
+            };
         }
 
         if (value > max) {
-            await this._setString('chemistry.ph.evaluation.status', 'high');
-            await this._setString(
-                'chemistry.ph.evaluation.recommendation',
-                I18n.translate(
-                    'pH value is too high. Check whether pH minus should be added according to the product instructions. Then circulate and measure again.',
-                ),
+            const recommendation = I18n.translate(
+                'pH value is too high. Check whether pH minus should be added according to the product instructions. Then circulate and measure again.',
             );
+
+            await this._setString('chemistry.ph.evaluation.status', 'high');
+            await this._setString('chemistry.ph.evaluation.recommendation', recommendation);
             await this._setBool('chemistry.ph.evaluation.action_required', true);
-            return;
+
+            return {
+                status: 'high',
+                actionRequired: true,
+                recommendation,
+            };
         }
 
+        const recommendation = I18n.translate('pH value is within the target range. No action is required.');
+
         await this._setString('chemistry.ph.evaluation.status', 'ok');
-        await this._setString(
-            'chemistry.ph.evaluation.recommendation',
-            I18n.translate('pH value is within the target range. No action is required.'),
-        );
+        await this._setString('chemistry.ph.evaluation.recommendation', recommendation);
         await this._setBool('chemistry.ph.evaluation.action_required', false);
+
+        return {
+            status: 'ok',
+            actionRequired: false,
+            recommendation,
+        };
     },
 
     async _startMixingRun() {
@@ -641,6 +873,30 @@ const chemistryPhHelper = {
         return Number.isFinite(value) ? value : null;
     },
 
+    async _readTimestampOrNull(id) {
+        const state = await this.adapter.getStateAsync(id);
+        const value = state?.val;
+        const numberValue = Number(value);
+
+        if (Number.isFinite(numberValue) && numberValue > 0) {
+            return numberValue; // FIX: numeric value.time timestamps are used directly.
+        }
+
+        const legacyDate = this._parseGermanDateTime(value); // FIX: keep backward compatibility for old string values.
+        return legacyDate ? legacyDate.getTime() : null;
+    },
+
+    async _readJsonArray(id) {
+        const state = await this.adapter.getStateAsync(id);
+
+        try {
+            const parsed = JSON.parse(String(state?.val || '[]'));
+            return Array.isArray(parsed) ? parsed : [];
+        } catch {
+            return [];
+        }
+    },
+
     async _readBoolean(id) {
         const state = await this.adapter.getStateAsync(id);
         return !!state?.val;
@@ -659,6 +915,11 @@ const chemistryPhHelper = {
         await this.adapter.setStateChangedAsync(id, { val: !!value, ack: true });
     },
 
+    _formatDelta(value) {
+        const rounded = Number(value) || 0;
+        return `${rounded >= 0 ? '+' : ''}${rounded.toFixed(2)}`;
+    },
+
     _formatDateTime(date) {
         return date.toLocaleString('de-DE', {
             year: 'numeric',
@@ -681,6 +942,15 @@ const chemistryPhHelper = {
         return new Date(Number(year), Number(month) - 1, Number(day), Number(hour), Number(minute), Number(second));
     },
 
+    _escapeHtml(value) {
+        return String(value ?? '')
+            .replace(/&/g, '&amp;')
+            .replace(/</g, '&lt;')
+            .replace(/>/g, '&gt;')
+            .replace(/"/g, '&quot;')
+            .replace(/'/g, '&#039;');
+    },
+
     cleanup() {
         if (this.evalTimer) {
             this.adapter.clearTimeout(this.evalTimer);

package/lib/helpers/chemistryTdsHelper.js

@@ -246,7 +246,7 @@ const chemistryTdsHelper = {
         const now = new Date();
         const value = Number(rawValue);
 
-        await this._setString('chemistry.tds.input.last_value_at', this._formatDateTime(now));
+        await this._setNumber('chemistry.tds.input.last_value_at', now.getTime()); // FIX: value.time uses numeric ms timestamp.
 
         if (source === 'manual') {
             await this._setBool('chemistry.tds.input.source_valid', true);
@@ -278,7 +278,7 @@ const chemistryTdsHelper = {
             await this._setString('chemistry.tds.evaluation.recommendation', measurementAllowed.recommendation);
             await this._setBool('chemistry.tds.evaluation.action_required', false);
             await this._setString('chemistry.tds.debug.last_reason', reason || measurementAllowed.reason);
-            await this._setString('chemistry.tds.debug.last_update', this._formatDateTime(now));
+            await this._setNumber('chemistry.tds.debug.last_update', now.getTime()); // FIX: value.time uses numeric ms timestamp.
             return;
         }
 
@@ -297,7 +297,7 @@ const chemistryTdsHelper = {
         await this._writeOutputs(value, reference, trend, evaluation);
 
         await this._setString('chemistry.tds.debug.last_reason', reason || 'value_processed');
-        await this._setString('chemistry.tds.debug.last_update', this._formatDateTime(now));
+        await this._setNumber('chemistry.tds.debug.last_update', now.getTime()); // FIX: value.time uses numeric ms timestamp.
     },
 
     async _checkMeasurementAllowed(now) {
@@ -357,22 +357,18 @@ const chemistryTdsHelper = {
 
     async _updateLastValues(value, now) {
         const lastValid = await this._readNumberOrNull('chemistry.tds.input.last_valid_value');
-        const lastValidAt = await this._readString('chemistry.tds.input.last_valid_value_at');
+        const lastValidAt = await this._readTimestampOrNull('chemistry.tds.input.last_valid_value_at'); // FIX: accept numeric ms timestamps and legacy German strings.
 
         if (lastValid !== null && lastValidAt) {
             await this._setNumber('chemistry.tds.input.previous_value', lastValid);
-            await this._setString('chemistry.tds.input.previous_value_at', lastValidAt);
+            await this._setNumber('chemistry.tds.input.previous_value_at', lastValidAt); // FIX: pass stored timestamp through unchanged.
 
-            const previousDate = this._parseGermanDateTime(lastValidAt);
-
-            if (previousDate) {
-                const minutes = Math.max(0, Math.round((now.getTime() - previousDate.getTime()) / 60000));
-                await this._setNumber('chemistry.tds.input.minutes_since_previous_value', minutes);
-            }
+            const minutes = Math.max(0, Math.round((now.getTime() - lastValidAt) / 60000)); // FIX: calculate from numeric ms timestamp.
+            await this._setNumber('chemistry.tds.input.minutes_since_previous_value', minutes);
         }
 
         await this._setNumber('chemistry.tds.input.last_valid_value', value);
-        await this._setString('chemistry.tds.input.last_valid_value_at', this._formatDateTime(now));
+        await this._setNumber('chemistry.tds.input.last_valid_value_at', now.getTime()); // FIX: value.time uses numeric ms timestamp.
     },
 
     async _updateHistory(value, now, forceSample) {
@@ -402,13 +398,15 @@ const chemistryTdsHelper = {
         await this._setNumber('chemistry.tds.history.samples_count', samples.length);
 
         if (samples.length) {
-            await this._setString(
+            await this._setNumber(
+                // FIX: history value.time states store sample timestamps, not readable strings.
                 'chemistry.tds.history.oldest_sample_at',
-                samples[0].time || this._formatDateTime(new Date(samples[0].ts)),
+                Number(samples[0].ts),
             );
-            await this._setString(
+            await this._setNumber(
+                // FIX: history value.time states store sample timestamps, not readable strings.
                 'chemistry.tds.history.newest_sample_at',
-                samples[samples.length - 1].time || this._formatDateTime(new Date(samples[samples.length - 1].ts)),
+                Number(samples[samples.length - 1].ts),
             );
         }
 
@@ -424,7 +422,7 @@ const chemistryTdsHelper = {
             initialSet = true;
 
             await this._setNumber('chemistry.tds.reference.initial_value', initialValue);
-            await this._setString('chemistry.tds.reference.initial_value_at', this._formatDateTime(now));
+            await this._setNumber('chemistry.tds.reference.initial_value_at', now.getTime()); // FIX: value.time uses numeric ms timestamp.
             await this._setBool('chemistry.tds.reference.initial_set', true);
         }
 
@@ -450,11 +448,11 @@ const chemistryTdsHelper = {
         const now = new Date();
 
         await this._setNumber('chemistry.tds.reference.initial_value', currentValue);
-        await this._setString('chemistry.tds.reference.initial_value_at', this._formatDateTime(now));
+        await this._setNumber('chemistry.tds.reference.initial_value_at', now.getTime()); // FIX: value.time uses numeric ms timestamp.
         await this._setBool('chemistry.tds.reference.initial_set', true);
         await this._setNumber('chemistry.tds.reference.delta_since_initial', 0);
         await this._setString('chemistry.tds.debug.last_reason', 'reference_reset');
-        await this._setString('chemistry.tds.debug.last_update', this._formatDateTime(now));
+        await this._setNumber('chemistry.tds.debug.last_update', now.getTime()); // FIX: value.time uses numeric ms timestamp.
 
         this._scheduleEvaluation('reference_reset', 500);
     },
@@ -560,15 +558,15 @@ const chemistryTdsHelper = {
 
     async _writeTrend(trend) {
         await this._setNumber('chemistry.tds.trend.reference_24h_value', trend.ref24h ? trend.ref24h.value : 0);
-        await this._setString('chemistry.tds.trend.reference_24h_at', trend.ref24h ? trend.ref24h.time : '');
+        await this._setNumber('chemistry.tds.trend.reference_24h_at', trend.ref24h ? trend.ref24h.ts : 0); // FIX
         await this._setNumber('chemistry.tds.trend.delta_24h', trend.ref24h ? trend.delta24h : 0);
 
         await this._setNumber('chemistry.tds.trend.reference_7d_value', trend.ref7d ? trend.ref7d.value : 0);
-        await this._setString('chemistry.tds.trend.reference_7d_at', trend.ref7d ? trend.ref7d.time : '');
+        await this._setNumber('chemistry.tds.trend.reference_7d_at', trend.ref7d ? trend.ref7d.ts : 0); // FIX
         await this._setNumber('chemistry.tds.trend.delta_7d', trend.ref7d ? trend.delta7d : 0);
 
         await this._setNumber('chemistry.tds.trend.reference_30d_value', trend.ref30d ? trend.ref30d.value : 0);
-        await this._setString('chemistry.tds.trend.reference_30d_at', trend.ref30d ? trend.ref30d.time : '');
+        await this._setNumber('chemistry.tds.trend.reference_30d_at', trend.ref30d ? trend.ref30d.ts : 0); // FIX
         await this._setNumber('chemistry.tds.trend.delta_30d', trend.ref30d ? trend.delta30d : 0);
 
         await this._setString('chemistry.tds.trend.direction', trend.direction);
@@ -718,7 +716,8 @@ const chemistryTdsHelper = {
         await this._setString('chemistry.tds.evaluation.recommendation', I18n.translate('TDS evaluation is disabled.'));
         await this._setBool('chemistry.tds.evaluation.action_required', false);
         await this._setString('chemistry.tds.debug.last_reason', reason);
-        await this._setString('chemistry.tds.debug.last_update', this._formatDateTime(new Date()));
+        const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+        await this._setNumber('chemistry.tds.debug.last_update', now.getTime()); // FIX
     },
 
     async _writeInvalid(recommendation, reason) {
@@ -730,7 +729,8 @@ const chemistryTdsHelper = {
         await this._setString('chemistry.tds.evaluation.recommendation', recommendation);
         await this._setBool('chemistry.tds.evaluation.action_required', false);
         await this._setString('chemistry.tds.debug.last_reason', reason);
-        await this._setString('chemistry.tds.debug.last_update', this._formatDateTime(new Date()));
+        const now = new Date(); // FIX: write value.time as numeric Unix timestamp in milliseconds.
+        await this._setNumber('chemistry.tds.debug.last_update', now.getTime()); // FIX
     },
 
     async _readString(id) {
@@ -750,6 +750,19 @@ const chemistryTdsHelper = {
         return Number.isFinite(value) ? value : null;
     },
 
+    async _readTimestampOrNull(id) {
+        const state = await this.adapter.getStateAsync(id);
+        const value = state?.val;
+        const numberValue = Number(value);
+
+        if (Number.isFinite(numberValue) && numberValue > 0) {
+            return numberValue; // FIX: numeric value.time timestamps are used directly.
+        }
+
+        const legacyDate = this._parseGermanDateTime(value); // FIX: keep backward compatibility for old string values.
+        return legacyDate ? legacyDate.getTime() : null;
+    },
+
     async _readBoolean(id) {
         const state = await this.adapter.getStateAsync(id);
         return !!state?.val;