Update runtimed to fix compatibility issue with the Ark kernel

[lionel-]

Closes #40888

This updates runtimed to the latest version, which handles the "starting" variant of execution_state. It actually handles a bunch of other variants that are not documented in the protocol (see https://jupyter-client.readthedocs.io/en/stable/messaging.html#kernel-status), like "starting", "terminating", etc. I added implementations for these variants as well.

Release Notes:

• Fixed issue that prevented the Ark kernel from working in Zed (Jupyter client can't unserialise status message with "starting" execution_state #40888).

[lionel-]

A build issue appeared with the update.

I don't understand the details here, but I fixed it following existing code at

zed/crates/client/src/proxy/http_proxy.rs

Lines 67 to 98 in 14b41b1

	#[cfg(any(target_os = "windows", target_os = "macos"))]
	async fn https_connect<T>(
	stream: T,
	target: (&str, u16),
	auth: Option<HttpProxyAuthorization<'_>>,
	proxy_domain: &str,
	) -> Result<Box<dyn AsyncReadWrite>>
	where
	T: AsyncReadWrite,
	{
	let tls_connector = TlsConnector::from(native_tls::TlsConnector::new()?);
	let stream = tls_connector.connect(proxy_domain, stream).await?;
	http_connect(stream, target, auth).await
	}
	#[cfg(not(any(target_os = "windows", target_os = "macos")))]
	async fn https_connect<T>(
	stream: T,
	target: (&str, u16),
	auth: Option<HttpProxyAuthorization<'_>>,
	proxy_domain: &str,
	) -> Result<Box<dyn AsyncReadWrite>>
	where
	T: AsyncReadWrite,
	{
	let proxy_domain = rustls_pki_types::ServerName::try_from(proxy_domain)
	.context("Address resolution failed")?
	.to_owned();
	let tls_connector = TlsConnector::from(std::sync::Arc::new(http_client_tls::tls_config()));
	let stream = tls_connector.connect(proxy_domain, stream).await?;
	http_connect(stream, target, auth).await
	}

[ConradIrwin]

Thanks! This seems reasonable, but so hard to know...

With my tin foil hat on I am convinced that there's a giant conspiracy by the authors of rust tls libraries to make it impossible to actually set up a project to use tls properly :D

[rgbkrk]

Guh. Yeah.

[lionel-]

Conrad's PR (runtimed/runtimed#182) was merged so we should now be able to depend on the main repo.

[ConradIrwin]

Does this correspond to an actual released version? Looks like it's tagged with sidecar-v0.8.0, tag: runt-cli-v0.3.0. If so, can we switch back to non-git dependencies?

[lionel-]

yep! now updated to latest releases

[ConradIrwin]

Ugh, that seems to have broken the tls config

[rgbkrk]

Is there something we need to change in https://github.com/runtimed/runtimed to make this compatible?

[rgbkrk]

Thank you so much for doing this absolutely necessary maintenance work to bring Zed up to the latest.

[rgbkrk]

Going to push up a change over to aws-lc-rs

[ConradIrwin]

@lionel- Ok, we pushed some changes to runtimelib to make sure the feature flags for async-tungstenite match what we use in Zed, to avoid needing to change the tls setup.

Assuming this fixes the build issues, will merge.

[rgbkrk]

Thanks @lionel- and @ConradIrwin!

[rgbkrk]

Tried it out this morning, it's looking great.

ark-kernel-zed.mp4