fabro/lib/apps
Bryan Helmkamp 64786c1932
refactor: pass the catalog by value and correct the RenderMode docs
- Take `Arc<Catalog>` by value again through the validation entry points.
  `AppState::catalog()` returns an owned `Arc`, so `&state.catalog()` was
  cloning, borrowing the temporary, then cloning again at the leaf. Every
  consumer ends up owning the `Arc`, so by-value is the honest shape and it
  drops one clone per call. The one caller holding the catalog in a field
  now says `Arc::clone(&self.catalog)` explicitly.
- Correct the `RenderMode` doc comment. It claimed `Strict` is "used by
  run-create", but run-create renders `Structural` and promotes the
  resulting warnings to errors itself; `Strict` has no production caller.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-07-28 14:29:03 -04:00
..
fabro-cli refactor: pass the catalog by value and correct the RenderMode docs 2026-07-28 14:29:03 -04:00
fabro-mcp-server refactor: pass the catalog by value and correct the RenderMode docs 2026-07-28 14:29:03 -04:00
fabro-server refactor: pass the catalog by value and correct the RenderMode docs 2026-07-28 14:29:03 -04:00
fabro-spa refactor: organize crates into three layers 2026-07-23 17:59:34 -04:00