Skip to content

style(runtime): unbreak main — cargo fmt for model_metadata.rs#3150

Merged
houko merged 1 commit into
mainfrom
emergency/main-fmt-fix
Apr 25, 2026
Merged

style(runtime): unbreak main — cargo fmt for model_metadata.rs#3150
houko merged 1 commit into
mainfrom
emergency/main-fmt-fix

Conversation

@houko

@houko houko commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

Emergency fix: main is red because the fmt residue from #3134 / #3140 got squash-merged in. 11 lines of pure formatting in model_metadata.rs. No behaviour change.

Emergency fix — main is red on Quality + Test/macOS. The fmt residue
came in via my rebase resolutions of #3134 / #3140 where I focused on
conflict markers and didn't run cargo fmt locally before push. Both
PRs were squash-merged into main, carrying the un-fmt'd lines along.

Pure formatting, no behaviour change.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@houko
houko merged commit 30b640d into main Apr 25, 2026
15 checks passed
@houko
houko deleted the emergency/main-fmt-fix branch April 25, 2026 14:25
@github-actions github-actions Bot added the ready-for-review PR is ready for maintainer review label Apr 25, 2026
@github-actions github-actions Bot added size/S 10-49 lines changed area/runtime Agent loop, LLM drivers, WASM sandbox and removed ready-for-review PR is ready for maintainer review labels Apr 25, 2026
houko added a commit that referenced this pull request Apr 25, 2026
)

Emergency fix — main is still red after #3150. clippy 1.95 added
`manual_pattern_char_comparison`, which flags the multi-`||` closure
form I wrote in #3139's `extract_url_host`. Switch to the
`.find(['/', '?', '#'])` array form clippy expects. Behaviour is
identical.
@houko houko mentioned this pull request Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/runtime Agent loop, LLM drivers, WASM sandbox size/S 10-49 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant