abmp-npm 2.0.76 → 2.0.78

package/.claude/worktrees/fix-save-button-disable/backend/contacts-methods.js

@@ -1,67 +0,0 @@
-const { contacts } = require('@wix/crm');
-const { auth } = require('@wix/essentials');
-
-const elevatedGetContact = auth.elevate(contacts.getContact);
-const elevatedUpdateContact = auth.elevate(contacts.updateContact);
-const elevatedCreateContact = auth.elevate(contacts.createContact);
-const elevatedDeleteContact = auth.elevate(contacts.deleteContact);
-
-const deleteSiteContact = contactId => elevatedDeleteContact(contactId);
-
-/**
- * Create a contact in Wix CRM
- * @param {Object} contactData - Contact data
- * @returns {Promise<Object>} - Contact data
- */
-async function createSiteContact(contactData) {
-  console.log('[createSiteContact]contactData', JSON.stringify(contactData, null, 2));
-  if (!contactData || !contactData.contactFormEmail) {
-    throw new Error('Contact data is required');
-  }
-  const contactInfo = {
-    name: {
-      first: contactData.firstName,
-      last: contactData.lastName,
-    },
-    emails: {
-      items: [{ email: contactData.contactFormEmail, primary: true }],
-    },
-  };
-  console.log('[createSiteContact]contactInfo', JSON.stringify(contactInfo, null, 2));
-  // Intentionally NOT passing allowDuplicates: Wix CRM enforces email uniqueness
-  // case-insensitively, which is the guard we want against two different members sharing an
-  // email. Same-member cases never reach here — they collapse to a single entity upstream.
-  const createContactResponse = await elevatedCreateContact(contactInfo);
-  console.log(
-    '[createSiteContact]createContactResponse',
-    JSON.stringify(createContactResponse, null, 2)
-  );
-  return createContactResponse.contact._id;
-}
-/**
- * Generic contact update helper function
- * @param {string} contactId - The contact ID in Wix CRM
- * @param {function} updateInfoCallback - Function that returns the updated info object
- * @param {string} operationName - Name of the operation for logging
- */
-async function updateContactInfo(contactId, updateInfoCallback, operationName) {
-  if (!contactId) {
-    throw new Error('Contact ID is required');
-  }
-
-  try {
-    const contact = await elevatedGetContact(contactId);
-    const currentInfo = contact.info;
-    const updatedInfo = updateInfoCallback(currentInfo);
-    await elevatedUpdateContact(contactId, updatedInfo, contact.revision);
-  } catch (error) {
-    console.error(`Error in ${operationName}:`, error);
-    throw new Error(`Failed to ${operationName}: ${error.message}`);
-  }
-}
-
-module.exports = {
-  createSiteContact,
-  updateContactInfo,
-  deleteSiteContact,
-};

package/.claude/worktrees/fix-save-button-disable/backend/daily-pull/bulk-process-methods.js

@@ -1,222 +0,0 @@
-const { bulkSaveMembers, getMemberBySlug, findMembersByIds } = require('../members-data-methods');
-const { extractUrlCounter } = require('../utils');
-
-const { generateUpdatedMemberData } = require('./process-member-methods');
-const {
-  changeWixMembersEmails,
-  summarizeLoginEmailOutcomes,
-  incrementUrlCounter,
-  extractBaseUrl,
-} = require('./utils');
-
-/**
- * Ensures unique URLs within a batch of members by deduplicating URLs
- * Groups members by their base URL (normalized) and assigns unique counters
- * Also checks database to handle cross-page conflicts
- * @param {Array} memberDataList - Array of processed member data
- * @returns {Promise<Array>} - Array of members with unique URLs assigned
- */
-async function ensureUniqueUrlsInBatch(memberDataList) {
-  if (!Array.isArray(memberDataList) || memberDataList.length === 0) {
-    return memberDataList;
-  }
-
-  // Group members by their normalized base URL (case-insensitive to avoid
-  // "John-12" and "john-12" being treated as different when slugs are matched case-insensitively)
-  const urlGroups = new Map();
-
-  memberDataList.forEach(member => {
-    if (!member || !member.url) {
-      return;
-    }
-
-    const baseUrl = extractBaseUrl(member.url);
-    const groupKey = baseUrl.toLowerCase();
-    if (!urlGroups.has(groupKey)) {
-      urlGroups.set(groupKey, { members: [], baseUrl });
-    }
-    urlGroups.get(groupKey).members.push(member);
-  });
-
-  // For each group, check database and assign unique URLs sequentially
-  for (const [, { members, baseUrl }] of urlGroups.entries()) {
-    if (members.length <= 1) {
-      // Single member - still check DB to ensure it doesn't conflict with other pages
-      const member = members[0];
-      if (member) {
-        const dbMember = await getMemberBySlug({
-          slug: baseUrl,
-          excludeDropped: false,
-          normalizeSlugForComparison: true,
-        });
-
-        if (dbMember && dbMember.url) {
-          // Conflict found in DB, need to add counter
-          member.url = incrementUrlCounter(dbMember.url, baseUrl);
-          console.log(
-            `Found DB conflict for single member with base URL "${baseUrl}", assigned: ${member.url}`
-          );
-        }
-      }
-      continue;
-    }
-
-    // Sort members to ensure consistent ordering
-    members.sort((a, b) => {
-      if (a.url && b.url) {
-        return String(a.url).localeCompare(String(b.url));
-      }
-      return 0;
-    });
-
-    // Check database for existing members with this base URL to find highest counter
-    const dbMember = await getMemberBySlug({
-      slug: baseUrl,
-      excludeDropped: false,
-      normalizeSlugForComparison: true,
-    });
-
-    const dbMaxCounter = extractUrlCounter(dbMember?.url);
-
-    // Find the highest existing counter among all members in this batch group
-    let batchMaxCounter = -1;
-    members.forEach(member => {
-      const originalUrl = member.url;
-      const urlCounter = extractUrlCounter(originalUrl);
-      if (urlCounter > batchMaxCounter) {
-        batchMaxCounter = urlCounter;
-      }
-    });
-
-    // Start index from the maximum of DB counter and batch counter + 1
-    const maxCounter = Math.max(dbMaxCounter, batchMaxCounter);
-    const startIndex = maxCounter + 1;
-
-    // Assign unique URLs: start from the appropriate index
-    members.forEach((member, index) => {
-      const assignedIndex = startIndex + index;
-      if (assignedIndex === 0) {
-        // Index 0 means no counter, use baseUrl
-        member.url = baseUrl;
-      } else {
-        // Index > 0 means add counter
-        member.url = `${baseUrl}-${assignedIndex}`;
-      }
-    });
-
-    console.log(
-      `Deduplicated ${
-        members.length
-      } members with base URL "${baseUrl}" (DB max: ${dbMaxCounter}, batch max: ${batchMaxCounter}, start: ${startIndex}): ${members
-        .map(m => m.url)
-        .join(', ')}`
-    );
-  }
-
-  return memberDataList;
-}
-
-/**
- * Processes and saves multiple member records in bulk
- * @param {Object} options - The options object
- * @param {Array} options.memberDataList - Array of member data from API
- * @param {number} options.currentPageNumber - Current page number being processed
- * @param {Array} memberDataList - Array of member data from API
- * @returns {Promise<Object>} - Bulk save operation result with statistics
- */
-const bulkProcessAndSaveMemberData = async ({ memberDataList, currentPageNumber }) => {
-  if (!Array.isArray(memberDataList) || memberDataList.length === 0) {
-    throw new Error('Invalid member data list provided');
-  }
-
-  const startTime = Date.now();
-
-  try {
-    // Prefetch all existing members for this page in a few chunked queries instead of
-    // one query per member — thousands of parallel lookups made the whole page fail
-    // whenever a single one hit a transient network error.
-    const existingMembersById = await findMembersByIds(
-      memberDataList.map(memberData => memberData?.memberid)
-    );
-
-    const processedMemberDataPromises = memberDataList.map(memberData =>
-      generateUpdatedMemberData({
-        inputMemberData: memberData,
-        currentPageNumber,
-        existingDbMember: existingMembersById.get(String(memberData?.memberid)) ?? null,
-      })
-    );
-
-    const processedMemberDataList = await Promise.all(processedMemberDataPromises);
-    const validMemberData = processedMemberDataList.filter(
-      data => data !== null && data !== undefined
-    );
-    if (validMemberData.length === 0) {
-      return {
-        totalProcessed: memberDataList.length,
-        totalSaved: 0,
-        totalFailed: memberDataList.length,
-        processingTime: Date.now() - startTime,
-      };
-    }
-    const newMembers = validMemberData.filter(data => data.isNewToDb);
-    const existingMembers = validMemberData.filter(data => !data.isNewToDb);
-    // Ensure unique URLs within the batch to prevent duplicates (also checks DB for cross-page conflicts)
-    const uniqueUrlsNewToDBMembersList = await ensureUniqueUrlsInBatch(newMembers);
-    const uniqueUrlsMembersData = [...uniqueUrlsNewToDBMembersList, ...existingMembers];
-
-    // Change Wix login emails FIRST so we know which succeeded before saving the CMS records.
-    const toChangeWixMembersEmails = uniqueUrlsMembersData.filter(
-      member => member.wixMemberId && member.isLoginEmailChanged
-    );
-    let failedLoginEmailIds = new Set();
-    let loginEmailFailures = [];
-    if (toChangeWixMembersEmails.length > 0) {
-      const outcomes = await changeWixMembersEmails(toChangeWixMembersEmails);
-      ({ failedMemberIds: failedLoginEmailIds, failures: loginEmailFailures } =
-        summarizeLoginEmailOutcomes(outcomes));
-    }
-
-    const toSaveMembersData = uniqueUrlsMembersData.map(member => {
-      // Transient flags only drive the sync above; never persist them.
-      const {
-        isLoginEmailChanged: _isLoginEmailChanged,
-        isNewToDb: _isNewToDb,
-        previousLoginEmail,
-        ...restMemberData
-      } = member;
-      // If the Wix login-email change failed, keep the CMS login email unchanged (revert to the
-      // previous value) so the CMS stays consistent with Wix; the failure is reported below for
-      // manual handling.
-      if (failedLoginEmailIds.has(member.memberId) && previousLoginEmail !== undefined) {
-        return { ...restMemberData, email: previousLoginEmail };
-      }
-      return restMemberData;
-    });
-    const saveResult = await bulkSaveMembers(toSaveMembersData);
-    const totalFailed = memberDataList.length - validMemberData.length;
-    const processingTime = Date.now() - startTime;
-
-    if (loginEmailFailures.length > 0) {
-      console.error(
-        `[loginEmailSync] ${loginEmailFailures.length} login-email change(s) failed on page ${currentPageNumber}; CMS login email left unchanged for memberIds: [${loginEmailFailures
-          .map(failure => failure.memberId)
-          .join(', ')}]`
-      );
-    }
-
-    return {
-      ...saveResult,
-      totalProcessed: memberDataList.length,
-      totalSaved: validMemberData.length,
-      totalFailed: totalFailed,
-      processingTime: processingTime,
-      loginEmailFailedCount: loginEmailFailures.length,
-      loginEmailFailures,
-    };
-  } catch (error) {
-    throw new Error(`Bulk operation failed: ${error.message}`);
-  }
-};
-
-module.exports = { bulkProcessAndSaveMemberData, ensureUniqueUrlsInBatch };

