Avoid copying dependencies in the vast majority of dispatches

[jviotti]

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

[cubic-dev-ai]

No issues found across 9 files

[github-actions]

Benchmark Index (community)

Details

Benchmark suite	Current: d503322	Previous: d4b4243	Ratio
Add one schema (0 existing)	22 ms	19 ms	1.16
Add one schema (100 existing)	79 ms	56 ms	1.41
Add one schema (1000 existing)	648 ms	393 ms	1.65
Cached rebuild (1 existing)	13 ms	11 ms	1.18
Cached rebuild (101 existing)	52 ms	34 ms	1.53
Cached rebuild (1001 existing)	404 ms	251 ms	1.61

This comment was automatically generated by workflow using github-action-benchmark.

[github-actions]

Benchmark Index (enterprise)

Details

Benchmark suite	Current: d503322	Previous: d4b4243	Ratio
Add one schema (0 existing)	25 ms	26 ms	0.96
Add one schema (100 existing)	81 ms	87 ms	0.93
Add one schema (1000 existing)	640 ms	674 ms	0.95
Cached rebuild (1 existing)	14 ms	15 ms	0.93
Cached rebuild (101 existing)	52 ms	55 ms	0.95
Cached rebuild (1001 existing)	405 ms	421 ms	0.96

This comment was automatically generated by workflow using github-action-benchmark.

[augmentcode]

🤖 Augment PR Summary

Summary: This PR optimizes the build dispatch pipeline to avoid copying dependency paths in most cases.

Changes:

• Refactors sourcemeta::one::Build dependency tracking to store static and dynamic dependencies separately.
• Introduces reference-based dependency passing (std::reference_wrapper<const std::filesystem::path>) so handlers can read dependency paths without extra allocations/copies.
• Adds a variadic Build::dispatch overload for the common case of a small, fixed number of static dependencies, plus a vector-based overload for bulk dependency sets.
• Centralizes dispatch completion logic in dispatch_commit and cache-hit evaluation in dispatch_is_cached.
• Updates dependency persistence (deps.txt) I/O to emit/read s/d lines for static/dynamic deps.
• Updates index/web generators and build e2e tests to use the new dependency representation (.get()) and new dispatch call shape.

Technical Notes: Cache checks now compare static-dependency lists directly and validate both static and dynamic dependency mtimes against the stored output mark to decide cache hits.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.