Skip to content

feat: support generic {ide}-remote connection types for space access#414

Merged
mollyheamazon merged 1 commit intoaws:mainfrom
earaghbidikashani:main
Apr 30, 2026
Merged

feat: support generic {ide}-remote connection types for space access#414
mollyheamazon merged 1 commit intoaws:mainfrom
earaghbidikashani:main

Conversation

@earaghbidikashani
Copy link
Copy Markdown
Contributor

feat: support generic {ide}-remote connection types for space access

Description

Updates the HyperPod CLI to support any {ide}-remote connection type pattern for space access, instead of only allowing the hardcoded vscode-remote and web-ui values. This enables kiro-remote, cursor-remote, windsurf-remote, and any future IDE remote connection types
without requiring CLI changes.

This change aligns the CLI with the server-side operator change in jupyter-k8s-infra#371 (feat: kiro and cursor support), which generalized the WorkspaceConnection validation from
a fixed enum to a regex-based {ide}-remote pattern.

Changes

SDK (hyperpod_space.py)

  • Replaced hardcoded {"vscode-remote", "web-ui"} allowlist with regex validation using the same pattern as the operator: ^[a-zA-Z0-9]+(?:-[a-zA-Z0-9]+)*-remote$
  • Updated docstring, error message, and usage examples

CLI (space_access.py)

  • Updated --connection-type help text to describe the {ide}-remote pattern

Documentation

  • README.md — Added kiro-remote and cursor-remote examples to CLI and SDK sections
  • doc/cli/space/cli_space.md — Updated parameter table and added example
  • doc/getting_started/space.md — Added examples to CLI and SDK tabs
  • examples/end_to_end_walkthrough/04-spaces/00-create-space.md — Added section for other IDE connections

Tests

  • test_space_access.py — Added CLI tests for kiro-remote and cursor-remote
  • test_hyperpod_space.py — Added SDK tests for kiro-remote, cursor-remote, and invalid pattern rejection (6 invalid inputs)

Testing

All space access tests pass (10/10 targeted, 67/67 total space-related tests). No regressions.

Comment thread test/unit_tests/cli/test_space_access.py
Comment thread src/sagemaker/hyperpod/space/hyperpod_space.py
@mollyheamazon mollyheamazon merged commit 8840eaa into aws:main Apr 30, 2026
6 checks passed
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.

5 participants