release-it 18.1.2 → 19.0.0-next.0

package/test/tasks.interactive.js

@@ -1,6 +1,7 @@
 import path from 'node:path';
+import { renameSync } from 'node:fs';
+import childProcess from 'node:child_process';
 import test from 'ava';
-import sh from 'shelljs';
 import _ from 'lodash';
 import sinon from 'sinon';
 import Log from '../lib/log.js';
@@ -9,6 +10,7 @@ import Prompt from '../lib/prompt.js';
 import Config from '../lib/config.js';
 import runTasks from '../lib/index.js';
 import Git from '../lib/plugin/git/Git.js';
+import { execOpts } from '../lib/util.js';
 import { mkTmpDir, gitAdd } from './util/helpers.js';
 import ShellStub from './stub/shell.js';
 import { interceptPublish as interceptGitLabPublish } from './stub/gitlab.js';
@@ -68,10 +70,10 @@ test.before(t => {
 test.serial.beforeEach(t => {
   const bare = mkTmpDir();
   const target = mkTmpDir();
-  sh.pushd('-q', bare);
-  sh.exec(`git init --bare .`);
-  sh.exec(`git clone ${bare} ${target}`);
-  sh.pushd('-q', target);
+  process.chdir(bare);
+  childProcess.execSync(`git init --bare .`, execOpts);
+  childProcess.execSync(`git clone ${bare} ${target}`, execOpts);
+  process.chdir(target);
   gitAdd('line', 'file', 'Add file');
   t.context = { bare, target };
 });
@@ -81,7 +83,7 @@ test.serial.afterEach(() => {
 });
 
 test.serial('should run tasks without throwing errors', async t => {
-  sh.mv('.git', 'foo');
+  renameSync('.git', 'foo');
   const { name, latestVersion, version } = await runTasks({}, getContainer());
   t.is(version, '0.0.1');
   t.true(log.obtrusive.firstCall.args[0].includes(`release ${name} (currently at ${latestVersion})`));
@@ -118,7 +120,7 @@ test.serial('should not run hooks for cancelled release-cycle methods', async t
   const { target } = t.context;
   const pkgName = path.basename(target);
   gitAdd(`{"name":"${pkgName}","version":"1.0.0"}`, 'package.json', 'Add package.json');
-  sh.exec('git tag 1.0.0');
+  childProcess.execSync('git tag 1.0.0', execOpts);
 
   const hooks = getHooks(['version', 'git', 'github', 'gitlab', 'npm']);
   const inquirer = { prompt: sandbox.stub().callsFake(([options]) => ({ [options.name]: false })) };
@@ -159,7 +161,7 @@ test.serial('should run "after:*:release" plugin hooks', async t => {
   const pkgName = path.basename(target);
   const owner = path.basename(path.dirname(bare));
   gitAdd(`{"name":"${pkgName}","version":"1.0.0"}`, 'package.json', 'Add package.json');
-  sh.exec('git tag 1.0.0');
+  childProcess.execSync('git tag 1.0.0', execOpts);
   const sha = gitAdd('line', 'file', 'More file');
 
   const git = factory(Git);

package/test/tasks.js

@@ -1,7 +1,8 @@
 import path from 'node:path';
+import childProcess from 'node:child_process';
+import { appendFileSync, mkdirSync, renameSync } from 'node:fs';
 import test from 'ava';
 import semver from 'semver';
-import sh from 'shelljs';
 import _ from 'lodash';
 import sinon from 'sinon';
 import Log from '../lib/log.js';
@@ -9,6 +10,7 @@ import Spinner from '../lib/spinner.js';
 import Config from '../lib/config.js';
 import runTasks from '../lib/index.js';
 import Git from '../lib/plugin/git/Git.js';
+import { execOpts } from '../lib/util.js';
 import { mkTmpDir, gitAdd, getArgs } from './util/helpers.js';
 import ShellStub from './stub/shell.js';
 import {
@@ -60,10 +62,10 @@ test.before(t => {
 test.serial.beforeEach(t => {
   const bare = mkTmpDir();
   const target = mkTmpDir();
-  sh.pushd('-q', bare);
-  sh.exec(`git init --bare .`);
-  sh.exec(`git clone ${bare} ${target}`);
-  sh.pushd('-q', target);
+  process.chdir(bare);
+  childProcess.execSync(`git init --bare .`, execOpts);
+  childProcess.execSync(`git clone ${bare} ${target}`, execOpts);
+  process.chdir(target);
   gitAdd('line', 'file', 'Add file');
   t.context = { bare, target };
 });
@@ -73,28 +75,30 @@ test.serial.afterEach(() => {
 });
 
 test.serial('should run tasks without throwing errors', async t => {
-  sh.mv('.git', 'foo');
+  renameSync('.git', 'foo');
   const { name, latestVersion, version } = await runTasks({}, getContainer());
   t.true(log.obtrusive.firstCall.args[0].includes(`release ${name} (${latestVersion}...${version})`));
   t.regex(log.log.lastCall.args[0], /Done \(in [0-9]+s\.\)/);
 });
 
 test.serial('should run tasks without package.json', async t => {
-  sh.exec('git tag 1.0.0');
+  childProcess.execSync('git tag 1.0.0', execOpts);
   gitAdd('line', 'file', 'Add file');
   const { name } = await runTasks({}, getContainer({ increment: 'major', git: { commit: false } }));
   t.true(log.obtrusive.firstCall.args[0].includes(`release ${name} (1.0.0...2.0.0)`));
   t.regex(log.log.lastCall.args[0], /Done \(in [0-9]+s\.\)/);
   t.is(log.warn.callCount, 0);
   {
-    const { stdout } = sh.exec('git describe --tags --match=* --abbrev=0');
+    const stdout = childProcess.execSync('git describe --tags --match=* --abbrev=0', {
+      encoding: 'utf-8'
+    });
     t.is(stdout.trim(), '2.0.0');
   }
 });
 
 test.serial('should disable plugins', async t => {
   gitAdd('{"name":"my-package","version":"1.2.3"}', 'package.json', 'Add package.json');
-  sh.exec('git tag 1.2.3');
+  childProcess.execSync('git tag 1.2.3', execOpts);
   gitAdd('line', 'file', 'Add file');
   const container = getContainer({ increment: 'minor', git: false, npm: false });
   const { latestVersion, version } = await runTasks({}, container);
@@ -105,12 +109,12 @@ test.serial('should disable plugins', async t => {
 
 test.serial('should run tasks with minimal config and without any warnings/errors', async t => {
   gitAdd('{"name":"my-package","version":"1.2.3"}', 'package.json', 'Add package.json');
-  sh.exec('git tag 1.2.3');
+  childProcess.execSync('git tag 1.2.3', execOpts);
   gitAdd('line', 'file', 'More file');
   await runTasks({}, getContainer({ increment: 'patch' }));
   t.true(log.obtrusive.firstCall.args[0].includes('release my-package (1.2.3...1.2.4)'));
   t.regex(log.log.lastCall.args[0], /Done \(in [0-9]+s\.\)/);
-  const { stdout } = sh.exec('git describe --tags --match=* --abbrev=0');
+  const stdout = childProcess.execSync('git describe --tags --match=* --abbrev=0', { encoding: 'utf-8' });
   t.is(stdout.trim(), '1.2.4');
 });
 
@@ -119,22 +123,22 @@ test.serial('should use pkg.version', async t => {
   await runTasks({}, getContainer({ increment: 'minor' }));
   t.true(log.obtrusive.firstCall.args[0].includes('release my-package (1.2.3...1.3.0)'));
   t.regex(log.log.lastCall.args[0], /Done \(in [0-9]+s\.\)/);
-  const { stdout } = sh.exec('git describe --tags --match=* --abbrev=0');
+  const stdout = childProcess.execSync('git describe --tags --match=* --abbrev=0', { encoding: 'utf-8' });
   t.is(stdout.trim(), '1.3.0');
 });
 
 test.serial('should use pkg.version (in sub dir) w/o tagging repo', async t => {
   gitAdd('{"name":"root-package","version":"1.0.0"}', 'package.json', 'Add package.json');
-  sh.exec('git tag 1.0.0');
-  sh.mkdir('my-package');
-  sh.pushd('-q', 'my-package');
+  childProcess.execSync('git tag 1.0.0', execOpts);
+  mkdirSync('my-package');
+  process.chdir('my-package');
   gitAdd('{"name":"my-package","version":"1.2.3"}', 'package.json', 'Add package.json');
   const container = getContainer({ increment: 'minor', git: { tag: false } });
   const exec = sinon.spy(container.shell, 'exec');
   await runTasks({}, container);
   t.true(log.obtrusive.firstCall.args[0].endsWith('release my-package (1.2.3...1.3.0)'));
   t.regex(log.log.lastCall.args[0], /Done \(in [0-9]+s\.\)/);
-  const { stdout } = sh.exec('git describe --tags --match=* --abbrev=0');
+  const stdout = childProcess.execSync('git describe --tags --match=* --abbrev=0', { encoding: 'utf-8' });
   t.is(stdout.trim(), '1.0.0');
   const npmArgs = getArgs(exec.args, 'npm');
   t.is(npmArgs[5], 'npm version 1.3.0 --no-git-tag-version');
@@ -143,12 +147,12 @@ test.serial('should use pkg.version (in sub dir) w/o tagging repo', async t => {
 
 test.serial('should ignore version in pkg.version and use git tag instead', async t => {
   gitAdd('{"name":"my-package","version":"0.0.0"}', 'package.json', 'Add package.json');
-  sh.exec('git tag 1.1.1');
+  childProcess.execSync('git tag 1.1.1', execOpts);
   gitAdd('line', 'file', 'More file');
   await runTasks({}, getContainer({ increment: 'minor', npm: { ignoreVersion: true } }));
   t.true(log.obtrusive.firstCall.args[0].includes('release my-package (1.1.1...1.2.0)'));
   t.regex(log.log.lastCall.args[0], /Done \(in [0-9]+s\.\)/);
-  const { stdout } = sh.exec('git describe --tags --match=* --abbrev=0');
+  const stdout = childProcess.execSync('git describe --tags --match=* --abbrev=0', { encoding: 'utf-8' });
   t.is(stdout.trim(), '1.2.0');
 });
 
@@ -158,7 +162,7 @@ test.serial('should release all the things (basic)', async t => {
   const pkgName = path.basename(target);
   const owner = path.basename(path.dirname(bare));
   gitAdd(`{"name":"${pkgName}","version":"1.0.0"}`, 'package.json', 'Add package.json');
-  sh.exec('git tag 1.0.0');
+  childProcess.execSync('git tag 1.0.0', execOpts);
   const sha = gitAdd('line', 'file', 'More file');
 
   interceptGitHubAuthentication();
@@ -201,10 +205,10 @@ test.serial('should release with correct tag name', async t => {
   const project = path.basename(bare);
   const pkgName = path.basename(target);
   const owner = path.basename(path.dirname(bare));
-  const { stdout } = sh.exec('git rev-parse --abbrev-ref HEAD');
+  const stdout = childProcess.execSync('git rev-parse --abbrev-ref HEAD', { encoding: 'utf-8' });
   const branchName = stdout.trim();
   gitAdd(`{"name":"${pkgName}","version":"1.0.0"}`, 'package.json', 'Add package.json');
-  sh.exec(`git tag ${pkgName}-${branchName}-1.0.0`);
+  childProcess.execSync(`git tag ${pkgName}-${branchName}-1.0.0`, execOpts);
   const sha = gitAdd('line', 'file', 'More file');
 
   interceptGitHubCreate({
@@ -246,9 +250,9 @@ test.serial('should release all the things (pre-release, github, gitlab)', async
   const owner = path.basename(path.dirname(bare));
   const url = `https://gitlab.com/${owner}/${project}`;
   gitAdd(`{"name":"${pkgName}","version":"1.0.0"}`, 'package.json', 'Add package.json');
-  sh.exec('git tag v1.0.0');
+  childProcess.execSync('git tag v1.0.0', execOpts);
   const sha = gitAdd('line', 'file', 'More file');
-  sh.exec('git push --follow-tags');
+  childProcess.execSync('git push --follow-tags', execOpts);
   const git = factory(Git);
   const ref = (await git.getBranchName()) ?? 'HEAD';
 
@@ -327,13 +331,17 @@ test.serial('should release all the things (pre-release, github, gitlab)', async
     'npm publish . --tag alpha'
   ]);
 
-  const { stdout: commitMessage } = sh.exec('git log --oneline --format=%B -n 1 HEAD');
+  const commitMessage = childProcess.execSync('git log --oneline --format=%B -n 1 HEAD', {
+    encoding: 'utf-8'
+  });
   t.is(commitMessage.trim(), `Release 1.1.0-alpha.0 for ${pkgName} (from 1.0.0)`);
 
-  const { stdout: tagName } = sh.exec('git describe --tags --match=* --abbrev=0');
+  const tagName = childProcess.execSync('git describe --tags --match=* --abbrev=0', { encoding: 'utf-8' });
   t.is(tagName.trim(), 'v1.1.0-alpha.0');
 
-  const { stdout: tagAnnotation } = sh.exec('git for-each-ref refs/tags/v1.1.0-alpha.0 --format="%(contents)"');
+  const tagAnnotation = childProcess.execSync('git for-each-ref refs/tags/v1.1.0-alpha.0 --format="%(contents)"', {
+    encoding: 'utf-8'
+  });
   t.is(tagAnnotation.trim(), `${owner} ${owner}/${project} ${project}`);
 
   t.true(log.obtrusive.firstCall.args[0].endsWith(`release ${pkgName} (1.0.0...1.1.0-alpha.0)`));
@@ -349,7 +357,7 @@ test.serial('should publish pre-release without pre-id with different npm.tag',
   const { target } = t.context;
   const pkgName = path.basename(target);
   gitAdd(`{"name":"${pkgName}","version":"1.0.0"}`, 'package.json', 'Add package.json');
-  sh.exec('git tag v1.0.0');
+  childProcess.execSync('git tag v1.0.0', execOpts);
 
   const container = getContainer({ increment: 'major', preRelease: true, npm: { name: pkgName, tag: 'next' } });
   const exec = sinon.spy(container.shell, 'exec');
@@ -367,7 +375,7 @@ test.serial('should publish pre-release without pre-id with different npm.tag',
     'npm publish . --tag next'
   ]);
 
-  const { stdout } = sh.exec('git describe --tags --match=* --abbrev=0');
+  const stdout = childProcess.execSync('git describe --tags --match=* --abbrev=0', { encoding: 'utf-8' });
   t.is(stdout.trim(), 'v2.0.0-0');
   t.true(log.obtrusive.firstCall.args[0].endsWith(`release ${pkgName} (1.0.0...2.0.0-0)`));
   t.true(log.log.firstCall.args[0].endsWith(`https://www.npmjs.com/package/${pkgName}`));
@@ -459,7 +467,7 @@ test.serial('should use custom changelog command with context', async t => {
   const { bare } = t.context;
   const project = path.basename(bare);
   const owner = path.basename(path.dirname(bare));
-  sh.exec('git tag v1.0.0');
+  childProcess.execSync('git tag v1.0.0', execOpts);
   gitAdd('line', 'file', 'More file');
 
   interceptGitHubAuthentication();
@@ -499,10 +507,10 @@ test.serial('should use custom changelog command with context', async t => {
 {
   test.serial('should run all hooks', async t => {
     gitAdd(`{"name":"hooked","version":"1.0.0","type":"module"}`, 'package.json', 'Add package.json');
-    sh.exec(`npm install ${rootDir}`);
+    childProcess.execSync(`npm install ${rootDir}`, execOpts);
     const plugin = "import { Plugin } from 'release-it'; class MyPlugin extends Plugin {}; export default MyPlugin;";
-    sh.ShellString(plugin).toEnd('my-plugin.js');
 
+    appendFileSync('my-plugin.js', plugin);
     const hooks = {};
     ['before', 'after'].forEach(prefix => {
       ['version', 'git', 'npm', 'my-plugin'].forEach(ns => {

package/test/util/helpers.js

@@ -1,24 +1,26 @@
-import fs from 'node:fs';
+import { appendFileSync, mkdirSync, mkdtempSync, promises } from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
-import sh from 'shelljs';
+import childProcess from 'node:child_process';
+import { execOpts } from '../../lib/util.js';
 
 const mkTmpDir = () => {
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'release-it-'));
+  const dir = mkdtempSync(path.join(os.tmpdir(), 'release-it-'));
   return dir;
 };
 
-const readFile = file => fs.promises.readFile(path.resolve(file), 'utf8');
+const readFile = file => promises.readFile(path.resolve(file), 'utf8');
 
 const gitAdd = (content, filePath, message) => {
   const pathSegments = filePath.split('/').filter(Boolean);
   pathSegments.pop();
   if (pathSegments.length) {
-    sh.mkdir('-p', pathSegments.join('/'));
+    mkdirSync(path.resolve(pathSegments.join('/')), { mode: parseInt('0777', 8), recursive: true });
   }
-  sh.ShellString(content).toEnd(filePath);
-  sh.exec(`git add ${filePath}`);
-  const { stdout } = sh.exec(`git commit -m "${message}"`);
+
+  appendFileSync(filePath, content);
+  childProcess.execSync(`git add ${filePath}`, execOpts);
+  const stdout = childProcess.execSync(`git commit -m "${message}"`, { encoding: 'utf-8' });
   const match = stdout.match(/\[.+([a-z0-9]{7})\]/);
   return match ? match[1] : null;
 };

package/test/util/index.js

@@ -20,6 +20,7 @@ export let factory = (Definition, { namespace, options = {}, container = {} } =
   spinner.show.callsFake(({ enabled = true, task }) => (enabled ? task() : () => {}));
   container.spinner = spinner;
   container.shell = container.shell || new ShellStub({ container });
+
   container.prompt = container.prompt || new Prompt({ container });
   container.shell.cache = { set: () => {}, has: () => false };
 

package/test/util/setup.js

@@ -1,5 +1,3 @@
-import shelljs from 'shelljs';
 import nock from 'nock';
 
-shelljs.config.silent = true;
 nock.disableNetConnect();

package/test/util/sh.js

@@ -0,0 +1,18 @@
+import { spawnSync } from 'node:child_process';
+
+const getCommandAndArgs = input => {
+  if (typeof input !== 'string' || !input.trim()) {
+    throw new Error('Invalid input: expected a non-empty string.');
+  }
+
+  const [command, ...args] = input.trim().split(/\s+/);
+
+  return [command, args];
+};
+
+const exec = (command, opts = { stdio: 'inherit' }) => {
+  const [cmd, args] = getCommandAndArgs(command);
+  return spawnSync(cmd, args, opts);
+};
+
+export default { getCommandAndArgs, exec };