@dappermountain/payload-plugin-achievements 0.2.10 → 0.2.12

package/README.md

@@ -97,7 +97,7 @@ Slugs are prefixed with `achievement-` by default so they don’t collide with y
 | `metric.delta` | Metric change log type (requires metric + change amount) |
 | `achievement.granted` | Written when a grant is created |
 | `achievement.revoked` | Written when a grant is deleted |
-| `tier.changed` | Written when derived tier moves |
+| `tier.changed` | Written when derived tier moves (including demotions when a tier request leaves `approved`) |
 
 Event types can require an **actor** and/or a **metric + change**. Admin only prompts for those fields when the type needs them.
 
@@ -110,7 +110,7 @@ Unlock (tiers), eligibility, and completion (achievements) share the same rule t
 - Built-in leaves → `tier-at-least`, `achievement-complete`, `metric-minimum`, `event-count`, `elapsed-since`
 - Custom leaves → register via `extensions.ruleTypes`
 
-**Current tier** is computed by walking tiers in `rank` order and evaluating each tier’s unlock rules (`resolveCurrentTier`). Tiers with **`requiresReview`** only become current after an **approved tier request**. **Next-tier fill** uses fractional progress on the same trees (`resolveTierProgress` / `evaluateRuleProgress`):
+**Current tier** is computed by walking tiers in `rank` order and evaluating each tier’s unlock rules (`resolveCurrentTier`). Tiers with **`requiresReview`** only become current after an **approved tier request**. Turning **`requiresReview` off** approves pending tier requests and writes missing `tier.changed` logs for members who already meet unlock rules (including those who never had a pending request). **Next-tier fill** uses fractional progress on the same trees (`resolveTierProgress` / `evaluateRuleProgress`):
 
 | Combinator / leaf | Progress behavior |
 | --- | --- |

package/dist/collections/TierRequests/hooks/prepareTierRequest.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prepareTierRequest.d.ts","sourceRoot":"","sources":["../../../../src/collections/TierRequests/hooks/prepareTierRequest.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,yBAAyB,EACzB,4BAA4B,EAE7B,MAAM,SAAS,CAAA;AAYhB;;;GAGG;AACH,eAAO,MAAM,kBAAkB,EAAE,4BAyChC,CAAA;AAED,eAAO,MAAM,sBAAsB,EAAE,yBA0BpC,CAAA"}
+{"version":3,"file":"prepareTierRequest.d.ts","sourceRoot":"","sources":["../../../../src/collections/TierRequests/hooks/prepareTierRequest.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,yBAAyB,EACzB,4BAA4B,EAE7B,MAAM,SAAS,CAAA;AAYhB;;;GAGG;AACH,eAAO,MAAM,kBAAkB,EAAE,4BAyChC,CAAA;AAED,eAAO,MAAM,sBAAsB,EAAE,yBAoBpC,CAAA"}

package/dist/collections/TierRequests/hooks/prepareTierRequest.js

@@ -46,17 +46,14 @@ export const prepareTierRequest = async ({ data, operation, req, originalDoc, })
     }
     return data;
 };
