refactor: promote logger to its own package (#493)

Refs: #217 

- creates a new `@ocap/logger` package
- moves the `Logger` class from `@ocap/utils` to `@ocap/logger`
- replaces deprecated `makeLogger` calls with `new Logger` in refactored
packages
- standardizes log labels to kebab-lower-case in affected logger
construction routines
- removes the deprecated `makeLogger` factory from the monorepo

Author: grypez

package.json

@@ -114,7 +114,9 @@
       "vite>sass>@parcel/watcher": false,
       "vitest>@vitest/browser>webdriverio>@wdio/utils>edgedriver": false,
       "vitest>@vitest/browser>webdriverio>@wdio/utils>geckodriver": false,
-      "vitest>@vitest/mocker>msw": false
+      "vitest>@vitest/mocker>msw": false,
+      "@ocap/cli>@ocap/logger>@ocap/streams": true,
+      "eslint-plugin-import-x>unrs-resolver": false
     }
   },
   "resolutions": {

packages/cli/package.json

@@ -38,6 +38,7 @@
     "@endo/promise-kit": "^1.1.10",
     "@metamask/snaps-utils": "^9.1.0",
     "@metamask/utils": "^11.4.0",
+    "@ocap/logger": "workspace:^",
     "@ocap/shims": "workspace:^",
     "@ocap/utils": "workspace:^",
     "@types/node": "^22.13.1",

packages/cli/src/app.ts

@@ -1,6 +1,6 @@
 import '@endo/init';
 
-import { Logger } from '@ocap/utils';
+import { Logger } from '@ocap/logger';
 import path from 'node:path';
 import yargs from 'yargs';
 import { hideBin } from 'yargs/helpers';

packages/cli/src/commands/bundle.test.ts

@@ -1,4 +1,4 @@
-import { Logger } from '@ocap/utils';
+import type { Logger } from '@ocap/logger';
 import { readFile, rm } from 'fs/promises';
 import { basename } from 'path';
 import { describe, it, expect, vi, beforeEach, afterAll } from 'vitest';
@@ -13,11 +13,14 @@ import { fileExists } from '../file.ts';
 const mocks = vi.hoisted(() => {
   return {
     endoBundleSource: vi.fn(),
-    Logger: vi.fn(() => ({
-      info: vi.fn(),
-      error: vi.fn(),
-      subLogger: vi.fn(),
-    })),
+    Logger: vi.fn(
+      () =>
+        ({
+          info: vi.fn(),
+          error: vi.fn(),
+          subLogger: vi.fn(),
+        }) as unknown as Logger,
+    ),
     isDirectory: vi.fn(),
   };
 });
@@ -28,7 +31,7 @@ vi.mock('@endo/bundle-source', () => ({
 
 vi.mock('@endo/init', () => ({}));
 
-vi.mock('@ocap/utils', () => ({
+vi.mock('@ocap/logger', () => ({
   Logger: mocks.Logger,
 }));
 
@@ -54,7 +57,7 @@ describe('bundle', async () => {
   beforeEach(async () => {
     await deleteTestBundles();
     vi.resetModules();
-    logger = new Logger();
+    logger = mocks.Logger();
     vi.resetAllMocks();
   });
 

packages/cli/src/commands/bundle.ts

@@ -1,6 +1,6 @@
 import '@endo/init';
 import endoBundleSource from '@endo/bundle-source';
-import { Logger } from '@ocap/utils';
+import { Logger } from '@ocap/logger';
 import { glob } from 'glob';
 import { writeFile } from 'node:fs/promises';
 import { resolve, join } from 'node:path';

packages/cli/src/commands/watch.test.ts

@@ -1,6 +1,6 @@
 import '@ocap/shims/endoify';
 
-import { Logger } from '@ocap/utils';
+import { Logger } from '@ocap/logger';
 import { watch } from 'chokidar';
 import type { FSWatcher } from 'chokidar';
 import type { Stats } from 'fs';

packages/cli/src/commands/watch.ts

@@ -1,5 +1,5 @@
 import { makePromiseKit } from '@endo/promise-kit';
-import { Logger } from '@ocap/utils';
+import { Logger } from '@ocap/logger';
 import { watch } from 'chokidar';
 import type { FSWatcher, MatchFunction } from 'chokidar';
 import { unlink } from 'fs/promises';

packages/cli/tsconfig.build.json

@@ -9,6 +9,7 @@
     "types": ["ses", "node"]
   },
   "references": [
+    { "path": "../logger/tsconfig.build.json" },
     { "path": "../utils/tsconfig.build.json" },
     { "path": "../shims/tsconfig.build.json" }
   ],

packages/cli/tsconfig.json

@@ -6,7 +6,11 @@
     "types": ["ses", "vitest", "node"],
     "noEmit": true
   },
-  "references": [{ "path": "../test-utils" }, { "path": "../utils" }],
+  "references": [
+    { "path": "../logger" },
+    { "path": "../test-utils" },
+    { "path": "../utils" }
+  ],
   "include": [
     "../../vitest.config.ts",
     "./src/**/*.ts",

packages/extension/package.json

@@ -50,6 +50,7 @@
     "@metamask/utils": "^11.4.0",
     "@ocap/errors": "workspace:^",
     "@ocap/kernel": "workspace:^",
+    "@ocap/logger": "workspace:^",
     "@ocap/rpc-methods": "workspace:^",
     "@ocap/shims": "workspace:^",
     "@ocap/store": "workspace:^",