@atlaskit/icon-object 6.11.0 → 7.0.0

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # @atlaskit/icon-object
 
+## 7.0.0
+
+### Major Changes
+
+- [#117363](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/117363)
+  [`10a0f7f6c2027`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/10a0f7f6c2027) -
+  This package's `peerDependencies` have been adjusted for `react` and/or `react-dom` to reflect the
+  status of only supporting React 18 going forward. No explicit breaking change to React support has
+  been made in this release, but this is to signify going forward, breaking changes for React 16 or
+  React 17 may come via non-major semver releases.
+
+  Please refer this community post for more details:
+  https://community.developer.atlassian.com/t/rfc-78-dropping-support-for-react-16-and-rendering-in-a-react-18-concurrent-root-in-jira-and-confluence/87026
+
+### Patch Changes
+
+- Updated dependencies
+
+## 6.11.1
+
+### Patch Changes
+
+- [#116456](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/116456)
+  [`b1978c1e70a88`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/b1978c1e70a88) -
+  Remove old codemod and update dependncies.
+
 ## 6.11.0
 
 ### Minor Changes

package/afm-jira/tsconfig.json

@@ -20,6 +20,9 @@
 		{
 			"path": "../../icon/afm-jira/tsconfig.json"
 		},
+		{
+			"path": "../../icon-lab/afm-jira/tsconfig.json"
+		},
 		{
 			"path": "../../../platform/feature-flags/afm-jira/tsconfig.json"
 		}

package/afm-post-office/tsconfig.json

@@ -20,6 +20,9 @@
 		{
 			"path": "../../icon/afm-post-office/tsconfig.json"
 		},
+		{
+			"path": "../../icon-lab/afm-post-office/tsconfig.json"
+		},
 		{
 			"path": "../../../platform/feature-flags/afm-post-office/tsconfig.json"
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/icon-object",
-	"version": "6.11.0",
+	"version": "7.0.0",
 	"description": "An object icon is used to represent an Atlassian-specific content type.",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -37,24 +37,23 @@
 		"build-glyphs": "ts-node --project ../../../tsconfig.node.json ./build/index.tsx"
 	},
 	"dependencies": {
-		"@atlaskit/icon": "^23.9.0",
-		"@atlaskit/icon-lab": "^2.4.0",
-		"@atlaskit/platform-feature-flags": "^1.0.0",
+		"@atlaskit/icon": "^24.0.0",
+		"@atlaskit/icon-lab": "^3.0.0",
+		"@atlaskit/platform-feature-flags": "^1.1.0",
 		"@babel/runtime": "^7.0.0"
 	},
 	"peerDependencies": {
-		"react": "^16.8.0 || ^17.0.0 || ^18.0.0"
+		"react": "^18.2.0"
 	},
 	"devDependencies": {
 		"@af/formatting": "*",
 		"@af/icon-build-process": "^2.4.0",
-		"@atlaskit/textfield": "^6.8.0",
-		"@atlaskit/tooltip": "^19.1.0",
+		"@atlaskit/textfield": "^8.0.0",
+		"@atlaskit/tooltip": "^20.0.0",
 		"@atlaskit/visual-regression": "*",
 		"@atlassian/codegen": "^0.1.2",
 		"@emotion/styled": "^11.0.0",
 		"fs-extra": "^4.0.2",
-		"jscodeshift": "^0.13.0",
 		"pkg-dir": "^4.2.0",
 		"react-dom": "^18.2.0",
 		"ts-node": "^10.9.1"

package/codemods/6.1.0-metadata-entry.tsx

@@ -1,111 +0,0 @@
-import type {
-	API,
-	default as core,
-	FileInfo,
-	ImportSpecifier,
-	ASTPath,
-	ImportDefaultSpecifier,
-	Options,
-} from 'jscodeshift';
-import { type Collection } from 'jscodeshift/src/Collection';
-
-export const packageName = '@atlaskit/icon-object';
-
-export default function transformer(file: FileInfo, api: API, options: Options) {
-	const j = api.jscodeshift;
-	const root = j(file.source, { quote: 'auto' });
-	if (hasImportDeclaration(j, root, packageName)) {
-		const imports = root.find(j.ImportSpecifier, {
-			imported: { name: 'metadata' },
-		});
-		imports.forEach((p: ASTPath<ImportSpecifier | ImportDefaultSpecifier>) => {
-			if (p.parentPath.parentPath.node.source.value !== packageName) {
-				return;
-			}
-			if (p.parentPath.node.specifiers.length === 1) {
-				p.node.type = 'ImportDefaultSpecifier';
-				p.parentPath.node.source.value = `${packageName}/metadata`;
-			} else if (p.node.local) {
-				tryCreateImport({
-					j,
-					base: root,
-					packageName: `${packageName}/metadata`,
-					relativeToPackage: packageName,
-				});
-				addToImport({
-					j,
-					base: root,
-					packageName: `${packageName}/metadata`,
-					importSpecifier: j.importDefaultSpecifier(j.identifier(p.node.local.name)),
-				});
-				p.replace();
-			}
-		});
-		return root.toSource(options.printOptions || { quote: 'single' });
-	}
-	return file.source;
-}
-
-function hasImportDeclaration(
-	j: core.JSCodeshift,
-	source: ReturnType<typeof j>,
-	importPath: string,
-) {
-	return !!source.find(j.ImportDeclaration).filter((path) => path.node.source.value === importPath)
-		.length;
-}
-
-function tryCreateImport({
-	j,
-	base,
-	relativeToPackage,
-	packageName,
-}: {
-	j: core.JSCodeshift;
-	base: Collection<any>;
-	relativeToPackage: string;
-	packageName: string;
-}) {
-	const exists: boolean =
-		base.find(j.ImportDeclaration).filter((path) => path.value.source.value === packageName)
-			.length > 0;
-
-	if (exists) {
-		return;
-	}
-
-	base
-		.find(j.ImportDeclaration)
-		.filter((path) => path.value.source.value === relativeToPackage)
-		.insertBefore(j.importDeclaration([], j.literal(packageName)));
-}
-
-function addToImport({
-	j,
-	base,
-	importSpecifier,
-	packageName,
-}: {
-	j: core.JSCodeshift;
-	base: Collection<any>;
-	importSpecifier: ImportSpecifier | ImportDefaultSpecifier;
-	packageName: string;
-}) {
-	base
-		.find(j.ImportDeclaration)
-		.filter((path) => path.value.source.value === packageName)
-		.replaceWith((declaration) => {
-			return j.importDeclaration(
-				[
-					// we are appending to the existing specifiers
-					// We are doing a filter hear because sometimes specifiers can be removed
-					// but they hand around in the declaration
-					...(declaration.value.specifiers || []).filter(
-						(item) => item.type === 'ImportSpecifier' && item.imported != null,
-					),
-					importSpecifier,
-				],
-				j.literal(packageName),
-			);
-		});
-}

package/codemods/__tests__/6.1.0-metadata-entry.tsx

@@ -1,126 +0,0 @@
-jest.disableAutomock();
-
-import transformer, { packageName } from '../6.1.0-metadata-entry';
-
-const defineInlineTest = require('jscodeshift/dist/testUtils').defineInlineTest;
-
-describe('Update imports', () => {
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import { metadata } from '${packageName}';
-      console.log(metadata);
-    `,
-		`
-      import metadata from '${packageName}/metadata';
-      console.log(metadata);
-    `,
-		'should replace named metadata import from main entry point with default import from dedicated entry point',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import { metadata as data } from '${packageName}';
-      console.log(data);
-    `,
-		`
-      import data from '${packageName}/metadata';
-      console.log(data);
-    `,
-		'should preserve renaming when converting from named import to default import',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import { a, metadata, b } from '${packageName}';
-      console.log(metadata);
-    `,
-		`
-      import metadata from '${packageName}/metadata';
-      import { a, b } from '${packageName}';
-      console.log(metadata);
-    `,
-		'should preserve other named imports',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import { a, metadata as data, b } from '${packageName}';
-      console.log(data);
-    `,
-		`
-      import data from '${packageName}/metadata';
-      import { a, b } from '${packageName}';
-      console.log(data);
-    `,
-		'should preserve other named imports, as well as metadata renaming',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import Icon, { a, metadata as data, b } from '${packageName}';
-      console.log(a, data, b);
-    `,
-		`
-      import data from '${packageName}/metadata';
-      import Icon, { a, b } from '${packageName}';
-      console.log(a, data, b);
-    `,
-		'should preserve other named imports and default import, as well as metadata renaming',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import metadata from '${packageName}/metadata';
-      console.log(metadata);
-    `,
-		`
-      import metadata from '${packageName}/metadata';
-      console.log(metadata);
-    `,
-		'should not affect existing good imports',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import { metadata } from 'not-${packageName}';
-    `,
-		`
-      import { metadata } from 'not-${packageName}';
-    `,
-		'should not affect other packages',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import { metadata } from '${packageName}/foo';
-    `,
-		`
-      import { metadata } from '${packageName}/foo';
-    `,
-		'should not affect other entrypoints',
-	);
-	defineInlineTest(
-		{ default: transformer, parser: 'tsx' },
-		{},
-		`
-      import { metadata } from '@atlaskit/icon';
-      import { metadata as objectIconMetadata } from '@atlaskit/icon-object';
-      import { metadata as fileTypeIconMetadata } from '@atlaskit/icon-file-type';
-    `,
-		`
-      import { metadata } from '@atlaskit/icon';
-      import objectIconMetadata from '@atlaskit/icon-object/metadata';
-      import { metadata as fileTypeIconMetadata } from '@atlaskit/icon-file-type';
-    `,
-		'should work as expected on a real-world case',
-	);
-});