Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

Split remote.ts by topic, starting with migrations

未关闭
#1,066 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 2 天内回复

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
55/100
Issue 类型
重构
描述清晰度
基本清楚
活跃度
冷清
技术栈
typescript
领域
devtools

调研方向

从 src/remote/remote.ts 开始,阅读 migrateLegacyAuthority、migrateToSecretsStorage、migrateSessionTokenFile 和 migrateSessionAuthFromFiles,以及它们在 setup 和 setupCoderRemote 中的调用方。将这些普通方法提取到 src/remote/migrations.ts,同时保留行为、删除时限和 telemetry span 的传递;迁移流程应继续保持不变地工作。

由索引模型根据 Issue 内容生成。

描述

Improvement tech-debt vs-code

`src/remote/remote.ts` is 1,207 lines and mixes every concern of remote setup in one class. Recent work (per-editor SSH configs, legacy authority migration) keeps growing it.

Topics currently mixed into the class

  • Setup orchestration: `setup`, `setupCoderRemote`
  • Migrations: `migrateLegacyAuthority` (authority), `migrateToSecretsStorage`, `migrateSessionTokenFile`, `migrateSessionAuthFromFiles` (credentials)
  • SSH config assembly: `updateSSHConfig`, `writeRemoteSshConfig`, `buildProxyCommand`, `getSshConfigPath`, `getLogArgs`, `getLogDir`
  • Workspace lifecycle: `lookupWorkspace`, `waitForWorkspaceReady`, `resolveAgent`
  • Auth/session: `ensureLoggedInAndRetry`, `showSessionExpiredDialog`, `watchRemoteSessionAuth`
  • Binary management: `resolveRemoteBinary`, `checkCompatibility`
  • Window/UI glue: `createAgentMetadataStatusBar`, `watchSettings`, `registerLabelFormatter`, `closeRemote`, `reloadWindow`

Proposal

  1. Extract migrations first, e.g. into `src/remote/migrations.ts`. They are self-contained, interact little with the rest of setup, and each has a deletion horizon; keeping them together makes them easy to find and eventually remove.
  2. If that lands well, extract SSH config assembly next. It is the most test-heavy area and only needs the path resolver, settings, and logger.
  3. Keep orchestration (`setup`/`setupCoderRemote`) in `remote.ts`.

Non-goals: no behavior changes and no new abstractions; move plain methods by topic and keep the telemetry span threading as is.

Filed by Claude Code for @EhabY.

主要语言
TypeScript
星标
131
派生
48
平均合并
1 天 19 小时
30 天内合并 PR
15

环境准备

在 Codespaces 中打开

在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

coder/vscode-coder 的其他 Issue

查看 coder/vscode-coder 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。