Restore legacy lastUpdatedAt-only behavior; mkdir state_dir early

Author: clean6378-max-it

scripts/export.py

@@ -179,6 +179,7 @@ def main() -> None:
     workspace_path = resolve_workspace_path(override=opts.get("base_dir"))
 
     state_dir = get_global_state_dir()
+    os.makedirs(state_dir, exist_ok=True)
     state_path = os.path.join(state_dir, "export_state.json")
     last_export = read_last_export_ms(since, state_path=state_path)
 
@@ -253,7 +254,6 @@ def main() -> None:
         "exportedCount": count,
         "exportDir": out_dir,
     }
-    os.makedirs(state_dir, exist_ok=True)
     with open(os.path.join(state_dir, "export_state.json"), "w", encoding="utf-8") as f:
         json.dump(state, f, indent=2)
 

services/export_engine.py

@@ -253,11 +253,7 @@ def _collect_ide_export_entries(
         if not isinstance(headers, list) or not headers:
             continue
 
-        updated_at = to_epoch_ms(cd.get("lastUpdatedAt")) or to_epoch_ms(
-            cd.get("createdAt"),
-        )
-        # Intentional behavior change vs legacy CLI: fall back to createdAt when
-        # lastUpdatedAt is absent (affects timestamps, filenames, and --since last).
+        updated_at = to_epoch_ms(cd.get("lastUpdatedAt"))
         if since == "last" and updated_at <= last_export_ms:
             continue
 

tests/test_export_engine.py

@@ -182,22 +182,26 @@ def _collect(
                     out_dir=self.tmp_out,
                 )
 
-    def test_created_at_fallback_when_last_updated_missing(self):
+    def test_last_updated_at_only_no_created_at_fallback(self):
         created_ms = 1739200000000
-        exported = self._collect({
-            "name": "Created-only chat",
-            "modelConfig": {},
-            "fullConversationHeadersOnly": [{"bubbleId": "bubble-1", "type": 1}],
-            "createdAt": created_ms,
-        })
+        fixed_now = datetime(2026, 6, 22, 12, 0, 0)
+        with patch("services.export_engine.datetime") as mock_dt:
+            mock_dt.now.return_value = fixed_now
+            mock_dt.fromtimestamp = datetime.fromtimestamp
+            exported = self._collect({
+                "name": "Created-only chat",
+                "modelConfig": {},
+                "fullConversationHeadersOnly": [{"bubbleId": "bubble-1", "type": 1}],
+                "createdAt": created_ms,
+            })
         self.assertEqual(len(exported), 1)
         entry = exported[0]
-        self.assertEqual(entry["updatedAt"], created_ms)
-        ts_str = datetime.fromtimestamp(created_ms / 1000).strftime("%Y-%m-%dT%H-%M-%S")
+        self.assertEqual(entry["updatedAt"], 0)
+        ts_str = fixed_now.strftime("%Y-%m-%dT%H-%M-%S")
         self.assertIn(ts_str, entry["rel_path"])
-        self.assertEqual(
+        self.assertNotIn(
+            datetime.fromtimestamp(created_ms / 1000).strftime("%Y-%m-%dT%H-%M-%S"),
             entry["rel_path"],
-            os.path.relpath(entry["out_path"], self.tmp_out),
         )
 
     def test_display_name_falls_back_to_slug_of_workspace_id_prefix(self):