fix: version update issue (#70)

Co-authored-by: jiangzs <2373806028@qq.com>

Author: Blithe-Chiang

src/fs.ts

@@ -42,7 +42,7 @@ export async function readJsoncFile(name: string, cwd: string): Promise<JsonFile
 export async function writeJsoncFile(file: JsonFile): Promise<void> {
   let newJSON = file.text
   for (const [key, value] of file.modified) {
-    const edit = (jsonc.modify(file.text, key, value, {}))
+    const edit = (jsonc.modify(newJSON, key, value, {}))
     newJSON = jsonc.applyEdits(newJSON, edit)
   }
 

src/update-files.ts

@@ -37,7 +37,7 @@ export async function updateFiles(operation: Operation): Promise<Operation> {
  * @returns - `true` if the file was actually modified
  */
 async function updateFile(relPath: string, operation: Operation): Promise<boolean> {
-  if (!existsSync(relPath)) {
+  if (!existsSync(path.join(operation.options.cwd, relPath))) {
     return false
   }
 

test/update-files.test.ts

@@ -29,3 +29,49 @@ it('should skip to modify the manifest file if version field is not specified',
   const updatedPackageJSON = await readFile(join(cwd(), 'test', 'update-files', 'testdata', 'package.json'), 'utf8')
   expect(JSON.parse(updatedPackageJSON)).toMatchObject({})
 })
+
+it('should update the manifest file correctly', async () => {
+  await writeFile(join(cwd(), 'test', 'update-files', 'testdata', 'package-lock.json'), JSON.stringify(
+    {
+      "name": "example",
+      "version": "1.0.43",
+      "lockfileVersion": 2,
+      "requires": true,
+      "packages": {
+        "": {
+          "name": "example",
+          "version": "1.0.43",
+          "hasInstallScript": true,
+          "dependencies": {}
+        }
+      }
+    }, null, 2
+  ), '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-lock.json'), 'utf8')
+  expect(JSON.parse(updatedPackageJSON)).toMatchObject({
+    "name": "example",
+    "version": "2.0.0",
+    "lockfileVersion": 2,
+    "requires": true,
+    "packages": {
+      "": {
+        "name": "example",
+        "version": "2.0.0",
+        "hasInstallScript": true,
+        "dependencies": {}
+      }
+    }
+  })
+})
+