feat: ignore manifest files without version property (#49)

Author: nekomeowww

src/update-files.ts

@@ -74,7 +74,13 @@ async function updateManifestFile(relPath: string, operation: Operation): Promis
 
   const file = await readJsoncFile(relPath, cwd)
 
-  if (isManifest(file.data) && file.data.version !== newVersion) {
+  if (!isManifest(file.data)) {
+    return modified
+  }
+  if (file.data.version == null) {
+    return modified
+  }
+  if (file.data.version !== newVersion) {
     file.modified.push([['version'], newVersion])
     if (isPackageLockManifest(file.data))
       file.modified.push([['packages', '', 'version'], newVersion])

test/update-files.test.ts

@@ -0,0 +1,31 @@
+import { mkdir, readFile, rmdir, writeFile } from 'node:fs/promises'
+import { join } from 'node:path'
+import { cwd } from 'node:process'
+import { afterEach, beforeEach, expect, it } from 'vitest'
+import { Operation } from '../src/operation'
+import { updateFiles } from '../src/update-files'
+
+beforeEach(async () => {
+  mkdir(join(cwd(), 'test', 'update-files', 'testdata'), { recursive: true }).catch(() => { })
+})
+
+afterEach(async () => {
+  rmdir(join(cwd(), 'test', 'update-files', 'testdata'), { recursive: true }).catch(() => { })
+})
+
+it('should skip to modify the manifest file if version field is not specified', async () => {
+  await writeFile(join(cwd(), 'test', 'update-files', 'testdata', 'package.json'), JSON.stringify({}), 'utf8')
+
+  const operation = await Operation.start({
+    cwd: join(cwd(), 'test', 'update-files', 'testdata'),
+    currentVersion: '1.0.0',
+  })
+
+  operation.update({
+    newVersion: '2.0.0',
+  })
+
+  await updateFiles(operation)
+  const updatedPackageJSON = await readFile(join(cwd(), 'test', 'update-files', 'testdata', 'package.json'), 'utf8')
+  expect(JSON.parse(updatedPackageJSON)).toMatchObject({})
+})