-export const afterTierRequestChange = async ({ doc, operation, previousDoc, req, }) => {
-    const status = doc.status;
-    if (status !== 'approved')
-        return doc;
-    const previousStatus = operation === 'update' ? previousDoc?.status : undefined;
-    if (previousStatus === 'approved')
-        return doc;
+export const afterTierRequestChange = async ({ doc, req, }) => {
     const userId = relationId(doc.user);
     if (!userId)
         return doc;
     const scopeId = scopeIdOf(doc);
+    // Any status transition (or create as pending/rejected/approved) can change the
+    // live ladder — including demotions when approved → pending/rejected. Sync is
+    // idempotent when the latest audit already matches resolveCurrentTier.
     await syncTierChangedLog({
         payload: req.payload,
         req: req,

package/dist/collections/Tiers/hooks/afterRequiresReviewDisabled.d.ts

@@ -1,7 +1,8 @@
 import type { CollectionAfterChangeHook } from 'payload';
 /**
- * When Requires review is turned off, approve pending tier requests for this step
- * so each waiting user gets a tier.changed audit via existing request hooks.
+ * When Requires review is turned off, approve pending tier requests and sync
+ * `tier.changed` for members who already met unlock rules (including those with
+ * no pending request — otherwise the UI advances with no audit until Repair).
  */
 export declare const afterRequiresReviewDisabled: CollectionAfterChangeHook;
 //# sourceMappingURL=afterRequiresReviewDisabled.d.ts.map

package/dist/collections/Tiers/hooks/afterRequiresReviewDisabled.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"afterRequiresReviewDisabled.d.ts","sourceRoot":"","sources":["../../../../src/collections/Tiers/hooks/afterRequiresReviewDisabled.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,SAAS,CAAA;AAOxD;;;GAGG;AACH,eAAO,MAAM,2BAA2B,EAAE,yBAmBzC,CAAA"}
+{"version":3,"file":"afterRequiresReviewDisabled.d.ts","sourceRoot":"","sources":["../../../../src/collections/Tiers/hooks/afterRequiresReviewDisabled.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,SAAS,CAAA;AAOxD;;;;GAIG;AACH,eAAO,MAAM,2BAA2B,EAAE,yBAmBzC,CAAA"}

package/dist/collections/Tiers/hooks/afterRequiresReviewDisabled.js

@@ -1,7 +1,8 @@
 import { didDisableRequiresReview, releasePendingTierReviews, } from '../../../services/achievement/releaseReviewGate.js';
 /**
- * When Requires review is turned off, approve pending tier requests for this step
- * so each waiting user gets a tier.changed audit via existing request hooks.
+ * When Requires review is turned off, approve pending tier requests and sync
+ * `tier.changed` for members who already met unlock rules (including those with
+ * no pending request — otherwise the UI advances with no audit until Repair).
  */
 export const afterRequiresReviewDisabled = async ({ doc, operation, previousDoc, req, }) => {
     if (operation !== 'update')

package/dist/services/achievement/releaseReviewGate.d.ts

@@ -1,6 +1,8 @@
 import type { PayloadRequest } from 'payload';
 export type ReleaseReviewGateResult = {
     requestsApproved: number;
+    /** `tier.changed` rows written for users who newly resolve past the released gate. */
+    tierLogsWritten?: number;
 };
 /**
  * When a definition stops requiring review, approve each pending request for that
@@ -10,6 +12,13 @@ export declare function releasePendingAchievementReviews(args: {
     req: PayloadRequest;
     achievementId: string;
 }): Promise<ReleaseReviewGateResult>;
+/**
+ * When a tier stops requiring review:
+ * 1. Approve pending tier requests (their afterChange hooks sync `tier.changed`).
+ * 2. Sync `tier.changed` for every (user, scope) that may now resolve to this rung —
+ *    including members who already met unlock rules but never had a pending request
+ *    (UI would otherwise show the new rank with no audit log until Repair).
+ */
 export declare function releasePendingTierReviews(args: {
     req: PayloadRequest;
     tierId: string;

package/dist/services/achievement/releaseReviewGate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"releaseReviewGate.d.ts","sourceRoot":"","sources":["../../../src/services/achievement/releaseReviewGate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,SAAS,CAAA;AAK7C,MAAM,MAAM,uBAAuB,GAAG;IACpC,gBAAgB,EAAE,MAAM,CAAA;CACzB,CAAA;AAED;;;GAGG;AACH,wBAAsB,gCAAgC,CAAC,IAAI,EAAE;IAC3D,GAAG,EAAE,cAAc,CAAA;IACnB,aAAa,EAAE,MAAM,CAAA;CACtB,GAAG,OAAO,CAAC,uBAAuB,CAAC,CAgCnC;AAED,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,GAAG,EAAE,cAAc,CAAA;IACnB,MAAM,EAAE,MAAM,CAAA;CACf,GAAG,OAAO,CAAC,uBAAuB,CAAC,CA6BnC;AAED,uEAAuE;AACvE,wBAAgB,wBAAwB,CACtC,WAAW,EAAE;IAAE,cAAc,CAAC,EAAE,OAAO,GAAG,IAAI,CAAA;CAAE,GAAG,IAAI,GAAG,SAAS,EACnE,GAAG,EAAE;IAAE,cAAc,CAAC,EAAE,OAAO,GAAG,IAAI,CAAA;CAAE,GAAG,IAAI,GAAG,SAAS,GAC1D,OAAO,CAET"}
+{"version":3,"file":"releaseReviewGate.d.ts","sourceRoot":"","sources":["../../../src/services/achievement/releaseReviewGate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAS,MAAM,SAAS,CAAA;AAMpD,MAAM,MAAM,uBAAuB,GAAG;IACpC,gBAAgB,EAAE,MAAM,CAAA;IACxB,sFAAsF;IACtF,eAAe,CAAC,EAAE,MAAM,CAAA;CACzB,CAAA;AAcD;;;GAGG;AACH,wBAAsB,gCAAgC,CAAC,IAAI,EAAE;IAC3D,GAAG,EAAE,cAAc,CAAA;IACnB,aAAa,EAAE,MAAM,CAAA;CACtB,GAAG,OAAO,CAAC,uBAAuB,CAAC,CAgCnC;AAED;;;;;;GAMG;AACH,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,GAAG,EAAE,cAAc,CAAA;IACnB,MAAM,EAAE,MAAM,CAAA;CACf,GAAG,OAAO,CAAC,uBAAuB,CAAC,CA0FnC;AAED,uEAAuE;AACvE,wBAAgB,wBAAwB,CACtC,WAAW,EAAE;IAAE,cAAc,CAAC,EAAE,OAAO,GAAG,IAAI,CAAA;CAAE,GAAG,IAAI,GAAG,SAAS,EACnE,GAAG,EAAE;IAAE,cAAc,CAAC,EAAE,OAAO,GAAG,IAAI,CAAA;CAAE,GAAG,IAAI,GAAG,SAAS,GAC1D,OAAO,CAET"}

package/dist/services/achievement/releaseReviewGate.js

@@ -1,5 +1,13 @@
 import { collectionOf } from '../../collections/helpers.js';
+import { syncTierChangedLog } from './recordEvent.js';
 import { relationId } from './relationId.js';
+function addUserScopePair(pairs, userId, scopeId) {
+    if (!userId)
+        return;
+    const key = `${userId}::${scopeId ?? ''}`;
+    if (!pairs.has(key))
+        pairs.set(key, { userId, scopeId });
+}
 /**
  * When a definition stops requiring review, approve each pending request for that
  * definition so existing grant / tier.changed hooks run for that user + scope only.
@@ -36,6 +44,13 @@ export async function releasePendingAchievementReviews(args) {
     }
     return { requestsApproved };
 }
+/**
+ * When a tier stops requiring review:
+ * 1. Approve pending tier requests (their afterChange hooks sync `tier.changed`).
+ * 2. Sync `tier.changed` for every (user, scope) that may now resolve to this rung —
+ *    including members who already met unlock rules but never had a pending request
+ *    (UI would otherwise show the new rank with no audit log until Repair).
+ */
 export async function releasePendingTierReviews(args) {
     const pending = await args.req.payload.find({
         collection: collectionOf('tierRequests'),
@@ -63,7 +78,56 @@ export async function releasePendingTierReviews(args) {
         });
         requestsApproved += 1;
     }
-    return { requestsApproved };
+    const tier = (await args.req.payload.findByID({
+        collection: collectionOf('tiers'),
+        id: args.tierId,
+        depth: 0,
+        overrideAccess: true,
+        req: args.req,
+    }));
+    const tierScopeId = relationId(tier?.scope);
+    const pairs = new Map();
+    const allRequests = await args.req.payload.find({
+        collection: collectionOf('tierRequests'),
+        depth: 0,
+        limit: 5000,
+        pagination: false,
+        overrideAccess: true,
+        req: args.req,
+        where: { tier: { equals: args.tierId } },
+    });
+    for (const doc of allRequests.docs) {
+        addUserScopePair(pairs, relationId(doc.user), tierScopeId ?? relationId(doc.scope));
+    }
+    const grantWhere = tierScopeId
+        ? { scope: { equals: tierScopeId } }
+        : {
+            or: [{ scope: { exists: false } }, { scope: { equals: null } }],
+        };
+    const grants = await args.req.payload.find({
+        collection: collectionOf('grants'),
+        depth: 0,
+        limit: 5000,
+        pagination: false,
+        overrideAccess: true,
+        req: args.req,
+        where: grantWhere,
+    });
+    for (const doc of grants.docs) {
+        addUserScopePair(pairs, relationId(doc.user), tierScopeId ?? relationId(doc.scope));
+    }
+    let tierLogsWritten = 0;
+    for (const pair of pairs.values()) {
+        const log = await syncTierChangedLog({
+            payload: args.req.payload,
+            req: args.req,
+            userId: pair.userId,
+            scopeId: pair.scopeId,
+        });
+        if (log)
+            tierLogsWritten += 1;
+    }
+    return { requestsApproved, tierLogsWritten };
 }
 /** True when `requiresReview` flipped from required → not required. */
 export function didDisableRequiresReview(previousDoc, doc) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dappermountain/payload-plugin-achievements",
-  "version": "0.2.10",
+  "version": "0.2.12",
   "description": "A robust, Admin-configured achievements engine for Payload: composable unlock rules, derived rank progress, request/review grants, metrics, and scoped multi-tenant support.",
   "license": "MIT",
   "type": "module",