Merge pull request #8 from Daniel-Ric/feature/2026-01-28/implement-new-endpoint-for-item-count-by-creator

creator-mapping

Author: Daniel-Ric

README.md

@@ -227,7 +227,7 @@ Optional Marketplace headers (when enabled):
 | ------ | --------------------------------------------- | --------------------------------------------------- |
 | GET    | `/inventory/balances`                         | Player virtual currency balances                    |
 | GET    | `/inventory/entitlements?includeReceipt=true` | Player entitlements (optionally with receipts)      |
-| GET    | `/inventory/creators?includeUnknown=true`     | Inventory counts grouped by creator (optional)      |
+| GET    | `/inventory/creators?includeUnknown=true`     | Inventory counts grouped by creator display name    |
 | GET    | `/inventory/playfab/items?filter=<query>`     | PlayFab inventory items with official filters       |
 
 #### Purchase
@@ -273,7 +273,7 @@ Headers: `authorization` (JWT), `x-mc-token` (required). Returns `{ count, entit
 
 #### `GET /inventory/creators?includeUnknown=<bool>`
 
-Headers: `authorization` (JWT), `x-mc-token` (required). Returns `{ count, totalItems, unknownCount, creators }` with creator ownership counts derived from entitlements.
+Headers: `authorization` (JWT), `x-mc-token` (required). Returns `{ count, totalItems, unknownCount, creators }` with creator ownership counts derived from entitlements and mapped to creator display names.
 
 #### `GET /inventory/playfab/items?filter=<query>`
 

src/routes/inventory.routes.js

@@ -3,6 +3,7 @@ import {jwtMiddleware} from "../utils/jwt.js";
 import {asyncHandler} from "../utils/async.js";
 import {badRequest} from "../utils/httpError.js";
 import {getBalances, getInventory} from "../services/minecraft.service.js";
+import {getCreators} from "../services/marketplace.service.js";
 import {getEntityTokenForPlayer, getInventoryItems} from "../services/playfab.service.js";
 import {summarizeCreators} from "../utils/inventoryCreators.js";
 
@@ -29,7 +30,9 @@ router.get("/creators", jwtMiddleware, asyncHandler(async (req, res) => {
     const includeReceipt = String(req.query.includeReceipt || "false") === "true";
     const includeUnknown = String(req.query.includeUnknown || "false") === "true";
     const entitlements = await getInventory(mcToken, includeReceipt);
-    const {totalItems, unknownCount, creators} = summarizeCreators(entitlements, {includeUnknown});
+    const creatorMap = await getCreators(mcToken);
+    const creatorLookup = Object.fromEntries(Object.entries(creatorMap).map(([name, id]) => [String(id), name]));
+    const {totalItems, unknownCount, creators} = summarizeCreators(entitlements, {includeUnknown, creatorLookup});
     res.json({count: Object.keys(creators).length, totalItems, unknownCount, creators});
 }));
 

src/routes/purchase.routes.js

@@ -240,7 +240,9 @@ router.get("/inventory/creators", asyncHandler(async (req, res) => {
     if (!mcToken && st) mcToken = await getMCToken(st);
     if (!mcToken) throw badRequest("x-mc-token or x-playfab-session is required");
     const entitlements = await getInventory(mcToken, includeReceipt);
-    const {totalItems, unknownCount, creators} = summarizeCreators(entitlements, {includeUnknown});
+    const creatorMap = await getCreators(mcToken);
+    const creatorLookup = Object.fromEntries(Object.entries(creatorMap).map(([name, id]) => [String(id), name]));
+    const {totalItems, unknownCount, creators} = summarizeCreators(entitlements, {includeUnknown, creatorLookup});
     res.json({count: Object.keys(creators).length, totalItems, unknownCount, creators});
 }));
 

src/utils/inventoryCreators.js

@@ -58,16 +58,22 @@ function resolveCreator(entitlement) {
     return "";
 }
 
-export function summarizeCreators(entitlements, {includeUnknown = false, unknownKey = "unknown"} = {}) {
+export function summarizeCreators(entitlements, {
+    includeUnknown = false,
+    unknownKey = "unknown",
+    creatorLookup = null
+} = {}) {
     const items = Array.isArray(entitlements) ? entitlements : [];
     const counts = new Map();
     let unknownCount = 0;
     const unknownLabel = String(unknownKey || "").trim() || "unknown";
+    const lookup = creatorLookup && typeof creatorLookup === "object" ? creatorLookup : null;
 
     for (const entitlement of items) {
-        const creator = resolveCreator(entitlement);
-        if (creator) {
-            counts.set(creator, (counts.get(creator) || 0) + 1);
+        const resolved = resolveCreator(entitlement);
+        if (resolved) {
+            const mapped = lookup && lookup[resolved] ? lookup[resolved] : resolved;
+            counts.set(mapped, (counts.get(mapped) || 0) + 1);
             continue;
         }
         unknownCount += 1;

src/utils/swagger.js

@@ -169,7 +169,7 @@ const options = {
                 get: {
                     tags: ["Inventory"],
                     summary: "Get inventory creators",
-                    description: "Returns owned inventory counts grouped by creator.",
+                    description: "Returns owned inventory counts grouped by creator display name.",
                     parameters: [{
                         in: "header",
                         name: "x-mc-token",
@@ -463,7 +463,7 @@ const options = {
                 get: {
                     tags: ["Purchase"],
                     summary: "Get inventory creators via purchase flow",
-                    description: "Returns owned inventory counts grouped by creator using either a Minecraft token or a PlayFab session.",
+                    description: "Returns owned inventory counts grouped by creator display name using either a Minecraft token or a PlayFab session.",
                     parameters: [{
                         in: "header",
                         name: "x-mc-token",

test/inventoryCreators.test.js

@@ -37,3 +37,14 @@ test("summarizeCreators can include unknown bucket", () => {
     assert.equal(summary.unknownCount, 1);
     assert.deepEqual(summary.creators, {Alpha: 1, unknown: 1});
 });
+
+test("summarizeCreators maps creator ids using lookup", () => {
+    const entitlements = [{creatorId: "master_player_account:abc"}, {creatorId: "master_player_account:def"}];
+    const summary = summarizeCreators(entitlements, {
+        creatorLookup: {
+            "master_player_account:abc": "Alpha Studio",
+            "master_player_account:def": "Beta Studio"
+        }
+    });
+    assert.deepEqual(summary.creators, {"Alpha Studio": 1, "Beta Studio": 1});
+});