package/.claude/worktrees/fix-save-button-disable/backend/daily-pull/consts.js

@@ -1,39 +0,0 @@
-const MEMBER_ACTIONS = {
-  UPDATE: 'update',
-  NEW: 'new',
-  DROP: 'drop',
-  NONE: 'none',
-};
-
-/**
- * Address visibility configuration options
- */
-const ADDRESS_VISIBILITY_OPTIONS = {
-  ALL: 'all',
-  NONE: 'none',
-};
-
-/**
- * Default display settings for member profiles
- */
-const DEFAULT_MEMBER_DISPLAY_SETTINGS = {
-  showLicenseNo: true,
-  showName: true,
-  showBookingUrl: false,
-  showWebsite: false,
-  showWixUrl: true,
-};
-
-const PAC_ASSOCIATIONS = {
-  ABMP: 'ABMP',
-  ASCP: 'ASCP',
-  ANP: 'ANP',
-  AHP: 'AHP',
-};
-
-module.exports = {
-  MEMBER_ACTIONS,
-  ADDRESS_VISIBILITY_OPTIONS,
-  DEFAULT_MEMBER_DISPLAY_SETTINGS,
-  PAC_ASSOCIATIONS,
-};

package/.claude/worktrees/fix-save-button-disable/backend/daily-pull/index.js

