fix(formatter): skip test-call layout when arguments have comments by leaysgur · Pull Request #27119 · oxc-project/oxc

leaysgur · 2026-09-28T06:49:29Z

leaysgur · 2026-09-28T06:49:43Z

fix(formatter): skip test-call layout when arguments have comments #27119 👈 (View in Graphite)
main

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

_{An organization admin has enabled the Graphite Merge Queue in this repository.} _{Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.}

This stack of pull requests is managed by Graphite. Learn more about stacking.

oxc-guard · 2026-09-28T06:52:46Z

Oxfmt Ecosystem CI

suite	oxfmt@latest	main	refs/pull/27119/head branch
monkeytypegame/monkeytype	❌	✅	✅
eggjs/egg	✅	✅	✅
fastify/fastify-vite	❌	✅	✅
vuejs/pinia	❌	✅	✅
dyad-sh/dyad	❌	🟨	✅
actualbudget/actual	❌	✅	✅
Comfy-Org/ComfyUI_frontend	❌	✅	✅
formatjs/formatjs	✅	✅	✅
AmanVarshney01/create-better-t-stack	✅	✅	✅
fuma-nama/fumadocs	✅	✅	✅
tale/headplane	❌	✅	✅
sveltejs/kit	✅	✅	✅
cloudflare/kumo	✅	✅	✅
vercel/turborepo	❌	✅	✅
lichess-org/lila	✅	✅	✅
npmx-dev/npmx.dev	✅	✅	✅
getsentry/sentry-javascript	✅	✅	✅
cloudflare/workers-sdk	✅	✅	✅
mantinedev/mantine	❌	✅	✅
huggingface/huggingface.js	❌	✅	✅
openclaw/openclaw	✅	✅	✅
vuejs/core	✅	✅	❌
cloudflare/agents	❌	✅	✅
cnpm/cnpmcore	❌	✅	✅
mastodon/mastodon	✅	✅	✅
aidenybai/react-grab	❌	✅	✅
rolldown/rolldown	✅	✅	✅

💥 = panic, ⚠️ = error e.g. parse error (main/branch columns: only new errors), ❌ = diff or exit code mismatch, 🟨 = differs from oxfmt@latest (merged-but-unreleased changes)

codspeed · 2026-09-28T06:53:51Z

Merging this PR will not alter performance

✅ 77 untouched benchmarks
⏩ 19 skipped benchmarks¹

_{Comparing 09-28-fix_formatter_skip_test-call_layout_when_arguments_have_comments (2cbb3b2) with main (a5290a6)²}

19 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. ↩
No successful run was found on main (e9b6ee0) during the generation of this report, so a5290a6 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

leaysgur · 2026-09-28T07:12:30Z

Merge activity

Sep 28, 7:12 AM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
Sep 28, 7:12 AM UTC: leaysgur added this pull request to the Graphite merge queue.
Sep 28, 7:16 AM UTC: Merged by the Graphite merge queue.

…27119) Fixes #24960, closes #26926.

