@helyx/module-reactionroles 1.0.9 → 1.0.11

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # @helyx/module-reactionroles
 
+## 1.0.11
+
+### Patch Changes
+
+- Report missing managed panel messages correctly during Server Health inspections.
+
+## 1.0.10
+
+### Patch Changes
+
+- Harden dashboard authentication, privacy execution, control routing, database transactions, and self-host configuration.
+
+  Add bounded multi-value record reads and use them to batch Polls and Reaction Roles dashboard queries. Improve dashboard presentation, Forms workflows, release safety, and operational tooling across hosted and self-hosted deployments.
+
+- Updated dependencies
+  - @helyx/sdk@0.7.0
+
 ## 1.0.9
 
 ### Patch Changes

package/dist/index.js

@@ -400,7 +400,7 @@ export const reactionRolesModule = defineModule({
             healthMessage = cache.saturated()
                 ? `Reaction Roles loaded the bounded maximum of ${count} active panels`
                 : cleanup.failed > 0 || cleanup.abandoned > 0
-                    ? `Reaction Roles loaded ${count} active panels; ${cleanup.failed + cleanup.abandoned} message cleanups need attention`
+                    ? `Reaction Roles could not clean up ${cleanup.failed + cleanup.abandoned} old panel message${cleanup.failed + cleanup.abandoned === 1 ? "" : "s"}.`
                     : `Reaction Roles loaded ${count} active panels`;
             context.logger.info({
                 event: "reactionroles.started",

package/dist/records.js

@@ -240,7 +240,7 @@ export class ReactionRolePanelRepository {
             limit: input.limit,
         });
         return {
-            panels: await Promise.all(page.records.map((panel) => this.#withEntries(panel))),
+            panels: await this.#withEntriesForPanels(page.records),
             nextCursor: page.nextCursor,
         };
     }
@@ -287,7 +287,7 @@ export class ReactionRolePanelRepository {
             ],
             limit: Math.min(100, Math.max(1, limit)),
         });
-        return Promise.all(page.records.map((panel) => this.#withEntries(panel)));
+        return this.#withEntriesForPanels(page.records);
     }
     async listByRole(guildId, roleId, limit = 100) {
         return this.#listByEntryField(guildId, "role_id", roleId, limit);
@@ -309,7 +309,7 @@ export class ReactionRolePanelRepository {
             limit: input.limit,
         });
         return {
-            panels: await Promise.all(page.records.map((panel) => this.#withEntries(panel))),
+            panels: await this.#withEntriesForPanels(page.records),
             nextCursor: page.nextCursor,
         };
     }
@@ -444,6 +444,32 @@ export class ReactionRolePanelRepository {
         });
         return mapPanel(panel, entries.records);
     }
+    async #withEntriesForPanels(panels) {
+        if (panels.length === 0)
+            return [];
+        const panelIds = panels.map((panel) => requiredString(panel, "panel_id"));
+        const entries = await this.records.findManyByValues({
+            moduleId: MODULE_ID,
+            collection: "entries",
+            select: entrySelect,
+            where: {},
+            whereAny: { field: "panel_id", values: panelIds },
+            orderBy: [
+                { field: "position", direction: "asc" },
+                { field: "entry_id", direction: "asc" },
+            ],
+        });
+        const entriesByPanel = new Map();
+        for (const entry of entries) {
+            const panelId = requiredString(entry, "panel_id");
+            const grouped = entriesByPanel.get(panelId);
+            if (grouped)
+                grouped.push(entry);
+            else
+                entriesByPanel.set(panelId, [entry]);
+        }
+        return panels.map((panel, index) => mapPanel(panel, entriesByPanel.get(panelIds[index]) ?? []));
+    }
     async #listByEntryField(guildId, field, value, limit) {
         const entries = await this.records.findMany({
             moduleId: MODULE_ID,
@@ -456,8 +482,27 @@ export class ReactionRolePanelRepository {
             ],
             limit: Math.min(100, Math.max(1, limit)),
         });
-        const panels = await Promise.all(entries.records.map((entry) => this.findById(guildId, requiredString(entry, "panel_id"))));
-        return panels.filter((panel) => panel !== null);
+        const panelIds = entries.records.map((entry) => requiredString(entry, "panel_id"));
+        const uniquePanelIds = [...new Set(panelIds)];
+        if (uniquePanelIds.length === 0)
+            return [];
+        const panels = await this.records.findManyByValues({
+            moduleId: MODULE_ID,
+            collection: "panels",
+            select: panelSelect,
+            where: { guild_id: guildId },
+            whereAny: { field: "panel_id", values: uniquePanelIds },
+            orderBy: [
+                { field: "updated_at", direction: "desc" },
+                { field: "panel_id", direction: "asc" },
+            ],
+        });
+        const hydrated = await this.#withEntriesForPanels(panels);
+        const byId = new Map(hydrated.map((panel) => [panel.panelId, panel]));
+        return uniquePanelIds.flatMap((panelId) => {
+            const panel = byId.get(panelId);
+            return panel ? [panel] : [];
+        });
     }
 }
 function panelValues(input) {

package/manifest.json

@@ -3,7 +3,7 @@
   "releaseStatus": "live",
   "id": "helyx.reactionroles",
   "name": "Reaction Roles",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "description": "Let members select approved roles themselves.",
   "entry": "./dist/index.js",
   "framework": "^0.1.0",

package/migrations/0004_entry_lookup_indexes.sql

@@ -0,0 +1,6 @@
+CREATE INDEX IF NOT EXISTS reactionroles_entries_role_panel_idx
+  ON reactionroles_entries (role_id, panel_id);
+
+CREATE INDEX IF NOT EXISTS reactionroles_entries_emoji_panel_idx
+  ON reactionroles_entries (emoji_id, panel_id)
+  WHERE emoji_id IS NOT NULL;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@helyx/module-reactionroles",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "description": "Configurable self-service Discord role panels for Helyx.",
   "keywords": [
     "helyx",
@@ -50,6 +50,6 @@
   },
   "dependencies": {
     "@helyx/module-manifest": ">=0.6.0 <0.7.0",
-    "@helyx/sdk": ">=0.6.0 <0.7.0"
+    "@helyx/sdk": ">=0.7.0 <0.8.0"
   }
 }