Skip to content

Commit c3fd01c

Browse files
authored
feat(ocap-kernel): Implement hierarchical logging (#504)
Closes: #217 This PR constitutes the majority of the lift for the referenced logging ticket. ### `@metamask/logger` - implements a protocol for serialized transfer of log entries over a `@metamask/streams` `DuplexStream` - adds the `injectStream` method to the `Logger` for subscribing to logger notifications - adds `makeArrayTransport` which returns a transport which appends log entries to the supplied array argument ### `@MetaMask/ocap-kernel` - The logger passed to the `Kernel` constructor is plumbed to subcomponents all the way to the vat user code's console endowment and logger power. ### `@ocap/kernel-test` - no longer relies on intercepting `process.stdout` to assert expected test vat operations - utilizes logger tags for distinguishing different vat operations
1 parent 521272f commit c3fd01c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

49 files changed

+1369
-649
lines changed

eslint.config.mjs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -216,9 +216,12 @@ const config = createConfig([
216216
},
217217

218218
{
219-
files: ['packages/nodejs/test/workers/**/*'],
219+
files: [
220+
'packages/nodejs/test/workers/**/*',
221+
'packages/logger/test/workers/**/*',
222+
],
220223
rules: {
221-
// Test node worker files can resolve these imports, even if eslint cannot.
224+
// Test worker files can resolve these imports, even if eslint cannot.
222225
'import-x/no-unresolved': 'off',
223226
},
224227
},

packages/extension/src/iframe.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ async function main(): Promise<void> {
3232
new VatSupervisor({
3333
id: vatId,
3434
kernelStream,
35+
logger: logger.subLogger(vatId),
3536
});
3637

3738
logger.info('VatSupervisor initialized with vatId:', vatId);

packages/kernel-test/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@
5656
"@metamask/kernel-shims": "workspace:^",
5757
"@metamask/kernel-store": "workspace:^",
5858
"@metamask/kernel-utils": "workspace:^",
59+
"@metamask/logger": "workspace:^",
5960
"@metamask/ocap-kernel": "workspace:^",
6061
"@metamask/streams": "workspace:^",
6162
"@metamask/utils": "^11.4.0",

0 commit comments

Comments
 (0)