Skip to content

feat(oxfmt): Support vite-plus/resolveConfig for vite.config.ts#22454

Merged
graphite-app[bot] merged 1 commit into
mainfrom
05-15-feat_oxfmt_support_vite-plus_resolveconfig
May 20, 2026
Merged

feat(oxfmt): Support vite-plus/resolveConfig for vite.config.ts#22454
graphite-app[bot] merged 1 commit into
mainfrom
05-15-feat_oxfmt_support_vite-plus_resolveconfig

Conversation

@leaysgur

@leaysgur leaysgur commented May 15, 2026

Copy link
Copy Markdown
Member

Closes #21603, fixes #21418

To simplify the review process, I'll first add support to Oxfmt.

In the next PR, I'll move that implementation to shared.

leaysgur commented May 15, 2026

Copy link
Copy Markdown
Member Author

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.

@github-actions github-actions Bot added A-cli Area - CLI A-formatter Area - Formatter labels May 15, 2026
@leaysgur leaysgur changed the title feat(oxfmt): Support vite-plus resolveConfig feat(oxfmt): Support vite-plus/resolveConfig for vite.config.ts May 15, 2026
@leaysgur leaysgur force-pushed the 05-15-refactor_oxlint_oxfmt_use_shared_importjsconfig branch from f47dc94 to ae332c0 Compare May 18, 2026 01:33
@leaysgur leaysgur force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch from a5b40e1 to 751aba7 Compare May 18, 2026 01:33
@github-actions github-actions Bot added the A-linter Area - Linter label May 18, 2026
@leaysgur leaysgur force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch from add6094 to 9c6ae32 Compare May 18, 2026 02:18
@leaysgur leaysgur force-pushed the 05-15-refactor_oxlint_oxfmt_use_shared_importjsconfig branch from ae332c0 to 7ec9098 Compare May 18, 2026 02:18
@leaysgur leaysgur marked this pull request as ready for review May 18, 2026 03:54
@graphite-app graphite-app Bot force-pushed the 05-15-refactor_oxlint_oxfmt_use_shared_importjsconfig branch from 7ec9098 to e3c5885 Compare May 18, 2026 08:00
@graphite-app graphite-app Bot force-pushed the 05-15-refactor_oxlint_oxfmt_use_shared_importjsconfig branch from e3c5885 to 42a1036 Compare May 18, 2026 08:01
@graphite-app graphite-app Bot force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch from 9c6ae32 to 6f0634e Compare May 18, 2026 08:01
@graphite-app graphite-app Bot changed the base branch from 05-15-refactor_oxlint_oxfmt_use_shared_importjsconfig to graphite-base/22454 May 18, 2026 08:05
@graphite-app graphite-app Bot force-pushed the graphite-base/22454 branch from 42a1036 to dcc6d38 Compare May 18, 2026 08:09
@graphite-app graphite-app Bot force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch from 6f0634e to 669b2d7 Compare May 18, 2026 08:09
@graphite-app graphite-app Bot changed the base branch from graphite-base/22454 to main May 18, 2026 08:10
@graphite-app graphite-app Bot force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch from 669b2d7 to d9b2a4d Compare May 18, 2026 08:10
@leaysgur leaysgur force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch from d9b2a4d to 934b8a1 Compare May 19, 2026 00:46
@leaysgur leaysgur force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch 2 times, most recently from 82be4cd to b6a967a Compare May 20, 2026 02:54
@graphite-app graphite-app Bot added the 0-merge Merge with Graphite Merge Queue label May 20, 2026
@graphite-app

graphite-app Bot commented May 20, 2026

Copy link
Copy Markdown
Contributor

Merge activity

…2454)

Closes #21603, fixes #21418

To simplify the review process, I'll first add support to Oxfmt.

In the next PR, I'll move that implementation to `shared`.
@graphite-app graphite-app Bot force-pushed the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch from b6a967a to 16b8058 Compare May 20, 2026 08:21
@graphite-app graphite-app Bot merged commit 16b8058 into main May 20, 2026
27 checks passed
@graphite-app graphite-app Bot removed the 0-merge Merge with Graphite Merge Queue label May 20, 2026
@graphite-app graphite-app Bot deleted the 05-15-feat_oxfmt_support_vite-plus_resolveconfig branch May 20, 2026 08:25
Dunqing added a commit that referenced this pull request May 26, 2026
# Oxlint
### 🚀 Features

- 10da26b linter: `no-misleading-character-class` add suggestions for
regex literal (#22681) (Sysix)
- b84941e linter/vue: Implement no-expose-after-await rule (#22675)
(bab)
- 98b98c1 linter/vue: Implement no-computed-properties-in-data rule
(#22674) (bab)
- 868e2e8 linter: Add suggestion for `no-misleading-character-class`
(#22631) (Sysix)
- 2d4c919 oxlint: Support `vite-plus/resolveConfig` for vite.config.ts
(#22456) (leaysgur)
- 2a60012 linter/vue: Implement require-render-return rule (#22613)
(bab)
- 9f227fd linter/vue: Implement no-deprecated-props-default-this rule
(#21892) (bab)
- 9cd28b3 linter: Add debug option to print files to be linted (#22546)
(camchenry)
- 87f065e linter/vue: Implement return-in-emits-validator rule (#21935)
(bab)
- ea0380c linter/unicorn: Implement `import-style` rule (#22173) (Hao
Chen)
- dde40fe linter/vue: Implement no-watch-after-await rule (#22006) (bab)
- a735eb0 linter/vue: Implement valid-next-tick rule (#22531) (bab)
- 6dc615d linter/vue: Implement no-shared-component-data rule (#21842)
(bab)
- a656418 linter/vue: Implement valid-define-options rule (#22107) (bab)
- bb6f1b2 linter/vue: Implement require-slots-as-functions rule (#22244)
(bab)
- 5fa4774 linter/n: Implement `callback-return` rule (#22470) (Mikhail
Baev)

### 🐛 Bug Fixes

- 52bd016 linter: Respect allow unused disable directives in LSP
(#22715) (camc314)
- fa7c463 semantic: Correct TS enum member symbol spans (#22689)
(camc314)
- ed445ba linter: Respect flags overrides for `RegExp(/regex/i, "u")`
(#22678) (Sysix)
- 26b9396 semantic: Resolve parameter decorators outside parameter scope
(#22623) (camc314)
- 203952d linter: `no-misleading-character-class` fix
`is_unicode_code_point_escape` check (#22655) (Sysix)
- 2f64d3d linter: `no-misleading-character-class` own diagnostic for
surrogate pairs without u flag (#22654) (Sysix)
- 0c6ebc2 linter/eslint/no-lone-blocks: Do not flag empty loops (#22649)
(Mikhail Baev)
- 2a7562e linter/no-focused-tests: Mark fixer as a suggestion (#22645)
(camc314)
- dbe644f linter: Respect args none for unused rest parameters (#22627)
(camc314)
- d0211b0 linter: Allow undefined in DummyRuleMap index (#22626)
(camc314)
- 36fc0ec oxlint/lsp: "ignore this" actions merge with existing
directive (#22604) (Sysix)
- f7f370e linter/vitest/prefer-expect-type-of: Recommend `toBeTypeOf`
instead of `expectTypeOf` (#22612) (Mikhail Baev)
- 77063e5 linter/consistent-indexed-object-style: Preserve interface
modifiers in fixes (#22579) (camc314)
- 4f33aa7 linter: Treat `TSGlobalDeclaration` as ambient in
`has_ambient_typescript_ancestor` (#22577) (camc314)

### ⚡ Performance

- c22938d linter/no-async-endpoint-handlers: Populate node types
(#22601) (camc314)
- 607486e linter/no-negated-condition: Populate node types (#22602)
(camc314)
- 4dcaa59 linter/consistent-type-imports: Populate node types (#22600)
(camc314)
- 5bd3b25 linter/no-unused-vars: Avoid cloned ancestor iterator (#22598)
(camc314)
- 97fe9ba linter/no-extra-non-null-assertion: Reduce node matches
(#22588) (camc314)
- ae98296 linter/consistent-indexed-object-style: Populate node types
(#22578) (camc314)
# Oxfmt
### 🚀 Features

- 16b8058 oxfmt: Support `vite-plus/resolveConfig` for vite.config.ts
(#22454) (leaysgur)

### 🐛 Bug Fixes

- 5a26479 formatter: Preserve import phases (#22692) (Cameron)

### ⚡ Performance

- 78cf83f formatter: Pre-size output buffer using source text length
(#22594) (Dunqing)

Co-authored-by: Dunqing <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cli Area - CLI A-formatter Area - Formatter A-linter Area - Linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

oxfmt: Support full vite.config.ts syntax

2 participants