🤖 refactor: put the workflows modules that agent-loop files import behind Effect services
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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-importsrule and its frozen pairs).
Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high • Cost: $31.10
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.1k
- Forks
- 140
- Merge medio
- 3 h 23 min
- PR fusionados (30 d)
- 953
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de coder/xum
-
🤖 tests: lock in git status outgoing-count rename equivalence and clarify index-lock commentAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
🤖 fix: Daytona startAndWait can start a sandbox after its deadline and then report a timeoutAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
coder/xum#7358 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
Issues similares
-
backend refactor
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
NIAEFEUP/tts#927 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
callstackincubator/react-native-harness#296 ·
Los mantenedores suelen responder en 1 día
-
Editor bug with grouped nodesAbiertostatus:in-linear
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
n8n-io/n8n#42293 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
verified
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Los mantenedores suelen responder en 5 días
-
Add inputmode attribute support (forwarded to the inner input)Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Neovici/cosmoz-input#467 ·
Los mantenedores suelen responder en 1 día