@atlaskit/collab-provider 39.0.6 → 39.1.0

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # @atlaskit/collab-provider
 
+## 39.1.0
+
+### Minor Changes
+
+- [`88f6eda39f497`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/88f6eda39f497) -
+  Resolve agent telepointer colours from supplied display names and stable named IDs instead of
+  machine types. Under `confluence_ncs_step_diffing_version_history`, collab-provider carries
+  `agentId`, `agentName` and `agentNamedId` from received steps into local agent presence, retaining
+  known metadata when later steps omit it. This path uses no profile lookups. Missing presentation
+  metadata uses the generic agent palette; human participants keep their existing colours.
+
+  Telepointers identify agent participants from these explicit identity fields rather than
+  `agentType` or a human participant's display name.
+
+  `getParticipantColor(id, agentName?, agentNamedId?)` and
+  `collabEdit.actions.getAvatarColor(id, agentName?, agentNamedId?)` now accept a resolved display
+  name in the second argument. Callers previously passing a machine type should supply the agent's
+  name instead, and pass stable named IDs in the third argument. With
+  `confluence_ncs_step_diffing_version_history` enabled, named brand metadata takes precedence over
+  the display name. With the gate off, brand lookup is disabled. Human callers should continue to
+  omit agent metadata.
+
+### Patch Changes
+
+- Updated dependencies
+
+## 39.0.7
+
+### Patch Changes
+
+- [`0f850e01177aa`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/0f850e01177aa) -
+  Remove the abandoned collab_edit_via_websocket_only_for_gcp feature gate and its WebSocket-only
+  transport path.
+- Updated dependencies
+
 ## 39.0.6
 
 ### Patch Changes

package/dist/cjs/document/document-service.js

