bkper-js 2.31.1 → 2.31.2

package/lib/model/Book.js

@@ -989,9 +989,27 @@ export class Book extends ResourceProperty {
             }
         }
     }
-    /** @internal */
+    /**
+     * Ensures every group has a cached `.accounts` Map after accounts are loaded.
+     *
+     * `linkAccountsAndGroups()` populates `.accounts` on groups that have accounts
+     * linked to them. Groups with zero accounts keep `.accounts = undefined`. This
+     * method fills that gap by initializing an empty Map on those groups.
+     *
+     * Without this, `Group.getAccounts()` sees `undefined`, skips the cache check,
+     * and falls through to a network request per group — causing hundreds of
+     * redundant `/groups/{id}/accounts` calls on books with many groups.
+     *
+     * IMPORTANT: This method must NOT be guarded by `allGroupsLoaded` or any
+     * similar flag. It runs inside `mapAccounts()`, which is always called after
+     * `mapGroups()` sets `allGroupsLoaded = true`. A guard like
+     * `if (!this.allGroupsLoaded)` would make this method dead code, silently
+     * reintroducing the redundant network requests.
+     *
+     * @internal
+     */
     ensureGroupsAccountMapsLoaded() {
-        if (this.idGroupMap && !this.allGroupsLoaded) {
+        if (this.idGroupMap) {
             for (const group of this.idGroupMap.values()) {
                 if (group.accounts == null) {
                     group.accounts = new Map();

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "bkper-js",
-    "version": "2.31.1",
+    "version": "2.31.2",
     "description": "Javascript client for Bkper REST API",
     "main": "./lib/index.js",
     "module": "./lib/index.js",