@i4e/invest4edu-access-core 0.42.0 → 0.43.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@i4e/invest4edu-access-core",
-  "version": "0.42.0",
+  "version": "0.43.0",
   "description": "Shared access-control primitives for NeoFindesk: tenant keystone, role capabilities, reportee tree, entity/role vocabulary, mobile app identity, feature flags, and the unified access engine (registry schema, snapshot resolver, visibleWhen).",
   "type": "module",
   "exports": {

package/src/client-profile/profile-service.js

@@ -56,10 +56,10 @@ export function createClientProfileService({ getDb, getSettings, getSideContext
    * `mode` tells the caller whether to act on it: "enforce" → show the existing client instead
    * of creating; "audit" → carry on as before, the match was only reported.
    */
-  async function findExisting({ pan, profileType, session }) {
+  async function findExisting({ pan, profileType, distributorId = null, session }) {
     const settings = getSettings();
     if (!isEvaluated(settings, "resolve_by_pan")) return { profile: null, mode: "off" };
-    const profile = pickProfileOfType(await findActiveProfilesByPan(pan, { session }), profileType);
+    const profile = pickProfileOfType(await findActiveProfilesByPan(pan, { session }), profileType, { distributorId, settings, ctx: getSideContext() });
     const mode = isEnforced(settings, "resolve_by_pan") ? "enforce" : "audit";
     if (profile && mode === "audit") onAudit({ kind: "resolve_by_pan", client_id: profile._id, profile_type: profileType });
     return { profile, mode };
@@ -128,7 +128,10 @@ export function createClientProfileService({ getDb, getSettings, getSideContext
   async function roleNamesOf(userId, session) {
     const maps = await db().collection("userentitymappings").find({ user_id: userId, status: { $ne: 0 }, role_id: { $ne: null } }, { session, projection: { role_id: 1 } }).toArray();
     if (!maps.length) return [];
-    const roles = await db().collection("roles").find({ _id: { $in: maps.map((m) => m.role_id) } }, { session, projection: { role_name: 1 } }).toArray();
+    // role_id is an ARRAY on real mappings; `$in` must get the role ids themselves, not the arrays
+    // (an array inside `$in` only matches an equal array, so every caller read as role-less).
+    const roleIds = maps.flatMap((m) => (Array.isArray(m.role_id) ? m.role_id : [m.role_id])).filter(Boolean);
+    const roles = await db().collection("roles").find({ _id: { $in: roleIds } }, { session, projection: { role_name: 1 } }).toArray();
     return roles.map((r) => r.role_name).filter(Boolean);
   }
 

package/src/client-profile/profiles.js

@@ -65,9 +65,25 @@ export function profileTypeForCaller(callerDistributor, settings) {
   return profileTypeFor(callerDistributor, settings);
 }
 
+/** Corporate profiles are one per corporate firm and never shared (settings.corporate). */
+function corporateSiloed(settings) {
+  return settings?.corporate?.one_profile_per_distributor !== false;
+}
+
+/** The side that owns a profile: its primary mapping, else its distributor_id. */
+function primarySideOfProfile(profile, ctx) {
+  return sideKeyOf((profile?.distributor_mappings ?? []).find((m) => m?.is_primary)?.distributor_id ?? profile?.distributor_id, ctx);
+}
+
 /** Of a person's active profiles, the one of this type (oldest first if the data has duplicates). */
-export function pickProfileOfType(profiles = [], profileType) {
-  const ofType = profiles.filter((p) => (p.profile_type ?? PROFILE_TYPES.MAIN) === profileType);
+export function pickProfileOfType(profiles = [], profileType, { distributorId = null, settings, ctx } = {}) {
+  let ofType = profiles.filter((p) => (p.profile_type ?? PROFILE_TYPES.MAIN) === profileType);
+  // A corporate profile belongs to ONE corporate firm: only that firm's own profile is "the existing
+  // one". Another firm (or a caller with no firm) finds nothing and creates its own.
+  if (profileType === PROFILE_TYPES.CORPORATE && corporateSiloed(settings)) {
+    const side = distributorId ? sideKeyOf(distributorId, ctx) : null;
+    ofType = side ? ofType.filter((p) => primarySideOfProfile(p, ctx) === side) : [];
+  }
   ofType.sort((a, b) => new Date(a.created_date ?? 0) - new Date(b.created_date ?? 0));
   return ofType[0] ?? null;
 }
@@ -90,9 +106,13 @@ export function attachDecision({ profile, actor = {}, via, at = new Date(), sett
   if (actor.callerClass === CALLER_CLASSES.PARTNER) {
     const side = sideKeyOf(actor.distributorId, ctx);
     if (side === DIRECT_SIDE) return { kind: "none", reason: "partner_without_firm" };
-    const primarySide = sideKeyOf((profile.distributor_mappings ?? []).find((m) => m?.is_primary)?.distributor_id ?? profile.distributor_id, ctx);
+    const primarySide = primarySideOfProfile(profile, ctx);
     const mapped = side === primarySide || (profile.distributor_mappings ?? []).some((m) => m && m.status !== "inactive" && sideKeyOf(m.distributor_id, ctx) === side);
     if (mapped) return { kind: "none", reason: "already_mapped" };
+    // A corporate profile is its firm's alone: no other firm is ever added to it.
+    if ((profile.profile_type ?? PROFILE_TYPES.MAIN) === PROFILE_TYPES.CORPORATE && corporateSiloed(settings)) {
+      return { kind: "none", reason: "corporate_silo" };
+    }
     return {
       kind: "mapping",
       reason: "new_side",

package/src/client-profile/settings.js

@@ -77,6 +77,12 @@ export const DEFAULT_CLIENT_PROFILE_SETTINGS = deepFreeze({
   },
   corporate_distributor_types: [],
   house_distributor_codes: [],
+  corporate: {
+    // Each corporate distributor holds its OWN profile of a person (one per corporate firm), and
+    // works on it alone: a PAN lookup never surfaces another firm's corporate profile, and no
+    // other firm is ever attached to one.
+    one_profile_per_distributor: true,
+  },
   primary: {
     lock_on_first_order: true,
     ignore_order_statuses: [],
@@ -192,6 +198,7 @@ export const CLIENT_PROFILE_SETTING_DESCRIPTIONS = Object.freeze({
   "modes.detail_scope": "A client's full record by id only for callers whose client list includes them; everyone else gets the limited card (R15).",
   "modes.linked_visibility": "Clients reached only through a secondary mapping or a linked user appear in lists, as the limited card.",
   corporate_distributor_types: "distributors.distributor_type values whose clients get a corporate profile.",
+  "corporate.one_profile_per_distributor": "Each corporate firm has its own, siloed profile of a person: no cross-firm lookup or attach.",
   house_distributor_codes: "Distributor codes treated as direct (InvestValue's own house distributors).",
   "primary.lock_on_first_order": "The first non-ignored order locks the primary distributor.",
   "primary.ignore_order_statuses": "Order statuses that do not count as first business (e.g. cancelled).",