@strapi/upgrade 0.0.0-experimental.defd8568ae03ef8d52f86e1f3541979f953c3941 → 0.0.0-experimental.e02b4637b3906c6d31048d00600d09a23a0edc3d
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/LICENSE +19 -4
- package/dist/cli.js +73 -36
- package/dist/cli.js.map +1 -1
- package/dist/index.js +79 -40
- package/dist/index.js.map +1 -1
- package/dist/index.mjs +78 -40
- package/dist/index.mjs.map +1 -1
- package/dist/modules/file-scanner/scanner.d.ts.map +1 -1
- package/dist/modules/format/formats.d.ts +2 -1
- package/dist/modules/format/formats.d.ts.map +1 -1
- package/dist/modules/project/constants.d.ts +6 -5
- package/dist/modules/project/constants.d.ts.map +1 -1
- package/dist/modules/project/project.d.ts +16 -2
- package/dist/modules/project/project.d.ts.map +1 -1
- package/dist/modules/project/types.d.ts +3 -0
- package/dist/modules/project/types.d.ts.map +1 -1
- package/dist/modules/runner/json/transform.d.ts.map +1 -1
- package/dist/tasks/upgrade/upgrade.d.ts.map +1 -1
- package/package.json +8 -7
- package/resources/codemods/5.0.0/comment-out-lifecycle-files.code.ts +63 -0
- package/resources/codemods/5.0.0/dependency-upgrade-react-and-react-dom.json.ts +67 -0
- package/resources/codemods/5.0.0/dependency-upgrade-styled-components.json.ts +49 -0
- package/resources/codemods/5.0.0/deprecate-helper-plugin.code.ts +192 -0
- package/resources/codemods/5.0.0/entity-service-document-service.code.ts +68 -5
- package/resources/utils/change-import.ts +118 -0
- package/resources/utils/replace-jsx.ts +49 -0
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"transform.d.ts","sourceRoot":"","sources":["../../../../src/modules/runner/json/transform.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AAE3C,OAAO,KAAK,EAAE,uBAAuB,EAAuC,MAAM,SAAS,CAAC;AAE5F,eAAO,MAAM,aAAa,gBACX,MAAM,SACZ,MAAM,EAAE,UACP,uBAAuB,KAC9B,QAAQ,aAAa,
|
|
1
|
+
{"version":3,"file":"transform.d.ts","sourceRoot":"","sources":["../../../../src/modules/runner/json/transform.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AAE3C,OAAO,KAAK,EAAE,uBAAuB,EAAuC,MAAM,SAAS,CAAC;AAE5F,eAAO,MAAM,aAAa,gBACX,MAAM,SACZ,MAAM,EAAE,UACP,uBAAuB,KAC9B,QAAQ,aAAa,CAiFvB,CAAC"}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"upgrade.d.ts","sourceRoot":"","sources":["../../../src/tasks/upgrade/upgrade.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAE9C,eAAO,MAAM,OAAO,YAAmB,cAAc,
|
|
1
|
+
{"version":3,"file":"upgrade.d.ts","sourceRoot":"","sources":["../../../src/tasks/upgrade/upgrade.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAE9C,eAAO,MAAM,OAAO,YAAmB,cAAc,kBAuDpD,CAAC"}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@strapi/upgrade",
|
|
3
|
-
"version": "0.0.0-experimental.
|
|
3
|
+
"version": "0.0.0-experimental.e02b4637b3906c6d31048d00600d09a23a0edc3d",
|
|
4
4
|
"description": "CLI to upgrade Strapi applications effortless",
|
|
5
5
|
"keywords": [
|
|
6
6
|
"strapi",
|
|
@@ -59,13 +59,13 @@
|
|
|
59
59
|
"watch": "pack-up watch"
|
|
60
60
|
},
|
|
61
61
|
"dependencies": {
|
|
62
|
-
"@strapi/utils": "0.0.0-experimental.
|
|
62
|
+
"@strapi/utils": "0.0.0-experimental.e02b4637b3906c6d31048d00600d09a23a0edc3d",
|
|
63
63
|
"chalk": "4.1.2",
|
|
64
64
|
"cli-table3": "0.6.2",
|
|
65
65
|
"commander": "8.3.0",
|
|
66
66
|
"esbuild-register": "3.5.0",
|
|
67
|
-
"
|
|
68
|
-
"
|
|
67
|
+
"fast-glob": "3.3.2",
|
|
68
|
+
"fs-extra": "11.2.0",
|
|
69
69
|
"jscodeshift": "0.15.1",
|
|
70
70
|
"lodash": "4.17.21",
|
|
71
71
|
"memfs": "4.6.0",
|
|
@@ -76,14 +76,15 @@
|
|
|
76
76
|
},
|
|
77
77
|
"devDependencies": {
|
|
78
78
|
"@strapi/pack-up": "5.0.0",
|
|
79
|
-
"@strapi/types": "0.0.0-experimental.
|
|
79
|
+
"@strapi/types": "0.0.0-experimental.e02b4637b3906c6d31048d00600d09a23a0edc3d",
|
|
80
|
+
"@types/fs-extra": "11.0.4",
|
|
80
81
|
"@types/jscodeshift": "0.11.10",
|
|
81
|
-
"eslint-config-custom": "0.0.0-experimental.
|
|
82
|
+
"eslint-config-custom": "0.0.0-experimental.e02b4637b3906c6d31048d00600d09a23a0edc3d",
|
|
82
83
|
"rimraf": "5.0.5"
|
|
83
84
|
},
|
|
84
85
|
"engines": {
|
|
85
86
|
"node": ">=18.0.0 <=20.x.x",
|
|
86
87
|
"npm": ">=6.0.0"
|
|
87
88
|
},
|
|
88
|
-
"gitHead": "
|
|
89
|
+
"gitHead": "e02b4637b3906c6d31048d00600d09a23a0edc3d"
|
|
89
90
|
}
|
|
@@ -0,0 +1,63 @@
|
|
|
1
|
+
import { Transform } from 'jscodeshift';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* comments out lifecycles.js/ts files and adds a description for the reason at the top
|
|
5
|
+
*/
|
|
6
|
+
const transform: Transform = (file, api) => {
|
|
7
|
+
const j = api.jscodeshift;
|
|
8
|
+
const root = j(file.source);
|
|
9
|
+
|
|
10
|
+
// check if file path follows this pattern `content-types/[content-type-name]/lifecycles`
|
|
11
|
+
if (/content-types\/[^/]+\/lifecycles\.(js|ts)$/.test(file.path)) {
|
|
12
|
+
// Get the entire source code as a string
|
|
13
|
+
const sourceCode = root.toSource();
|
|
14
|
+
|
|
15
|
+
// Split the source code into lines and prepend // to each line
|
|
16
|
+
// we are using line comments instead of block comments so we don't face issues with existing block comments
|
|
17
|
+
const commentedCode = sourceCode
|
|
18
|
+
.split('\n')
|
|
19
|
+
.map((line) => `// ${line}`)
|
|
20
|
+
.join('\n');
|
|
21
|
+
|
|
22
|
+
// Add a header comment at the top to explain why the file is commented out
|
|
23
|
+
const headerComment = `
|
|
24
|
+
/*
|
|
25
|
+
*
|
|
26
|
+
* ============================================================
|
|
27
|
+
* WARNING: THIS FILE HAS BEEN COMMENTED OUT
|
|
28
|
+
* ============================================================
|
|
29
|
+
*
|
|
30
|
+
* CONTEXT:
|
|
31
|
+
*
|
|
32
|
+
* The lifecycles.js file has been commented out to prevent unintended side effects when starting Strapi 5 for the first time after migrating to the document service.
|
|
33
|
+
*
|
|
34
|
+
* STRAPI 5 introduces a new document service that handles lifecycles differently compared to previous versions. Without migrating your lifecycles to document service middlewares, you may experience issues such as:
|
|
35
|
+
*
|
|
36
|
+
* - \`unpublish\` actions triggering \`delete\` lifecycles for every locale with a published entity, which differs from the expected behavior in v4.
|
|
37
|
+
* - \`discardDraft\` actions triggering both \`create\` and \`delete\` lifecycles, leading to potential confusion.
|
|
38
|
+
*
|
|
39
|
+
* MIGRATION GUIDE:
|
|
40
|
+
*
|
|
41
|
+
* For a thorough guide on migrating your lifecycles to document service middlewares, please refer to the following link:
|
|
42
|
+
* [Document Services Middlewares Migration Guide](https://docs.strapi.io/dev-docs/migration/v4-to-v5/breaking-changes/lifecycle-hooks-document-service)
|
|
43
|
+
*
|
|
44
|
+
* IMPORTANT:
|
|
45
|
+
*
|
|
46
|
+
* Simply uncommenting this file without following the migration guide may result in unexpected behavior and inconsistencies. Ensure that you have completed the migration process before re-enabling this file.
|
|
47
|
+
*
|
|
48
|
+
* ============================================================
|
|
49
|
+
*/
|
|
50
|
+
`;
|
|
51
|
+
|
|
52
|
+
// Combine the header comment with the commented-out code
|
|
53
|
+
const finalCode = `${headerComment}\n${commentedCode}`;
|
|
54
|
+
|
|
55
|
+
return finalCode;
|
|
56
|
+
}
|
|
57
|
+
|
|
58
|
+
return root.toSource();
|
|
59
|
+
};
|
|
60
|
+
|
|
61
|
+
export const parser = 'tsx';
|
|
62
|
+
|
|
63
|
+
export default transform;
|
|
@@ -0,0 +1,67 @@
|
|
|
1
|
+
import path from 'node:path';
|
|
2
|
+
import semver from 'semver';
|
|
3
|
+
|
|
4
|
+
import type { modules } from '../../../dist';
|
|
5
|
+
|
|
6
|
+
const REACT_DEP_NAME = 'react';
|
|
7
|
+
const REACT_DEP_PATH = `dependencies.${REACT_DEP_NAME}`;
|
|
8
|
+
|
|
9
|
+
const REACT_DOM_DEP_NAME = 'react-dom';
|
|
10
|
+
const REACT_DOM_DEP_PATH = `dependencies.${REACT_DOM_DEP_NAME}`;
|
|
11
|
+
|
|
12
|
+
const DEP_NEW_VERSION_RANGE = '^18.0.0';
|
|
13
|
+
|
|
14
|
+
/**
|
|
15
|
+
* Specifically targets the root package.json and updates the react and react-dom dependency version.
|
|
16
|
+
*
|
|
17
|
+
* We first check if the react and react-dom dependency is listed in the package.json. If the dependency is
|
|
18
|
+
* found, we verify its version.
|
|
19
|
+
*
|
|
20
|
+
* If the detected version does not satisfy the new version range, we replace it with the new one.
|
|
21
|
+
*
|
|
22
|
+
* Conversely, if no react or react-dom dependency is listed, we add it with the new version range.
|
|
23
|
+
*/
|
|
24
|
+
const transform: modules.runner.json.JSONTransform = (file, params) => {
|
|
25
|
+
const { cwd, json } = params;
|
|
26
|
+
|
|
27
|
+
const rootPackageJsonPath = path.join(cwd, 'package.json');
|
|
28
|
+
|
|
29
|
+
if (file.path !== rootPackageJsonPath) {
|
|
30
|
+
return file.json;
|
|
31
|
+
}
|
|
32
|
+
|
|
33
|
+
const j = json(file.json);
|
|
34
|
+
|
|
35
|
+
if (j.has(REACT_DEP_PATH) && j.has(REACT_DOM_DEP_PATH)) {
|
|
36
|
+
const currentReactVersion = j.get(REACT_DEP_PATH);
|
|
37
|
+
const currentReactDOMVersion = j.get(REACT_DOM_DEP_PATH);
|
|
38
|
+
|
|
39
|
+
// If the current version is not a string, then something is wrong, abort
|
|
40
|
+
if (typeof currentReactVersion !== 'string' || typeof currentReactDOMVersion !== 'string') {
|
|
41
|
+
return j.root();
|
|
42
|
+
}
|
|
43
|
+
|
|
44
|
+
const currentSatisfiesNew =
|
|
45
|
+
semver.satisfies(currentReactVersion, DEP_NEW_VERSION_RANGE) &&
|
|
46
|
+
semver.satisfies(currentReactDOMVersion, DEP_NEW_VERSION_RANGE);
|
|
47
|
+
|
|
48
|
+
// if the current version satisfies the new range, keep it as is and abort
|
|
49
|
+
if (currentSatisfiesNew) {
|
|
50
|
+
return j.root();
|
|
51
|
+
}
|
|
52
|
+
|
|
53
|
+
// else, update the version with the new one
|
|
54
|
+
j.set(REACT_DEP_PATH, DEP_NEW_VERSION_RANGE);
|
|
55
|
+
j.set(REACT_DOM_DEP_PATH, DEP_NEW_VERSION_RANGE);
|
|
56
|
+
}
|
|
57
|
+
|
|
58
|
+
// If the dependency is not listed yet, add it
|
|
59
|
+
else {
|
|
60
|
+
j.set(REACT_DEP_PATH, DEP_NEW_VERSION_RANGE);
|
|
61
|
+
j.set(REACT_DOM_DEP_PATH, DEP_NEW_VERSION_RANGE);
|
|
62
|
+
}
|
|
63
|
+
|
|
64
|
+
return j.root();
|
|
65
|
+
};
|
|
66
|
+
|
|
67
|
+
export default transform;
|
|
@@ -0,0 +1,49 @@
|
|
|
1
|
+
import path from 'node:path';
|
|
2
|
+
import semver from 'semver';
|
|
3
|
+
|
|
4
|
+
import type { modules } from '../../../dist';
|
|
5
|
+
|
|
6
|
+
const DEP_NAME = 'styled-components';
|
|
7
|
+
const DEP_PATH = `dependencies.${DEP_NAME}`;
|
|
8
|
+
|
|
9
|
+
const DEP_NEW_VERSION_RANGE = '^6.0.0';
|
|
10
|
+
|
|
11
|
+
const transform: modules.runner.json.JSONTransform = (file, params) => {
|
|
12
|
+
const { cwd, json } = params;
|
|
13
|
+
|
|
14
|
+
const rootPackageJsonPath = path.join(cwd, 'package.json');
|
|
15
|
+
|
|
16
|
+
if (file.path !== rootPackageJsonPath) {
|
|
17
|
+
return file.json;
|
|
18
|
+
}
|
|
19
|
+
|
|
20
|
+
const j = json(file.json);
|
|
21
|
+
|
|
22
|
+
if (j.has(DEP_PATH)) {
|
|
23
|
+
const currentVersion = j.get(DEP_PATH);
|
|
24
|
+
|
|
25
|
+
// If the current version is not a string, then something is wrong, abort
|
|
26
|
+
if (typeof currentVersion !== 'string') {
|
|
27
|
+
return j.root();
|
|
28
|
+
}
|
|
29
|
+
|
|
30
|
+
const currentSatisfiesNew = semver.satisfies(currentVersion, DEP_NEW_VERSION_RANGE);
|
|
31
|
+
|
|
32
|
+
// if the current version satisfies the new range, keep it as is and abort
|
|
33
|
+
if (currentSatisfiesNew) {
|
|
34
|
+
return j.root();
|
|
35
|
+
}
|
|
36
|
+
|
|
37
|
+
// else, update the version with the new one
|
|
38
|
+
j.set(DEP_PATH, DEP_NEW_VERSION_RANGE);
|
|
39
|
+
}
|
|
40
|
+
|
|
41
|
+
// If the dependency is not listed yet, add it
|
|
42
|
+
else {
|
|
43
|
+
j.set(DEP_PATH, DEP_NEW_VERSION_RANGE);
|
|
44
|
+
}
|
|
45
|
+
|
|
46
|
+
return j.root();
|
|
47
|
+
};
|
|
48
|
+
|
|
49
|
+
export default transform;
|
|
@@ -0,0 +1,192 @@
|
|
|
1
|
+
import type { Transform } from 'jscodeshift';
|
|
2
|
+
import { changeImportSpecifier } from '../../utils/change-import';
|
|
3
|
+
import { replaceJSXElement } from '../../utils/replace-jsx';
|
|
4
|
+
|
|
5
|
+
/**
|
|
6
|
+
* This codemods automates all the imports and naming changes
|
|
7
|
+
* for methods or components that used to be imported from '@strapi/helper-plugin'
|
|
8
|
+
*/
|
|
9
|
+
const transform: Transform = (file, api) => {
|
|
10
|
+
const { j } = api;
|
|
11
|
+
|
|
12
|
+
const root = j.withParser('tsx')(file.source);
|
|
13
|
+
|
|
14
|
+
type Replacement = {
|
|
15
|
+
oldName: string;
|
|
16
|
+
oldDependency: string;
|
|
17
|
+
toReplace: boolean;
|
|
18
|
+
toChangeImportSpecifier: boolean;
|
|
19
|
+
newDependency?: string;
|
|
20
|
+
newName?: string;
|
|
21
|
+
newImport?: string;
|
|
22
|
+
};
|
|
23
|
+
|
|
24
|
+
const replacements: Replacement[] = [
|
|
25
|
+
{
|
|
26
|
+
oldName: 'AnErrorOccurred',
|
|
27
|
+
newImport: 'Page',
|
|
28
|
+
newName: 'Page.Error',
|
|
29
|
+
oldDependency: '@strapi/helper-plugin',
|
|
30
|
+
newDependency: '@strapi/strapi/admin',
|
|
31
|
+
toReplace: true,
|
|
32
|
+
toChangeImportSpecifier: true,
|
|
33
|
+
},
|
|
34
|
+
{
|
|
35
|
+
oldName: 'CheckPagePermissions',
|
|
36
|
+
newImport: 'Page',
|
|
37
|
+
newName: 'Page.Protect',
|
|
38
|
+
oldDependency: '@strapi/helper-plugin',
|
|
39
|
+
newDependency: '@strapi/strapi/admin',
|
|
40
|
+
toReplace: true,
|
|
41
|
+
toChangeImportSpecifier: true,
|
|
42
|
+
},
|
|
43
|
+
{
|
|
44
|
+
oldName: 'ConfirmDialog',
|
|
45
|
+
oldDependency: '@strapi/helper-plugin',
|
|
46
|
+
newDependency: '@strapi/strapi/admin',
|
|
47
|
+
toChangeImportSpecifier: true,
|
|
48
|
+
toReplace: false,
|
|
49
|
+
},
|
|
50
|
+
{
|
|
51
|
+
oldName: 'DateTimePicker',
|
|
52
|
+
oldDependency: '@strapi/helper-plugin',
|
|
53
|
+
newDependency: '@strapi/design-system',
|
|
54
|
+
toChangeImportSpecifier: true,
|
|
55
|
+
toReplace: false,
|
|
56
|
+
},
|
|
57
|
+
{
|
|
58
|
+
oldName: 'getFetchClient',
|
|
59
|
+
oldDependency: '@strapi/helper-plugin',
|
|
60
|
+
newDependency: '@strapi/strapi/admin',
|
|
61
|
+
toChangeImportSpecifier: true,
|
|
62
|
+
toReplace: false,
|
|
63
|
+
},
|
|
64
|
+
{
|
|
65
|
+
oldName: 'LoadingIndicatorPage',
|
|
66
|
+
newImport: 'Page',
|
|
67
|
+
newName: 'Page.Loading',
|
|
68
|
+
oldDependency: '@strapi/helper-plugin',
|
|
69
|
+
newDependency: '@strapi/strapi/admin',
|
|
70
|
+
toReplace: true,
|
|
71
|
+
toChangeImportSpecifier: true,
|
|
72
|
+
},
|
|
73
|
+
{
|
|
74
|
+
oldName: 'NoContent',
|
|
75
|
+
newImport: 'EmptyStateLayout',
|
|
76
|
+
newName: 'EmptyStateLayout',
|
|
77
|
+
oldDependency: '@strapi/helper-plugin',
|
|
78
|
+
newDependency: '@strapi/design-system',
|
|
79
|
+
toReplace: true,
|
|
80
|
+
toChangeImportSpecifier: true,
|
|
81
|
+
},
|
|
82
|
+
{
|
|
83
|
+
oldName: 'NoPermissions',
|
|
84
|
+
newImport: 'Page',
|
|
85
|
+
newName: 'Page.NoPermissions',
|
|
86
|
+
oldDependency: '@strapi/helper-plugin',
|
|
87
|
+
newDependency: '@strapi/strapi/admin',
|
|
88
|
+
toReplace: true,
|
|
89
|
+
toChangeImportSpecifier: true,
|
|
90
|
+
},
|
|
91
|
+
{
|
|
92
|
+
oldName: 'Status',
|
|
93
|
+
oldDependency: '@strapi/helper-plugin',
|
|
94
|
+
newDependency: '@strapi/design-system',
|
|
95
|
+
toChangeImportSpecifier: true,
|
|
96
|
+
toReplace: false,
|
|
97
|
+
},
|
|
98
|
+
{
|
|
99
|
+
oldName: 'translatedErrors',
|
|
100
|
+
oldDependency: '@strapi/helper-plugin',
|
|
101
|
+
newDependency: '@strapi/strapi/admin',
|
|
102
|
+
toChangeImportSpecifier: true,
|
|
103
|
+
toReplace: false,
|
|
104
|
+
},
|
|
105
|
+
{
|
|
106
|
+
oldName: 'useAPIErrorHandler',
|
|
107
|
+
oldDependency: '@strapi/helper-plugin',
|
|
108
|
+
newDependency: '@strapi/strapi/admin',
|
|
109
|
+
toChangeImportSpecifier: true,
|
|
110
|
+
toReplace: false,
|
|
111
|
+
},
|
|
112
|
+
{
|
|
113
|
+
oldName: 'useCallbackRef',
|
|
114
|
+
oldDependency: '@strapi/helper-plugin',
|
|
115
|
+
newDependency: '@strapi/design-system',
|
|
116
|
+
toChangeImportSpecifier: true,
|
|
117
|
+
toReplace: false,
|
|
118
|
+
},
|
|
119
|
+
{
|
|
120
|
+
oldName: 'useCollator',
|
|
121
|
+
oldDependency: '@strapi/helper-plugin',
|
|
122
|
+
newDependency: '@strapi/design-system',
|
|
123
|
+
toChangeImportSpecifier: true,
|
|
124
|
+
toReplace: false,
|
|
125
|
+
},
|
|
126
|
+
{
|
|
127
|
+
oldName: 'useFetchClient',
|
|
128
|
+
oldDependency: '@strapi/helper-plugin',
|
|
129
|
+
newDependency: '@strapi/strapi/admin',
|
|
130
|
+
toChangeImportSpecifier: true,
|
|
131
|
+
toReplace: false,
|
|
132
|
+
},
|
|
133
|
+
{
|
|
134
|
+
oldName: 'useFilter',
|
|
135
|
+
oldDependency: '@strapi/helper-plugin',
|
|
136
|
+
newDependency: '@strapi/design-system',
|
|
137
|
+
toChangeImportSpecifier: true,
|
|
138
|
+
toReplace: false,
|
|
139
|
+
},
|
|
140
|
+
{
|
|
141
|
+
oldName: 'useQueryParams',
|
|
142
|
+
oldDependency: '@strapi/helper-plugin',
|
|
143
|
+
newDependency: '@strapi/strapi/admin',
|
|
144
|
+
toChangeImportSpecifier: true,
|
|
145
|
+
toReplace: false,
|
|
146
|
+
},
|
|
147
|
+
{
|
|
148
|
+
oldName: 'useRBAC',
|
|
149
|
+
oldDependency: '@strapi/helper-plugin',
|
|
150
|
+
newDependency: '@strapi/strapi/admin',
|
|
151
|
+
toChangeImportSpecifier: true,
|
|
152
|
+
toReplace: false,
|
|
153
|
+
},
|
|
154
|
+
{
|
|
155
|
+
oldName: 'SearchURLQuery',
|
|
156
|
+
oldDependency: '@strapi/helper-plugin',
|
|
157
|
+
newDependency: '@strapi/strapi/admin',
|
|
158
|
+
toChangeImportSpecifier: true,
|
|
159
|
+
toReplace: false,
|
|
160
|
+
},
|
|
161
|
+
{
|
|
162
|
+
oldName: 'useSettingsForm',
|
|
163
|
+
oldDependency: '@strapi/helper-plugin',
|
|
164
|
+
newDependency: '@strapi/strapi/admin',
|
|
165
|
+
toChangeImportSpecifier: true,
|
|
166
|
+
toReplace: false,
|
|
167
|
+
},
|
|
168
|
+
];
|
|
169
|
+
|
|
170
|
+
replacements.forEach((replacement) => {
|
|
171
|
+
if (replacement.toReplace && replacement.newName) {
|
|
172
|
+
replaceJSXElement(root, j, {
|
|
173
|
+
oldElementName: replacement.oldName,
|
|
174
|
+
newElementName: replacement.newName,
|
|
175
|
+
oldDependency: replacement.oldDependency,
|
|
176
|
+
});
|
|
177
|
+
}
|
|
178
|
+
|
|
179
|
+
if (replacement.toChangeImportSpecifier && replacement.newDependency) {
|
|
180
|
+
changeImportSpecifier(root, j, {
|
|
181
|
+
oldMethodName: replacement.oldName,
|
|
182
|
+
newMethodName: replacement.newImport,
|
|
183
|
+
oldDependency: replacement.oldDependency,
|
|
184
|
+
newDependency: replacement.newDependency,
|
|
185
|
+
});
|
|
186
|
+
}
|
|
187
|
+
});
|
|
188
|
+
|
|
189
|
+
return root.toSource();
|
|
190
|
+
};
|
|
191
|
+
|
|
192
|
+
export default transform;
|
|
@@ -142,12 +142,39 @@ const objectParam_10 = [...objectParam_10_2];
|
|
|
142
142
|
|
|
143
143
|
strapi.entityService.findOne(...[...objectParam_10]);
|
|
144
144
|
|
|
145
|
+
Case: find, create, update, delete with entityId as first argument
|
|
146
|
+
|
|
147
|
+
strapi.entityService.findMany(uid, {
|
|
148
|
+
fields: ["id", "name", "description"],
|
|
149
|
+
populate: ["author", "comments"],
|
|
150
|
+
publicationState: "preview",
|
|
151
|
+
});
|
|
152
|
+
|
|
153
|
+
strapi.entityService.create(uid, {
|
|
154
|
+
data: {
|
|
155
|
+
name: "John Doe",
|
|
156
|
+
age: 30,
|
|
157
|
+
},
|
|
158
|
+
});
|
|
159
|
+
|
|
160
|
+
strapi.entityService.update(uid, entityId, {
|
|
161
|
+
data: {
|
|
162
|
+
name: "John Doe",
|
|
163
|
+
age: 30,
|
|
164
|
+
},
|
|
165
|
+
});
|
|
166
|
+
|
|
167
|
+
strapi.entityService.delete(uid, entityId);
|
|
168
|
+
strapi.entityService.findOne(uid, entityId);
|
|
169
|
+
|
|
145
170
|
*/
|
|
146
171
|
|
|
147
|
-
const movedFunctions = ['findOne', '
|
|
172
|
+
const movedFunctions = ['findOne', 'findMany', 'count', 'create', 'update', 'delete'];
|
|
173
|
+
|
|
174
|
+
const functionsWithEntityId = ['findOne', 'update', 'delete'];
|
|
148
175
|
|
|
149
176
|
const transformDeclaration = (path: ASTPath<any>, name: any, j: JSCodeshift) => {
|
|
150
|
-
const declaration =
|
|
177
|
+
const declaration = findClosestDeclaration(path, name, j);
|
|
151
178
|
|
|
152
179
|
if (!declaration) {
|
|
153
180
|
return;
|
|
@@ -222,7 +249,7 @@ const transformObjectParam = (path: ASTPath<any>, expression: ObjectExpression,
|
|
|
222
249
|
break;
|
|
223
250
|
}
|
|
224
251
|
case j.Identifier.check(prop.value): {
|
|
225
|
-
const declaration =
|
|
252
|
+
const declaration = findClosestDeclaration(path, prop.value.name, j);
|
|
226
253
|
|
|
227
254
|
if (!declaration) {
|
|
228
255
|
return;
|
|
@@ -253,7 +280,7 @@ const transformObjectParam = (path: ASTPath<any>, expression: ObjectExpression,
|
|
|
253
280
|
});
|
|
254
281
|
};
|
|
255
282
|
|
|
256
|
-
const
|
|
283
|
+
const findClosestDeclaration = (path: ASTPath<any>, name: string, j) => {
|
|
257
284
|
// find Identifier declaration
|
|
258
285
|
const scope = path.scope.lookup(name);
|
|
259
286
|
|
|
@@ -318,7 +345,7 @@ const transform: Transform = (file, api) => {
|
|
|
318
345
|
case j.Identifier.check(arg.argument): {
|
|
319
346
|
const identifier = arg.argument;
|
|
320
347
|
|
|
321
|
-
const declaration =
|
|
348
|
+
const declaration = findClosestDeclaration(path, identifier.name, j);
|
|
322
349
|
|
|
323
350
|
if (!declaration) {
|
|
324
351
|
return arg;
|
|
@@ -351,6 +378,42 @@ const transform: Transform = (file, api) => {
|
|
|
351
378
|
|
|
352
379
|
const [docUID, ...rest] = resolvedArgs;
|
|
353
380
|
|
|
381
|
+
// function with entityId as first argument
|
|
382
|
+
if (
|
|
383
|
+
j.Identifier.check(path.value.callee.property) &&
|
|
384
|
+
functionsWithEntityId.includes(path.value.callee.property.name)
|
|
385
|
+
) {
|
|
386
|
+
rest.splice(0, 1);
|
|
387
|
+
|
|
388
|
+
// in case no extra params are passed in the function e.g delete(uid, entityId)
|
|
389
|
+
if (rest.length === 0) {
|
|
390
|
+
rest.push(
|
|
391
|
+
j.objectExpression.from({
|
|
392
|
+
properties: [],
|
|
393
|
+
})
|
|
394
|
+
);
|
|
395
|
+
}
|
|
396
|
+
|
|
397
|
+
const params = rest[0];
|
|
398
|
+
|
|
399
|
+
const placeholder = j.objectProperty(j.identifier('documentId'), j.literal('__TODO__'));
|
|
400
|
+
|
|
401
|
+
// add documentId to params with a placeholder
|
|
402
|
+
if (j.ObjectExpression.check(params)) {
|
|
403
|
+
params.properties.unshift(placeholder);
|
|
404
|
+
} else if (j.Identifier.check(params)) {
|
|
405
|
+
const declaration = findClosestDeclaration(path, params.name, j);
|
|
406
|
+
|
|
407
|
+
if (!declaration) {
|
|
408
|
+
return;
|
|
409
|
+
}
|
|
410
|
+
|
|
411
|
+
if (j.ObjectExpression.check(declaration.init)) {
|
|
412
|
+
declaration.init.properties.unshift(placeholder);
|
|
413
|
+
}
|
|
414
|
+
}
|
|
415
|
+
}
|
|
416
|
+
|
|
354
417
|
path.value.arguments.forEach((arg) => {
|
|
355
418
|
transformElement(path, arg, j);
|
|
356
419
|
});
|
|
@@ -0,0 +1,118 @@
|
|
|
1
|
+
import type { ImportDeclaration, JSCodeshift, Collection } from 'jscodeshift';
|
|
2
|
+
|
|
3
|
+
export const changeImportSpecifier = (
|
|
4
|
+
root: Collection,
|
|
5
|
+
j: JSCodeshift,
|
|
6
|
+
options: {
|
|
7
|
+
oldDependency: string;
|
|
8
|
+
newDependency: string;
|
|
9
|
+
oldMethodName: string;
|
|
10
|
+
newMethodName?: string;
|
|
11
|
+
}
|
|
12
|
+
): void => {
|
|
13
|
+
const { oldMethodName, newMethodName, oldDependency, newDependency } = options;
|
|
14
|
+
const methodNameToReplace = newMethodName ?? oldMethodName;
|
|
15
|
+
|
|
16
|
+
// Flag to check if the method was imported from the old dependency
|
|
17
|
+
let methodImportedFromOldDependency = false;
|
|
18
|
+
const methodAliases: string[] = [];
|
|
19
|
+
|
|
20
|
+
// Remove the method from the old dependency and check if it was imported
|
|
21
|
+
root
|
|
22
|
+
.find(j.ImportDeclaration)
|
|
23
|
+
.filter((path) => path.node.source.value === oldDependency)
|
|
24
|
+
.forEach((path) => {
|
|
25
|
+
const importDeclaration: ImportDeclaration = path.node;
|
|
26
|
+
|
|
27
|
+
// Check if the method is imported from the old dependency
|
|
28
|
+
const methodSpecifiers = importDeclaration.specifiers?.filter(
|
|
29
|
+
(specifier) =>
|
|
30
|
+
specifier.type === 'ImportSpecifier' && specifier.imported.name === oldMethodName
|
|
31
|
+
);
|
|
32
|
+
|
|
33
|
+
if (methodSpecifiers && methodSpecifiers.length > 0) {
|
|
34
|
+
methodImportedFromOldDependency = true;
|
|
35
|
+
|
|
36
|
+
// Collect all aliases for the method
|
|
37
|
+
methodSpecifiers.forEach((specifier) => {
|
|
38
|
+
if (specifier.local && specifier.local.name !== oldMethodName) {
|
|
39
|
+
methodAliases.push(specifier.local.name);
|
|
40
|
+
} else {
|
|
41
|
+
methodAliases.push(methodNameToReplace);
|
|
42
|
+
}
|
|
43
|
+
});
|
|
44
|
+
|
|
45
|
+
// Remove the method specifiers from the old import
|
|
46
|
+
const updatedSpecifiers = importDeclaration.specifiers?.filter(
|
|
47
|
+
(specifier) =>
|
|
48
|
+
specifier.type !== 'ImportSpecifier' || specifier.imported.name !== oldMethodName
|
|
49
|
+
);
|
|
50
|
+
|
|
51
|
+
if (updatedSpecifiers && updatedSpecifiers.length > 0) {
|
|
52
|
+
// Replace the import with the updated specifiers if there are other imports left
|
|
53
|
+
j(path).replaceWith(j.importDeclaration(updatedSpecifiers, j.literal(oldDependency)));
|
|
54
|
+
} else {
|
|
55
|
+
// Remove the entire import statement if the specified method was the only import
|
|
56
|
+
j(path).remove();
|
|
57
|
+
}
|
|
58
|
+
}
|
|
59
|
+
});
|
|
60
|
+
|
|
61
|
+
// Add new import dependency if the method was imported from the old dependency
|
|
62
|
+
if (methodImportedFromOldDependency) {
|
|
63
|
+
const dependencies = root
|
|
64
|
+
.find(j.ImportDeclaration)
|
|
65
|
+
.filter((path) => path.node.source.value === newDependency);
|
|
66
|
+
|
|
67
|
+
if (dependencies.length > 0) {
|
|
68
|
+
// we have to use a flag to prevent adding the method to multiple imports
|
|
69
|
+
let methodAdded = false;
|
|
70
|
+
dependencies.forEach((path) => {
|
|
71
|
+
const importDeclaration: ImportDeclaration = path.node;
|
|
72
|
+
if (!methodAdded) {
|
|
73
|
+
methodAliases.forEach((alias) => {
|
|
74
|
+
// Check if the methodNameToReplace or its alias is already imported
|
|
75
|
+
const specifiersArray = importDeclaration.specifiers || [];
|
|
76
|
+
const methodAlreadyExists = specifiersArray.some(
|
|
77
|
+
(specifier) =>
|
|
78
|
+
specifier.type === 'ImportSpecifier' &&
|
|
79
|
+
specifier.imported.name === methodNameToReplace && // Check if imported method matches
|
|
80
|
+
specifier.local?.name === alias // Check if local alias matches
|
|
81
|
+
);
|
|
82
|
+
|
|
83
|
+
if (!methodAlreadyExists) {
|
|
84
|
+
// If method does not exist, add it
|
|
85
|
+
const newSpecifier = j.importSpecifier(
|
|
86
|
+
j.identifier(methodNameToReplace),
|
|
87
|
+
j.identifier(alias)
|
|
88
|
+
);
|
|
89
|
+
path.get('specifiers').replace([...specifiersArray, newSpecifier]);
|
|
90
|
+
methodAdded = true;
|
|
91
|
+
}
|
|
92
|
+
});
|
|
93
|
+
}
|
|
94
|
+
});
|
|
95
|
+
} else {
|
|
96
|
+
const newSpecifiers = methodAliases.map((alias) =>
|
|
97
|
+
j.importSpecifier(j.identifier(methodNameToReplace), j.identifier(alias))
|
|
98
|
+
);
|
|
99
|
+
|
|
100
|
+
const newImportDeclaration = j.importDeclaration(newSpecifiers, j.literal(newDependency));
|
|
101
|
+
|
|
102
|
+
// Find the index of the first non-import declaration
|
|
103
|
+
const body = root.get().node.program.body;
|
|
104
|
+
const lastImportIndex = body.findIndex((node) => node.type !== 'ImportDeclaration');
|
|
105
|
+
|
|
106
|
+
if (lastImportIndex > -1) {
|
|
107
|
+
// Insert the new import declaration just before the first non-import node
|
|
108
|
+
body.splice(lastImportIndex, 0, newImportDeclaration);
|
|
109
|
+
} else {
|
|
110
|
+
// Check if 'use strict' exists at the beginning
|
|
111
|
+
const hasUseStrict =
|
|
112
|
+
body[0]?.type === 'ExpressionStatement' && body[0]?.expression?.value === 'use strict';
|
|
113
|
+
// Add the new import after 'use strict' if it exists, otherwise at the beginning
|
|
114
|
+
body.splice(hasUseStrict ? 1 : 0, 0, newImportDeclaration);
|
|
115
|
+
}
|
|
116
|
+
}
|
|
117
|
+
}
|
|
118
|
+
};
|