@fabriktor/fx 0.0.147 → 0.0.148

package/dist/src/chat/forms.js

@@ -1,5 +1,5 @@
 import { normalizeIDs, resolvedID } from "../core/id.js";
-import { isFabriktorUser, userSupportsMode } from "../users/users.js";
+import { isFabriktorUser, isOtherUserID, userSupportsMode, } from "../users/users.js";
 // Recipients are all other members of the conversation, resolved before use.
 // Sender modes apply now; recipient modes describe what their identity supports.
 // Do not depend on the recipient's current mode, which Public() omits.
@@ -10,7 +10,7 @@ export function canSendChatForm(policy, actor, recipients) {
         (policy.allow_group || recipients.length === 1) &&
         normalizeIDs(recipients.map((recipient) => recipient.id)).length ===
             recipients.length &&
-        recipients.every((recipient) => recipient.id !== actor.owner_id &&
+        recipients.every((recipient) => isOtherUserID(recipient.id, actor.owner_id) &&
             isFabriktorUser(recipient) &&
             policy.recipient_modes.some((mode) => userSupportsMode(recipient, mode))));
 }

package/dist/src/users/users.d.ts

@@ -17,6 +17,7 @@ export type UpdateExternalUserOptions = {
 };
 export declare function userSupportsMode(user: Pick<User, "account_identity"> | null | undefined, mode: User["active_mode"]): boolean;
 export declare function isFabriktorUser(user: Pick<User, "account_identity"> | null | undefined): boolean;
+export declare function isOtherUserID(user_id: string | null | undefined, owner_id: string | null | undefined): boolean;
 export declare function canManageExternalUser(user: User | null | undefined, owner_id: string): boolean;
 export type SearchExternalUsersOptions = SearchUsersOptions;
 export type SearchExternalUsersPage = SearchUsersPage;

package/dist/src/users/users.js

@@ -44,6 +44,13 @@ export function userSupportsMode(user, mode) {
 export function isFabriktorUser(user) {
     return userSupportsMode(user, UserTypeClient);
 }
+// Client capability and being a different party are separate requirements.
+// Apply this to counterparties, not to ownership or employee scope filters.
+export function isOtherUserID(user_id, owner_id) {
+    const userID = resolvedID(user_id ?? undefined);
+    const ownerID = resolvedID(owner_id ?? undefined);
+    return !!userID && !!ownerID && userID !== ownerID;
+}
 export function canManageExternalUser(user, owner_id) {
     return (!!resolvedID(owner_id) &&
         user?.account_identity === UserTypeExternal &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fabriktor/fx",
-  "version": "0.0.147",
+  "version": "0.0.148",
   "description": "![Fabriktor Logo](./img/fabriktor-character.gif)",
   "type": "module",
   "exports": {