Skip to content

fix(report): write run data to project-local target/piano/runs/#77

Merged
rocketman-code merged 1 commit intomainfrom
fix/project-scoped-report
Feb 24, 2026
Merged

fix(report): write run data to project-local target/piano/runs/#77
rocketman-code merged 1 commit intomainfrom
fix/project-scoped-report

Conversation

@rocketman-code
Copy link
Owner

Summary

  • Add shutdown_to(dir) to the runtime for writing to a specified directory
  • CLI injects shutdown_to with <project>/target/piano/runs/ instead of shutdown()
  • piano report and piano tag check ./target/piano/runs/ first, falling back to ~/.piano/runs/
  • PIANO_RUNS_DIR env var still takes priority for testing and user overrides

Before: all run data went to global ~/.piano/runs/ regardless of project.
After: run data is project-local in target/piano/runs/, eliminating cross-project confusion.

Closes #71

Add shutdown_to(dir) to the runtime that writes to a specified directory.
The CLI now injects shutdown_to with the project's target/piano/runs/ path
instead of shutdown(), making run data project-scoped by default.

piano report and piano tag check ./target/piano/runs/ first, falling back
to ~/.piano/runs/ for backward compatibility with existing global runs.

PIANO_RUNS_DIR env var still takes priority in both shutdown and
shutdown_to for testing and user overrides.

Closes #71
@rocketman-code rocketman-code force-pushed the fix/project-scoped-report branch from c2aa80b to fdf391a Compare February 24, 2026 16:43
@rocketman-code rocketman-code merged commit c3dee0b into main Feb 24, 2026
5 checks passed
@rocketman-code rocketman-code deleted the fix/project-scoped-report branch February 24, 2026 16:55
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.

piano report is globally scoped, not project-scoped

1 participant