internal: remove version/preversion scripts (#4603)

these are now covered by "npm run release:prepare"

-- removes need to add `ignore-scripts=false` when running `npm version`

Author: yaacovCR

package.json

@@ -29,8 +29,6 @@
     "node": "^20.0.0 || ^22.0.0 || ^24.0.0 || >=25.0.0"
   },
   "scripts": {
-    "preversion": "bash -c '. ./resources/checkgit.sh && npm ci --ignore-scripts'",
-    "version": "node --import ./resources/register-ts-node.js  resources/gen-version.ts && npm test && git add src/version.ts",
     "fuzzonly": "mocha --full-trace src/**/__tests__/**/*-fuzz.ts",
     "changelog": "node --import ./resources/register-ts-node.js  resources/gen-changelog.ts",
     "release:prepare": "node --import ./resources/register-ts-node.js resources/release-prepare.ts",

resources/checkgit.sh

@@ -9,9 +9,24 @@ try {
   process.exit(1);
 }
 
-console.log('Running npm version without creating a tag...');
+console.log('Installing dependencies...');
+npm().ci('--ignore-scripts');
+
+console.log('Bumping package version without creating a tag...');
 npm().version(...args.npmVersionArgs, '--no-git-tag-version');
 
+console.log('Updating src/version.ts...');
+npm().exec(
+  '--',
+  'node',
+  '--import',
+  './resources/register-ts-node.js',
+  'resources/gen-version.ts',
+);
+
+console.log('Running test suite...');
+npm().run('test');
+
 const { version } = readPackageJSON();
 console.log(`Generating changelog for v${version}...`);
 const changelogArgs = ['--silent', 'changelog'];
@@ -97,6 +112,13 @@ function validateBranchState(releaseBranch: string): void {
       'Working directory must be clean before running release:prepare.',
     );
   }
+  const branchStatus = git().status('--porcelain', '--branch');
+  const branchSummary = branchStatus.split('\n')[0] ?? '';
+  if (/\[[^\]]+\]/.test(branchSummary)) {
+    throw new Error(
+      `Current branch "${checkedBranch}" is not up to date with its upstream.`,
+    );
+  }
 
   let releaseBranchHead: string;
   try {