🤖 fix: renaming a Daytona workspace fails while its sandbox is stopped
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 68/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- typescript
- 領域
- backend
調査の方向性
Start with WorkspaceService.rename() and src/node/runtime/DaytonaSSHRuntime.ts, especially the ensureReady() and renameWorkspace() paths described in the issue. Read how terminals and turns ensure the runtime is ready, then add the runtime test for a stopped sandbox. Done means the sandbox starts once and the workspace rename succeeds.
索引モデルが issue の本文から書いたものです。
説明
Follow-up from the Codex review of #7100 (thread: https://github.com/coder/xum/pull/7100#discussion_r4353981200, src/node/runtime/DaytonaSSHRuntime.ts:510).
Problem
Renaming a Daytona workspace whose sandbox auto-stopped (30 min idle) or was archived by Daytona fails. WorkspaceService.rename() calls the inherited SSH rename (a remote mv) directly. The Daytona runtime restarts a stopped sandbox only inside ensureReady(), and its one-shot SSH recovery refuses to start a sandbox for a command that did not go through ensureReady(). So the rename fails until the user runs a turn or opens a terminal, which starts the sandbox.
Proposed fix
Bring the sandbox up before the remote move: either WorkspaceService.rename() calls runtime.ensureReady() for runtimes that need it (as terminals and turns do), or DaytonaSSHRuntime overrides renameWorkspace() to run ensureReady() first. Add a runtime test with a stopped sandbox that expects one start and a successful rename.
Why it is deferred
#7100 runs its review loop under a P0/P1-only rule; P2 findings get a tracked follow-up.
- 主要言語
- TypeScript
- スター
- 2.1k
- フォーク
- 140
- 平均マージ
- 4時間 57分
- マージ済み PR（30日）
- 946
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coder/xum のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
🤖 fix: a Daytona connection test started before a settings save can show the old key as connectedオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
bug require-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
expressjs/expressjs.com#2549 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
umami-software/umami#4686 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
[Bug]: Device screenshot fails with 500 on iPhone Duo (Xcode 27.2): serve-sim calls simctl without --display対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
pingdotgg/t3code#24900 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
inu-appcenter/UNIDorm-web#411 ·
メンテナーはふだん 1 日以内に返信
-
🐛 bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 2 日以内に返信