@atlaskit/rovo-triggers 10.38.0 → 11.1.0

package/CHANGELOG.md

@@ -1,5 +1,51 @@
 # @atlaskit/rovo-triggers
 
+## 11.1.0
+
+### Minor Changes
+
+- [`c575d86f82b4a`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/c575d86f82b4a) -
+  Add an opt-in deferred seeded-chat opening lifecycle gated by
+  `jira_rovo_defer_view_chat_open_until_seeded`.
+- [`ee866df2b4a26`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/ee866df2b4a26) -
+  Add optional `statusScope` and `isNewStatus` to the Jira workflow wizard ADD_STATUS payload.
+
+### Patch Changes
+
+- Updated dependencies
+
+## 11.0.0
+
+### Major Changes
+
+- [`de8897276a43e`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/de8897276a43e) -
+  Replace the `rovo-ext_context_bridge` feature gate with the `rovo-ext_context_bridge_exp`
+  experiment, read through its boolean `isEnabled` parameter.
+
+  **Breaking:** `@atlaskit/rovo-triggers/extension-context-bridge/constants` no longer exports
+  `BRIDGE_FEATURE_GATE`. Use `EXT_CONTEXT_BRIDGE_EXPERIMENT` instead, which holds the experiment key
+  rather than the gate key:
+
+  ```diff
+  - import { BRIDGE_FEATURE_GATE } from '@atlaskit/rovo-triggers/extension-context-bridge/constants';
+  + import { EXT_CONTEXT_BRIDGE_EXPERIMENT } from '@atlaskit/rovo-triggers/extension-context-bridge/constants';
+  ```
+
+  `ExtensionContextBridgeHost`, `ExtensionContextBridgeClient` and `ChatOpenerSubscriber` read the
+  experiment through `UNSAFE_expValNoExposure`, so none of them log exposure: the Host mounts at the
+  root of every page of an opted-in product, and exposure there would count product page loads
+  rather than extension users. The Rovo browser extension owns the single exposure call.
+
+  No behaviour change for consumers beyond the assignment source. The
+  `extensionContextBridgeEnabled` prop on `ChatOpenerSubscriber` is unchanged and still required to
+  opt in.
+
+### Minor Changes
+
+- [`487372cfb38b7`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/487372cfb38b7) -
+  Send the `remix_custom_edit` experience, source page ID, and required editor context for editor
+  Custom Remix requests behind the `aifc_remix_custom_page_edit` gate.
+
 ## 10.38.0
 
 ### Minor Changes

package/dist/cjs/common/utils/extension-context-bridge/ExtensionContextBridgeClient.js

@@ -7,7 +7,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.ExtensionContextBridgeClient = void 0;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _react = require("react");
-var _fg = require("@atlaskit/platform-feature-flags/fg");
+var _unsafeExpValNoExposure = require("@atlaskit/platform-feature-experiments/unsafe-exp-val-no-exposure");
 var _main = require("../../../main");
 var _constants = require("./constants");
 var _policy = require("./policy");
