Merge pull request #40 from camrun91/staging

Staging

Author: gharezlak

.eslintignore

@@ -0,0 +1,10 @@
+node_modules/
+dist/
+coverage/
+build/
+*.min.js
+**/*.min.js
+templates/
+styles/
+
+

.eslintrc.cjs

@@ -0,0 +1,62 @@
+/** @type {import('eslint').Linter.Config} */
+module.exports = {
+  root: true,
+  env: {
+    browser: true,
+    es2022: true,
+    node: true
+  },
+  parserOptions: {
+    ecmaVersion: "latest",
+    sourceType: "module"
+  },
+  extends: [
+    "eslint:recommended",
+    "plugin:prettier/recommended"
+  ],
+  plugins: [
+    "prettier"
+  ],
+  // Foundry VTT globals (v13) used throughout the module
+  globals: {
+    foundry: "readonly",
+    game: "readonly",
+    ui: "readonly",
+    CONFIG: "readonly",
+    Hooks: "readonly",
+    canvas: "readonly",
+    Actor: "readonly",
+    Item: "readonly",
+    Scene: "readonly",
+    fromUuid: "readonly",
+    CONST: "readonly",
+    // Additional Foundry/core globals referenced in legacy areas
+    JournalEntry: "readonly",
+    Folder: "readonly",
+    DocumentSheetConfig: "readonly",
+    jQuery: "readonly"
+  },
+  rules: {
+    "prettier/prettier": "warn",
+    "no-unused-vars": ["warn", { argsIgnorePattern: "^_", varsIgnorePattern: "^_" }],
+    // Allow empty catch blocks (common in Foundry-safe guards) but warn elsewhere
+    "no-empty": ["warn", { "allowEmptyCatch": true }],
+    // Many code paths intentionally use constant conditions for guards; keep as warn
+    "no-constant-condition": ["warn", { "checkLoops": false }],
+    // Strings built for regexes sometimes intentionally escape chars; treat as warn
+    "no-useless-escape": "warn"
+  },
+  env: {
+    browser: true,
+    es2022: true,
+    node: true,
+    jquery: true
+  },
+  ignorePatterns: [
+    "node_modules/",
+    "dist/",
+    "coverage/"
+  ]
+};
+
+

CHANGELOG.md

@@ -5,6 +5,20 @@ All notable changes to the Archivist Sync module will be documented in this file
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+
+### Fixed
+- Restored compatibility with PopOut! by cloning sidebar base options before overriding defaults so Combat Tracker and other stock tabs keep their pop-out buttons when realtime sync is enabled.
+
+### Changed
+- Recap custom sheet now displays the session date as MM/DD/YYYY.
+- After importing Recaps via the Sync dialog, Recaps in the Recaps folder are normalized to sort by the `sessionDate` flag ascending (oldest → newest); undated Recaps are placed at the end.
+
+## [1.3.9] - 2025-12-06
+
+### Fixed
+- Recap “lock” (save) action could wipe the summary in Foundry and on the Archivist API if the editor selector failed to resolve; editor detection has been made robust for the Recap layout, local page updates only occur when content is actually read, and the API summary field is only included when content is present.
+
 ## [1.3.7] - 2025-12-05
 
 ### Fixed

module.json

@@ -8,7 +8,7 @@
       "email": "cameron.b.llewellyn@gmail.com"
     }
   ],
-  "version": "1.3.7",
+  "version": "1.3.9",
   "compatibility": {
     "minimum": "13.341",
     "verified": "13.346"

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "archivist-sync",
-  "version": "1.3.7",
+  "version": "1.3.9",
   "description": "A simple Foundry VTT module for fetching world data from an API endpoint using an API key.",
   "type": "module",
   "scripts": {
@@ -29,4 +29,4 @@
     "eslint-plugin-prettier": "^5.1.3",
     "prettier": "^3.2.5"
   }
-}
+}

package.json

@@ -1196,14 +1196,30 @@ function installRealtimeSyncListeners() {
       const html = Utils.extractPageHtml(page);
       const payload = { description: Utils.toMarkdownIfHtml?.(html) || html };
 
-      if (sheetType === 'pc' || sheetType === 'npc' || sheetType === 'character') {
-        res = await archivistApi.updateCharacter(apiKey, flags.archivistId, payload);
+      if (
+        sheetType === 'pc' ||
+        sheetType === 'npc' ||
+        sheetType === 'character'
+      ) {
+        res = await archivistApi.updateCharacter(
+          apiKey,
+          flags.archivistId,
+          payload
+        );
       } else if (sheetType === 'item') {
         res = await archivistApi.updateItem(apiKey, flags.archivistId, payload);
       } else if (sheetType === 'location') {
-        res = await archivistApi.updateLocation(apiKey, flags.archivistId, payload);
+        res = await archivistApi.updateLocation(
+          apiKey,
+          flags.archivistId,
+          payload
+        );
       } else if (sheetType === 'faction') {
-        res = await archivistApi.updateFaction(apiKey, flags.archivistId, payload);
+        res = await archivistApi.updateFaction(
+          apiKey,
+          flags.archivistId,
+          payload
+        );
       } else if (sheetType === 'recap') {
         await archivistApi.updateSession(apiKey, flags.archivistId, {
           title: parent?.name || page.name,