Skip to content

Conversation

@grypez
Copy link
Contributor

@grypez grypez commented Oct 10, 2024

Changes

@ocap/extension

  • Moves the kernel storage logic out of the kernel worker into a new sqlite-kernel-store file.

@ocap/kernel

  • Introduces a KernelStore interface to be supplied to the Kernel constructor.
  • Relocates the handling of kernel messages from @ocap/extension's kernel worker into Kernel.

@grypez grypez changed the base branch from main to grypez/move-kernel-to-dedicated-worker-with-message-channel October 10, 2024 17:26
@grypez grypez force-pushed the grypez/move-kernel-to-dedicated-worker-with-message-channel branch 3 times, most recently from cb6e33f to 70e43cc Compare October 10, 2024 18:37
@grypez grypez force-pushed the grypez/offscreen-doc-with-message-channel branch from b3b1fa4 to edda13e Compare October 10, 2024 19:07
@grypez grypez changed the title Close offscreen doc issue with message channel refactor(extension,kernel): Move kernel message handling to Kernel Oct 10, 2024
@grypez
Copy link
Contributor Author

grypez commented Oct 10, 2024

#98
Kernel.ts:60 ends with a call to .catch.
Screenshot 2024-10-10 at 2 35 32 PM

Copy link
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice 👌

Comment on lines 29 to 30
// Hopefully removed when we get to n+1 vats.
readonly #defaultVat: PromiseKit<Vat>;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤣

@grypez grypez force-pushed the grypez/move-kernel-to-dedicated-worker-with-message-channel branch from 217cb81 to cc274b8 Compare October 10, 2024 22:36
Base automatically changed from grypez/move-kernel-to-dedicated-worker-with-message-channel to main October 10, 2024 22:38
@grypez grypez force-pushed the grypez/offscreen-doc-with-message-channel branch 5 times, most recently from b2ff125 to 16f1b1f Compare October 10, 2024 23:27
@grypez grypez force-pushed the grypez/offscreen-doc-with-message-channel branch from 16f1b1f to af6ee52 Compare October 10, 2024 23:34
@grypez grypez marked this pull request as ready for review October 10, 2024 23:37
@grypez grypez requested a review from a team as a code owner October 10, 2024 23:37
Copy link
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@grypez grypez merged commit f29baa4 into main Oct 11, 2024
@grypez grypez deleted the grypez/offscreen-doc-with-message-channel branch October 11, 2024 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants