@tenonhq/sincronia-core 0.0.82 → 0.0.83

package/dist/allScopesCommands.js

@@ -90,7 +90,7 @@ async function processManifestForScope(manifest, sourceDirectory, forceWrite = f
                 const recordDirName = record.name || recordName;
                 const recordPath = path.join(tablePath, recordDirName);
                 // Check if metadata file exists in the files from server
-                const hasMetadataFromServer = record.files?.some((f) => f.name === 'metaData' && f.type === 'json');
+                const hasMetadataFromServer = record.files && record.files.some((f) => f.name === 'metaData' && f.type === 'json');
                 // Ensure the record directory exists
                 await fsp.mkdir(recordPath, { recursive: true });
                 // Process each file in the record
@@ -243,7 +243,7 @@ async function processScope(scopeName, scopeConfig, apiDelay = 0) {
                 }
             }
         }
-        const tableCount = Object.keys(manifest?.tables || {}).length;
+        const tableCount = Object.keys((manifest && manifest.tables) || {}).length;
         Logger_1.logger.info("Writing " + tableCount + " tables for " + scopeName + "...");
         await processManifestForScope(manifest, sourceDirectory, true);
         // Create the scope-specific manifest structure
@@ -325,8 +325,8 @@ async function initScopesCommand(args) {
             }
             else {
                 failCount++;
-                const error = result.status === "rejected" ? result.reason : result.value?.error;
-                Logger_1.logger.error(`Failed to process ${scopeName}: ${error?.message || "Unknown error"}`);
+                const error = result.status === "rejected" ? result.reason : (result.value && result.value.error);
+                Logger_1.logger.error(`Failed to process ${scopeName}: ${(error && error.message) || "Unknown error"}`);
             }
         });
         // Write per-scope manifest files instead of a single combined one

package/dist/appUtils.js

@@ -61,19 +61,46 @@ const getUpdateSetConfig = () => {
     }
     return {};
 };
+// Merge _lastUpdatedOn into the server-provided metadata content. Preserves all
+// record fields (sys_id, sys_scope, field value/display_value pairs, etc.) so
+// the local metaData.json is a full snapshot of the record, not just a stub.
+const stampMetadataContent = (file) => {
+    if (file.name !== "metaData" || file.type !== "json")
+        return file;
+    const stamp = new Date().toISOString();
+    if (!file.content) {
+        return { ...file, content: JSON.stringify({ _lastUpdatedOn: stamp }, null, 2) };
+    }
+    try {
+        const metadata = JSON.parse(file.content);
+        if (metadata.sys_updated_on && metadata.sys_updated_on.value) {
+            metadata._lastUpdatedOn = metadata.sys_updated_on.value;
+        }
+        else {
+            metadata._lastUpdatedOn = stamp;
+        }
+        return { ...file, content: JSON.stringify(metadata, null, 2) };
+    }
+    catch (e) {
+        // Content isn't JSON — leave as-is, it will be written verbatim.
+        return file;
+    }
+};
+const hasServerMetadata = (files) => files.some((f) => f.name === "metaData" && f.type === "json" && !!f.content);
 const processFilesInManRec = async (recPath, rec, forceWrite) => {
     FileLogger_1.fileLogger.debug("Processing record: " + rec.name + " (" + rec.files.length + " files)");
     const fileWrite = fUtils.writeSNFileCurry(forceWrite);
-    // Create metadata file with current timestamp
-    const metadataFile = {
-        name: "metaData",
-        type: "json",
-        content: JSON.stringify({
-            _lastUpdatedOn: new Date().toISOString()
-        }, null, 2)
-    };
-    await fileWrite(metadataFile, recPath);
-    const writeResults = await (0, genericUtils_1.allSettledBatched)(rec.files, constants_1.CONCURRENCY_FILES, function (file) { return fileWrite(file, recPath); });
+    // If the server did not provide a metadata file, fall back to a timestamp-only
+    // stub so the record directory always has a metaData.json.
+    if (!hasServerMetadata(rec.files)) {
+        const stubMetadata = {
+            name: "metaData",
+            type: "json",
+            content: JSON.stringify({ _lastUpdatedOn: new Date().toISOString() }, null, 2),
+        };
+        await fileWrite(stubMetadata, recPath);
+    }
+    const writeResults = await (0, genericUtils_1.allSettledBatched)(rec.files, constants_1.CONCURRENCY_FILES, function (file) { return fileWrite(stampMetadataContent(file), recPath); });
     const writeFailures = writeResults.filter((r) => r.status === "rejected");
     if (writeFailures.length > 0) {
         writeFailures.forEach((f) => {
@@ -461,7 +488,22 @@ const refreshAllFiles = async (newManifest, sourcePath, options = {}) => {
             await (0, genericUtils_1.processBatched)(recKeys, constants_1.CONCURRENCY_RECORDS, async function (recKey) {
                 var rec = recs[recKey];
                 var recPath = path_1.default.join(tablePath, rec.name);
-                var results = await (0, genericUtils_1.allSettledBatched)(rec.files, constants_1.CONCURRENCY_FILES, async function (file) {
+                // Split server-provided metadata off from the regular files so we can
+                // track whether any regular file actually changed — metaData shouldn't
+                // be the trigger for "this record changed" since we stamp it on every
+                // touch.
+                var metadataFiles = [];
+                var regularFiles = [];
+                for (var mi = 0; mi < rec.files.length; mi++) {
+                    var rf = rec.files[mi];
+                    if (rf.name === "metaData" && rf.type === "json") {
+                        metadataFiles.push(rf);
+                    }
+                    else {
+                        regularFiles.push(rf);
+                    }
+                }
+                var results = await (0, genericUtils_1.allSettledBatched)(regularFiles, constants_1.CONCURRENCY_FILES, async function (file) {
                     if (forceWrite) {
                         await forceWriter(file, recPath);
                         return true;
@@ -483,15 +525,23 @@ const refreshAllFiles = async (newManifest, sourcePath, options = {}) => {
                         unchangedCount++;
                     }
                 }
-                // Only touch metaData when at least one file in the record actually
-                // changed. Avoids rewriting _lastUpdatedOn for records that were
-                // already in sync with the instance.
+                // Only touch metaData when at least one regular file in the record
+                // actually changed. Avoids rewriting _lastUpdatedOn for records that
+                // were already in sync with the instance. Prefer the server-provided
+                // metadata (full field snapshot) over a stub; fall back to a stub only
+                // when the server didn't send metadata at all.
                 if (anyChanged || forceWrite) {
-                    const metadataFile = {
-                        name: "metaData",
-                        type: "json",
-                        content: JSON.stringify({ _lastUpdatedOn: new Date().toISOString() }, null, 2),
-                    };
+                    let metadataFile;
+                    if (metadataFiles.length > 0 && metadataFiles[0].content) {
+                        metadataFile = stampMetadataContent(metadataFiles[0]);
+                    }
+                    else {
+                        metadataFile = {
+                            name: "metaData",
+                            type: "json",
+                            content: JSON.stringify({ _lastUpdatedOn: new Date().toISOString() }, null, 2),
+                        };
+                    }
                     await forceWriter(metadataFile, recPath);
                 }
                 // Strip content from manifest entries to keep memory bounded.

package/dist/updateSetCommands.js

@@ -283,7 +283,7 @@ async function switchUpdateSetCommand(args) {
         // Extract the actual values from display_value objects
         const sysId = typeof targetUpdateSet.sys_id === 'object' && targetUpdateSet.sys_id !== null ? targetUpdateSet.sys_id.value : targetUpdateSet.sys_id;
         const name = typeof targetUpdateSet.name === 'object' && targetUpdateSet.name !== null ? targetUpdateSet.name.value : targetUpdateSet.name;
-        const applicationName = targetUpdateSet.application?.display_value || (typeof targetUpdateSet.application === 'object' && targetUpdateSet.application !== null ? targetUpdateSet.application.value : targetUpdateSet.application);
+        const applicationName = (targetUpdateSet.application && targetUpdateSet.application.display_value) || (typeof targetUpdateSet.application === 'object' && targetUpdateSet.application !== null ? targetUpdateSet.application.value : targetUpdateSet.application);
         const applicationScope = typeof targetUpdateSet.application === 'object' && targetUpdateSet.application !== null ? targetUpdateSet.application.value : targetUpdateSet.application;
         // Switch to the selected update set
         await switchToUpdateSet(sysId, name, args.scope || applicationScope);
@@ -334,7 +334,7 @@ async function listUpdateSetsCommand(args) {
             const name = typeof updateSet.name === 'object' && updateSet.name !== null ? updateSet.name.value : updateSet.name;
             const description = typeof updateSet.description === 'object' && updateSet.description !== null ? updateSet.description.value : updateSet.description;
             const createdBy = typeof updateSet.sys_created_by === 'object' && updateSet.sys_created_by !== null ? updateSet.sys_created_by.display_value || updateSet.sys_created_by.value : updateSet.sys_created_by;
-            const applicationName = updateSet.application?.display_value || (typeof updateSet.application === 'object' && updateSet.application !== null ? updateSet.application.value : updateSet.application);
+            const applicationName = (updateSet.application && updateSet.application.display_value) || (typeof updateSet.application === 'object' && updateSet.application !== null ? updateSet.application.value : updateSet.application);
             const isCurrent = sysId === currentUpdateSetId;
             const marker = isCurrent ? chalk_1.default.green("► ") : "  ";
             const displayName = isCurrent ? chalk_1.default.green(name) : name;
@@ -616,7 +616,7 @@ async function selectUpdateSet(nameFilter, scopeFilter) {
     const choices = filteredSets.map(us => {
         const name = typeof us.name === 'object' && us.name !== null ? us.name.value : us.name;
         const description = typeof us.description === 'object' && us.description !== null ? us.description.value : us.description;
-        const applicationName = us.application?.display_value || (typeof us.application === 'object' && us.application !== null ? us.application.value : us.application);
+        const applicationName = (us.application && us.application.display_value) || (typeof us.application === 'object' && us.application !== null ? us.application.value : us.application);
         return {
             name: `${name}${applicationName ? ` (${applicationName})` : ""} - ${description || "No description"}`,
             value: us

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tenonhq/sincronia-core",
-  "version": "0.0.82",
+  "version": "0.0.83",
   "description": "Next-gen file syncer",
   "license": "GPL-3.0",
   "main": "./dist/index.js",
@@ -38,7 +38,7 @@
     "typescript": "^5.2.2"
   },
   "dependencies": {
-    "@tenonhq/sincronia-clickup": "^0.0.1",
+    "@tenonhq/sincronia-clickup": "^0.0.6",
     "@tenonhq/sincronia-schema": "^0.0.1",
     "axios": "^1.5.1",
     "axios-cookiejar-support": "^4.0.7",