Skip to content

sbx: add claude desktop ssh callout - #25670

Merged
craig-osterhout merged 1 commit into
docker:mainfrom
craig-osterhout:docs/claude-desktop-ssh-credentials-warning
Jul 29, 2026
Merged

sbx: add claude desktop ssh callout#25670
craig-osterhout merged 1 commit into
docker:mainfrom
craig-osterhout:docs/claude-desktop-ssh-credentials-warning

Conversation

@craig-osterhout

@craig-osterhout craig-osterhout commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Description

  • Added callout about Claude Desktop SSH implementation.
  • Added another troubleshooting section for token refresh.

Preview: https://deploy-preview-25670--docsdocker.netlify.app/ai/sandboxes/integrations/claude-desktop/

Related issues or tickets

https://docker.slack.com/archives/C09E0594URW/p1785344408912809

Reviews

  • Technical review
  • Editorial review
  • Product review

@craig-osterhout
craig-osterhout requested a review from a team July 29, 2026 17:26
@craig-osterhout
craig-osterhout requested a review from dvdksn as a code owner July 29, 2026 17:26
@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label Jul 29, 2026
@netlify

netlify Bot commented Jul 29, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 09eea6a
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a6a3afd856a3d00089e8a27
😎 Deploy Preview https://deploy-preview-25670--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

The PR adds a WARNING callout to the Claude Desktop SSH integration page. The callout syntax and Markdown formatting are correct, and the information (credential transmission risk + token-refresh breakage) is relevant and worth surfacing. Two medium-severity clarity issues were found in the new callout text: an opaque workaround that doesn't explain why the command helps, and an ambiguous pronoun that leaves unclear which of the two stated problems the workaround addresses.

Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated
Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated
Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated
Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md
Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

The added WARNING callout correctly identifies two real issues with Claude Desktop SSH integration (credential exposure and token refresh failures). Three style/clarity issues were found in the new text — all introduced by this PR.

Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated
Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated
Comment thread content/manuals/ai/sandboxes/integrations/claude-desktop.md Outdated
@craig-osterhout
craig-osterhout force-pushed the docs/claude-desktop-ssh-credentials-warning branch from d542098 to 3af4d29 Compare July 29, 2026 17:36
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@craig-osterhout
craig-osterhout force-pushed the docs/claude-desktop-ssh-credentials-warning branch from 3af4d29 to 09eea6a Compare July 29, 2026 17:40
@craig-osterhout
craig-osterhout merged commit f976a33 into docker:main Jul 29, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ai status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants