WORK-483
ID:WORK-483Status:ready

snippet / expand / file-ref readers delegate I/O to ProjectFiles

Close the main gap in SPEC-113: read-file.ts (and the snippet/expand/file-ref pipelines) call node:fs directly today, bypassing the injectable seam — so tree mode is not actually fs-free for snippets. Delegate the I/O to the WORK-481 provider, keeping line-slicing and diagnostics intact.

Priority:highComplexity:moderateMilestone:v0.27.0Source:SPEC-113
claude/v0-27-0-planning-sgq68k View source

Criteria completion

Criteria completion: 0 of 3 (0%) checked; tracking started on Jun 25, no incremental history yet0%25%50%75%100%Jun 25Jul 11

Tracking started Jun 25 — check back for trends.

Relationships

Implements 1
Related 3
Branches 3
claude/v0-27-0-planning-sgq68k current ready
changeset-release/main doneclaude/milestone-v0-27-0-yqiu8v donemain done
History 1
  1. 4eab338
    Created (ready)by bjornolofandersson

Scope

  • read-file.ts — keep readSnippetFile/readWholeSandboxedFile line-slicing, range parsing, and structured diagnostics; replace the node:fs reads (statSync/readFileSync/realpathSync) with ProjectFiles calls. The containment logic moves into fsProjectFiles (WORK-481), so this module stops re-implementing it.
  • Consumers — snippet-pipeline.ts, expand-pipeline.ts, file-ref-resolve.ts obtain the provider from the preprocess context rather than importing fs-bound readers.
  • Tests — the existing snippet/expand sandbox test suites pass against both fsProjectFiles and memoryProjectFiles.

Acceptance Criteria

  • snippet / expand / file-ref readers delegate I/O to the provider; line-slicing and diagnostics are unchanged.
  • Their existing sandbox tests pass against both providers; a snippet resolves in pure memoryProjectFiles mode (no fs access).
  • read-file.ts no longer re-implements path containment (that now lives in fsProjectFiles).

Dependencies

  • WORK-481 — the provider (incl. promoted containment).

References

  • SPEC-113 §3 (snippet/expand/file-ref) — packages/runes/src/{snippet-pipeline,expand-pipeline,file-ref-resolve}.ts, packages/runes/src/lib/read-file.ts.