# Oxlint ### 🚀 Features - 9d80eed linter/react/only-export-components: Support `allowCompoundComponents` (#27117) (Kuroda Kayn) - e05b155 linter: Add typescript/no-generated-empty-object-type (#26958) (camc314) ### 🐛 Bug Fixes - 42dfbb5 linter/eslint/one-var: Keep `declare` when splitting declarations (#27081) (Cheolhee Lee) - 2ba7e33 linter/eslint/require-await: Count `await using` as an await (#27080) (Cheolhee Lee) - 611e4ed linter/plugins: Include executing selectors in JS plugin rule timings (#27111) (overlookmotel) - 2cac66f react-compiler: Handle recursive function expressions (#26796) (Brennan Butler) - e996e6c react-compiler: Treat zero-argument new Date as impure (#26894) (Boshen) - 571cfa3 oxlint: Skip type-aware lint rules in type-check-only mode (#27076) (camc314) - d0b2462 oxlint: Skip undefined children in CFG walker (#27075) (camc314) - 5186328 parser: Handle HTML comment values (#22933) (Boshen) - 0b630e8 linter/typescript/unified-signatures: Align rule with upstream (#26956) (camc314) - ebb22f1 linter/node/no-exports-assign: Change category from style to suspicious (#26555) (Bartok) - cce28a0 linter/import/no-duplicates: Distinguish import attributes (#26936) (camc314) - feb733b linter/unicorn/prefer-spread: Stop checking string split calls (#26935) (camc314) - 929e154 linter/eslint/no-unused-vars: Honor ignore patterns inside array rest bindings (#26923) (camc314) - 5bdb9b8 linter/eslint/no-unused-vars: Recognize consumed update expressions (#26782) (camc314) - 5c05bef linter/eslint/prefer-const: Ignore embedded assignments (#26920) (camc314) ### ⚡ Performance - ded4c29 linter/eslint/no-unused-vars: Skip sequence checks when absent (#26921) (camc314) # Oxfmt ### 🚀 Features - e0b1f9f oxfmt: Bump bundled Prettier version to 3.9.9 (#27002) (leaysgur) - 342527d oxfmt: Bump bundled Prettier version to 3.9.8 (#26999) (leaysgur) ### 🐛 Bug Fixes - eeba1db formatter: Skip test-call layout when arguments have comments (#27119) (leaysgur) - 1f7b8ad oxfmt: Allow repeated CLI calls in the same process (#27051) (Liang) - 7bcb807 formatter_markdown: Keep blank line between HTML and nested list (#27112) (leaysgur) - 10b10c5 formatter: Keep comments around `=` on their side and line (#27041) (leaysgur) - 9aad365 formatter: Keep comments deferred before an assignment operator (#26997) (waltu) - 8fbddb1 formatter/jsdoc: More alignment with original plugin (#27039) (leaysgur) - 50be18e formatter: Keep trailing spaces on normal block comments (#27037) (leaysgur) - 56d1880 formatter: Nestle adjacent block comments (#27036) (leaysgur) - 3be5d94 formatter: Treat `/***` comments as JSDoc (#27035) (leaysgur) - cd45f71 formatter: Keep trailing double spaces on JSDoc lines (#26861) (John Costa) - fd695f4 formatter_markdown: Fix more mismatches found in ecosystem-ci repos (#27003) (leaysgur) - 4e77d59 formatter_markdown: Keep a math span after a kept line break from opening a block (#27001) (leaysgur) - 584b8b0 formatter_markdown: Keep a shape line after a multi-line inline node or link title (#27000) (leaysgur) - b939645 formatter_markdown: Keep a line break before an inline liquid tag under preserve (#26998) (leaysgur)