@@ -971,7 +971,7 @@ var DocumentService = exports.DocumentService = /*#__PURE__*/function () {
       }
       try {
         var _this$analyticsHelper34;
-        var agentTypeByProviderId = new Map();
+        var agentsByProviderId = new Map();
         var agentIds = new Set();
         var agentTypes = new Set();
         var agentStepCount = 0;
@@ -985,11 +985,19 @@ var DocumentService = exports.DocumentService = /*#__PURE__*/function () {
               continue;
             }
             agentStepCount++;
-            var _ref10 = step,
-              agentId = _ref10.agentId,
-              agentType = _ref10.agentType;
+            var _ref11 = step,
+              agentId = _ref11.agentId,
+              agentType = _ref11.agentType,
+              agentName = _ref11.agentName,
+              agentNamedId = _ref11.agentNamedId;
             if (agentType) {
-              agentTypeByProviderId.set(providerId, agentType);
+              var previous = agentsByProviderId.get(providerId);
+              agentsByProviderId.set(providerId, {
+                agentType: agentType,
+                agentId: (agentId ? (0, _utils.normalizeAgentId)(agentId.trim()) : undefined) || (previous === null || previous === void 0 ? void 0 : previous.agentId),
+                agentName: (agentName === null || agentName === void 0 ? void 0 : agentName.trim()) || (previous === null || previous === void 0 ? void 0 : previous.agentName),
+                agentNamedId: (agentNamedId === null || agentNamedId === void 0 ? void 0 : agentNamedId.trim()) || (previous === null || previous === void 0 ? void 0 : previous.agentNamedId)
+              });
               agentTypes.add(agentType);
             }
             if (agentId) {
@@ -1002,7 +1010,7 @@ var DocumentService = exports.DocumentService = /*#__PURE__*/function () {
         } finally {
           _iterator.f();
         }
-        if (agentTypeByProviderId.size === 0) {
+        if (agentsByProviderId.size === 0) {
           return;
         }
         // Anchor signal that the agent-presence logic kicked off for a received transaction. Fires
@@ -1010,13 +1018,21 @@ var DocumentService = exports.DocumentService = /*#__PURE__*/function () {
         // non-PII: agent kinds, actor ids, and counts — never document content.
         (_this$analyticsHelper34 = this.analyticsHelper) === null || _this$analyticsHelper34 === void 0 || _this$analyticsHelper34.sendActionEvent(_const.EVENT_ACTION.AGENT_EDIT_RECEIVED, _const.EVENT_STATUS.SUCCESS, {
           agentIds: (0, _toConsumableArray2.default)(agentIds),
-          agentCount: agentTypeByProviderId.size,
+          agentCount: agentsByProviderId.size,
           agentTypes: (0, _toConsumableArray2.default)(agentTypes),
           agentStepCount: agentStepCount,
           totalStepCount: steps.length
         });
-        agentTypeByProviderId.forEach(function (agentType, providerId) {
-          return _this3.participantsService.upsertAIProviderParticipantLocally(providerId, agentType);
+        agentsByProviderId.forEach(function (_ref10, providerId) {
+          var agentType = _ref10.agentType,
+            agentId = _ref10.agentId,
+            agentName = _ref10.agentName,
+            agentNamedId = _ref10.agentNamedId;
+          return _this3.participantsService.upsertAIProviderParticipantLocally(providerId, agentType, (0, _fg.fg)('confluence_ncs_step_diffing_version_history') ? {
+            agentId: agentId,
+            agentName: agentName,
+            agentNamedId: agentNamedId
+          } : undefined);
         });
       } catch (error) {
         var _this$analyticsHelper35;
@@ -1044,10 +1060,10 @@ var DocumentService = exports.DocumentService = /*#__PURE__*/function () {
      * @param root0.clientId
      * @example
      */
-    function setup(_ref11) {
-      var getState = _ref11.getState,
-        onSyncUpError = _ref11.onSyncUpError,
-        clientId = _ref11.clientId;
+    function setup(_ref12) {
+      var getState = _ref12.getState,
+        onSyncUpError = _ref12.onSyncUpError,
+        clientId = _ref12.clientId;
       this.getState = getState;
       this.onSyncUpError = onSyncUpError || noop;
       this.clientId = clientId;

package/dist/cjs/participants/participants-service.js

@@ -138,32 +138,41 @@ var ParticipantsService = exports.ParticipantsService = /*#__PURE__*/function ()
      *
      * Deliberately does NOT go through the `getUser` hydration path: agents have no human profile,
      * so `getUser('agent:<id>')` would 404/throw (and could drop the participant) or add latency;
-     * the agent's display identity is resolved downstream in the facepile.
+     * supplied step presentation can be used directly; avatars are resolved downstream.
      *
-     * `presence` is emitted only on the FIRST add for an agent. Subsequent steps for the same agent
-     * refresh `lastActive` and reset the sliding inactivity timer silently, to avoid churning every
-     * presence consumer on each streamed step.
+     * Unchanged active agents refresh `lastActive` and the inactivity timer silently. Emit presence
+     * when an agent is added or its supplied identity changes, so consumers get the latest metadata.
      *
      * @param providerId `agent:<aaid|type>` id of the agent
      * @param agentType kind of the agent that authored the step
+     * @param identity optional display metadata already supplied by the agent's steps
      */
-    (0, _defineProperty2.default)(this, "upsertAIProviderParticipantLocally", function (providerId, agentType) {
+    (0, _defineProperty2.default)(this, "upsertAIProviderParticipantLocally", function (providerId, agentType, identity) {
+      var _identity$agentId, _identity$agentName, _identity$agentNamedI;
       var payload = _this.buildAIProviderPresencePayload(providerId, agentType);
       var sessionId = payload.sessionId;
       var existing = _this.participantsState.getBySessionId(sessionId);
       var hasAgentTypeChanged = (existing === null || existing === void 0 ? void 0 : existing.agentType) !== agentType;
+      var agentId = (0, _fg.fg)('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 || (_identity$agentId = identity.agentId) === null || _identity$agentId === void 0 ? void 0 : _identity$agentId.trim()) || (existing === null || existing === void 0 ? void 0 : existing.agentId) : undefined;
+      var name = (0, _fg.fg)('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 || (_identity$agentName = identity.agentName) === null || _identity$agentName === void 0 ? void 0 : _identity$agentName.trim()) || (existing === null || existing === void 0 ? void 0 : existing.name) || '' : '';
+      var agentNamedId = (0, _fg.fg)('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 || (_identity$agentNamedI = identity.agentNamedId) === null || _identity$agentNamedI === void 0 ? void 0 : _identity$agentNamedI.trim()) || (existing === null || existing === void 0 ? void 0 : existing.agentNamedId) : undefined;
       var wasInactive = existing ? Date.now() - existing.lastActive >= AGENT_PRESENCE_INACTIVE_MS : false;
-      var participant = _objectSpread(_objectSpread(_objectSpread({}, payload), (0, _fg.fg)('platform_move_presence_agents') ? {
+      var participant = _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({}, payload), (0, _fg.fg)('platform_move_presence_agents') ? {
         presenceActivity: 'editor'
       } : {}), {}, {
         // `payload.userId` is typed `string | undefined`; here it is always the (string)
         // providerId, so pin it to satisfy ProviderParticipant.userId (string).
         userId: providerId,
         lastActive: payload.timestamp,
-        // name/avatar/email are intentionally empty: an agent has no human profile here. Its display
-        // identity (name + avatar) is resolved downstream in the facepile from the agent id, so the
-        // provider stays identity-free (no getUser/assistance lookup on this hot path).
-        name: '',
+        // Use presentation already supplied on the step; never fetch a human profile on this path.
+        name: name
+      }, agentId ? {
+        agentId: agentId
+      } : {}), name ? {
+        agentName: name
+      } : {}), agentNamedId ? {
+        agentNamedId: agentNamedId
+      } : {}), {}, {
         avatar: '',
         email: ''
       });
@@ -171,7 +180,7 @@ var ParticipantsService = exports.ParticipantsService = /*#__PURE__*/function ()
 
       // Active refreshes stay silent. Re-emit when an inactive agent becomes active so consumers
       // can update its presentation immediately.
-      if (!existing || hasAgentTypeChanged || (0, _fg.fg)('platform_move_presence_agents') && wasInactive) {
+      if (!existing || hasAgentTypeChanged || existing.name !== name || existing.agentId !== agentId || existing.agentName !== (name || undefined) || existing.agentNamedId !== agentNamedId || (0, _fg.fg)('platform_move_presence_agents') && wasInactive) {
         _this.emitPresence({
           joined: [participant]
         }, 'adding agent participant from remote step');

package/dist/cjs/socket-io-provider.js

@@ -56,8 +56,6 @@ function createSocketIOSocket(url, auth, productInfo, isPresenceOnly, analyticsH
       if ((0, _fg.fg)('platform_editor_gcp_collab_ws_first')) {
         transports = ['websocket', 'polling'];
         webSocketFirst = true;
-      } else if ((0, _fg.fg)('collab_edit_via_websocket_only_for_gcp')) {
-        transports = ['websocket'];
       }
       break;
     default:

package/dist/cjs/version-wrapper.js

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.version = exports.nextMajorVersion = exports.name = void 0;
 var name = exports.name = "@atlaskit/collab-provider";
-var version = exports.version = "39.0.5";
+var version = exports.version = "39.0.7";
 var nextMajorVersion = exports.nextMajorVersion = function nextMajorVersion() {
   return [Number(version.split('.')[0]) + 1, 0, 0].join('.');
 };

package/dist/es2019/document/document-service.js

@@ -847,7 +847,7 @@ export class DocumentService {
     }
     try {
       var _this$analyticsHelper34;
-      const agentTypeByProviderId = new Map();
+      const agentsByProviderId = new Map();
       const agentIds = new Set();
       const agentTypes = new Set();
       let agentStepCount = 0;
@@ -859,10 +859,18 @@ export class DocumentService {
         agentStepCount++;
         const {
           agentId,
-          agentType
+          agentType,
+          agentName,
+          agentNamedId
         } = step;
         if (agentType) {
-          agentTypeByProviderId.set(providerId, agentType);
+          const previous = agentsByProviderId.get(providerId);
+          agentsByProviderId.set(providerId, {
+            agentType,
+            agentId: (agentId ? normalizeAgentId(agentId.trim()) : undefined) || (previous === null || previous === void 0 ? void 0 : previous.agentId),
+            agentName: (agentName === null || agentName === void 0 ? void 0 : agentName.trim()) || (previous === null || previous === void 0 ? void 0 : previous.agentName),
+            agentNamedId: (agentNamedId === null || agentNamedId === void 0 ? void 0 : agentNamedId.trim()) || (previous === null || previous === void 0 ? void 0 : previous.agentNamedId)
+          });
           agentTypes.add(agentType);
         }
         if (agentId) {
@@ -870,7 +878,7 @@ export class DocumentService {
         }
       }
       // No agent-authored steps in this batch — nothing to register or report.
-      if (agentTypeByProviderId.size === 0) {
+      if (agentsByProviderId.size === 0) {
         return;
       }
       // Anchor signal that the agent-presence logic kicked off for a received transaction. Fires
@@ -878,12 +886,21 @@ export class DocumentService {
       // non-PII: agent kinds, actor ids, and counts — never document content.
       (_this$analyticsHelper34 = this.analyticsHelper) === null || _this$analyticsHelper34 === void 0 ? void 0 : _this$analyticsHelper34.sendActionEvent(EVENT_ACTION.AGENT_EDIT_RECEIVED, EVENT_STATUS.SUCCESS, {
         agentIds: [...agentIds],
-        agentCount: agentTypeByProviderId.size,
+        agentCount: agentsByProviderId.size,
         agentTypes: [...agentTypes],
         agentStepCount,
         totalStepCount: steps.length
       });
-      agentTypeByProviderId.forEach((agentType, providerId) => this.participantsService.upsertAIProviderParticipantLocally(providerId, agentType));
+      agentsByProviderId.forEach(({
+        agentType,
+        agentId,
+        agentName,
+        agentNamedId
+      }, providerId) => this.participantsService.upsertAIProviderParticipantLocally(providerId, agentType, fg('confluence_ncs_step_diffing_version_history') ? {
+        agentId,
+        agentName,
+        agentNamedId
+      } : undefined));
     } catch (error) {
       var _this$analyticsHelper35;
       (_this$analyticsHelper35 = this.analyticsHelper) === null || _this$analyticsHelper35 === void 0 ? void 0 : _this$analyticsHelper35.sendErrorEvent(error, 'Error while adding agent presence from steps');

package/dist/es2019/participants/participants-service.js

@@ -123,22 +123,26 @@ export class ParticipantsService {
      *
      * Deliberately does NOT go through the `getUser` hydration path: agents have no human profile,
      * so `getUser('agent:<id>')` would 404/throw (and could drop the participant) or add latency;
-     * the agent's display identity is resolved downstream in the facepile.
+     * supplied step presentation can be used directly; avatars are resolved downstream.
      *
-     * `presence` is emitted only on the FIRST add for an agent. Subsequent steps for the same agent
-     * refresh `lastActive` and reset the sliding inactivity timer silently, to avoid churning every
-     * presence consumer on each streamed step.
+     * Unchanged active agents refresh `lastActive` and the inactivity timer silently. Emit presence
+     * when an agent is added or its supplied identity changes, so consumers get the latest metadata.
      *
      * @param providerId `agent:<aaid|type>` id of the agent
      * @param agentType kind of the agent that authored the step
+     * @param identity optional display metadata already supplied by the agent's steps
      */
-    _defineProperty(this, "upsertAIProviderParticipantLocally", (providerId, agentType) => {
+    _defineProperty(this, "upsertAIProviderParticipantLocally", (providerId, agentType, identity) => {
+      var _identity$agentId, _identity$agentName, _identity$agentNamedI;
       const payload = this.buildAIProviderPresencePayload(providerId, agentType);
       const {
         sessionId
       } = payload;
       const existing = this.participantsState.getBySessionId(sessionId);
       const hasAgentTypeChanged = (existing === null || existing === void 0 ? void 0 : existing.agentType) !== agentType;
+      const agentId = fg('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 ? void 0 : (_identity$agentId = identity.agentId) === null || _identity$agentId === void 0 ? void 0 : _identity$agentId.trim()) || (existing === null || existing === void 0 ? void 0 : existing.agentId) : undefined;
+      const name = fg('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 ? void 0 : (_identity$agentName = identity.agentName) === null || _identity$agentName === void 0 ? void 0 : _identity$agentName.trim()) || (existing === null || existing === void 0 ? void 0 : existing.name) || '' : '';
+      const agentNamedId = fg('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 ? void 0 : (_identity$agentNamedI = identity.agentNamedId) === null || _identity$agentNamedI === void 0 ? void 0 : _identity$agentNamedI.trim()) || (existing === null || existing === void 0 ? void 0 : existing.agentNamedId) : undefined;
       const wasInactive = existing ? Date.now() - existing.lastActive >= AGENT_PRESENCE_INACTIVE_MS : false;
       const participant = {
         ...payload,
@@ -149,10 +153,17 @@ export class ParticipantsService {
         // providerId, so pin it to satisfy ProviderParticipant.userId (string).
         userId: providerId,
         lastActive: payload.timestamp,
-        // name/avatar/email are intentionally empty: an agent has no human profile here. Its display
-        // identity (name + avatar) is resolved downstream in the facepile from the agent id, so the
-        // provider stays identity-free (no getUser/assistance lookup on this hot path).
-        name: '',
+        // Use presentation already supplied on the step; never fetch a human profile on this path.
+        name,
+        ...(agentId ? {
+          agentId
+        } : {}),
+        ...(name ? {
+          agentName: name
+        } : {}),
+        ...(agentNamedId ? {
+          agentNamedId
+        } : {}),
         avatar: '',
         email: ''
       };
@@ -160,7 +171,7 @@ export class ParticipantsService {
 
       // Active refreshes stay silent. Re-emit when an inactive agent becomes active so consumers
       // can update its presentation immediately.
-      if (!existing || hasAgentTypeChanged || fg('platform_move_presence_agents') && wasInactive) {
+      if (!existing || hasAgentTypeChanged || existing.name !== name || existing.agentId !== agentId || existing.agentName !== (name || undefined) || existing.agentNamedId !== agentNamedId || fg('platform_move_presence_agents') && wasInactive) {
         this.emitPresence({
           joined: [participant]
         }, 'adding agent participant from remote step');

package/dist/es2019/socket-io-provider.js

@@ -45,8 +45,6 @@ export function createSocketIOSocket(url, auth, productInfo, isPresenceOnly, ana
       if (fg('platform_editor_gcp_collab_ws_first')) {
         transports = ['websocket', 'polling'];
         webSocketFirst = true;
-      } else if (fg('collab_edit_via_websocket_only_for_gcp')) {
-        transports = ['websocket'];
       }
       break;
     default:

package/dist/es2019/version-wrapper.js

@@ -1,5 +1,5 @@
 export const name = "@atlaskit/collab-provider";
-export const version = "39.0.5";
+export const version = "39.0.7";
 export const nextMajorVersion = () => {
   return [Number(version.split('.')[0]) + 1, 0, 0].join('.');
 };

package/dist/esm/document/document-service.js

@@ -966,7 +966,7 @@ export var DocumentService = /*#__PURE__*/function () {
       }
       try {
         var _this$analyticsHelper34;
-        var agentTypeByProviderId = new Map();
+        var agentsByProviderId = new Map();
         var agentIds = new Set();
         var agentTypes = new Set();
         var agentStepCount = 0;
@@ -980,11 +980,19 @@ export var DocumentService = /*#__PURE__*/function () {
               continue;
             }
             agentStepCount++;
-            var _ref10 = step,
-              agentId = _ref10.agentId,
-              agentType = _ref10.agentType;
+            var _ref11 = step,
+              agentId = _ref11.agentId,
+              agentType = _ref11.agentType,
+              agentName = _ref11.agentName,
+              agentNamedId = _ref11.agentNamedId;
             if (agentType) {
-              agentTypeByProviderId.set(providerId, agentType);
+              var previous = agentsByProviderId.get(providerId);
+              agentsByProviderId.set(providerId, {
+                agentType: agentType,
+                agentId: (agentId ? normalizeAgentId(agentId.trim()) : undefined) || (previous === null || previous === void 0 ? void 0 : previous.agentId),
+                agentName: (agentName === null || agentName === void 0 ? void 0 : agentName.trim()) || (previous === null || previous === void 0 ? void 0 : previous.agentName),
+                agentNamedId: (agentNamedId === null || agentNamedId === void 0 ? void 0 : agentNamedId.trim()) || (previous === null || previous === void 0 ? void 0 : previous.agentNamedId)
+              });
               agentTypes.add(agentType);
             }
             if (agentId) {
@@ -997,7 +1005,7 @@ export var DocumentService = /*#__PURE__*/function () {
         } finally {
           _iterator.f();
         }
-        if (agentTypeByProviderId.size === 0) {
+        if (agentsByProviderId.size === 0) {
           return;
         }
         // Anchor signal that the agent-presence logic kicked off for a received transaction. Fires
@@ -1005,13 +1013,21 @@ export var DocumentService = /*#__PURE__*/function () {
         // non-PII: agent kinds, actor ids, and counts — never document content.
         (_this$analyticsHelper34 = this.analyticsHelper) === null || _this$analyticsHelper34 === void 0 || _this$analyticsHelper34.sendActionEvent(EVENT_ACTION.AGENT_EDIT_RECEIVED, EVENT_STATUS.SUCCESS, {
           agentIds: _toConsumableArray(agentIds),
-          agentCount: agentTypeByProviderId.size,
+          agentCount: agentsByProviderId.size,
           agentTypes: _toConsumableArray(agentTypes),
           agentStepCount: agentStepCount,
           totalStepCount: steps.length
         });
-        agentTypeByProviderId.forEach(function (agentType, providerId) {
-          return _this3.participantsService.upsertAIProviderParticipantLocally(providerId, agentType);
+        agentsByProviderId.forEach(function (_ref10, providerId) {
+          var agentType = _ref10.agentType,
+            agentId = _ref10.agentId,
+            agentName = _ref10.agentName,
+            agentNamedId = _ref10.agentNamedId;
+          return _this3.participantsService.upsertAIProviderParticipantLocally(providerId, agentType, fg('confluence_ncs_step_diffing_version_history') ? {
+            agentId: agentId,
+            agentName: agentName,
+            agentNamedId: agentNamedId
+          } : undefined);
         });
       } catch (error) {
         var _this$analyticsHelper35;
@@ -1039,10 +1055,10 @@ export var DocumentService = /*#__PURE__*/function () {
      * @param root0.clientId
      * @example
      */
-    function setup(_ref11) {
-      var getState = _ref11.getState,
-        onSyncUpError = _ref11.onSyncUpError,
-        clientId = _ref11.clientId;
+    function setup(_ref12) {
+      var getState = _ref12.getState,
+        onSyncUpError = _ref12.onSyncUpError,
+        clientId = _ref12.clientId;
       this.getState = getState;
       this.onSyncUpError = onSyncUpError || noop;
       this.clientId = clientId;

package/dist/esm/participants/participants-service.js

@@ -131,32 +131,41 @@ export var ParticipantsService = /*#__PURE__*/function () {
      *
      * Deliberately does NOT go through the `getUser` hydration path: agents have no human profile,
      * so `getUser('agent:<id>')` would 404/throw (and could drop the participant) or add latency;
-     * the agent's display identity is resolved downstream in the facepile.
+     * supplied step presentation can be used directly; avatars are resolved downstream.
      *
-     * `presence` is emitted only on the FIRST add for an agent. Subsequent steps for the same agent
-     * refresh `lastActive` and reset the sliding inactivity timer silently, to avoid churning every
-     * presence consumer on each streamed step.
+     * Unchanged active agents refresh `lastActive` and the inactivity timer silently. Emit presence
+     * when an agent is added or its supplied identity changes, so consumers get the latest metadata.
      *
      * @param providerId `agent:<aaid|type>` id of the agent
      * @param agentType kind of the agent that authored the step
+     * @param identity optional display metadata already supplied by the agent's steps
      */
-    _defineProperty(this, "upsertAIProviderParticipantLocally", function (providerId, agentType) {
+    _defineProperty(this, "upsertAIProviderParticipantLocally", function (providerId, agentType, identity) {
+      var _identity$agentId, _identity$agentName, _identity$agentNamedI;
       var payload = _this.buildAIProviderPresencePayload(providerId, agentType);
       var sessionId = payload.sessionId;
       var existing = _this.participantsState.getBySessionId(sessionId);
       var hasAgentTypeChanged = (existing === null || existing === void 0 ? void 0 : existing.agentType) !== agentType;
+      var agentId = fg('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 || (_identity$agentId = identity.agentId) === null || _identity$agentId === void 0 ? void 0 : _identity$agentId.trim()) || (existing === null || existing === void 0 ? void 0 : existing.agentId) : undefined;
+      var name = fg('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 || (_identity$agentName = identity.agentName) === null || _identity$agentName === void 0 ? void 0 : _identity$agentName.trim()) || (existing === null || existing === void 0 ? void 0 : existing.name) || '' : '';
+      var agentNamedId = fg('confluence_ncs_step_diffing_version_history') ? (identity === null || identity === void 0 || (_identity$agentNamedI = identity.agentNamedId) === null || _identity$agentNamedI === void 0 ? void 0 : _identity$agentNamedI.trim()) || (existing === null || existing === void 0 ? void 0 : existing.agentNamedId) : undefined;
       var wasInactive = existing ? Date.now() - existing.lastActive >= AGENT_PRESENCE_INACTIVE_MS : false;
-      var participant = _objectSpread(_objectSpread(_objectSpread({}, payload), fg('platform_move_presence_agents') ? {
+      var participant = _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({}, payload), fg('platform_move_presence_agents') ? {
         presenceActivity: 'editor'
       } : {}), {}, {
         // `payload.userId` is typed `string | undefined`; here it is always the (string)
         // providerId, so pin it to satisfy ProviderParticipant.userId (string).
         userId: providerId,
         lastActive: payload.timestamp,
-        // name/avatar/email are intentionally empty: an agent has no human profile here. Its display
-        // identity (name + avatar) is resolved downstream in the facepile from the agent id, so the
-        // provider stays identity-free (no getUser/assistance lookup on this hot path).
-        name: '',
+        // Use presentation already supplied on the step; never fetch a human profile on this path.
+        name: name
+      }, agentId ? {
+        agentId: agentId
+      } : {}), name ? {
+        agentName: name
+      } : {}), agentNamedId ? {
+        agentNamedId: agentNamedId
+      } : {}), {}, {
         avatar: '',
         email: ''
       });
@@ -164,7 +173,7 @@ export var ParticipantsService = /*#__PURE__*/function () {
 
       // Active refreshes stay silent. Re-emit when an inactive agent becomes active so consumers
       // can update its presentation immediately.
-      if (!existing || hasAgentTypeChanged || fg('platform_move_presence_agents') && wasInactive) {
+      if (!existing || hasAgentTypeChanged || existing.name !== name || existing.agentId !== agentId || existing.agentName !== (name || undefined) || existing.agentNamedId !== agentNamedId || fg('platform_move_presence_agents') && wasInactive) {
         _this.emitPresence({
           joined: [participant]
         }, 'adding agent participant from remote step');

package/dist/esm/socket-io-provider.js

@@ -47,8 +47,6 @@ export function createSocketIOSocket(url, auth, productInfo, isPresenceOnly, ana
       if (fg('platform_editor_gcp_collab_ws_first')) {
         transports = ['websocket', 'polling'];
         webSocketFirst = true;
-      } else if (fg('collab_edit_via_websocket_only_for_gcp')) {
-        transports = ['websocket'];
       }
       break;
     default:

package/dist/esm/version-wrapper.js

@@ -1,5 +1,5 @@
 export var name = "@atlaskit/collab-provider";
-export var version = "39.0.5";
+export var version = "39.0.7";
 export var nextMajorVersion = function nextMajorVersion() {
   return [Number(version.split('.')[0]) + 1, 0, 0].join('.');
 };

package/dist/types/participants/participants-service.d.ts

@@ -1,4 +1,4 @@
-import type { CollabEventPresenceData, CollabTelepointerPayload, CollabPresenceActivityChangePayload, ProviderParticipant, StepJson, UserPermitType } from '@atlaskit/editor-common/collab';
+import type { BaseStepPM, CollabEventPresenceData, CollabTelepointerPayload, CollabPresenceActivityChangePayload, ProviderParticipant, StepJson, UserPermitType } from '@atlaskit/editor-common/collab';
 import type AnalyticsHelper from '../analytics/analytics-helper';
 import type { CollabEventDisconnectedData, ChannelEvent, PresenceData, PresencePayload, TelepointerPayload, FetchAnonymousAsset } from '../types';
 import type { BatchProps, GetUserType } from './participants-helper';
@@ -76,16 +76,16 @@ export declare class ParticipantsService {
      *
      * Deliberately does NOT go through the `getUser` hydration path: agents have no human profile,
      * so `getUser('agent:<id>')` would 404/throw (and could drop the participant) or add latency;
-     * the agent's display identity is resolved downstream in the facepile.
+     * supplied step presentation can be used directly; avatars are resolved downstream.
      *
-     * `presence` is emitted only on the FIRST add for an agent. Subsequent steps for the same agent
-     * refresh `lastActive` and reset the sliding inactivity timer silently, to avoid churning every
-     * presence consumer on each streamed step.
+     * Unchanged active agents refresh `lastActive` and the inactivity timer silently. Emit presence
+     * when an agent is added or its supplied identity changes, so consumers get the latest metadata.
      *
      * @param providerId `agent:<aaid|type>` id of the agent
      * @param agentType kind of the agent that authored the step
+     * @param identity optional display metadata already supplied by the agent's steps
      */
-    upsertAIProviderParticipantLocally: (providerId: string, agentType?: string) => void;
+    upsertAIProviderParticipantLocally: (providerId: string, agentType?: string, identity?: Pick<BaseStepPM, 'agentId' | 'agentName' | 'agentNamedId'>) => void;
     /**
      * (Re)starts the sliding expiry timer for an agent participant. The enriched lifecycle retains
      * agents for 30 seconds active plus five minutes inactive; the legacy lifecycle removes at 30 seconds.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/collab-provider",
-	"version": "39.0.6",
+	"version": "39.1.0",
 	"description": "A provider for collaborative editing.",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -25,7 +25,7 @@
 		"@atlaskit/atlassian-context": "^2.0.0",
 		"@atlaskit/editor-json-transformer": "^10.0.0",
 		"@atlaskit/editor-prosemirror": "^9.0.0",
-		"@atlaskit/feature-gate-js-client": "^7.1.0",
+		"@atlaskit/feature-gate-js-client": "^8.0.0",
 		"@atlaskit/platform-feature-experiments": "^0.3.0",
 		"@atlaskit/platform-feature-flags": "^2.2.0",
 		"@atlaskit/prosemirror-collab": "^1.1.0",
@@ -69,12 +69,9 @@
 		},
 		"platform_editor_presence_ws_restore_after_upgrade": {
 			"type": "boolean"
-		},
-		"collab_edit_via_websocket_only_for_gcp": {
-			"type": "boolean"
 		}
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^134.3.0"
+		"@atlaskit/editor-common": "^134.4.0"
 	}
 }