Skip to content

fix(mcp): retain RagMcpAgent stub to satisfy migration history (patch) - #112

Merged
liplus-lin-lay merged 1 commit into
mainfrom
111-retain-stub
Apr 23, 2026
Merged

liplus-lin-lay merged 1 commit into
mainfrom
111-retain-stub

Conversation

@liplus-lin-lay

Copy link
Copy Markdown
Member

Closes #111
Refs #109

#109 の merge 後 deploy が code 10064 ("script does not export class 'RagMcpAgent'") で失敗した件の修正。旧 RagMcpAgent を空の DurableObject stub として残し、Cloudflare の「過去 migration で declare されたクラスは script に存在し続けること」制約を満たす。stub には binding が無いため traffic を受けない。

Post-merge deploy of #109 failed with:
  New version of script does not export class 'RagMcpAgent' which is
  depended on by existing Durable Objects. [code: 10064]

The v1 migration still declares RagMcpAgent as an sqlite class, and Cloudflare
requires every declared class to exist in the script (or be removed via a
deleted_classes migration). Since the goal of the rename was to reach a fresh
DO namespace for RagMcpAgentV2, a renamed_classes migration would defeat that
by preserving state. Instead, keep RagMcpAgent as an empty DurableObject stub
that receives no traffic (no binding points to it).

Follow-up work: delete RagMcpAgent entirely via deleted_classes once the V2
switch is confirmed stable.

Closes #111
Refs #109
@liplus-lin-lay liplus-lin-lay self-assigned this Apr 23, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Apr 23, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
github-rag-mcp a03f152 Apr 23 2026, 02:49 PM

@liplus-lin-lay liplus-lin-lay left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

AI self-review: stub class 追加による script export 要件の充足、bindingに影響なし、local tsc/wrangler dry-run pass、CI/test green。Workers Builds は #110 と同じ versioned-deploy 起因で落ちるが非 required なので無視。auto mode → squash merge へ。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(mcp): retain RagMcpAgent stub class to satisfy migration history

1 participant