Skip to content

feat(sagemaker): add cursor support for parker#8769

Open
aws-ajangg wants to merge 4 commits into
aws:masterfrom
aws-ajangg:feat-cursor-support-parker
Open

feat(sagemaker): add cursor support for parker#8769
aws-ajangg wants to merge 4 commits into
aws:masterfrom
aws-ajangg:feat-cursor-support-parker

Conversation

@aws-ajangg
Copy link
Copy Markdown
Contributor

@aws-ajangg aws-ajangg commented Apr 29, 2026

Description

  • Extends HyperPod connection support to the Cursor IDE. Cursor uses an IDE-specific suffix (c) to distinguish its SSH connections from VS Code's (e.g., smc_ for Studio). This PR applies the same suffix pattern to HyperPod, giving Cursor the smhpc_ prefix while VS Code/Kiro continue using smhp_. It also fixes the HyperPod SSH config to use %n (preserving hostname casing) and the SageMaker SSH config template (no IdentitiesOnly), matching what SageMaker Studio already does.

Changes

  • Add ide suffix coverage to hyperpod connections, now inclusive of smhpc_ for cursor
  • Updated regex in connect scripts to cover smhpc_ prefix
  • Aligned hyperpod ssh config entry with jorus template to use %n in ProxyCommand and remove IdentitiesOnly

Testing

  • Added unit tests
  • Manual testing with vsix on macOS and Windows

  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@aws-ajangg aws-ajangg requested a review from a team as a code owner April 29, 2026 00:58
@amazon-inspector-ohio
Copy link
Copy Markdown

⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done

@github-actions
Copy link
Copy Markdown

  • This pull request implements a feat or fix, so it must include a changelog entry (unless the fix is for an unreleased feature). Review the changelog guidelines.
    • Note: beta or "experiment" features that have active users should announce fixes in the changelog.
    • If this is not a feature or fix, use an appropriate type from the title guidelines. For example, telemetry-only changes should use the telemetry type.

@amazon-inspector-ohio
Copy link
Copy Markdown

✅ I finished the code review, and didn't find any security or code quality issues.

Comment thread packages/core/src/shared/sshConfig.ts
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.

2 participants