@@ -1,5 +0,0 @@
-module.exports = {
-  ...require('./sync-to-cms-methods'),
-  ...require('./consts'), //TODO: remove it once we finish NPM movement
-  ...require('./bulk-process-methods'),
-};

package/.claude/worktrees/fix-save-button-disable/backend/daily-pull/process-member-methods.js

@@ -1,213 +0,0 @@
-const { ADDRESS_STATUS_TYPES } = require('../../public/consts');
-const { findMemberById, getMemberBySlug } = require('../members-data-methods');
-const { isValidArray, generateGeoHash } = require('../utils');
-
-const { MEMBER_ACTIONS, DEFAULT_MEMBER_DISPLAY_SETTINGS } = require('./consts');
-const { incrementUrlCounter } = require('./utils');
-const { validateCoreMemberData, containsNonEnglish, createFullName } = require('./utils');
-
-/**
- * Ensures a URL is unique by appending a counter if necessary
- * @param {Object} options - The options object
- * @param {string} options.url - The base URL to make unique
- * @param {string|number} options.memberId - The member ID requesting this URL
- * @param {string} options.fullName - The full name of the member
- * @returns {Promise<string>} - A unique URL
- */
-const ensureUniqueUrl = async ({ url, memberId, fullName }) => {
-  const baseUrl = url;
-  let uniqueUrl = url;
-  if (!url) {
-    console.log(`member with id ${memberId} has no url, creating one`);
-    const fullNameWithoutSpace = fullName?.replace(/ /g, '');
-    if (!fullNameWithoutSpace || containsNonEnglish(fullNameWithoutSpace)) {
-      console.log(
-        `member with id ${memberId} has non-english full name, will use fallback url: 'firstNameLastName'`
-      );
-      uniqueUrl = 'firstNameLastName';
-    } else {
-      uniqueUrl = fullNameWithoutSpace; //fallback if there is no full name for this user
-    }
-    console.log(
-      `member with id ${memberId} and  no API provided url, will have this initial url ${uniqueUrl}`
-    );
-  }
-  if (!memberId) throw new Error('Member ID is required');
-
-  const existingMember = await getMemberBySlug({
-    slug: uniqueUrl,
-    excludeDropped: false,
-    excludeSearchedMember: true,
-    memberId,
-    normalizeSlugForComparison: true,
-  });
-  if (existingMember && existingMember.url) {
-    console.log(
-      `Found member with same url ${existingMember.url} for memberId ${memberId} and URL ${uniqueUrl}, increasing counter by 1`
-    );
-    uniqueUrl = incrementUrlCounter(existingMember.url, uniqueUrl);
-  }
-  if (uniqueUrl !== baseUrl) {
-    console.log(`URL conflict resolved: ${baseUrl} -> ${uniqueUrl} for member ${memberId}`);
-  }
-  return uniqueUrl;
-};
-
-/**
- * Generates complete updated member data by combining existing and migration data
- * @param {Object} options - The options object
- * @param {Object} options.inputMemberData - Raw member data from API
- * @param {number} options.currentPageNumber - Current page number being processed
- * @param {Object|null} [options.existingDbMember] - Prefetched DB member (null if none exists).
- *   Pass it when processing members in bulk to avoid one DB query per member; when omitted,
- *   the member is looked up individually.
- * @returns {Promise<Object|null>} - Complete updated member data or null if validation fails
- */
-async function generateUpdatedMemberData({
-  inputMemberData,
-  currentPageNumber,
-  existingDbMember: prefetchedDbMember,
-}) {
-  if (!validateCoreMemberData(inputMemberData)) {
-    throw new Error(
-      'Invalid member data: memberid, email (valid string), and memberships (array) are required'
-    );
-  }
-
-  const existingDbMember =
-    prefetchedDbMember === undefined
-      ? await findMemberById(inputMemberData.memberid)
-      : prefetchedDbMember;
-
-  const updatedMemberData = await createCoreMemberData(
-    inputMemberData,
-    existingDbMember,
-    currentPageNumber
-  );
-
-  // If createCoreMemberData returns null due to validation failure, return null
-  if (!updatedMemberData) {
-    return null;
-  }
-
-  // Only add address data for new members
-  if (!existingDbMember && isValidArray(inputMemberData.addresses)) {
-    const normalizedAddresses = inputMemberData.addresses.map((address, index) => {
-      const key = address?.key || `address_${index}`;
-      const rawStatus = address?.addressStatus;
-      const resolvedStatus =
-        index === 0 && rawStatus === ADDRESS_STATUS_TYPES.DONT_SHOW
-          ? ADDRESS_STATUS_TYPES.STATE_CITY_ZIP
-          : rawStatus || ADDRESS_STATUS_TYPES.STATE_CITY_ZIP;
-      return {
-        ...address,
-        key,
-        addressStatus: resolvedStatus,
-      };
-    });
-    updatedMemberData.addresses = normalizedAddresses;
-    updatedMemberData.addressDisplayOption = [{ key: normalizedAddresses[0].key, isMain: true }];
-  }
-
-  return { ...updatedMemberData, isNewToDb: !existingDbMember };
-}
-/**
- * Helper function to get fields that should only be set for new members
- * @param {Object} inputMemberData - Raw member data from API
- * @param {Object} existingDbMember - Existing member data from database
- * @returns {Promise<Object>} - Object with fields that should only be set for new members
- */
-async function getNewMemberOnlyFields(inputMemberData, existingDbMember) {
-  if (existingDbMember) {
-    return {};
-  }
-
-  // Only set these fields for new members
-  const sanitizedFirstName = inputMemberData.firstname?.trim() || '';
-  const sanitizedLastName = inputMemberData.lastname?.trim() || '';
-  const fullName = createFullName(sanitizedFirstName, sanitizedLastName);
-
-  const uniqueUrl = await ensureUniqueUrl({
-    url: inputMemberData.url,
-    memberId: inputMemberData.memberid,
-    fullName,
-  });
-  return {
-    memberId: inputMemberData.memberid,
-    firstName: sanitizedFirstName,
-    lastName: sanitizedLastName,
-    fullName,
-    phones: inputMemberData.phones || [],
-    optOut: false,
-    url: uniqueUrl,
-    showContactForm: true,
-    showABMP: false,
-    locHash: generateGeoHash(inputMemberData.addresses || []),
-    ...DEFAULT_MEMBER_DISPLAY_SETTINGS,
-  };
-}
-/**
- * Creates base member data structure with core properties
- * @param {Object} inputMemberData - Raw member data from API
- * @param {Object} existingDbMember - Existing member data from database
- * @param {number} currentPageNumber - Current page number being processed
- * @returns {Promise<Object|null>} - Structured base member data or null if required fields are missing
- */
-async function createCoreMemberData(inputMemberData, existingDbMember, currentPageNumber) {
-  if (!validateCoreMemberData(inputMemberData)) {
-    return null;
-  }
-
-  const getMemberEmails = () => {
-    // Update both loginEmail & contactFormEmail only for new members who don't exist in DB
-    // Note: PAC API member actions are not reliable, so we need to check if the member exists in DB to know if it's a new member or not
-    const newEmail = inputMemberData.email.trim();
-    const isMemberExistInDb = Boolean(existingDbMember?._id);
-    if (!isMemberExistInDb) {
-      return {
-        email: newEmail,
-        contactFormEmail: newEmail,
-      };
-    }
-    //If exists in DB then only update loginEmail for those who came in with action new based on logic below, otherwise we don't update emails
-    const isMemberReinstatedWithNewEmail =
-      inputMemberData.action === MEMBER_ACTIONS.NEW &&
-      newEmail &&
-      existingDbMember.email !== newEmail;
-    if (isMemberReinstatedWithNewEmail) {
-      // If exists in DB, and email was changed means this user was dropped before that's why it exists in DB, then only update loginEmail not contactFormEmail.
-      // previousLoginEmail lets the caller keep the old email if the Wix login-email change fails.
-      return {
-        email: newEmail,
-        isLoginEmailChanged: true,
-        previousLoginEmail: existingDbMember.email,
-      };
-    }
-    //If exists in DB but not reinstated with new email, then don't update emails
-    return {};
-  };
-
-  const newMemberFields = await getNewMemberOnlyFields(inputMemberData, existingDbMember);
-
-  return {
-    ...existingDbMember,
-
-    // Always update these core fields from API
-    action: inputMemberData.action,
-    licenses: inputMemberData.licenses || [],
-    memberships: inputMemberData.memberships,
-    pageNumber: currentPageNumber,
-    isVisible: inputMemberData.action !== MEMBER_ACTIONS.DROP,
-
-    // Handle Member emails
-    ...getMemberEmails(),
-
-    // Handle fields that should only be set for new members
-    ...newMemberFields,
-  };
-}
-
-module.exports = {
-  generateUpdatedMemberData,
-  ensureUniqueUrl,
-};

package/.claude/worktrees/fix-save-button-disable/backend/daily-pull/schedule-methods.js

@@ -1,41 +0,0 @@
-const { taskManager } = require('psdev-task-manager');
-
-const { TASKS_NAMES } = require('../tasks/consts');
-
-const { MEMBER_ACTIONS } = require('./consts');
-
-/**
- * Builds the per-action daily pull tasks. Single source of truth for what a
- * "daily pull" schedules, shared by the cron job and the execution-check
- * fallback so both produce identical tasks (same names, same environment flags).
- * @param {Object} [options]
- * @param {string} [options.backupDate] - Optional backup date (YYYY-MM-DD) to pull from the backup endpoint
- * @param {boolean} [options.isTestEnvironment=false] - Whether to pull from the test PAC API
- * @param {boolean} [options.includeNone=false] - Whether to also sync the NONE action
- * @returns {Array<Object>} - Task definitions for taskManager().scheduleInBulk
- */
-const buildDailyPullTasks = ({ backupDate, isTestEnvironment, includeNone } = {}) => {
-  const actionsToSync = includeNone
-    ? Object.values(MEMBER_ACTIONS)
-    : Object.values(MEMBER_ACTIONS).filter(action => action !== MEMBER_ACTIONS.NONE);
-  return actionsToSync.map(action => ({
-    name: TASKS_NAMES.ScheduleMembersDataPerAction,
-    data: {
-      action,
-      ...(backupDate ? { backupDate } : {}),
-      ...(isTestEnvironment ? { isTestEnvironment } : {}),
-      ...(includeNone ? { includeNone } : {}),
-    },
-    type: 'scheduled',
-  }));
-};
-
-/**
- * Schedules the daily pull (one ScheduleMembersDataPerAction task per action).
- * @param {Object} [options] - Same options as buildDailyPullTasks
- * @returns {Promise<any>}
- */
-const scheduleDailyPullTasks = options =>
-  taskManager().scheduleInBulk(buildDailyPullTasks(options));
-
-module.exports = { buildDailyPullTasks, scheduleDailyPullTasks };