@helyx/module-account-gate 2.1.1 → 2.1.3

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## 2.1.3
+
+### Patch Changes
+
+- Respect status filters and bounded cursor paging for Access Control gate panels shown in Server Overview. Reuse canonical managed-resource permissions and detail views.
+- Exclude bot and unconfirmed account types from member tables, searches and direct member details while preserving retained summaries.
+
+## 2.1.2
+
+### Patch Changes
+
+- Updated dependencies
+  - @helyx/sdk@0.17.0
+  - @helyx/module-manifest@0.14.0
+
 ## 2.1.1
 
 ### Patch Changes

package/dist/contracts.d.ts

@@ -2,7 +2,7 @@ export declare const ACCOUNT_GATE_IDENTITY: {
     readonly packageName: "@helyx/module-account-gate";
     readonly moduleId: "helyx.account-gate";
     readonly name: "Access Control";
-    readonly version: "2.1.1";
+    readonly version: "2.1.3";
     readonly releaseStatus: "live";
     readonly category: "safety";
     readonly icon: "shield-check";

package/dist/contracts.js

@@ -2,7 +2,7 @@ export const ACCOUNT_GATE_IDENTITY = {
     packageName: "@helyx/module-account-gate",
     moduleId: "helyx.account-gate",
     name: "Access Control",
-    version: "2.1.1",
+    version: "2.1.3",
     releaseStatus: "live",
     category: "safety",
     icon: "shield-check",

package/dist/member-resource.js

@@ -31,7 +31,9 @@ export const accountGateMemberResource = {
         });
         const identities = await resolveLiveIdentities(context, page.records.map((row) => stringValue(row.member_user_id)));
         return {
-            items: page.records.map((row) => memberSummary(row, identities.get(stringValue(row.member_user_id)))),
+            items: page.records
+                .filter((row) => identities.has(stringValue(row.member_user_id)))
+                .map((row) => memberSummary(row, identities.get(stringValue(row.member_user_id)))),
             ...(page.nextCursor
                 ? {
                     nextCursor: encodeRecordCursor(context.guildId, stringValue(page.nextCursor[1])),
@@ -45,6 +47,8 @@ export const accountGateMemberResource = {
         if (!row)
             return null;
         const identities = await resolveLiveIdentities(context, [resourceId]);
+        if (!identities.has(resourceId))
+            return null;
         return memberDetail(row, identities.get(resourceId));
     },
     preview() {
@@ -104,6 +108,8 @@ async function searchKnownMembers(context, search, cursor) {
         if (!row)
             return { items: [] };
         const identities = await resolveLiveIdentities(context, [search]);
+        if (!identities.has(search))
+            return { items: [] };
         return { items: [memberSummary(row, identities.get(search))] };
     }
     const offset = cursor ? decodeSearchCursor(cursor, search) : 0;
@@ -113,7 +119,7 @@ async function searchKnownMembers(context, search, cursor) {
         limit: SEARCH_RESULT_LIMIT,
     });
     const known = (await Promise.all(matches
-        .filter(({ isBot }) => !isBot)
+        .filter(({ isBot }) => isBot === false)
         .map(async (identity) => ({
         identity: {
             userId: identity.userId,
@@ -138,7 +144,7 @@ async function resolveLiveIdentities(context, userIds) {
         guildId: context.guildId,
         userIds,
     });
-    return new Map(resolved.flatMap((member) => member.displayName === null
+    return new Map(resolved.flatMap((member) => member.isBot !== false
         ? []
         : [
             [
@@ -165,8 +171,8 @@ function memberSummary(row, identity) {
         attributes: {
             memberUserId,
             displayName: identity?.displayName ?? null,
-            avatarUrl: identity?.avatarUrl ?? null,
-            memberPresent: identity !== undefined,
+            avatarUrl: identity?.avatarUrl || null,
+            memberPresent: identity?.displayName != null,
             completedAt: completedAt?.toISOString() ?? null,
         },
         updatedAt: updatedAt.toISOString(),
@@ -178,7 +184,7 @@ function memberDetail(row, identity) {
         ...summary,
         value: {
             memberUserId: summary.id,
-            memberPresent: identity !== undefined,
+            memberPresent: identity?.displayName != null,
         },
     };
 }

package/dist/member-summary.d.ts

@@ -33,7 +33,7 @@ export declare function summaryValues(input: {
     pending_reason: "none" | "account_too_new";
     last_failure_terminal_id: string;
     left_at: Date;
-    last_decision: "passed" | "failed" | "review" | "kicked" | "pending" | "finalizing" | "kick_pending" | "dismissed" | "revoked";
+    last_decision: "pending" | "passed" | "review" | "failed" | "revoked" | "finalizing" | "kicked" | "kick_pending" | "dismissed";
     revision: number;
     retention_until: Date;
     created_at: Date;

package/dist/panel-resource.js

@@ -13,12 +13,19 @@ const MAX_GATE_PANELS = 10;
 export const gatePanelResource = {
     id: "gate-panel",
     availableWhileDisabled: true,
-    async list(context) {
+    async list(context, input) {
+        const offset = Number(input.cursor ?? "0");
+        if (!Number.isSafeInteger(offset) || offset < 0 || offset > MAX_GATE_PANELS)
+            throw new DashboardActionValidationError("Invalid panel cursor. Reload the list.");
+        const limit = Math.min(MAX_GATE_PANELS, Math.max(1, input.limit));
         const page = await records(context).findMany({
             moduleId: ACCOUNT_GATE_IDENTITY.moduleId,
             collection: "gate_panels",
             select: accountGateRecords.gate_panels.readableFields,
-            where: { guild_id: context.guildId },
+            where: {
+                guild_id: context.guildId,
+                ...(input.filters?.status ? { status: input.filters.status } : {}),
+            },
             orderBy: [
                 { field: "guild_id", direction: "asc" },
                 { field: "created_at", direction: "asc" },
@@ -26,8 +33,15 @@ export const gatePanelResource = {
             ],
             limit: MAX_GATE_PANELS,
         });
-        const details = await Promise.all(page.records.map((row) => inspectedDetail(context, row)));
-        return { items: details.map(managedResourceSummary) };
+        const details = await Promise.all(page.records
+            .slice(offset, offset + limit)
+            .map((row) => inspectedDetail(context, row)));
+        return {
+            items: details.map(managedResourceSummary),
+            ...(offset + details.length < page.records.length
+                ? { nextCursor: String(offset + details.length) }
+                : {}),
+        };
     },
     async read(context, resourceId) {
         const row = await records(context).findOne({

package/manifest.json

@@ -11,7 +11,7 @@
   "healthExposure": "direct",
   "id": "helyx.account-gate",
   "name": "Access Control",
-  "version": "2.1.1",
+  "version": "2.1.3",
   "description": "Require new members to complete configurable Discord interaction checks before gaining access.",
   "entry": "./dist/index.js",
   "framework": "^0.2.0",
@@ -618,7 +618,26 @@
               "equals": "password"
             }
           }
-        ]
+        ],
+        "query": {
+          "filters": [
+            {
+              "key": "status",
+              "label": "State",
+              "type": "select",
+              "options": [
+                {
+                  "value": "posted",
+                  "label": "Posted"
+                },
+                {
+                  "value": "missing",
+                  "label": "Missing"
+                }
+              ]
+            }
+          ]
+        }
       },
       {
         "id": "members",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@helyx/module-account-gate",
-  "version": "2.1.1",
+  "version": "2.1.3",
   "description": "Require configurable Discord interaction checks before new members gain access.",
   "keywords": [
     "helyx",
@@ -50,7 +50,7 @@
     "test": "vitest run --root ../.. modules/account-gate/tests"
   },
   "dependencies": {
-    "@helyx/module-manifest": ">=0.13.0 <0.14.0",
-    "@helyx/sdk": ">=0.16.0 <0.17.0"
+    "@helyx/module-manifest": ">=0.14.0 <0.15.0",
+    "@helyx/sdk": ">=0.17.1 <0.18.0"
   }
 }