discord-html-transcripts-fix 1.7.1 → 1.8.0

package/dist/generator/index.js

@@ -150,7 +150,13 @@ async function render(_a) {
             // Non-blocking inline JSON: type="application/json" so the parser doesn't pause
             (0, jsx_runtime_1.jsx)("script", { id: "dht-data", type: "application/json", dangerouslySetInnerHTML: { __html: (0, utils_1.safeJsonForScript)(globalData) } }),
             (0, jsx_runtime_1.jsx)("script", { dangerouslySetInnerHTML: { __html: 'try{globalThis.$discordMessage=JSON.parse(document.getElementById("dht-data").textContent);}catch(e){globalThis.$discordMessage={};}' } }),
-            !options.hydrate && (0, jsx_runtime_1.jsx)("script", { type: "module", src: `https://cdn.jsdelivr.net/npm/@skyra/discord-components-core@${discordComponentsVersion}/+esm`, crossOrigin: "anonymous" }),
+            // Always load the @skyra/discord-components-core ES module, even
+            // when `hydrate: true` is requested. Lit SSR does not reliably
+            // expand every <discord-*> custom element to plain HTML, and the
+            // browser-side definitions are also needed to hydrate the SSR'd
+            // markup (re-render with state, click handlers, etc.). Without
+            // this tag the page is just static unstyled custom elements.
+            (0, jsx_runtime_1.jsx)("script", { type: "module", src: `https://cdn.jsdelivr.net/npm/@skyra/discord-components-core@${discordComponentsVersion}/+esm`, crossOrigin: "anonymous" }),
         ] }),
         (0, jsx_runtime_1.jsxs)("body", { style: { margin: 0, minHeight: '100vh' }, children: [
             (0, jsx_runtime_1.jsx)(transcript_1.default, Object.assign({ messages, channel, callbacks }, options, renderContext)),

package/dist/generator/transcript.js

@@ -52,21 +52,24 @@ async function DiscordMessages(_a) {
     ] }));
 }
 
-// Legacy "Exported X message(s). Powered by …" line. The new stats footer
-// (see renderStatsFooter in generator/index.js) already shows the message
-// count, so this line is suppressed by default and only renders when:
-//   - options.statsFooter === false (caller opted out of the new footer), or
-//   - options.footerText is explicitly set to a string, or
-//   - options.poweredBy === true (caller wants the credit link).
+// Legacy "Exported X message(s). Powered by …" footer line.
+//
+// The new stats footer (see renderStatsFooter in generator/index.js) already
+// shows the message count and more. To prevent two footers stacking on the
+// same transcript, the legacy footer is now mutually exclusive with the
+// stats footer: it ONLY renders when the caller has explicitly disabled the
+// stats footer via `statsFooter: false` (or `{ enabled: false }`). Setting
+// `footerText` or `poweredBy` alone is no longer enough — that used to
+// produce a duplicate footer because the stats footer kept rendering.
 function renderLegacyFooter(messages, options) {
     const statsDisabled = options.statsFooter === false ||
         (options.statsFooter && options.statsFooter.enabled === false);
+    if (!statsDisabled) return null;
     const customText = (typeof options.footerText === 'string' && options.footerText.length > 0)
         ? options.footerText.replaceAll('{number}', messages.length.toString()).replaceAll('{s}', messages.length > 1 ? 's' : '')
         : null;
     const wantsPowered = !!options.poweredBy;
-    // Default render only if stats footer is off AND no custom text was set.
-    const defaultText = statsDisabled && !customText
+    const defaultText = !customText
         ? `Exported ${messages.length} message${messages.length > 1 ? 's' : ''}.`
         : null;
     const shown = customText || defaultText;

package/dist/utils/buildProfiles.js

@@ -97,9 +97,15 @@ async function buildAllContext(messages, channel) {
             : (displayHex && displayHex !== '#000000' ? displayHex : null);
 
         // legacy profile (used by <discord-mention>)
+        //
+        // forceStatic: Discord CDN returns HTTP 415 for `.gif` on some animated
+        // avatar hashes (commonly bot avatars whose animation was removed but
+        // whose hash still carries the `a_` prefix). The static variant (.webp
+        // or .png) always works, so render avatars as static in transcripts.
+        // Avatars in a historical archive don't gain much from being animated.
         profiles[id] = {
             author: member?.nickname ?? author.displayName ?? author.username,
-            avatar: member?.displayAvatarURL?.({ size: 64 }) ?? author.displayAvatarURL?.({ size: 64 }),
+            avatar: member?.displayAvatarURL?.({ size: 64, forceStatic: true }) ?? author.displayAvatarURL?.({ size: 64, forceStatic: true }),
             roleColor: effectiveColor,
             roleIcon: member?.roles?.icon?.iconURL?.() ?? undefined,
             roleName: member?.roles?.hoist?.name ?? undefined,
@@ -124,7 +130,7 @@ async function buildAllContext(messages, channel) {
             username: author.username,
             displayName: member?.nickname || author.displayName || author.username,
             globalName: author.globalName || null,
-            avatar: member?.displayAvatarURL?.({ size: 128 }) || author.displayAvatarURL?.({ size: 128 }) || null,
+            avatar: member?.displayAvatarURL?.({ size: 128, forceStatic: true }) || author.displayAvatarURL?.({ size: 128, forceStatic: true }) || null,
             bannerColor: effectiveColor,
             bot: !!author.bot,
             system: !!author.system,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "discord-html-transcripts-fix",
-  "version": "1.7.1",
+  "version": "1.8.0",
   "description": "A nicely formatted html transcript generator for discord.js. Bugfix fork with support for the latest discord.js and Components v2.",
   "main": "dist/index.js",
   "types": "./dist/index.d.ts",