@napi-rs/cli 3.7.2 → 3.7.3

package/src/utils/__tests__/__snapshots__/version.spec.ts.md

@@ -0,0 +1,22 @@
+# Snapshot report for `src/utils/__tests__/version.spec.ts`
+
+The actual snapshot is saved in `version.spec.ts.snap`.
+
+Generated by [AVA](https://avajs.dev).
+
+## should generate correct napi engine requirement
+
+> Snapshot 1
+
+    [
+      '>= 8.6.0 < 9 || >= 9.0.0 < 10 || >= 10.0.0',
+      '>= 8.10.0 < 9 || >= 9.3.0 < 10 || >= 10.0.0',
+      '>= 6.14.2 < 7 || >= 8.11.2 < 9 || >= 9.11.0 < 10 || >= 10.0.0',
+      '>= 10.16.0 < 11 || >= 11.8.0 < 12 || >= 12.0.0',
+      '>= 10.17.0 < 11 || >= 12.11.0 < 13 || >= 13.0.0',
+      '>= 10.20.0 < 11 || >= 12.17.0 < 13 || >= 14.0.0',
+      '>= 10.23.0 < 11 || >= 12.19.0 < 13 || >= 14.12.0 < 15 || >= 15.0.0',
+      '>= 12.22.0 < 13 || >= 14.17.0 < 15 || >= 15.12.0 < 16 || >= 16.0.0',
+      '>= 18.17.0 < 19 || >= 20.3.0 < 21 || >= 21.1.0',
+      '>= 22.14.0 < 23 || >= 23.6.0',
+    ]

package/src/utils/__tests__/config.spec.ts

@@ -0,0 +1,81 @@
+import { unlink, writeFile } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+
+import ava, { type TestFn } from 'ava'
+
+import {
+  type CommonPackageJsonFields,
+  type UserNapiConfig,
+  readNapiConfig,
+} from '../config.js'
+
+const NON_EXISTENT_FILE = 'non-existent-file'
+
+const test = ava as TestFn<{
+  configPath: string
+  packageJson: string
+  pkgJson: CommonPackageJsonFields
+  config: UserNapiConfig
+}>
+
+test.before(async (t) => {
+  const tmp = tmpdir()
+  const configPath = join(tmp, 'napi.json')
+  const packageJson = join(tmp, 'package.json')
+  const pkgJson = {
+    name: '@napi-rs/testing',
+    version: '0.0.0',
+    napi: {
+      binaryName: 'testing',
+      packageName: '@napi-rs/testing',
+      targets: [
+        'x86_64-unknown-linux-gnu',
+        'x86_64-pc-windows-msvc',
+        'x86_64-apple-darwin',
+      ],
+    },
+  }
+  await writeFile(packageJson, JSON.stringify(pkgJson, null, 2))
+  const config = {
+    binaryName: 'testing',
+    packageName: '@node-rs/testing',
+    targets: [
+      'x86_64-unknown-linux-gnu',
+      'x86_64-apple-darwin',
+      'aarch64-apple-darwin',
+    ],
+  }
+  await writeFile(configPath, JSON.stringify(config, null, 2))
+  t.context = { configPath, config, packageJson, pkgJson }
+})
+
+test.after(async (t) => {
+  await unlink(t.context.configPath)
+  await unlink(t.context.packageJson)
+})
+
+test('should throw if package.json not found', async (t) => {
+  await t.throwsAsync(() => readNapiConfig(NON_EXISTENT_FILE), {
+    message: `package.json not found at ${NON_EXISTENT_FILE}`,
+  })
+})
+
+test('should throw if napi.json not found', async (t) => {
+  const { packageJson } = t.context
+  await t.throwsAsync(() => readNapiConfig(packageJson, NON_EXISTENT_FILE), {
+    message: `NAPI-RS config not found at ${NON_EXISTENT_FILE}`,
+  })
+})
+
+test('should be able to read config from package.json', async (t) => {
+  const { packageJson } = t.context
+  const config = await readNapiConfig(packageJson)
+  t.snapshot(config)
+})
+
+test('should be able to read config from napi.json', async (t) => {
+  const { packageJson, configPath } = t.context
+  const config = await readNapiConfig(packageJson, configPath)
+  t.snapshot(config)
+})

package/src/utils/__tests__/metadata.spec.ts

@@ -0,0 +1,28 @@
+import { unlink, writeFile } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+
+import test from 'ava'
+
+import { parseMetadata } from '../metadata.js'
+
+test('should surface child process startup errors', async (t) => {
+  const manifestPath = join(
+    tmpdir(),
+    `napi-rs-metadata-${process.pid}-${Date.now()}.toml`,
+  )
+  const originalPath = process.env.PATH
+
+  await writeFile(manifestPath, '[package]\nname = "test"\nversion = "0.0.0"\n')
+  process.env.PATH = ''
+
+  try {
+    const error = await t.throwsAsync(() => parseMetadata(manifestPath))
+    t.truthy(error)
+    t.is(error?.message, 'cargo metadata failed to run')
+    t.is((error?.cause as NodeJS.ErrnoException | undefined)?.code, 'ENOENT')
+  } finally {
+    process.env.PATH = originalPath
+    await unlink(manifestPath)
+  }
+})

package/src/utils/__tests__/misc.spec.ts

@@ -0,0 +1,56 @@
+import { existsSync } from 'node:fs'
+import { mkdtemp, readFile, rm, writeFile } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+
+import ava, { type TestFn } from 'ava'
+
+import { updatePackageJson } from '../misc.js'
+
+const test = ava as TestFn<{
+  tmpDir: string
+}>
+
+test.beforeEach(async (t) => {
+  t.context = {
+    tmpDir: await mkdtemp(join(tmpdir(), 'napi-rs-misc-spec-')),
+  }
+})
+
+test.afterEach.always(async (t) => {
+  if (existsSync(t.context.tmpDir)) {
+    await rm(t.context.tmpDir, { recursive: true, force: true })
+  }
+})
+
+test('updatePackageJson merges nested objects instead of overwriting them', async (t) => {
+  const packageJsonPath = join(t.context.tmpDir, 'package.json')
+
+  await writeFile(
+    packageJsonPath,
+    JSON.stringify(
+      {
+        name: 'fixture',
+        version: '1.0.0',
+        optionalDependencies: {
+          fsevents: '^2.3.3',
+        },
+      },
+      null,
+      2,
+    ),
+  )
+
+  await updatePackageJson(packageJsonPath, {
+    optionalDependencies: {
+      '@napi-rs/fixture-darwin-arm64': '1.0.1',
+    },
+  })
+
+  const packageJson = JSON.parse(await readFile(packageJsonPath, 'utf8'))
+
+  t.deepEqual(packageJson.optionalDependencies, {
+    fsevents: '^2.3.3',
+    '@napi-rs/fixture-darwin-arm64': '1.0.1',
+  })
+})

package/src/utils/__tests__/target.spec.ts

@@ -0,0 +1,19 @@
+import os from 'os'
+
+import test from 'ava'
+
+import {
+  parseTriple,
+  getSystemDefaultTarget,
+  AVAILABLE_TARGETS,
+} from '../target.js'
+
+test('should parse triple correctly', (t) => {
+  t.snapshot(AVAILABLE_TARGETS.map(parseTriple))
+})
+
+test('should get system default target correctly', (t) => {
+  const target = getSystemDefaultTarget()
+
+  t.is(target.platform, os.platform())
+})

package/src/utils/__tests__/typegen.spec.ts

@@ -0,0 +1,85 @@
+import { join } from 'path'
+import { fileURLToPath } from 'url'
+
+import test from 'ava'
+
+import { correctStringIdent, processTypeDef } from '../typegen.js'
+
+test('should ident string correctly', (t) => {
+  const input = `
+  /**
+   * should keep
+   * class A {
+   * foo = () => {}
+   *   bar = () => {}
+   * }
+   */
+  class A {
+    foo() {
+      a = b
+    }
+
+  bar = () => {
+
+  }
+      boz = 1
+    }
+
+  namespace B {
+      namespace C {
+  type D = A
+      }
+  }
+`
+  t.snapshot(correctStringIdent(input, 0), 'original ident is 0')
+  t.snapshot(correctStringIdent(input, 2), 'original ident is 2')
+})
+
+test('should process type def correctly', async (t) => {
+  const { dts } = await processTypeDef(
+    join(
+      fileURLToPath(import.meta.url),
+      '../',
+      '__fixtures__',
+      'napi_type_def',
+    ),
+    true,
+  )
+
+  t.snapshot(dts)
+})
+
+test('should process type def with noConstEnum correctly', async (t) => {
+  const { dts } = await processTypeDef(
+    join(
+      fileURLToPath(import.meta.url),
+      '../',
+      '__fixtures__',
+      'napi_type_def',
+    ),
+    false,
+  )
+
+  t.snapshot(dts)
+})
+
+// The next two tests use a minimal fixture (one numeric + one string
+// enum) to keep snapshots small and focused on the flag's behavior.
+const flagFixture = join(
+  fileURLToPath(import.meta.url),
+  '../',
+  '__fixtures__',
+  'runtime_string_enum_flag',
+)
+
+test('should process type def with noConstEnum and runtimeStringEnum correctly', async (t) => {
+  const { dts } = await processTypeDef(flagFixture, false, true)
+
+  t.snapshot(dts)
+})
+
+test('runtimeStringEnum is a no-op when constEnum is set', async (t) => {
+  const { dts } = await processTypeDef(flagFixture, true, true)
+
+  t.snapshot(dts)
+})

package/src/utils/__tests__/version.spec.ts

@@ -0,0 +1,7 @@
+import test from 'ava'
+
+import { napiEngineRequirement, SUPPORTED_NAPI_VERSIONS } from '../version.js'
+
+test('should generate correct napi engine requirement', (t) => {
+  t.snapshot(SUPPORTED_NAPI_VERSIONS.map(napiEngineRequirement))
+})

package/src/utils/version.ts

@@ -8,6 +8,7 @@ export enum NapiVersion {
   Napi7,
   Napi8,
   Napi9,
+  Napi10,
 }
 
 /// because node support new napi version in some minor version updates, so we might meet such situation:
@@ -23,6 +24,7 @@ const NAPI_VERSION_MATRIX = new Map<NapiVersion, string>([
   [NapiVersion.Napi7, '10.23.0 | 12.19.0 | 14.12.0 | 15.0.0'],
   [NapiVersion.Napi8, '12.22.0 | 14.17.0 | 15.12.0 | 16.0.0'],
   [NapiVersion.Napi9, '18.17.0 | 20.3.0 | 21.1.0'],
+  [NapiVersion.Napi10, '22.14.0 | 23.6.0'],
 ])
 
 export const SUPPORTED_NAPI_VERSIONS = Object.values(NapiVersion).filter(