@bobfrankston/rmfmail 1.2.278 → 1.2.280

package/.commitmsg

@@ -1,29 +1,19 @@
-TinyMCE 8.6.0 → 8.8.2
+About: show the bundled editor version
 
-Routine maintenance releases — no security advisories, no breaking changes
-(all within major 8). Four of the fixes land on paths mailx actually uses:
+CORRECTION to 1.2.279. That release added the resolved tinymce / tinymce-i18n
+versions to build-tinymce.js's console output, on the claim that this put the
+answer in the build transcript every publish archives. It does not:
+npmglobalize swallows a sub-build's stdout and reports only "✓ Build
+succeeded", so the line never reaches c:\temp\rebuild\. The echo is still
+correct when the script is run directly, but the build log is not where this
+question gets answered.
 
-- The menubar could wrap onto multiple rows on narrow viewports or at high
-  browser zoom, leaving little or no visible editable area (8.8.1). Same class
-  of failure as the v1.2.264 "body must fill the iframe" bug, and compose is
-  routinely used zoomed.
-- CSS custom property names and colour values in a style attribute were
-  lowercased when parsed (8.7.0) — that is quoted mail passing through a
-  reply/forward.
-- Copying or dragging in an image didn't set width/height on the tag (8.7.0),
-  which the inline-image → cid path then had to carry.
-- The link plugin threw a querySelectorAll syntax error on anchors containing
-  semicolons (8.8.0). Mail bodies are full of odd links.
+About… now carries an "Editor bundle" row — "TinyMCE 8.8.2 · 61 languages" —
+read at display time from client/lib/tinymce/package.json and langs/index.json,
+the same tree the editor loads from, so it cannot drift from what is actually
+running. That answers it on the machine showing the symptom, which is where it
+matters, rather than requiring a checkout of the matching tag.
 
-Also: undo not restoring the selection, deleting a newline in a list item
-eating the item, and the emoticons dialog's 300-item cap (we load that plugin).
-
-Verified before publishing: all 17 plugins we request still exist in 8.8.2, the
-dom model is unchanged, and the package still ships no langs/ directory — the
-61 packs added in 1.2.277 come from tinymce-i18n and are copied after the tree,
-so the re-copy keeps them (62 files present, manifest included).
-
-The spec moves from ^8.5.0 to ^8.8.2. The old floor already permitted 8.8.2;
-the tree had simply drifted at 8.6.0 with nothing to notice it, which is what
-prompted this. Raising the floor makes the intent explicit rather than leaving
-a fresh install free to resolve lower.
+Why this keeps mattering: both specs are caret ranges, so the resolved version
+moves whenever npm re-resolves the tree — tinymce sat at 8.6.0 for weeks under
+a ^8.5.0 spec with nothing to notice.

package/bin/build-tinymce.js

@@ -18,6 +18,21 @@ const dst = path.join(root, "client", "lib", "tinymce");
 const langSrc = path.join(root, "node_modules", "tinymce-i18n", "langs8");
 const langDst = path.join(dst, "langs");
 
+// Which versions went into this build. The spec is a caret range, so the
+// resolved version moves on its own whenever npm re-resolves the tree — it sat
+// at 8.6.0 for weeks under a ^8.5.0 spec with nothing to notice (Bob
+// 2026-08-23: "do you track updates for TinyMCE?"). Pinning would only trade
+// silent drift for silent staleness. Printing it puts the answer in the build
+// transcript every publish archives, at no cost and with no network call.
+// (Claude Code 2026-08-24)
+function pkgVersion(name) {
+    try {
+        return JSON.parse(fs.readFileSync(path.join(root, "node_modules", name, "package.json"), "utf-8")).version;
+    } catch {
+        return "unknown";
+    }
+}
+
 if (!fs.existsSync(src)) {
     console.error(`build-tinymce: source not found at ${src} — run npm install first`);
     process.exit(1);
@@ -53,7 +68,7 @@ if (fs.existsSync(langSrc)) {
     }
     langs.sort();
     fs.writeFileSync(path.join(langDst, "index.json"), JSON.stringify(langs, null, 2) + "\n");
-    console.log(`build-tinymce: copied ${langs.length} language packs → client/lib/tinymce/langs/`);
+    console.log(`build-tinymce: copied ${langs.length} language packs (tinymce-i18n ${pkgVersion("tinymce-i18n")}) → client/lib/tinymce/langs/`);
 } else {
     // Never silently ship an editor whose language menu is empty.
     console.error(`build-tinymce: WARNING no language packs at ${langSrc} — run npm install; the editor will be English-only`);
@@ -66,4 +81,4 @@ function tally(p) {
     else { count++; bytes += st.size; }
 }
 tally(dst);
-console.log(`build-tinymce: copied ${count} files (${(bytes / 1024 / 1024).toFixed(1)} MB) → client/lib/tinymce/`);
+console.log(`build-tinymce: copied tinymce ${pkgVersion("tinymce")} — ${count} files (${(bytes / 1024 / 1024).toFixed(1)} MB) → client/lib/tinymce/`);

package/client/app.bundle.js

@@ -13315,9 +13315,27 @@ async function openAboutDialog() {
     if (e.target === backdrop) close();
   });
   try {
-    const [v, accounts] = await Promise.all([
+    const editorBundle = async () => {
+      const readJson = async (url) => {
+        try {
+          const r = await fetch(url);
+          return r.ok ? await r.json() : null;
+        } catch {
+          return null;
+        }
+      };
+      const [pkg, langs] = await Promise.all([
+        readJson("lib/tinymce/package.json"),
+        readJson("lib/tinymce/langs/index.json")
+      ]);
+      if (!pkg?.version) return "";
+      const n = langs?.length || 0;
+      return n ? `TinyMCE ${pkg.version} \xB7 ${n} languages` : `TinyMCE ${pkg.version}`;
+    };
+    const [v, accounts, editorInfo] = await Promise.all([
       getVersion().catch(() => ({})),
-      getAccounts().catch(() => [])
+      getAccounts().catch(() => []),
+      editorBundle()
     ]);
     const storage = v.storage || {};
     const isApp2 = typeof mailxapi !== "undefined" && mailxapi?.isApp;
@@ -13336,6 +13354,7 @@ async function openAboutDialog() {
     if (storage.folderId) rows.push(["Drive folderId", storage.folderId]);
     if (storage.folderOwner) rows.push(["Drive owner", storage.folderOwner]);
     if (storage.configDir) rows.push(["Config dir", storage.configDir]);
+    if (editorInfo) rows.push(["Editor bundle", editorInfo]);
     rows.push(["Accounts", String((accounts || []).length)]);
     rows.push(["User agent", navigator.userAgent]);
     rows.push(["Screen", `${screen.width}\xD7${screen.height}`]);