🔵 **minor** · oxfmt This PR contains the following updates: | Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) | |---|---|---|---| | [oxfmt](https://oxc.rs/docs/guide/usage/formatter) ([source](https://github.com/oxc-project/oxc/tree/HEAD/npm/oxfmt)) | [`0.70.0` → `0.71.0`](https://renovatebot.com/diffs/npm/oxfmt/0.70.0/0.71.0) | ![age](https://developer.mend.io/api/mc/badges/age/npm/oxfmt/0.71.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/oxfmt/0.70.0/0.71.0?slim=true) | | [oxfmt](https://oxc.rs/docs/guide/usage/formatter) ([source](https://github.com/oxc-project/oxc/tree/HEAD/npm/oxfmt)) | `0.70.0` → `0.71.0` | ![age](https://developer.mend.io/api/mc/badges/age/npm/oxfmt/0.71.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/oxfmt/0.70.0/0.71.0?slim=true) | --- ### Release Notes <details> <summary>oxc-project/oxc (oxfmt)</summary> ### [`v0.71.0`](https://github.com/oxc-project/oxc/releases/tag/oxfmt_v0.71.0): oxfmt v0.71.0 [Compare Source](oxc-project/oxc@oxfmt_v0.70.0...oxfmt_v0.71.0) ##### 🚀 Features - [`e0b1f9f`](oxc-project/oxc@e0b1f9f) oxfmt: Bump bundled Prettier version to 3.9.9 ([#27002](oxc-project/oxc#27002)) (leaysgur) - [`342527d`](oxc-project/oxc@342527d) oxfmt: Bump bundled Prettier version to 3.9.8 ([#26999](oxc-project/oxc#26999)) (leaysgur) ##### 🐛 Bug Fixes - [`eeba1db`](oxc-project/oxc@eeba1db) formatter: Skip test-call layout when arguments have comments ([#27119](oxc-project/oxc#27119)) (leaysgur) - [`1f7b8ad`](oxc-project/oxc@1f7b8ad) oxfmt: Allow repeated CLI calls in the same process ([#27051](oxc-project/oxc#27051)) (Liang) - [`7bcb807`](oxc-project/oxc@7bcb807) formatter\_markdown: Keep blank line between HTML and nested list ([#27112](oxc-project/oxc#27112)) (leaysgur) - [`10b10c5`](oxc-project/oxc@10b10c5) formatter: Keep comments around `=` on their side and line ([#27041](oxc-project/oxc#27041)) (leaysgur) - [`9aad365`](oxc-project/oxc@9aad365) formatter: Keep comments deferred before an assignment operator ([#26997](oxc-project/oxc#26997)) (waltu) - [`8fbddb1`](oxc-project/oxc@8fbddb1) formatter/jsdoc: More alignment with original plugin ([#27039](oxc-project/oxc#27039)) (leaysgur) - [`50be18e`](oxc-project/oxc@50be18e) formatter: Keep trailing spaces on normal block comments ([#27037](oxc-project/oxc#27037)) (leaysgur) - [`56d1880`](oxc-project/oxc@56d1880) formatter: Nestle adjacent block comments ([#27036](oxc-project/oxc#27036)) (leaysgur) - [`3be5d94`](oxc-project/oxc@3be5d94) formatter: Treat `/***` comments as JSDoc ([#27035](oxc-project/oxc#27035)) (leaysgur) - [`cd45f71`](oxc-project/oxc@cd45f71) formatter: Keep trailing double spaces on JSDoc lines ([#26861](oxc-project/oxc#26861)) (John Costa) - [`fd695f4`](oxc-project/oxc@fd695f4) formatter\_markdown: Fix more mismatches found in ecosystem-ci repos ([#27003](oxc-project/oxc#27003)) (leaysgur) - [`4e77d59`](oxc-project/oxc@4e77d59) formatter\_markdown: Keep a math span after a kept line break from opening a block ([#27001](oxc-project/oxc#27001)) (leaysgur) - [`584b8b0`](oxc-project/oxc@584b8b0) formatter\_markdown: Keep a shape line after a multi-line inline node or link title ([#27000](oxc-project/oxc#27000)) (leaysgur) - [`b939645`](oxc-project/oxc@b939645) formatter\_markdown: Keep a line break before an inline liquid tag under preserve ([#26998](oxc-project/oxc#26998)) (leaysgur) </details> --- - [ ] If you want to rebase/retry this PR, check this box  Reviewed-on: https://git.dcunha.io/Exikle/Artemis-Cluster/pulls/2446

github-actions Bot added the A-formatter Area - Formatter label Sep 28, 2026

leaysgur added the run-oxfmt-ecosystem-ci label Sep 28, 2026

oxc-guard Bot removed the run-oxfmt-ecosystem-ci label Sep 28, 2026

leaysgur added the 0-merge Merge with Graphite Merge Queue label Sep 28, 2026

leaysgur mentioned this pull request Sep 28, 2026

fix(formatter): preserve comment order in test calls #26926

Closed

fix(formatter): skip test-call layout when arguments have comments (#…

eeba1db

…27119) Fixes #24960, closes #26926.

graphite-app Bot force-pushed the 09-28-fix_formatter_skip_test-call_layout_when_arguments_have_comments branch from 2cbb3b2 to eeba1db Compare September 28, 2026 07:13

graphite-app Bot merged commit eeba1db into main Sep 28, 2026
33 checks passed

graphite-app Bot removed the 0-merge Merge with Graphite Merge Queue label Sep 28, 2026

graphite-app Bot deleted the 09-28-fix_formatter_skip_test-call_layout_when_arguments_have_comments branch September 28, 2026 07:16

oxc-guard Bot mentioned this pull request Sep 28, 2026

release(apps): oxlint v1.86.0 && oxfmt v0.71.0 #27132

Merged

bachmacintosh mentioned this pull request Oct 1, 2026

Track oxlint/oxfmt changes between peerDependencies and devDependencies bachman-dev/oxc-config#19

Open

1 of 42 tasks

Navigation Menu

Uh oh!

fix(formatter): skip test-call layout when arguments have comments - #27119

fix(formatter): skip test-call layout when arguments have comments#27119
graphite-app[bot] merged 1 commit into
mainfrom
09-28-fix_formatter_skip_test-call_layout_when_arguments_have_comments

leaysgur commented Sep 28, 2026

Uh oh!

leaysgur commented Sep 28, 2026

Uh oh!

oxc-guard Bot commented Sep 28, 2026 •

edited

Loading

Uh oh!

codspeed Bot commented Sep 28, 2026

Uh oh!

leaysgur commented Sep 28, 2026 •

edited by graphite-app Bot

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

leaysgur commented Sep 28, 2026

Uh oh!

leaysgur commented Sep 28, 2026

How to use the Graphite Merge Queue

Uh oh!

oxc-guard Bot commented Sep 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Oxfmt Ecosystem CI

Uh oh!

codspeed Bot commented Sep 28, 2026

Merging this PR will not alter performance

Footnotes

Uh oh!

leaysgur commented Sep 28, 2026 • edited by graphite-app Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

oxc-guard Bot commented Sep 28, 2026 •

edited

Loading

leaysgur commented Sep 28, 2026 •

edited by graphite-app Bot

Loading