fix(tests): update test imports and fix NpmConfig mock

- Updated import paths in test files after file renames:
  - ecosystem/ecosystem.mts → ecosystem/types.mts
  - fs/fs.mts → fs/find-up.mts
  - git/git.mts → git/operations.mts
- Fixed NpmConfig mock to use function keyword instead of arrow
  function for proper constructor support with new operator
- Used vi.hoisted() for proper mock hoisting before imports

Author: jdalton

packages/cli/src/utils/ecosystem/types.test.mts

@@ -6,7 +6,7 @@ import {
   getEcosystemChoicesForMeow,
   isValidEcosystem,
   parseEcosystems,
-} from './ecosystem.mts'
+} from './types.mts'
 
 describe('ecosystem utilities', () => {
   describe('ALL_ECOSYSTEMS', () => {

packages/cli/src/utils/fs/fs.test.mts

@@ -4,7 +4,7 @@ import path from 'node:path'
 
 import { afterEach, beforeEach, describe, expect, it } from 'vitest'
 
-import { findUp } from './fs.mts'
+import { findUp } from './find-up.mts'
 
 describe('fs utilities', () => {
   describe('findUp', () => {

packages/cli/src/utils/git/operations.test.mts

@@ -12,7 +12,7 @@ import {
   gitPushBranch,
   gitResetHard,
   parseGitRemoteUrl,
-} from './git.mts'
+} from './operations.mts'
 
 // Mock spawn.
 vi.mock('@socketsecurity/lib/spawn', () => ({

packages/cli/src/utils/npm/config.test.mts

@@ -1,22 +1,30 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 
-import { getNpmConfig } from './config.mts'
-
-// Mock @npmcli/config.
-vi.mock('@npmcli/config', () => {
-  const MockNpmConfig = vi.fn().mockImplementation(() => ({
+// Create mock constructor and instance in hoisted scope
+const { MockNpmConfig, mockNpmConfigInstance } = vi.hoisted(() => {
+  const mockNpmConfigInstance = {
     load: vi.fn().mockResolvedValue(undefined),
     flat: {
       registry: 'https://registry.npmjs.org/',
       cache: '/home/user/.npm',
       prefix: '/usr/local',
     },
-  }))
-  return {
-    default: MockNpmConfig,
   }
+
+  const MockNpmConfig = vi.fn().mockImplementation(function () {
+    return mockNpmConfigInstance
+  })
+
+  return { MockNpmConfig, mockNpmConfigInstance }
 })
 
+// Mock @npmcli/config.
+vi.mock('@npmcli/config', () => ({
+  default: MockNpmConfig,
+}))
+
+import { getNpmConfig } from './config.mts'
+
 // Mock @npmcli/config/lib/definitions.
 vi.mock('@npmcli/config/lib/definitions', () => ({
   definitions: {},
@@ -31,7 +39,9 @@ vi.mock('./paths.mts', () => ({
 
 describe('npm-config utilities', () => {
   beforeEach(() => {
-    vi.clearAllMocks()
+    // Clear only the mock calls, not the implementation
+    MockNpmConfig.mockClear()
+    vi.mocked(mockNpmConfigInstance.load).mockClear()
   })
 
   describe('getNpmConfig', () => {
@@ -48,48 +58,48 @@ describe('npm-config utilities', () => {
     })
 
     it('uses custom cwd option', async () => {
-      const NpmConfig = (await import('@npmcli/config')).default
+      // Use MockNpmConfig directly
 
       await getNpmConfig({ cwd: '/custom/path' })
 
-      expect(NpmConfig).toHaveBeenCalledWith(
+      expect(MockNpmConfig).toHaveBeenCalledWith(
         expect.objectContaining({
           cwd: '/custom/path',
         }),
       )
     })
 
     it('uses custom env option', async () => {
-      const NpmConfig = (await import('@npmcli/config')).default
+      // Use MockNpmConfig directly
       const customEnv = { NODE_ENV: 'test', FOO: 'bar' }
 
       await getNpmConfig({ env: customEnv })
 
-      expect(NpmConfig).toHaveBeenCalledWith(
+      expect(MockNpmConfig).toHaveBeenCalledWith(
         expect.objectContaining({
           env: customEnv,
         }),
       )
     })
 
     it('uses custom npmPath option', async () => {
-      const NpmConfig = (await import('@npmcli/config')).default
+      // Use MockNpmConfig directly
 
       await getNpmConfig({ npmPath: '/custom/npm/path' })
 
-      expect(NpmConfig).toHaveBeenCalledWith(
+      expect(MockNpmConfig).toHaveBeenCalledWith(
         expect.objectContaining({
           npmPath: '/custom/npm/path',
         }),
       )
     })
 
     it('uses custom platform option', async () => {
-      const NpmConfig = (await import('@npmcli/config')).default
+      // Use MockNpmConfig directly
 
       await getNpmConfig({ platform: 'win32' })
 
-      expect(NpmConfig).toHaveBeenCalledWith(
+      expect(MockNpmConfig).toHaveBeenCalledWith(
         expect.objectContaining({
           platform: 'win32',
         }),
@@ -113,11 +123,11 @@ describe('npm-config utilities', () => {
     })
 
     it('handles execPath option', async () => {
-      const NpmConfig = (await import('@npmcli/config')).default
+      // Use MockNpmConfig directly
 
       await getNpmConfig({ execPath: '/usr/bin/node' })
 
-      expect(NpmConfig).toHaveBeenCalledWith(
+      expect(MockNpmConfig).toHaveBeenCalledWith(
         expect.objectContaining({
           execPath: '/usr/bin/node',
         }),
@@ -127,11 +137,12 @@ describe('npm-config utilities', () => {
     it('calls config.load()', async () => {
       const mockLoad = vi.fn().mockResolvedValue(undefined)
       vi.mocked((await import('@npmcli/config')).default).mockImplementation(
-        () =>
-          ({
+        function () {
+          return {
             load: mockLoad,
             flat: { test: 'value' },
-          }) as any,
+          }
+        } as any,
       )
 
       await getNpmConfig()