style: rustfmt the LC-78/LC-152 bridge files (LC-187) #250

Merged

nrupard merged 1 commit from fix/lc-187-fmt-bridge-files into main

2026-05-28 16:21:58 +02:00

nrupard commented

2026-05-28 16:21:14 +02:00

Owner

LC-187 standing CI fmt issue. CI Check workflow flagged formatting drift across the LC-78 / LC-152 bridge surfaces plus adjacent tests.

Source: the LC-78-AVATAR-PROXY (#245) and LC-152 SSRF (#246) merges landed without cargo fmt --all over the whole workspace, so one-line shapes that the workspace-wide rustfmt wants wrapped slipped in.

This PR is a pure cargo fmt --all run, no semantic changes. cargo check -p lets-chat-server clean afterwards.

21 files (8 src + 13 tests) under server/src/{bridge_avatar.rs, db/bridge_avatar_proxies.rs, db/bridges.rs, http_client.rs, routes/admin.rs, routes/api.rs, routes/bridge_avatar_media.rs, routes/room.rs} and server/tests/{bridge_avatar_fetch.rs, db_bridge_avatar_proxies.rs, lc152_.rs, outgoing_webhook_actor_payload.rs, routes_admin_bridges.rs, routes_api_bridge_.rs, routes_api_pagination.rs, routes_bridge_role_isolation.rs, routes_media_bridge_avatar_proxy*.rs, spike_image_decoder_hostile_corpus.rs}.

Note: there is a separate pre-existing clippy too_many_arguments violation on resolve_msg_author in server/src/routes/mod.rs:154 that the CI Check workflow also flags. Out of scope for this PR (different fix shape - either #[allow] or a struct-of-args refactor); will follow up in a separate commit under LC-187.

🤖 Generated with Claude Code

LC-187 standing CI fmt issue. CI Check workflow flagged formatting drift across the LC-78 / LC-152 bridge surfaces plus adjacent tests. Source: the LC-78-AVATAR-PROXY (#245) and LC-152 SSRF (#246) merges landed without `cargo fmt --all` over the whole workspace, so one-line shapes that the workspace-wide rustfmt wants wrapped slipped in. This PR is a pure `cargo fmt --all` run, no semantic changes. `cargo check -p lets-chat-server` clean afterwards. 21 files (8 src + 13 tests) under server/src/{bridge_avatar.rs, db/bridge_avatar_proxies.rs, db/bridges.rs, http_client.rs, routes/admin.rs, routes/api.rs, routes/bridge_avatar_media.rs, routes/room.rs} and server/tests/{bridge_avatar_fetch.rs, db_bridge_avatar_proxies.rs, lc152_*.rs, outgoing_webhook_actor_payload.rs, routes_admin_bridges.rs, routes_api_bridge_*.rs, routes_api_pagination.rs, routes_bridge_role_isolation.rs, routes_media_bridge_avatar_proxy*.rs, spike_image_decoder_hostile_corpus.rs}. Note: there is a separate pre-existing clippy `too_many_arguments` violation on `resolve_msg_author` in `server/src/routes/mod.rs:154` that the CI Check workflow also flags. Out of scope for this PR (different fix shape - either `#[allow]` or a struct-of-args refactor); will follow up in a separate commit under LC-187. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

nrupard added 1 commit

2026-05-28 16:21:14 +02:00

style: rustfmt the LC-78 / LC-152 bridge files (LC-187)

check-secrets / Nosey parker (push) Successful in 4s

Details

check-secrets / TruffleHog (push) Successful in 5s

Details

check-secrets / Kingfisher (push) Successful in 7s

Details

check-secrets / Nosey parker (pull_request) Successful in 4s

Details

check-secrets / TruffleHog (pull_request) Successful in 6s

Details

check-secrets / Kingfisher (pull_request) Successful in 6s

Details

Check / clippy + fmt + tests (pull_request) Failing after 34s

Details

Create release / Create release from merged PR (pull_request) Has been skipped

Details

73f6b06bc5

CI Check workflow flagged formatting drift on the bridge-avatar / SSRF surfaces and several adjacent test files. Pre-existing on main from the LC-78-AVATAR-PROXY (#245) and LC-152 (#246) merges - neither commit ran `cargo fmt --all` over the whole workspace before pushing, so they landed with one-line shapes that the workspace-wide rustfmt then wants to wrap.

Ran `cargo fmt --all` (the only mechanical fix per the standing rule: workspace-wide, not per-crate). No semantic change; `cargo check -p lets-chat-server` clean afterwards.

Files: server/src/{bridge_avatar.rs, db/bridge_avatar_proxies.rs, db/bridges.rs, http_client.rs, routes/admin.rs, routes/api.rs, routes/bridge_avatar_media.rs, routes/room.rs} + server/tests/{bridge_avatar_fetch.rs, db_bridge_avatar_proxies.rs, lc152_*.rs, outgoing_webhook_actor_payload.rs, routes_admin_bridges.rs, routes_api_bridge_*.rs, routes_api_pagination.rs, routes_bridge_role_isolation.rs, routes_media_bridge_avatar_proxy*.rs, spike_image_decoder_hostile_corpus.rs}.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>