Skip to content

fix(jupyter): support IPC connection files#35202

Open
quillaid wants to merge 1 commit into
denoland:mainfrom
quillaid:quillaid/jupyter-ipc-transport
Open

fix(jupyter): support IPC connection files#35202
quillaid wants to merge 1 commit into
denoland:mainfrom
quillaid:quillaid/jupyter-ipc-transport

Conversation

@quillaid

Copy link
Copy Markdown

Summary

  • Support Jupyter connection files that set transport to ipc.
  • Bind Deno Jupyter kernel sockets as Unix sockets using the Jupyter IPC <ip>-<port> convention.
  • Update the integration-test client to accept complete ipc://... endpoints and add Unix-only IPC coverage for heartbeat and kernel_info_reply.

Validation

  • ./tools/format.js cli/js/jupyter_kernel.js tests/integration/jupyter_client.rs tests/integration/jupyter_tests.rs passed before the host cleanup.
  • git diff --check passed.
  • Full local integration validation was not completed. Building target/debug/deno for the targeted integration test was stopped because host disk pressure dropped free space below 8G while Deno target/ grew large. No further local Deno build/test was run after cleanup.

Closes #35201

Disclosure: this PR was AI-assisted.

@deno-cla-assistant

Copy link
Copy Markdown

Deno Individual Contributor License Agreement

The following contributors need to sign the CLA before this PR can be merged:

Click here to review and sign the CLA | Re-run CLA check


This is an automated message from CLA Assistant

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.

deno jupyter should support IPC connection files

1 participant