fix(html-formatter): remove extra space before > when bracketSameLine…

[leno23]

… is true and slash is absent

Summary

Closes #9524

Test Plan

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: 3677804

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages

Name	Type
@biomejs/biome	Patch
@biomejs/cli-win32-x64	Patch
@biomejs/cli-win32-arm64	Patch
@biomejs/cli-darwin-x64	Patch
@biomejs/cli-darwin-arm64	Patch
@biomejs/cli-linux-x64	Patch
@biomejs/cli-linux-arm64	Patch
@biomejs/cli-linux-x64-musl	Patch
@biomejs/cli-linux-arm64-musl	Patch
@biomejs/wasm-web	Patch
@biomejs/wasm-bundler	Patch
@biomejs/wasm-nodejs	Patch
@biomejs/backend-jsonrpc	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[ematipico]

Please comply to the contribution guidelines:

• use our PR template
• add a changeset
• test your changes and add a new test if necessary

[coderabbitai]

Walkthrough

Adjusts whitespace logic when formatting self-closing HTML elements with bracket_same_line = true: a hard space before the next token is now written only if a slash will be present. The will_have_slash condition is computed from self_close_void_elements and the presence of slash_token, with is_void derived from node.is_void_element().unwrap_or_default(). Other attribute, line-break and slash-handling logic is unchanged.

Possibly related PRs

• fix(format/html): add space before /> #8557: Changes spacing logic in the same self_closing_element.rs codepath for inserting a space before the self-closing slash/closing bracket.

Suggested reviewers

• dyc3
• ematipico

🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The PR description is vague and incomplete, lacking meaningful context about the changeset despite referencing issue #9524.	Expand the description to explain the bug being fixed, why it occurs, and what the intended behaviour is. Include details about the test plan demonstrating the fix.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly addresses the core fix: removing extra space before > when bracketSameLine is true and slash is absent, matching the changeset and code modifications.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codspeed-hq]

Merging this PR will not alter performance

✅ 64 untouched benchmarks
⏩ 152 skipped benchmarks¹

---

_{Comparing leno23:fix/html-bracket-same-line-spacing (3677804) with main (4df86ca)²}

Footnotes

1. 152 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

2. No successful run was found on main (7fe43c8) during the generation of this report, so 4df86ca was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