@atlaskit/app-provider 1.6.0 → 1.7.0

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @atlaskit/app-provider
 
+## 1.7.0
+
+### Minor Changes
+
+- [#105257](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/105257)
+  [`16af4057ef04e`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/16af4057ef04e) -
+  Fixes a race condition with setting the theme. This change is no longer behind a feature flag.
+
+## 1.6.1
+
+### Patch Changes
+
+- [#103996](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/103996)
+  [`e97a60e120280`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/e97a60e120280) -
+  Update dependencies and remove unused internal exports.
+
 ## 1.6.0
 
 ### Minor Changes

package/dist/cjs/app-provider.js

@@ -5,7 +5,6 @@ var _typeof = require("@babel/runtime/helpers/typeof");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.AppProvider = AppProvider;
 exports.default = void 0;
 var _react = _interopRequireWildcard(require("react"));
 var _routerLinkProvider = _interopRequireDefault(require("./router-link-provider"));

package/dist/cjs/router-link-provider/index.js

@@ -8,6 +8,7 @@ exports.default = exports.RouterLinkProviderContext = void 0;
 var _react = _interopRequireWildcard(require("react"));
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof(e) && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
+// eslint-disable-next-line @repo/internal/react/require-jsdoc
 var RouterLinkProviderContext = exports.RouterLinkProviderContext = /*#__PURE__*/(0, _react.createContext)({});
 /**
  * __RouterLinkProvider__

package/dist/cjs/theme-provider.js

@@ -5,7 +5,6 @@ var _typeof = require("@babel/runtime/helpers/typeof");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.ThemeProvider = ThemeProvider;
 exports.UNSAFE_useColorModeForMigration = UNSAFE_useColorModeForMigration;
 exports.default = void 0;
 exports.useColorMode = useColorMode;
@@ -145,20 +144,8 @@ function ThemeProvider(_ref) {
       return _objectSpread(_objectSpread({}, theme), nextTheme);
     });
   }, []);
-  (0, _react.useEffect)(function () {
-    if ((0, _platformFeatureFlags.fg)('platform_dst_fix_set_theme_race')) {
-      return;
-    }
-    (0, _tokens.setGlobalTheme)(_objectSpread(_objectSpread({}, theme), {}, {
-      colorMode: reconciledColorMode
-    }));
-  }, [theme, reconciledColorMode]);
   var lastSetGlobalThemePromiseRef = (0, _react.useRef)(null);
   (0, _react.useEffect)(function () {
-    if (!(0, _platformFeatureFlags.fg)('platform_dst_fix_set_theme_race')) {
-      return;
-    }
-
     /**
      * We need to wait for any previous `setGlobalTheme` calls to finish before calling it again.
      * This is to prevent race conditions as `setGlobalTheme` is async and mutates the DOM (e.g. sets the

package/dist/es2019/app-provider.js

@@ -9,7 +9,7 @@ const InsideAppProviderContext = /*#__PURE__*/createContext(false);
  *
  * Place it at the root of your application.
  */
-export function AppProvider({
+function AppProvider({
   children,
   defaultColorMode = 'light',
   defaultTheme,

package/dist/es2019/router-link-provider/index.js

@@ -1,4 +1,5 @@
 import React, { createContext } from 'react';
+// eslint-disable-next-line @repo/internal/react/require-jsdoc
 export const RouterLinkProviderContext = /*#__PURE__*/createContext({});
 /**
  * __RouterLinkProvider__

package/dist/es2019/theme-provider.js

@@ -94,7 +94,7 @@ function getReconciledColorMode(colorMode) {
  *
  * @internal
  */
-export function ThemeProvider({
+function ThemeProvider({
   children,
   defaultColorMode,
   defaultTheme
@@ -115,21 +115,8 @@ export function ThemeProvider({
       ...nextTheme
     }));
   }, []);
-  useEffect(() => {
-    if (fg('platform_dst_fix_set_theme_race')) {
-      return;
-    }
-    setGlobalTheme({
-      ...theme,
-      colorMode: reconciledColorMode
-    });
-  }, [theme, reconciledColorMode]);
   const lastSetGlobalThemePromiseRef = useRef(null);
   useEffect(() => {
-    if (!fg('platform_dst_fix_set_theme_race')) {
-      return;
-    }
-
     /**
      * We need to wait for any previous `setGlobalTheme` calls to finish before calling it again.
      * This is to prevent race conditions as `setGlobalTheme` is async and mutates the DOM (e.g. sets the

package/dist/esm/app-provider.js

@@ -9,7 +9,7 @@ var InsideAppProviderContext = /*#__PURE__*/createContext(false);
  *
  * Place it at the root of your application.
  */
-export function AppProvider(_ref) {
+function AppProvider(_ref) {
   var children = _ref.children,
     _ref$defaultColorMode = _ref.defaultColorMode,
     defaultColorMode = _ref$defaultColorMode === void 0 ? 'light' : _ref$defaultColorMode,

package/dist/esm/router-link-provider/index.js

@@ -1,4 +1,5 @@
 import React, { createContext } from 'react';
+// eslint-disable-next-line @repo/internal/react/require-jsdoc
 export var RouterLinkProviderContext = /*#__PURE__*/createContext({});
 /**
  * __RouterLinkProvider__

package/dist/esm/theme-provider.js

@@ -102,7 +102,7 @@ function getReconciledColorMode(colorMode) {
  *
  * @internal
  */
-export function ThemeProvider(_ref) {
+function ThemeProvider(_ref) {
   var children = _ref.children,
     defaultColorMode = _ref.defaultColorMode,
     defaultTheme = _ref.defaultTheme;
@@ -129,20 +129,8 @@ export function ThemeProvider(_ref) {
       return _objectSpread(_objectSpread({}, theme), nextTheme);
     });
   }, []);
-  useEffect(function () {
-    if (fg('platform_dst_fix_set_theme_race')) {
-      return;
-    }
-    setGlobalTheme(_objectSpread(_objectSpread({}, theme), {}, {
-      colorMode: reconciledColorMode
-    }));
-  }, [theme, reconciledColorMode]);
   var lastSetGlobalThemePromiseRef = useRef(null);
   useEffect(function () {
-    if (!fg('platform_dst_fix_set_theme_race')) {
-      return;
-    }
-
     /**
      * We need to wait for any previous `setGlobalTheme` calls to finish before calling it again.
      * This is to prevent race conditions as `setGlobalTheme` is async and mutates the DOM (e.g. sets the

package/dist/types/app-provider.d.ts

@@ -35,5 +35,5 @@ interface AppProviderProps {
  *
  * Place it at the root of your application.
  */
-export declare function AppProvider({ children, defaultColorMode, defaultTheme, routerLinkComponent, UNSAFE_isThemingDisabled, }: AppProviderProps): JSX.Element;
+declare function AppProvider({ children, defaultColorMode, defaultTheme, routerLinkComponent, UNSAFE_isThemingDisabled, }: AppProviderProps): JSX.Element;
 export default AppProvider;

package/dist/types/router-link-provider/index.d.ts

@@ -31,7 +31,7 @@ export type RouterLinkProviderContextProps<RouterLinkConfig extends Record<strin
     routerLinkComponent?: RouterLinkComponent<RouterLinkConfig>;
 };
 export declare const RouterLinkProviderContext: React.Context<RouterLinkProviderContextProps<never>>;
-export type RouterLinkProviderProps = {
+type RouterLinkProviderProps = {
     /**
      * The rendering mechanism of router links within Design System components.
      */

package/dist/types/theme-provider.d.ts

@@ -2,7 +2,7 @@ import React from 'react';
 import { type ThemeState } from '@atlaskit/tokens';
 export type Theme = Omit<ThemeState, 'colorMode' | 'contrastMode'>;
 export type ColorMode = 'light' | 'dark' | 'auto';
-export type ReconciledColorMode = Exclude<ColorMode, 'auto'>;
+type ReconciledColorMode = Exclude<ColorMode, 'auto'>;
 /**
  * __UNSAFE_useColorModeForMigration()__
  *
@@ -47,5 +47,5 @@ interface ThemeProviderProps {
  *
  * @internal
  */
-export declare function ThemeProvider({ children, defaultColorMode, defaultTheme }: ThemeProviderProps): JSX.Element;
+declare function ThemeProvider({ children, defaultColorMode, defaultTheme }: ThemeProviderProps): JSX.Element;
 export default ThemeProvider;

package/dist/types-ts4.5/app-provider.d.ts

@@ -35,5 +35,5 @@ interface AppProviderProps {
  *
  * Place it at the root of your application.
  */
-export declare function AppProvider({ children, defaultColorMode, defaultTheme, routerLinkComponent, UNSAFE_isThemingDisabled, }: AppProviderProps): JSX.Element;
+declare function AppProvider({ children, defaultColorMode, defaultTheme, routerLinkComponent, UNSAFE_isThemingDisabled, }: AppProviderProps): JSX.Element;
 export default AppProvider;

package/dist/types-ts4.5/router-link-provider/index.d.ts

@@ -31,7 +31,7 @@ export type RouterLinkProviderContextProps<RouterLinkConfig extends Record<strin
     routerLinkComponent?: RouterLinkComponent<RouterLinkConfig>;
 };
 export declare const RouterLinkProviderContext: React.Context<RouterLinkProviderContextProps<never>>;
-export type RouterLinkProviderProps = {
+type RouterLinkProviderProps = {
     /**
      * The rendering mechanism of router links within Design System components.
      */

package/dist/types-ts4.5/theme-provider.d.ts

@@ -2,7 +2,7 @@ import React from 'react';
 import { type ThemeState } from '@atlaskit/tokens';
 export type Theme = Omit<ThemeState, 'colorMode' | 'contrastMode'>;
 export type ColorMode = 'light' | 'dark' | 'auto';
-export type ReconciledColorMode = Exclude<ColorMode, 'auto'>;
+type ReconciledColorMode = Exclude<ColorMode, 'auto'>;
 /**
  * __UNSAFE_useColorModeForMigration()__
  *
@@ -47,5 +47,5 @@ interface ThemeProviderProps {
  *
  * @internal
  */
-export declare function ThemeProvider({ children, defaultColorMode, defaultTheme }: ThemeProviderProps): JSX.Element;
+declare function ThemeProvider({ children, defaultColorMode, defaultTheme }: ThemeProviderProps): JSX.Element;
 export default ThemeProvider;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/app-provider",
-	"version": "1.6.0",
+	"version": "1.7.0",
 	"description": "A top level provider for the Design System.",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -40,7 +40,7 @@
 	},
 	"dependencies": {
 		"@atlaskit/platform-feature-flags": "^0.3.0",
-		"@atlaskit/tokens": "^3.0.0",
+		"@atlaskit/tokens": "^3.2.0",
 		"@babel/runtime": "^7.0.0",
 		"bind-event-listener": "^3.0.0"
 	},
@@ -49,16 +49,16 @@
 	},
 	"devDependencies": {
 		"@af/visual-regression": "*",
+		"@atlaskit/dropdown-menu": "^12.25.0",
 		"@atlaskit/ds-lib": "^3.3.0",
+		"@atlaskit/primitives": "^13.3.7",
 		"@atlaskit/ssr": "*",
 		"@atlaskit/visual-regression": "*",
 		"@testing-library/react": "^12.1.5",
-		"@testing-library/react-hooks": "^8.0.1",
 		"@testing-library/user-event": "^14.4.3",
 		"react-dom": "^16.8.0",
 		"react-resource-router": "^0.20.0",
-		"typescript": "~5.4.2",
-		"wait-for-expect": "^1.2.0"
+		"typescript": "~5.4.2"
 	},
 	"techstack": {
 		"@atlassian/frontend": {
@@ -93,9 +93,6 @@
 	},
 	"homepage": "https://atlassian.design/components/app-provider",
 	"platform-feature-flags": {
-		"platform_dst_fix_set_theme_race": {
-			"type": "boolean"
-		},
 		"platform-default-typography-modernized": {
 			"type": "boolean"
 		}