fix(matrix): ignore invalid device timestamps

steipete · steipete · commit 6804b7cb716b · 2026-05-30T05:59:10.000-04:00
diff --git a/extensions/matrix/src/cli.test.ts b/extensions/matrix/src/cli.test.ts
@@ -950,6 +950,28 @@ describe("matrix CLI verification commands", () => {
     expect(console.log).toHaveBeenCalledWith("- BritdXC6iL (OpenClaw Gateway)");
   });
 
+  it("omits invalid matrix device last seen timestamps", async () => {
+    listMatrixOwnDevicesMock.mockResolvedValue([
+      {
+        deviceId: "DEVICE123",
+        displayName: "OpenClaw Gateway",
+        lastSeenIp: "127.0.0.1",
+        lastSeenTs: 8_700_000_000_000_000,
+        current: true,
+      },
+    ]);
+    const program = buildProgram();
+
+    await program.parseAsync(["matrix", "devices", "list", "--account", "poe"], { from: "user" });
+
+    expect(console.log).toHaveBeenCalledWith("Account: poe");
+    expect(console.log).toHaveBeenCalledWith("- DEVICE123 (current, OpenClaw Gateway)");
+    expect(console.log).toHaveBeenCalledWith("  Last IP: 127.0.0.1");
+    expect(
+      consoleLogMock.mock.calls.some(([message]) => String(message).startsWith("  Last seen:")),
+    ).toBe(false);
+  });
+
   it("prunes stale matrix gateway devices", async () => {
     pruneMatrixStaleGatewayDevicesMock.mockResolvedValue({
       before: [
diff --git a/extensions/matrix/src/cli.ts b/extensions/matrix/src/cli.ts
@@ -1,6 +1,6 @@
 import type { Command } from "commander";
 import { normalizeAccountId } from "openclaw/plugin-sdk/account-id";
-import { parseStrictInteger } from "openclaw/plugin-sdk/number-runtime";
+import { parseStrictInteger, timestampMsToIsoString } from "openclaw/plugin-sdk/number-runtime";
 import type { ChannelSetupInput } from "openclaw/plugin-sdk/setup";
 import { resolveMatrixAccount, resolveMatrixAccountConfig } from "./matrix/accounts.js";
 import { listMatrixOwnDevices, pruneMatrixStaleGatewayDevices } from "./matrix/actions/devices.js";
@@ -216,8 +216,9 @@ function printMatrixOwnDevices(
     console.log(
       `- ${formatMatrixCliText(device.deviceId)}${labels.length ? ` (${labels.join(", ")})` : ""}`,
     );
-    if (device.lastSeenTs) {
-      printTimestamp("  Last seen", new Date(device.lastSeenTs).toISOString());
+    const lastSeenAt = timestampMsToIsoString(device.lastSeenTs);
+    if (lastSeenAt) {
+      printTimestamp("  Last seen", lastSeenAt);
     }
     if (device.lastSeenIp) {
       console.log(`  Last IP: ${formatMatrixCliText(device.lastSeenIp)}`);
