Revert "fix: avoid static napi cli type resolution"

This reverts commit 1083043.

Author: wan9chi

packages/cli/build.ts

@@ -26,28 +26,16 @@ import { dirname, join, relative } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { parseArgs } from 'node:util';
 
+import { createBuildCommand, NapiCli } from '@napi-rs/cli';
 import { format } from 'oxfmt';
 
 import { generateLicenseFile } from '../../scripts/generate-license.js';
 import corePkg from '../core/package.json' with { type: 'json' };
 
 const projectDir = dirname(fileURLToPath(import.meta.url));
-const requireFromBuildScript = createRequire(import.meta.url);
 const TEST_PACKAGE_NAME = 'vitest';
 const CORE_PACKAGE_NAME = '@voidzero-dev/vite-plus-core';
 
-type NapiBuildOptions = Record<string, unknown> & { dtsHeader?: string };
-type NapiBuildOutput = { kind: string; path: string };
-type NapiBuildCommand = { getOptions(): NapiBuildOptions };
-type NapiCliInstance = {
-  build(options: NapiBuildOptions): Promise<{ task: Promise<NapiBuildOutput[]> }>;
-};
-
-const napiRsCli = requireFromBuildScript('@napi-rs/cli') as {
-  createBuildCommand(this: void, args: string[]): NapiBuildCommand;
-  NapiCli: new () => NapiCliInstance;
-};
-
 // Browser providers projected under ./test/* and ./test/browser/providers/* so the
 // public surface matches what the deleted `@voidzero-dev/vite-plus-test` wrapper exposed.
 // Each entry maps the upstream package name to the short provider name used in the
@@ -155,10 +143,10 @@ await copyBundledDocs();
 await syncReadmeFromRoot();
 
 async function buildNapiBinding() {
-  const buildCommand = napiRsCli.createBuildCommand(napiArgs);
+  const buildCommand = createBuildCommand(napiArgs);
   const passedInOptions = buildCommand.getOptions();
 
-  const cli = new napiRsCli.NapiCli();
+  const cli = new NapiCli();
 
   const bindingFeatures = ['rolldown'];
   const { dtsHeader } = (
@@ -363,7 +351,8 @@ async function syncTestPackageExports() {
 
   // Resolve vitest's package.json via Node's resolver so we always read the
   // currently installed copy — packages/test/ no longer exists.
-  const testPkgPath = requireFromBuildScript.resolve(`${TEST_PACKAGE_NAME}/package.json`, {
+  const require = createRequire(import.meta.url);
+  const testPkgPath = require.resolve(`${TEST_PACKAGE_NAME}/package.json`, {
     paths: [projectDir],
   });
   const cliPkgPath = join(projectDir, 'package.json');
@@ -418,9 +407,7 @@ async function syncTestPackageExports() {
   for (const { pkg, short } of BROWSER_PROVIDER_PACKAGES) {
     let providerPkgPath: string;
     try {
-      providerPkgPath = requireFromBuildScript.resolve(`${pkg}/package.json`, {
-        paths: [projectDir],
-      });
+      providerPkgPath = require.resolve(`${pkg}/package.json`, { paths: [projectDir] });
     } catch (err) {
       console.warn(`  Skipping ${pkg} — not installed: ${(err as Error).message}`);
       continue;
@@ -491,7 +478,7 @@ async function syncTestPackageExports() {
   // `oxlint-plugin.ts` autofixes `@vitest/browser/client` →
   // `vite-plus/test/client` and `@vitest/browser/locators` →
   // `vite-plus/test/locators`, so the runtime targets MUST resolve.
-  const bareBrowserShims = await createBareBrowserShims(requireFromBuildScript, testDistDir);
+  const bareBrowserShims = await createBareBrowserShims(require, testDistDir);
   for (const [cliPath, exportValue] of Object.entries(bareBrowserShims)) {
     generatedExports[cliPath] = exportValue;
     console.log(`  Created ${cliPath}`);