Skip to content

chore(worker): clean up the retired Durable Object binding #230

Description

@liplus-lin-lay

目的

wrangler.tomlMCP_OBJECT バインディングが、退役した Durable Object クラス RagMcpAgentV2 のスタブを指したままになっている。整理する。

前提

#224 / #229 のステートレス移行で、McpAgent を継承していた RagMcpAgent / RagMcpAgentV2 は不要になった。ただし Cloudflare の制約により、DO クラスは deleted_classes migration を経ずに削除できない。そのため両クラスは src/retired-do.ts にスタブとして残され、wrangler.toml のバインディングもそのままになっている。

現状は安全側である。バインディングとクラス実体が対応しており、デプロイは通っている(#229 の Workers Builds pass、および merge 後の実機確認で確認済み)。

検討する内容

  • deleted_classes migration を切ってバインディングとスタブを両方落とすか
  • 落とさずスタブを維持し、retired-do.ts のコメントで意図を保持し続けるか

前者は migration の記述ミスが本番の DO に影響しうるため、慎重さの度合いが rules/model/subtractive-structural-beauty.md の blast radius でいう medium 以上に来る。後者は死にコードを抱え続けるが、影響はゼロ。

制約

  • ステートレス移行そのものは完了済み。本 issue は後片付けであり、機能に影響しない
  • migration を切る場合、本番 DO の状態に対する影響を事前に確認すること。Durable Object の削除は git revert で戻せない

対象ファイル

  • wrangler.tomlMCP_OBJECT バインディングと migrations セクション
  • src/retired-do.ts — スタブ本体
  • src/index.ts — スタブの re-export

関連

Metadata

Metadata

Assignees

No one assigned

    Labels

    backlog受け入れ済み、着手時期未定enhancement新機能・改善要望

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions