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 workflows modules that agent-loop files import behind Effect services

Đang mở
  • Có gì không ổn?
#7,243 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
45/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
backend

Hướng nghiên cứu

Read the phase 5 plan’s claim 3.5 and inspect the four listed modules: WorkflowRunStore.ts, agentWorkflowRunReferences.ts, workflowScriptResolver.ts, and WorkflowService.ts, alongside the named agent-loop import sites. Map each host reference to the listed Effect services and review the Node and test layers from phase 5. Done means host access is behind those services with behavior and persisted files unchanged, and the phase 5 end check lists only the expected 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 workflows group: workflow runs.

Modules

Module Host refs Host imports Imported by agent-loop files at
src/node/services/workflows/WorkflowRunStore.ts 30 fs/promises:3 utils/writeFileAtomic:8 aiService.ts:110 turnRequestBuilder.ts:215
src/node/services/agentWorkflowRunReferences.ts 4 fs/promises:1 utils/writeFileAtomic:4 tools/toolUtils.ts:10
src/node/services/workflows/workflowScriptResolver.ts 2 runtime/LocalRuntime:8 tools/workflow_run.ts:19 turnRequestBuilder.ts:217
src/node/services/workflows/WorkflowService.ts 1 runtime/runtimeHelpers:14 turnRequestBuilder.ts:221

Total: 4 module(s), 37 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
139
Merge trung bình
3 giờ 45 phút
Pull request đã merge (30 ngày)
953

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.