Skip to content

style: cargo fmt recently-merged code (repair main Quality fmt)#5853

Merged
houko merged 1 commit into
mainfrom
fix/wasm-skill-fmt
May 29, 2026
Merged

style: cargo fmt recently-merged code (repair main Quality fmt)#5853
houko merged 1 commit into
mainfrom
fix/wasm-skill-fmt

Conversation

@houko

@houko houko commented May 29, 2026

Copy link
Copy Markdown
Contributor

Problem

main fails the Quality job's cargo fmt --check step, which fails Quality on every open PR. The unformatted blocks were merged via #5835, #5839, #5848, and #5849 — method chains and multi-line statements that rustfmt collapses/reflows (the PRs were verified with clippy + scoped tests but cargo fmt --check was missed on those).

Fix

Pure cargo fmt output across the five affected files (no logic change):

  • crates/librefang-runtime/src/tool_runner/wasm_skill.rs
  • crates/librefang-runtime/src/context_engine/sidecar.rs
  • crates/librefang-kernel/src/goal_runner.rs
  • crates/librefang-memory/src/proactive.rs
  • crates/librefang-api/tests/memory_routes_integration.rs

Verification

cargo fmt --check on the result → clean (0 diffs), run in the dev container.

rustfmt flags several blocks merged via #5835, #5839, #5848, and #5849
(method chains and multi-line statements rustfmt collapses/reflows). main
was therefore red on the Quality "Check formatting" step, which fails the
Quality job on every open PR. Pure `cargo fmt` output — no logic change.
@houko
houko merged commit 2274197 into main May 29, 2026
17 checks passed
@github-actions github-actions Bot added the area/runtime Agent loop, LLM drivers, WASM sandbox label May 29, 2026
@houko
houko deleted the fix/wasm-skill-fmt branch May 29, 2026 01:26
@github-actions github-actions Bot added area/kernel Core kernel (scheduling, RBAC, workflows) size/M 50-249 lines changed labels May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/kernel Core kernel (scheduling, RBAC, workflows) area/runtime Agent loop, LLM drivers, WASM sandbox size/M 50-249 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant