fix: add EOL at the end of updated lock file

lib/file.ts

@@ -146,7 +146,8 @@ async function writeToImportMap(
 async function writeToLockfile(
   update: FileUpdate<"lockfile">,
 ) {
-  const original = await parseLockFileJson(update.path);
+  const content = await Deno.readTextFile(update.path);
+  const original = parseLockFileJson(content);
 
   for await (const dependency of update.dependencies) {
     const specifier = dependency.code.specifier;
@@ -192,7 +193,10 @@ async function writeToLockfile(
       );
     }
   }
-  await Deno.writeTextFile(update.path, JSON.stringify(original, replacer, 2));
+  await Deno.writeTextFile(
+    update.path,
+    JSON.stringify(original, replacer, 2) + (detectEOL(content) ?? EOL),
+  );
 }
 
 function replacer(

lib/lockfile.ts

@@ -68,16 +68,13 @@ export interface LockPart {
  * @param specifier - The URL or path to the lockfile.
  * @returns The parsed JSON object of the lockfile.
  */
-export async function parseLockFileJson(
-  specifier: URL | string,
-): Promise<LockFileJson> {
+export function parseLockFileJson(
+  content: string,
+): LockFileJson {
   try {
-    return ensure(
-      JSON.parse(await Deno.readTextFile(specifier)),
-      isLockFileJson,
-    );
+    return ensure(JSON.parse(content), isLockFileJson);
   } catch (cause) {
-    throw new Error(`Failed to parse lockfile: ${specifier}`, { cause });
+    throw new Error(`Failed to parse lockfile`, { cause });
   }
 }
 
@@ -87,12 +84,12 @@ export async function parseLockFileJson(
  * @param specifier - The URL or path to the lockfile.
  * @returns The parsed `LockFile` object.
  */
-export async function parseLockFile(
+export async function readLockFile(
   specifier: URL | string,
 ): Promise<LockFile> {
   return {
     path: toPath(specifier),
-    data: await parseLockFileJson(specifier),
+    data: parseLockFileJson(await Deno.readTextFile(specifier)),
   };
 }
 
@@ -149,7 +146,7 @@ export async function createLockPart(
   }
   return {
     specifier: dependency,
-    data: await parseLockFileJson(lock.path),
+    data: parseLockFileJson(await Deno.readTextFile(lock.path)),
   };
 }
 

lib/lockfile_test.ts

@@ -2,15 +2,17 @@ import {
   collectUpdateFromLockFile,
   createLockPart,
   createLockPartForEach,
-  parseLockFile,
   parseLockFileJson,
+  readLockFile,
 } from "./lockfile.ts";
 import { assertEquals, assertObjectMatch } from "./std/assert.ts";
 
 Deno.test("parseLockFileJson", async () =>
   assertObjectMatch(
-    await parseLockFileJson(
-      new URL("../test/data/lockfile/deno.updated.lock", import.meta.url),
+    parseLockFileJson(
+      await Deno.readTextFile(
+        new URL("../test/data/lockfile/deno.updated.lock", import.meta.url),
+      ),
     ),
     {
       version: "3",
@@ -55,7 +57,7 @@ Deno.test("createLockPart - npm:hono", async () => {
 
 Deno.test("createLockPartForEach", async () => {
   const updated = await createLockPartForEach(
-    await parseLockFile(
+    await readLockFile(
       new URL("../test/data/lockfile/deno.lock", import.meta.url),
     ),
   );
@@ -95,7 +97,7 @@ Deno.test("createLockPartForEach", async () => {
 
 Deno.test("createLockPartForEach - no updates", async () => {
   const updated = await createLockPartForEach(
-    await parseLockFile(
+    await readLockFile(
       new URL("../test/data/lockfile/deno.lock", import.meta.url),
     ),
     false,
@@ -143,7 +145,7 @@ Deno.test("createLockPartForEach - no updates", async () => {
 
 Deno.test("collectUpdateFromLockFile", async () => {
   const updates = await collectUpdateFromLockFile(
-    await parseLockFile(
+    await readLockFile(
       new URL("../test/data/lockfile/deno.lock", import.meta.url),
     ),
   );
@@ -190,7 +192,7 @@ Deno.test("collectUpdateFromLockFile", async () => {
 
 Deno.test("collectUpdateFromLockFile - with a patch", async () => {
   const updates = await collectUpdateFromLockFile(
-    await parseLockFile(
+    await readLockFile(
       new URL("../test/data/lockfile/deno.lock", import.meta.url),
     ),
     await createLockPart("npm:hono@^3"),

lib/update.ts

@@ -24,7 +24,7 @@ import {
   createLockPart,
   LockFile,
   LockPart,
-  parseLockFile,
+  readLockFile,
 } from "./lockfile.ts";
 
 export type SourceType = "import_map" | "module" | "lockfile";
@@ -186,7 +186,7 @@ export async function collect(
         ? await maybeFile(new URL("deno.lock", toUrl(importMapPath)))
         : undefined))
     : undefined;
-  const lockFile = lockFilePath ? await parseLockFile(lockFilePath) : undefined;
+  const lockFile = lockFilePath ? await readLockFile(lockFilePath) : undefined;
 
   const urls = [from].flat().map((path) => toUrl(path));
   const [jsons, esms] = partition(urls, isJsonPath);

test/snapshots/file_test.ts.snap

@@ -123,7 +123,8 @@ snapshot[`write - lockfile_not_importable/mod.ts 2`] = `
   "remote": {
     "https://deno.land/std@0.218.0/version.ts": "cfb73e2f2b628978b2b70585b941c5ef5ccbf1dc06e76967206115c74ecfce49"
   }
-}',
+}
+',
 ]
 `;
 
@@ -1017,7 +1018,8 @@ snapshot[`write - lockfile/deno.json 2`] = `
       "npm:hono@^3"
     ]
   }
-}',
+}
+',
 ]
 `;
 
@@ -1170,7 +1172,8 @@ snapshot[`write - lockfile/mod.ts 2`] = `
       "npm:hono@^3"
     ]
   }
-}',
+}
+',
 ]
 `;