HubSpot (patch) cache properties per HubSpot portal (#475)

* HubSpot (patch) cache properties per HubSpot portal

* Update src/appmixer/hubspot/commons.js

thanks 🐇

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

---------

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: jirihofman

src/appmixer/hubspot/commons.js

@@ -43,7 +43,9 @@ module.exports = {
         // Default cache TTL set to 1 minute as property definitions rarely change
         // Can be configured via context.config.objectPropertiesCacheTTL if needed
         const objectPropertiesCacheTTL = context.config.objectPropertiesCacheTTL || (60 * 1000);
-        const cacheKeyPrefix = 'hubspot_properties_' + objectType;
+        // Use hub_id from context to differentiate between different HubSpot portals/users.
+        const portalId = context.auth?.profileInfo?.hub_id || 'default';
+        const cacheKeyPrefix = 'hubspot_properties_' + objectType + '_' + portalId;
         let lock;
         try {
             lock = await context.lock(`hubspot_properties_${objectType}`);

test/hubspot/commons.test.js

@@ -12,6 +12,7 @@ describe('commons.js', () => {
 
         // Reset the context.
         context = testUtils.createMockContext();
+        context.auth = { profileInfo: { hub_id: '123456' } };
         context.messages = { in: { content: {} } };
         context.config = {
             objectPropertiesCacheTTL: 1
@@ -57,5 +58,9 @@ describe('commons.js', () => {
         assert(properties.length > 0);
         assert.equal(properties[0].label, 'Jirka Notes [custom]', 'Custom field should have [custom] suffix');
         assert.equal(properties[1].label, 'Job Title', 'Non custom field should not have [custom] suffix');
+
+        // Should use `hub_id` from context to differentiate between different HubSpot portals/users.
+        const setCacheArgs = context.staticCache.set.getCall(0).args;
+        assert.equal(setCacheArgs[0], 'hubspot_properties_contacts_123456_all', 'Cache key should contain hub_id');
     });
 });