@@ -60,8 +60,9 @@ var isChatFocused = function isChatFocused() {
 var ExtensionContextBridgeClient = exports.ExtensionContextBridgeClient = function ExtensionContextBridgeClient(_ref) {
   var transport = _ref.transport,
     allowedOrigins = _ref.allowedOrigins;
-  // Self-gated. Matches `BRIDGE_FEATURE_GATE` in `./constants`.
-  var active = (0, _fg.fg)('rovo-ext_context_bridge');
+  // Self-gated. Matches `EXT_CONTEXT_BRIDGE_EXPERIMENT` in `./constants`. No-exposure read: the
+  // extension decides whether to mount this and owns the exposure call.
+  var active = (0, _unsafeExpValNoExposure.UNSAFE_expValNoExposure)('rovo-ext_context_bridge_exp', 'isEnabled', false) === true;
   var publish = (0, _main.usePublish)('ai-mate');
   var lastKnownSelectionRef = (0, _react.useRef)(null);
   var activeTransport = (0, _react.useMemo)(function () {

package/dist/cjs/common/utils/extension-context-bridge/ExtensionContextBridgeHost.js

@@ -7,7 +7,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.ExtensionContextBridgeHost = void 0;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _react = require("react");
-var _fg = require("@atlaskit/platform-feature-flags/fg");
+var _unsafeExpValNoExposure = require("@atlaskit/platform-feature-experiments/unsafe-exp-val-no-exposure");
 var _main = require("../../../main");
 var _analytics = require("./analytics");
 var _constants = require("./constants");
@@ -56,8 +56,12 @@ var ExtensionContextBridgeHost = exports.ExtensionContextBridgeHost = function E
     isAiEnabled = _ref.isAiEnabled,
     onRelayDecision = _ref.onRelayDecision;
   // Self-gated (not taken from the consumer) so this cannot mount by mistake wherever
-  // `ChatOpenerSubscriber` renders. Matches `BRIDGE_FEATURE_GATE` in `./constants`.
-  var active = (0, _fg.fg)('rovo-ext_context_bridge');
+  // `ChatOpenerSubscriber` renders. Matches `EXT_CONTEXT_BRIDGE_EXPERIMENT` in `./constants`.
+  //
+  // No-exposure read on purpose: this component mounts on every page of an opted-in product, so
+  // firing exposure here would measure product page loads instead of extension users. The
+  // extension logs the one exposure. See `./constants`.
+  var active = (0, _unsafeExpValNoExposure.UNSAFE_expValNoExposure)('rovo-ext_context_bridge_exp', 'isEnabled', false) === true;
   var activeTransport = (0, _react.useMemo)(function () {
     if (!active) {
       return undefined;

package/dist/cjs/common/utils/extension-context-bridge/constants.js

@@ -3,13 +3,19 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.BRIDGE_TO_PRODUCT = exports.BRIDGE_TO_EXTENSION = exports.BRIDGE_SOURCE = exports.BRIDGE_MESSAGE_MARKER = exports.BRIDGE_FEATURE_GATE = exports.BRIDGE_CONTROL_REQUEST_CONTEXT = void 0;
+exports.EXT_CONTEXT_BRIDGE_EXPERIMENT = exports.BRIDGE_TO_PRODUCT = exports.BRIDGE_TO_EXTENSION = exports.BRIDGE_SOURCE = exports.BRIDGE_MESSAGE_MARKER = exports.BRIDGE_CONTROL_REQUEST_CONTEXT = void 0;
 /**
- * Feature gate for the extension context bridge; both Host and Client self-gate on it.
- * `fg()` requires a static literal, so call sites inline this value rather than importing it;
- * tests resolve the gate through this constant so the two cannot drift apart.
+ * Statsig experiment for the extension context bridge; both Host and Client self-gate on its
+ * boolean `isEnabled` parameter. Call sites inline this value rather than importing it, since the
+ * experiment runtime takes a static literal; tests resolve the experiment through this constant so
+ * the two cannot drift apart.
+ *
+ * Read here **without** logging exposure. Host mounts at the root of every page of an opted-in
+ * product, so an exposure-firing read would count product page loads rather than extension users.
+ * The extension owns the single exposure call, at the point the user is on a page the bridge can
+ * act on.
  */
-var BRIDGE_FEATURE_GATE = exports.BRIDGE_FEATURE_GATE = 'rovo-ext_context_bridge';
+var EXT_CONTEXT_BRIDGE_EXPERIMENT = exports.EXT_CONTEXT_BRIDGE_EXPERIMENT = 'rovo-ext_context_bridge_exp';
 
 /** Marker on every bridge `postMessage`, so listeners can ignore unrelated messages. */
 var BRIDGE_MESSAGE_MARKER = exports.BRIDGE_MESSAGE_MARKER = '__rovoExtContextBridge';

package/dist/es2019/common/utils/extension-context-bridge/ExtensionContextBridgeClient.js

@@ -1,5 +1,5 @@
 import { useEffect, useMemo, useRef } from 'react';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
+import { UNSAFE_expValNoExposure } from '@atlaskit/platform-feature-experiments/unsafe-exp-val-no-exposure';
 import { usePublish, useSubscribeAll } from '../../../main';
 import { BRIDGE_CONTROL_REQUEST_CONTEXT, BRIDGE_MESSAGE_MARKER, BRIDGE_SOURCE, BRIDGE_TO_EXTENSION, BRIDGE_TO_PRODUCT } from './constants';
 import { shouldRelayToProduct } from './policy';
@@ -52,8 +52,9 @@ export const ExtensionContextBridgeClient = ({
   transport,
   allowedOrigins
 }) => {
-  // Self-gated. Matches `BRIDGE_FEATURE_GATE` in `./constants`.
-  const active = fg('rovo-ext_context_bridge');
+  // Self-gated. Matches `EXT_CONTEXT_BRIDGE_EXPERIMENT` in `./constants`. No-exposure read: the
+  // extension decides whether to mount this and owns the exposure call.
+  const active = UNSAFE_expValNoExposure('rovo-ext_context_bridge_exp', 'isEnabled', false) === true;
   const publish = usePublish('ai-mate');
   const lastKnownSelectionRef = useRef(null);
   const activeTransport = useMemo(() => {

package/dist/es2019/common/utils/extension-context-bridge/ExtensionContextBridgeHost.js

@@ -1,5 +1,5 @@
 import { useEffect, useMemo, useRef } from 'react';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
+import { UNSAFE_expValNoExposure } from '@atlaskit/platform-feature-experiments/unsafe-exp-val-no-exposure';
 import { useSubscribeAll } from '../../../main';
 import { getContextKey, reportRelayDecision } from './analytics';
 import { BRIDGE_CONTROL_REQUEST_CONTEXT, BRIDGE_MESSAGE_MARKER, BRIDGE_SOURCE, BRIDGE_TO_EXTENSION, BRIDGE_TO_PRODUCT } from './constants';
@@ -49,8 +49,12 @@ export const ExtensionContextBridgeHost = ({
   onRelayDecision
 }) => {
   // Self-gated (not taken from the consumer) so this cannot mount by mistake wherever
-  // `ChatOpenerSubscriber` renders. Matches `BRIDGE_FEATURE_GATE` in `./constants`.
-  const active = fg('rovo-ext_context_bridge');
+  // `ChatOpenerSubscriber` renders. Matches `EXT_CONTEXT_BRIDGE_EXPERIMENT` in `./constants`.
+  //
+  // No-exposure read on purpose: this component mounts on every page of an opted-in product, so
+  // firing exposure here would measure product page loads instead of extension users. The
+  // extension logs the one exposure. See `./constants`.
+  const active = UNSAFE_expValNoExposure('rovo-ext_context_bridge_exp', 'isEnabled', false) === true;
   const activeTransport = useMemo(() => {
     if (!active) {
       return undefined;

package/dist/es2019/common/utils/extension-context-bridge/constants.js

@@ -1,9 +1,15 @@
 /**
- * Feature gate for the extension context bridge; both Host and Client self-gate on it.
- * `fg()` requires a static literal, so call sites inline this value rather than importing it;
- * tests resolve the gate through this constant so the two cannot drift apart.
+ * Statsig experiment for the extension context bridge; both Host and Client self-gate on its
+ * boolean `isEnabled` parameter. Call sites inline this value rather than importing it, since the
+ * experiment runtime takes a static literal; tests resolve the experiment through this constant so
+ * the two cannot drift apart.
+ *
+ * Read here **without** logging exposure. Host mounts at the root of every page of an opted-in
+ * product, so an exposure-firing read would count product page loads rather than extension users.
+ * The extension owns the single exposure call, at the point the user is on a page the bridge can
+ * act on.
  */
-export const BRIDGE_FEATURE_GATE = 'rovo-ext_context_bridge';
+export const EXT_CONTEXT_BRIDGE_EXPERIMENT = 'rovo-ext_context_bridge_exp';
 
 /** Marker on every bridge `postMessage`, so listeners can ignore unrelated messages. */
 export const BRIDGE_MESSAGE_MARKER = '__rovoExtContextBridge';

package/dist/esm/common/utils/extension-context-bridge/ExtensionContextBridgeClient.js

@@ -2,7 +2,7 @@ import _defineProperty from "@babel/runtime/helpers/defineProperty";
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 import { useEffect, useMemo, useRef } from 'react';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
+import { UNSAFE_expValNoExposure } from '@atlaskit/platform-feature-experiments/unsafe-exp-val-no-exposure';
 import { usePublish, useSubscribeAll } from '../../../main';
 import { BRIDGE_CONTROL_REQUEST_CONTEXT, BRIDGE_MESSAGE_MARKER, BRIDGE_SOURCE, BRIDGE_TO_EXTENSION, BRIDGE_TO_PRODUCT } from './constants';
 import { shouldRelayToProduct } from './policy';
@@ -54,8 +54,9 @@ var isChatFocused = function isChatFocused() {
 export var ExtensionContextBridgeClient = function ExtensionContextBridgeClient(_ref) {
   var transport = _ref.transport,
     allowedOrigins = _ref.allowedOrigins;
-  // Self-gated. Matches `BRIDGE_FEATURE_GATE` in `./constants`.
-  var active = fg('rovo-ext_context_bridge');
+  // Self-gated. Matches `EXT_CONTEXT_BRIDGE_EXPERIMENT` in `./constants`. No-exposure read: the
+  // extension decides whether to mount this and owns the exposure call.
+  var active = UNSAFE_expValNoExposure('rovo-ext_context_bridge_exp', 'isEnabled', false) === true;
   var publish = usePublish('ai-mate');
   var lastKnownSelectionRef = useRef(null);
   var activeTransport = useMemo(function () {

package/dist/esm/common/utils/extension-context-bridge/ExtensionContextBridgeHost.js

@@ -1,6 +1,6 @@
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import { useEffect, useMemo, useRef } from 'react';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
+import { UNSAFE_expValNoExposure } from '@atlaskit/platform-feature-experiments/unsafe-exp-val-no-exposure';
 import { useSubscribeAll } from '../../../main';
 import { getContextKey, reportRelayDecision } from './analytics';
 import { BRIDGE_CONTROL_REQUEST_CONTEXT, BRIDGE_MESSAGE_MARKER, BRIDGE_SOURCE, BRIDGE_TO_EXTENSION, BRIDGE_TO_PRODUCT } from './constants';
@@ -49,8 +49,12 @@ export var ExtensionContextBridgeHost = function ExtensionContextBridgeHost(_ref
     isAiEnabled = _ref.isAiEnabled,
     onRelayDecision = _ref.onRelayDecision;
   // Self-gated (not taken from the consumer) so this cannot mount by mistake wherever
-  // `ChatOpenerSubscriber` renders. Matches `BRIDGE_FEATURE_GATE` in `./constants`.
-  var active = fg('rovo-ext_context_bridge');
+  // `ChatOpenerSubscriber` renders. Matches `EXT_CONTEXT_BRIDGE_EXPERIMENT` in `./constants`.
+  //
+  // No-exposure read on purpose: this component mounts on every page of an opted-in product, so
+  // firing exposure here would measure product page loads instead of extension users. The
+  // extension logs the one exposure. See `./constants`.
+  var active = UNSAFE_expValNoExposure('rovo-ext_context_bridge_exp', 'isEnabled', false) === true;
   var activeTransport = useMemo(function () {
     if (!active) {
       return undefined;

package/dist/esm/common/utils/extension-context-bridge/constants.js

@@ -1,9 +1,15 @@
 /**
- * Feature gate for the extension context bridge; both Host and Client self-gate on it.
- * `fg()` requires a static literal, so call sites inline this value rather than importing it;
- * tests resolve the gate through this constant so the two cannot drift apart.
+ * Statsig experiment for the extension context bridge; both Host and Client self-gate on its
+ * boolean `isEnabled` parameter. Call sites inline this value rather than importing it, since the
+ * experiment runtime takes a static literal; tests resolve the experiment through this constant so
+ * the two cannot drift apart.
+ *
+ * Read here **without** logging exposure. Host mounts at the root of every page of an opted-in
+ * product, so an exposure-firing read would count product page loads rather than extension users.
+ * The extension owns the single exposure call, at the point the user is on a page the bridge can
+ * act on.
  */
-export var BRIDGE_FEATURE_GATE = 'rovo-ext_context_bridge';
+export var EXT_CONTEXT_BRIDGE_EXPERIMENT = 'rovo-ext_context_bridge_exp';
 
 /** Marker on every bridge `postMessage`, so listeners can ignore unrelated messages. */
 export var BRIDGE_MESSAGE_MARKER = '__rovoExtContextBridge';

package/dist/types/common/utils/extension-context-bridge/constants.d.ts

@@ -1,9 +1,15 @@
 /**
- * Feature gate for the extension context bridge; both Host and Client self-gate on it.
- * `fg()` requires a static literal, so call sites inline this value rather than importing it;
- * tests resolve the gate through this constant so the two cannot drift apart.
+ * Statsig experiment for the extension context bridge; both Host and Client self-gate on its
+ * boolean `isEnabled` parameter. Call sites inline this value rather than importing it, since the
+ * experiment runtime takes a static literal; tests resolve the experiment through this constant so
+ * the two cannot drift apart.
+ *
+ * Read here **without** logging exposure. Host mounts at the root of every page of an opted-in
+ * product, so an exposure-firing read would count product page loads rather than extension users.
+ * The extension owns the single exposure call, at the point the user is on a page the bridge can
+ * act on.
  */
-export declare const BRIDGE_FEATURE_GATE = "rovo-ext_context_bridge";
+export declare const EXT_CONTEXT_BRIDGE_EXPERIMENT = "rovo-ext_context_bridge_exp";
 /** Marker on every bridge `postMessage`, so listeners can ignore unrelated messages. */
 export declare const BRIDGE_MESSAGE_MARKER: '__rovoExtContextBridge';
 /** Read context flows product -> extension; actions flow extension -> product. */

package/dist/types/types.d.ts

@@ -100,7 +100,7 @@ export type ChatCreationContextParams = {
     source?: string;
     shouldUseExistingContent?: boolean;
     isViewMode?: boolean;
-    experience?: 'cwr' | 'cwr_type' | 'cwr_edit' | 'cwr_existing' | 'inline_edit' | 'remix' | 'remix_edit' | 'remix_object' | 'remix_custom' | 'chat_edit' | 'chat_view' | 'keep_existing_page_structure';
+    experience?: 'cwr' | 'cwr_type' | 'cwr_edit' | 'cwr_existing' | 'inline_edit' | 'remix' | 'remix_edit' | 'remix_object' | 'remix_custom' | 'remix_custom_edit' | 'chat_edit' | 'chat_view' | 'keep_existing_page_structure';
     contentTypes?: string[];
     blocks?: string[];
     spaceKey?: string;
@@ -137,6 +137,11 @@ export type ChatNewPayload = PayloadCore<'chat-new', {
     rovoJourneyId?: string;
     minionAlias?: string;
     skipCreatingSeededConversation?: boolean;
+    /**
+     * Opens the Rovo shell with loading while the seeded conversation is created, then selects
+     * that conversation once ready. Has no effect when `skipCreatingSeededConversation` is true.
+     */
+    deferOpenUntilSeeded?: boolean;
     selectedSpace?: {
         id: string;
         name: string;
@@ -487,6 +492,8 @@ export type AddStatusRovoPayload = {
     statusId: StatusId;
     statusName: string;
     statusCategory: StatusCategory;
+    statusScope?: 'GLOBAL' | 'PROJECT';
+    isNewStatus?: boolean;
 };
 export type UpdateStatusRovoPayload = {
     statusId: StatusId;

package/package.json

@@ -1,12 +1,17 @@
 {
 	"name": "@atlaskit/rovo-triggers",
-	"version": "10.38.0",
+	"version": "11.1.0",
 	"description": "Provides various trigger events to drive Rovo Chat functionality, such as a publish-subscribe and URL parameter hooks",
 	"author": "Atlassian Pty Ltd",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
 	},
 	"atlassian": {
+		"website": {
+			"tags": [
+				"platform-labs"
+			]
+		},
 		"react-compiler": {
 			"enabled": true,
 			"gating": {
@@ -28,7 +33,8 @@
 	],
 	"atlaskit:src": "src/index.ts",
 	"dependencies": {
-		"@atlaskit/adf-schema": "^57.4.0",
+		"@atlaskit/adf-schema": "^57.5.0",
+		"@atlaskit/platform-feature-experiments": "^0.3.0",
 		"@atlaskit/platform-feature-flags": "^2.2.0",
 		"@atlaskit/react-compiler-gating": "^0.2.0",
 		"@babel/runtime": "^7.0.0",
@@ -44,7 +50,8 @@
 		"@atlaskit/primitives": "^22.5.0",
 		"@atlaskit/ssr": "workspace:^",
 		"@atlassian/a11y-jest-testing": "^0.17.0",
-		"@atlassian/conversation-assistant-service-api": "^17.13.0",
+		"@atlassian/conversation-assistant-service-api": "^19.1.0",
+		"@atlassian/experiment-test-utils": "^0.2.0",
 		"@atlassian/feature-flags-test-utils": "^1.3.0",
 		"@atlassian/testing-library": "^0.11.0",
 		"@atlassian/workbench": "workspace:*",
@@ -54,9 +61,6 @@
 		"wait-for-expect": "^1.2.0"
 	},
 	"platform-feature-flags": {
-		"rovo-ext_context_bridge": {
-			"type": "boolean"
-		},
 		"rovo_chat_replace_url_param_history": {
 			"type": "boolean"
 		}