style: apply eslint 8 auto-fixes

Auto-fixed lint errors from ESLint 8 stricter rules:
- import/order: Sorted imports by group (builtin, external, internal)
- import/newline-after-import: Added newlines after imports
- no-else-return: Removed unnecessary else after return
- jest/prefer-to-be: Use toBe/toBeNull for primitive checks

Also fixed a buggy mock in updateVersionMonorepo.spec.js that was
attempting to spy on a non-existent property.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: Sarah Dayan

packages/shipjs-lib/rollup.config.js

@@ -1,6 +1,7 @@
 import { createRequire } from 'module';
-import resolve from '@rollup/plugin-node-resolve';
+
 import commonjs from '@rollup/plugin-commonjs';
+import resolve from '@rollup/plugin-node-resolve';
 
 const require = createRequire(import.meta.url);
 const pkg = require('./package.json');

packages/shipjs-lib/src/lib/config/__tests__/defaultConfig.spec.js

@@ -1,4 +1,5 @@
 import defaultConfig from '../defaultConfig.js';
+
 jest.mock('../../../version', () => '0.5.2');
 const {
   formatCommitMessage,

packages/shipjs-lib/src/lib/config/__tests__/loadConfig.spec.js

@@ -1,7 +1,8 @@
-import loadConfig from '../loadConfig.js';
-import defaultConfig from '../defaultConfig.js';
 import path from 'path';
 
+import defaultConfig from '../defaultConfig.js';
+import loadConfig from '../loadConfig.js';
+
 const baseConfig = expect.objectContaining({});
 
 const d = (dirName) => path.resolve(__dirname, dirName);

packages/shipjs-lib/src/lib/config/__tests__/mergeConfig.spec.js

@@ -20,7 +20,7 @@ describe('mergeConfig', () => {
         },
       }
     );
-    expect(config.slack.default.username).toEqual('Release');
+    expect(config.slack.default.username).toBe('Release');
     expect(config.slack.prepared).toBeInstanceOf(Function);
     expect(config.slack.releaseSuccess).toBeInstanceOf(Function);
   });
@@ -42,7 +42,7 @@ describe('mergeConfig', () => {
         },
       }
     );
-    expect(config.slack.default.username).toEqual('Ship.js');
+    expect(config.slack.default.username).toBe('Ship.js');
     expect(config.slack.prepared).toBeInstanceOf(Function);
     expect(config.slack.releaseSuccess).toBeNull();
   });

packages/shipjs-lib/src/lib/config/loadConfig.js

@@ -1,5 +1,5 @@
-import { resolve } from 'path';
 import { promises as fs, constants } from 'fs';
+import { resolve } from 'path';
 
 import defaultConfig from './defaultConfig.js';
 import mergeConfig from './mergeConfig.js';

packages/shipjs-lib/src/lib/git/__tests__/getLatestCommitMessage.spec.js

@@ -1,5 +1,5 @@
-import getLatestCommitMessage from '../getLatestCommitMessage.js';
 import silentExec from '../../shell/silentExec.js';
+import getLatestCommitMessage from '../getLatestCommitMessage.js';
 
 describe('getLatestCommitMessage', () => {
   it('gets latest commit message', () => {

packages/shipjs-lib/src/lib/git/__tests__/getRemoteBranches.spec.js

@@ -1,5 +1,6 @@
 import silentExec from '../../shell/silentExec.js';
 import getRemoteBranches from '../getRemoteBranches.js';
+
 jest.mock('../../shell/silentExec.js');
 
 describe('getRemoteBranches', () => {

packages/shipjs-lib/src/lib/git/__tests__/hasLocalBranch.spec.js

@@ -1,5 +1,5 @@
-import hasLocalBranch from '../hasLocalBranch.js';
 import silentExec from '../../shell/silentExec.js';
+import hasLocalBranch from '../hasLocalBranch.js';
 
 describe('hasLocalBranch', () => {
   beforeAll(() => {

packages/shipjs-lib/src/lib/git/__tests__/hasTag.spec.js

@@ -1,5 +1,5 @@
-import hasTag from '../hasTag.js';
 import silentExec from '../../shell/silentExec.js';
+import hasTag from '../hasTag.js';
 
 describe('hasTag', () => {
   it('checks if it does not have tag', () => {

packages/shipjs-lib/src/lib/git/__tests__/isWorkingTreeClean.spec.js

@@ -1,5 +1,5 @@
-import isWorkingTreeClean from '../isWorkingTreeClean.js';
 import silentExec from '../../shell/silentExec.js';
+import isWorkingTreeClean from '../isWorkingTreeClean.js';
 
 describe('isWorkingTreeClean', () => {
   it('checks if it is clean', () => {