🤖 fix: a Daytona connection test started before a settings save can show the old key as connected
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 79/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- react, typescript
調査の方向性
src/browser/features/Settings/Sections/DaytonaSettingsForms.tsx と、issue にリンクされている以前のレビュー議論から始めてください。runTest、settle、resetTest、保存または削除が実行中のテストをどのように処理するかを追い、説明されている UI テストを追加してください。テストを開始し、新しいキーを保存してから、古いテストを成功として解決します。古い結果に「Connected」と表示されなければ完了です。提案された修正のいずれかを選んでください。
索引モデルが issue の本文から書いたものです。
説明
Follow-up from the Codex review of #7100 (thread: https://github.com/coder/xum/pull/7100#discussion_r4354065080, src/browser/features/Settings/Sections/DaytonaSettingsForms.tsx:89).
Problem
In Settings > Daytona, a connection test result is tagged with the settings load generation (loads) it ran under. Saving or removing the API key calls resetTest(), which clears the current result but neither cancels the in-flight daytona.testConnection() call nor bumps loads (that happens only after the reload). Save stays enabled while a test runs. So a test started with the old key can finish after the save and show "Connected" for the previous credentials, even when the newly saved key is invalid.
Proposed fix
Either invalidate the test generation on every save and remove (a local counter that runTest captures and settle compares), or disable Save and Remove while a test is running. Add a UI test: start a test, save a new key, resolve the old test as ok, and expect no "Connected" state.
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時間 55分
- マージ済み PR（30日）
- 897
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coder/xum のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
🤖 fix: stream-error card header actions reach 2.6px into the card padding in a 170px VS Code sidebarオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
数据界面的横向滚动条缺失オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
zilliztech/attu#1037 ·
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P3
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
openclaw/openclaw#181212 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
apify/apify-mcp-server#1684 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
Automattic/data-liberation-agent#953 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
siteboon/claudecodeui#1642 ·
メンテナーはふだん 5 日以内に返信