Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

🤖 refactor: put the MCP modules that agent-loop files import behind Effect services

Đang mở
  • Có gì không ổn?
#7,239 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
42/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
backend, devtools

Hướng nghiên cứu

Start with the phase 5 plan’s claim 3.5 and the four listed modules: workspaceMcpOverridesService.ts, mcpAppResultStore.ts, mcpServerManager.ts, and agentPlugins/mcpConfig.ts. Trace their imports from aiService.ts and turnRequestBuilder.ts, then inspect the Effect services and Node and test layers added in phase 5. Done means all listed host access goes through those services with behavior and persisted files unchanged, and the phase 5 end check reports only its specified follow-up modules and log.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Summary

Agent-loop files import these modules, and the modules still reach the host directly (fs, os, child_process, writeFileAtomic, a runtime class or a runtime builder). The local/no-host-imports rule checks only the agent-loop files themselves, so it does not see this host access. Phase 5 of the agent-loop refactor does not move these modules: its plan (claim 3.5, decision "reach") keeps one follow-up issue per group. This issue is the MCP group: MCP servers, overrides and app results.

Modules

Module Host refs Host imports Imported by agent-loop files at
src/node/services/workspaceMcpOverridesService.ts 44 fs/promises:2 fs:3 runtime/runtimeFactory:15 runtime/runtimeHelpers:16 aiService.ts:104 turnRequestBuilder.ts:158
src/node/services/mcpAppResultStore.ts 12 fs/promises:2 utils/fs:15 utils/writeFileAtomic:16 turnRequestBuilder.ts:87
src/node/services/mcpServerManager.ts 10 fs/promises:5 runtime/DevcontainerRuntime:88 runtime/RemoteRuntime:89 runtime/runtimeFactory:104 turnRequestBuilder.ts:149
src/node/services/agentPlugins/mcpConfig.ts 4 fs:7 fs/promises:8 turnRequestBuilder.ts:81

Total: 4 module(s), 70 uses of host bindings. Counts and lines come from reach.mjs --phase2 and reachRefs.mjs of the phase 5 plan, at ba9e451cab (phase 1 changed no file under src/).

Work

Move the host access of each module behind the Effect services that phase 5 adds (HostFiles, HostPaths, HostProcesses, RuntimeFactory, SessionFiles, SessionLocks), with a Node layer and a test layer, and no change in behavior or persisted files.

Trigger

After phase 5 ends (stacks 5a and 5b merged). Phase 5 end check 7 expects reach.mjs --phase2 to list only the 30 follow-up modules of these 8 issues and log.

Refs

  • Phase 5 plan v4 ("Host I/O Services for the Xum Agent Loop", artifact phase5-plan.html), claim 3.5.
  • Refs #7207 (the local/no-host-imports rule and its frozen pairs).

Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high • Cost: $31.10

Ngôn ngữ chính
TypeScript
Star
2.1k
Fork
140
Merge trung bình
4 giờ 57 phút
Pull request đã merge (30 ngày)
946

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của coder/xum

Tất cả issue của coder/xum

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.