Skip to content

Commit 7076310

Browse files
fix(upgrade): Remove unused getMajorVersion function (#5138)
Co-authored-by: Nikos Douvlis <nikosdouvlis@gmail.com>
1 parent 7d19f51 commit 7076310

File tree

3 files changed

+5
-15
lines changed

3 files changed

+5
-15
lines changed

.changeset/tiny-moose-appear.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@clerk/upgrade': patch
3+
---
4+
5+
Remove an internal function that was executed but its return value wasn't used. In some instances this function threw an error.

packages/upgrade/src/app.js

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import { Header } from './components/Header.js';
66
import { Scan } from './components/Scan.js';
77
import { SDKWorkflow } from './components/SDKWorkflow.js';
88
import SDKS from './constants/sdks.js';
9-
import { getClerkMajorVersion } from './util/get-clerk-version.js';
109
import guessFrameworks from './util/guess-framework.js';
1110

1211
/**
@@ -34,15 +33,13 @@ export default function App(props) {
3433
const [sdkGuessConfirmed, setSdkGuessConfirmed] = useState(false);
3534
const [sdkGuessAttempted, setSdkGuessAttempted] = useState(false);
3635
const [fromVersion, setFromVersion] = useState(props.fromVersion);
37-
const [fromVersionGuessAttempted, setFromVersionGuessAttempted] = useState(false);
3836

3937
const [toVersion, setToVersion] = useState(props.toVersion);
4038
const [dir, setDir] = useState(props.dir);
4139
const [ignore, setIgnore] = useState(props.ignore ?? []);
4240
const [configComplete, setConfigComplete] = useState(false);
4341
const [configVerified, setConfigVerified] = useState(false);
4442
const [uuid, setUuid] = useState();
45-
let fromVersionGuess = false;
4643

4744
if (yolo) {
4845
setSdks(SDKS.map(s => s.value));
@@ -77,12 +74,6 @@ export default function App(props) {
7774
setSdkGuessAttempted(true);
7875
}
7976

80-
// We try to guess which version of Clerk they are using
81-
if (isEmpty(sdks) && !fromVersion && !fromVersionGuess && !fromVersionGuessAttempted) {
82-
fromVersionGuess = getClerkMajorVersion();
83-
setFromVersionGuessAttempted(true);
84-
}
85-
8677
// No support for v3 or below, sadly
8778
if (parseInt(fromVersion) < 4) {
8879
return <Text color='red'>We're so sorry, but this tool only supports migration from version 4 and above.</Text>;

packages/upgrade/src/util/get-clerk-version.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,6 @@
11
import { readPackageSync } from 'read-pkg';
22
import semverRegex from 'semver-regex';
33

4-
export function getClerkMajorVersion() {
5-
const pkg = readPackageSync();
6-
const clerk = pkg.dependencies.clerk;
7-
return clerk ? semverRegex().exec(clerk)[0][0] : false;
8-
}
9-
104
export function getClerkSdkVersion(sdk) {
115
const pkg = readPackageSync();
126
const clerkSdk = pkg.dependencies[`@clerk/${sdk}`];

0 commit comments

Comments
 (0)