mirror of https://github.com/fabro-sh/fabro.git synced 2026-09-07 08:27:12 +00:00

refactor(workflow): remove retro stage (#230 )

## Summary

Removes Fabro's automatic retro generation stage so workflow runs go
directly from execution to finalization and optional PR creation. This
drops the retro-specific crate, events, projection fields, config/API
knobs, and user-facing docs in favor of the existing durable run
observability surfaces.

## What Changed

- Deleted the `fabro-retro` crate and the workflow `retro` pipeline
phase, with finalization now consuming `Executed` state directly.
- Removed retro configuration and API surface area, including
`--no-retro`, `[run.execution].retros`, manifest `no_retro`,
`features.retros`, and run projection `retro*` fields.
- Retired typed `retro.*` events while keeping historical event logs
readable by deserializing retired retro event names as `Unknown`.
- Stopped appending retro sections to generated PR bodies and updated
docs, marketing copy, screenshots, and navigation to point users toward
observability/event-stream inspection.

## Testing

Not run during PR creation; this branch already contained the
implementation commit.

---

[![Compound
Engineering](https://img.shields.io/badge/Compound_Engineering-6366f1)](https://github.com/EveryInc/compound-engineering-plugin)
🤖 Generated with GPT-5 (unknown context, reasoning unspecified) via
[Codex](https://openai.com/codex)

2026-05-09 10:18:20 -04:00

2.1 KiB

Raw Permalink Blame History

Run Scratch Files

This document maps the files that still live under a run scratch directory. Durable run state lives in the run store and metadata branch; scratch is mostly local runtime state and caches.

Scope:

Scratch root: ~/.fabro/scratch/YYYYMMDD-{run_id}/
This covers local run files only
Persistent store keys live in lib/crates/fabro-store/src/keys.rs
Artifact object-store keys live in lib/crates/fabro-store/src/artifact_store.rs

There is no _init.json anymore. Run existence in the database is determined by stored run events, and local scratch directories are managed separately under scratch/.

Root-Level Files

File	Purpose	Source
`workflow_bundle.json`	Bundled workflow input used by `start` to restore `workflow_path` and bundled child workflows/files	Written during create from the resolved workflow bundle
`run.pid`	Legacy detached-run pid file from older runs	Legacy only; current flows do not rely on it

Local-Only Directories

These paths are local runtime state, not canonical event projections.

Path	Purpose
`worktree/`	Git worktree used by checkpointed runs
`runtime/server.log`	Worker tracing log for the run
`runtime/blobs/`	Materialized local blob payloads for file-backed `fabro+blob://` references
`nodes/{manager_node}_{visit}/child/`	Nested scratch root for manager-loop child workflows

Reconstructed / Exported Files

These names are still real, but they are no longer live scratch files by default:

Metadata branch files such as run.json, start.json, and checkpoint.json
fabro dump exports such as run.json, start.json, status.json, checkpoint.json, conclusion.json, events.jsonl, and per-node prompt/response/status/stdout/stderr files

Notes

Artifact binaries are no longer stored in the SlateDB keyspace. They live in ArtifactStore; the run scratch tree only contains local cached copies when a workflow stage writes them to disk.
Final diffs for checkpointed runs are projected from the run store; they are no longer written as scratch files.

2.1 KiB Raw Permalink Blame History