Merge tag '0.4.1'

LogTape 0.4.1

.github/workflows/main.yaml

@@ -21,6 +21,9 @@ jobs:
     - uses: denoland/setup-deno@v1
       with:
         deno-version: v1.x
+    - uses: oven-sh/setup-bun@v2
+      with:
+        bun-version: latest
     - run: deno task test --coverage=.cov --junit-path=.test-report.xml
     - uses: EnricoMi/publish-unit-test-result-action@v2
       if: runner.os == 'Linux' && always()
@@ -43,6 +46,9 @@ jobs:
         token: ${{ secrets.CODECOV_TOKEN }}
         slug: dahlia/logtape
         file: .cov.lcov
+    - run: deno task dnt
+    - run: bun run ./test_runner.js
+      working-directory: ${{ github.workspace }}/npm/
     - run: deno task check
 
   publish:

CHANGES.md

@@ -9,6 +9,15 @@ Version 0.5.0
 To be released.
 
 
+Version 0.4.1
+-------------
+
+Released on July 2, 2024.
+
+ -  Fix a bug where LogTape failed to load under Node.js when incoporated
+    in a project from JSR.  [[#3], [#4] by Kitson Kelly]
+
+
 Version 0.4.0
 -------------
 
@@ -19,6 +28,8 @@ Released on May 7, 2024.
 
 [#1]: https://github.com/dahlia/logtape/issues/1
 [#2]: https://github.com/dahlia/logtape/pull/2
+[#3]: https://github.com/dahlia/logtape/issues/3
+[#4]: https://github.com/dahlia/logtape/pull/4
 
 
 Version 0.3.1

deno.json

@@ -25,6 +25,7 @@
     "test": "deno test --allow-read --allow-write",
     "coverage": "rm -rf coverage && deno task test --coverage && deno coverage --html coverage",
     "dnt": "deno run -A dnt.ts",
+    "test-all": "deno task test && deno task dnt && cd npm/ && bun run ./test_runner.js && cd ../",
     "hooks:install": "deno run --allow-read=deno.json,.git/hooks/ --allow-write=.git/hooks/ jsr:@hongminhee/deno-task-hooks",
     "hooks:pre-commit": "deno task check",
     "hooks:pre-push": "deno task test"

dnt.ts

@@ -29,6 +29,7 @@ await build({
   entryPoints: ["./logtape/mod.ts"],
   importMap: "./deno.json",
   mappings: {
+    "./logtape/filesink.jsr.ts": "./logtape/filesink.node.ts",
     "./logtape/filesink.deno.ts": "./logtape/filesink.node.ts",
   },
   shims: {

logtape/filesink.deno.ts

@@ -24,8 +24,8 @@ export const denoDriver: RotatingFileSinkDriver<Deno.FsFile> = {
   closeSync(fd) {
     fd.close();
   },
-  statSync: Deno.statSync,
-  renameSync: Deno.renameSync,
+  statSync: globalThis?.Deno.statSync,
+  renameSync: globalThis?.Deno.renameSync,
 };
 
 /**

logtape/filesink.jsr.ts

@@ -0,0 +1,9 @@
+const filesink: Omit<typeof import("./filesink.deno.ts"), "denoDriver"> =
+  await ("Deno" in globalThis
+    ? import("./filesink.deno.ts")
+    : import("./filesink.node.ts"));
+
+export const getFileSink = filesink.getFileSink;
+export const getRotatingFileSink = filesink.getRotatingFileSink;
+
+// cSpell: ignore filesink

logtape/filesink.test.ts

@@ -26,7 +26,6 @@ Deno.test("getFileSink()", () => {
 
 Deno.test("getRotatingFileSink()", () => {
   const path = Deno.makeTempFileSync();
-  console.debug({ path });
   const sink: Sink & Disposable = getRotatingFileSink(path, {
     maxSize: 150,
   });

logtape/mod.ts

@@ -7,7 +7,7 @@ export {
   type LoggerConfig,
   reset,
 } from "./config.ts";
-export { getFileSink, getRotatingFileSink } from "./filesink.deno.ts";
+export { getFileSink, getRotatingFileSink } from "./filesink.jsr.ts";
 export {
   type Filter,
   type FilterLike,