feat(linter): implement useVueDefineMacrosOrder
#7712
Conversation
🦋 Changeset detectedLatest commit: e6afa6f The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
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 |
CodSpeed Performance ReportMerging #7712 will not alter performanceComparing Summary
Footnotes
|
WalkthroughAdds a new lint rule UseVueDefineMacrosOrder to the Biome nursery lints to enforce a configurable order for Vue <script setup> compiler macros and provide an autofix that reorders top-level module items. Introduces UseVueDefineMacrosOrderOptions with a default order, exports its module, and adds multiple Vue SFC test fixtures (valid/invalid and custom-order cases). Updates the changeset with the new rule entry and example. Suggested labels
Suggested reviewers
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (14)
crates/biome_cli/src/execute/migrate/eslint_any_rule_to_biome.rsis excluded by!**/migrate/eslint_any_rule_to_biome.rsand included by**crates/biome_configuration/src/analyzer/linter/rules.rsis excluded by!**/rules.rsand included by**crates/biome_diagnostics_categories/src/categories.rsis excluded by!**/categories.rsand included by**crates/biome_js_analyze/src/lint/nursery.rsis excluded by!**/nursery.rsand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vue.snapis excluded by!**/*.snapand included by**packages/@biomejs/backend-jsonrpc/src/workspace.tsis excluded by!**/backend-jsonrpc/src/workspace.tsand included by**packages/@biomejs/biome/configuration_schema.jsonis excluded by!**/configuration_schema.jsonand included by**
📒 Files selected for processing (14)
.changeset/bright-insects-ring.md(1 hunks)crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.json(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.json(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vue(1 hunks)crates/biome_rule_options/src/lib.rs(1 hunks)crates/biome_rule_options/src/use_vue_define_macros_order.rs(1 hunks)
🧰 Additional context used
📓 Path-based instructions (6)
crates/biome_*_{syntax,parser,formatter,analyze,factory,semantic}/**
📄 CodeRabbit inference engine (CLAUDE.md)
Maintain the per-language crate structure: biome_{lang}_{syntax,parser,formatter,analyze,factory,semantic}
Files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vuecrates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rscrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vue
crates/biome_*/**
📄 CodeRabbit inference engine (CLAUDE.md)
Place core crates under /crates/biome_*/
Files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.jsoncrates/biome_rule_options/src/lib.rscrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vuecrates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rscrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vuecrates/biome_rule_options/src/use_vue_define_macros_order.rs
**/tests/**
📄 CodeRabbit inference engine (CLAUDE.md)
Place test files under a tests/ directory in each crate
Files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vue
.changeset/**/*.md
📄 CodeRabbit inference engine (CONTRIBUTING.md)
.changeset/**/*.md: Create changesets using thejust new-changesetcommand; do not author them manually
In changeset markdown, only use headers #### or #####
Changeset descriptions must end every sentence with a full stop (.)
For bug fixes, start the changeset description with a linked issue reference like “Fixed #1234”
Prefer past tense for what was done and present tense for current behavior in changesets
Files:
.changeset/bright-insects-ring.md
**/*.{rs,toml}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Before committing, format Rust and TOML files (e.g., via
just f/just format)
Files:
crates/biome_rule_options/src/lib.rscrates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rscrates/biome_rule_options/src/use_vue_define_macros_order.rs
**/*.rs
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Document rules, assists, and options via inline rustdoc in Rust source
Files:
crates/biome_rule_options/src/lib.rscrates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rscrates/biome_rule_options/src/use_vue_define_macros_order.rs
🧠 Learnings (3)
📚 Learning: 2025-10-02T12:57:33.209Z
Learnt from: CR
PR: biomejs/biome#0
File: crates/biome_analyze/CONTRIBUTING.md:0-0
Timestamp: 2025-10-02T12:57:33.209Z
Learning: Applies to crates/biome_analyze/crates/biome_js_analyze/tests/specs/nursery/** : Place snapshot tests for new rules under tests/specs/nursery/<ruleName>/ with files prefixed by invalid* and valid*
Applied to files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.json
📚 Learning: 2025-08-05T14:43:29.581Z
Learnt from: dyc3
PR: biomejs/biome#7081
File: packages/@biomejs/biome/configuration_schema.json:7765-7781
Timestamp: 2025-08-05T14:43:29.581Z
Learning: The file `packages/biomejs/biome/configuration_schema.json` is auto-generated and should not be manually edited or reviewed for schema issues; any changes should be made at the code generation source.
Applied to files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.json
📚 Learning: 2025-10-02T12:57:33.209Z
Learnt from: CR
PR: biomejs/biome#0
File: crates/biome_analyze/CONTRIBUTING.md:0-0
Timestamp: 2025-10-02T12:57:33.209Z
Learning: Applies to crates/biome_analyze/crates/biome_rule_options/lib/*.rs : Define per-rule options in biome_rule_options/lib/<rule>.rs with a dedicated options struct/enum (camelCase serde names, deny_unknown_fields, default) and derive Serialize/Deserialize/Deserializable (and schemars JsonSchema when schema feature is on)
Applied to files:
crates/biome_rule_options/src/use_vue_define_macros_order.rs
🧬 Code graph analysis (2)
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs (3)
packages/@biomejs/backend-jsonrpc/src/workspace.ts (4)
FixKind(7941-7941)RuleDomain(904-912)TextRange(9292-9292)UseVueDefineMacrosOrderOptions(8297-8302)crates/biome_js_factory/src/generated/node_factory.rs (1)
js_module_item_list(6976-6987)crates/biome_js_syntax/src/generated/nodes.rs (1)
declarators(6878-6880)
crates/biome_rule_options/src/use_vue_define_macros_order.rs (1)
packages/@biomejs/backend-jsonrpc/src/workspace.ts (1)
UseVueDefineMacrosOrderOptions(8297-8302)
🪛 GitHub Actions: Pull request Markdown
.changeset/bright-insects-ring.md
[error] 13-13: markdownlint: MD047/single-trailing-newline. Files should end with a single newline character.
🪛 GitHub Check: lint
.changeset/bright-insects-ring.md
[failure] 13-13: Files should end with a single newline character
.changeset/bright-insects-ring.md:13:3 MD047/single-trailing-newline Files should end with a single newline character https://github.com/DavidAnson/markdownlint/blob/v0.38.0/doc/md047.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (14)
- GitHub Check: Test (depot-windows-2022-16)
- GitHub Check: Documentation
- GitHub Check: Test (depot-ubuntu-24.04-arm-16)
- GitHub Check: End-to-end tests
- GitHub Check: Bench (biome_js_analyze)
- GitHub Check: Lint project (depot-windows-2022)
- GitHub Check: Lint project (depot-ubuntu-24.04-arm-16)
- GitHub Check: Bench (biome_js_formatter)
- GitHub Check: Test Node.js API
- GitHub Check: Check Dependencies
- GitHub Check: Bench (biome_js_parser)
- GitHub Check: autofix
- GitHub Check: Check JS Files
- GitHub Check: Bench (biome_configuration)
🔇 Additional comments (7)
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.json (1)
1-20: LGTM!The custom order configuration correctly tests an extended macro sequence including
definePage.crates/biome_rule_options/src/lib.rs (1)
367-367: LGTM!Module export follows the established pattern and is correctly positioned alphabetically.
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.json (1)
1-20: LGTM!The configuration matches
invalid-custom.options.json, which is correct—the validity difference comes from whether the corresponding.vuefixture follows this order.crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vue (1)
1-18: LGTM!Test fixture correctly demonstrates invalid macro ordering with
withDefaultswrappingdefineProps.crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vue (1)
1-18: LGTM!Test fixture correctly demonstrates valid macro ordering with
definePropsbeforedefineEmits.crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vue (1)
1-20: LGTM!Test fixture correctly demonstrates invalid macro ordering and includes comments to test proper handling of interspersed annotations.
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vue (1)
1-17: LGTM!Test fixture correctly demonstrates invalid macro ordering in a comma-separated declaration, covering an important edge case.
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs
Outdated
Show resolved
Hide resolved
de7e548 to
a2748df
Compare
| let statement = statements.remove(from); | ||
|
|
||
| // Need to add newline after the moved statement, because it may not have one. | ||
| let trailing_trivia = make::token(T![;]) |
There was a problem hiding this comment.
Ideally this would add newline only when needed, but I am not sure how to check for it generally without needing to handle a lot of node types.
Element after moved macro can be any JS statement.
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (3)
.changeset/bright-insects-ring.md (1)
13-13: Trailing newline still required.The missing trailing newline has already been flagged and must be fixed before merge.
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs (2)
150-173: Critical: Call expression branch misses later out-of-order macros.Lines 170-172 do nothing when a macro is in correct order (same/higher
order_index), meaningfound_macrois never updated. This causes later out-of-order macros to slip through undetected.Example that fails:
defineModel() // order 0, found_macro = Some(order: 0) defineEmits() // order 2, NOT < 0, so no update (found_macro still order: 0) defineProps() // order 1, NOT < 0, so no update—bug! Should be caught.The declarator branch (lines 201-216) handles this correctly by always updating
found_macro. Apply the same fix here.- _ => { - // Same/higher order, do nothing + Some(c) => { + let has_violation = c.has_out_of_order_content_prior || non_macro_found; + found_macro = Some(FoundMacro { + order_index, + fixable_statement_index: Some(index), + range: statement.range(), + has_out_of_order_content_prior: has_violation, + }); }
226-236: Major: Fix placement moves macros to top instead of after last in-order macro.
skippable_top_statements_end_index(line 138) only tracks skippable pre-content, not the position of previously seen macros. When reordering macros, the fixer moves the out-of-order macro to the top (e.g., index 0 or 1) rather than inserting it after the last correctly ordered macro.Example:
defineModel() // index 0 defineEmits() // index 1, order 2 defineProps() // index 2, order 1—out of orderCurrent fix:
move_from_to = (2, 0)→ movesdefinePropsto the very top.
Expected fix:move_from_to = (2, 1)→ insertsdefinePropsafterdefineModel.Store each seen macro's index and
order_indexas you iterate. When you detect an out-of-order macro, scan backwards through those recorded macros to find the nearest one withorder_index ≤ current_order_index, and set the target tothat_macro_index + 1. Fall back toskippable_top_statements_end_indexonly if no such macro exists.
🧹 Nitpick comments (1)
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs (1)
278-284: Optional: Unconditional newline may create double spacing.Lines 278-281 always append a newline to the moved statement, even if trailing trivia already exists. This can result in double newlines.
Check whether the statement already has trailing newline trivia before adding one:
let trailing_trivia = statement.syntax().last_trailing_trivia(); let needs_newline = trailing_trivia .and_then(|t| t.pieces().last()) .map_or(true, |piece| piece.kind() != TriviaPieceKind::Newline); let statement = if needs_newline { let newline_trivia = make::token(T![;]) .with_trailing_trivia([(TriviaPieceKind::Newline, "\n")]) .trailing_trivia() .pieces(); statement.with_trailing_trivia_pieces(newline_trivia)? } else { statement };
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (16)
crates/biome_cli/src/execute/migrate/eslint_any_rule_to_biome.rsis excluded by!**/migrate/eslint_any_rule_to_biome.rsand included by**crates/biome_configuration/src/analyzer/linter/rules.rsis excluded by!**/rules.rsand included by**crates/biome_diagnostics_categories/src/categories.rsis excluded by!**/categories.rsand included by**crates/biome_js_analyze/src/lint/nursery.rsis excluded by!**/nursery.rsand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-c.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single-a.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue.snapis excluded by!**/*.snapand included by**crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vue.snapis excluded by!**/*.snapand included by**packages/@biomejs/backend-jsonrpc/src/workspace.tsis excluded by!**/backend-jsonrpc/src/workspace.tsand included by**packages/@biomejs/biome/configuration_schema.jsonis excluded by!**/configuration_schema.jsonand included by**
📒 Files selected for processing (16)
.changeset/bright-insects-ring.md(1 hunks)crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-c.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.json(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single-a.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.json(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue(1 hunks)crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vue(1 hunks)crates/biome_rule_options/src/lib.rs(1 hunks)crates/biome_rule_options/src/use_vue_define_macros_order.rs(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (7)
- crates/biome_rule_options/src/lib.rs
- crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.vue
- crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.options.json
- crates/biome_rule_options/src/use_vue_define_macros_order.rs
- crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-with-defaults.vue
- crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-a.vue
- crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-b.vue
🧰 Additional context used
📓 Path-based instructions (6)
crates/biome_*_{syntax,parser,formatter,analyze,factory,semantic}/**
📄 CodeRabbit inference engine (CLAUDE.md)
Maintain the per-language crate structure: biome_{lang}_{syntax,parser,formatter,analyze,factory,semantic}
Files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vuecrates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rscrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-c.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single-a.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue
crates/biome_*/**
📄 CodeRabbit inference engine (CLAUDE.md)
Place core crates under /crates/biome_*/
Files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vuecrates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rscrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-c.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single-a.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue
**/tests/**
📄 CodeRabbit inference engine (CLAUDE.md)
Place test files under a tests/ directory in each crate
Files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.jsoncrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-c.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single-a.vuecrates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue
.changeset/**/*.md
📄 CodeRabbit inference engine (CONTRIBUTING.md)
.changeset/**/*.md: Create changesets using thejust new-changesetcommand; do not author them manually
In changeset markdown, only use headers #### or #####
Changeset descriptions must end every sentence with a full stop (.)
For bug fixes, start the changeset description with a linked issue reference like “Fixed #1234”
Prefer past tense for what was done and present tense for current behavior in changesets
Files:
.changeset/bright-insects-ring.md
**/*.{rs,toml}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Before committing, format Rust and TOML files (e.g., via
just f/just format)
Files:
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs
**/*.rs
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Document rules, assists, and options via inline rustdoc in Rust source
Files:
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs
🧠 Learnings (1)
📚 Learning: 2025-10-02T12:57:33.228Z
Learnt from: CR
PR: biomejs/biome#0
File: crates/biome_analyze/CONTRIBUTING.md:0-0
Timestamp: 2025-10-02T12:57:33.228Z
Learning: Applies to crates/biome_analyze/crates/biome_js_analyze/tests/specs/nursery/** : Place snapshot tests for new rules under tests/specs/nursery/<ruleName>/ with files prefixed by invalid* and valid*
Applied to files:
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.json
🧬 Code graph analysis (1)
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs (3)
packages/@biomejs/backend-jsonrpc/src/workspace.ts (4)
FixKind(7941-7941)RuleDomain(904-912)TextRange(9292-9292)UseVueDefineMacrosOrderOptions(8297-8302)crates/biome_js_factory/src/generated/node_factory.rs (1)
js_module_item_list(6976-6987)crates/biome_js_syntax/src/generated/nodes.rs (1)
declarators(6878-6880)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Lint project (depot-windows-2022)
🔇 Additional comments (9)
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-custom.options.json (1)
1-20: LGTM!The configuration correctly specifies a custom macro order including
definePage, which aligns with the exploration mentioned in the PR objectives.crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single.vue (1)
1-17: LGTM!The test fixture correctly demonstrates an invalid macro order scenario where
defineEmitsprecedesdefinePropson the same line.crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid.vue (1)
1-18: LGTM!The test fixture correctly demonstrates valid macro ordering with
definePropsbeforedefineEmits.crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/valid-custom.vue (1)
1-20: LGTM!The test fixture correctly demonstrates a valid custom macro order including
definePage, matching the configuration invalid-custom.options.json(presumably).crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-single-a.vue (1)
2-2: Verify missing ref import is intentional.Similar to
invalid-c.vue, this test callsref(0)without importing it from Vue. Vue compiler macros are auto-imported, butrefis not.crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid.vue (1)
1-20: LGTM!The test fixture correctly demonstrates an invalid macro order scenario with comments interspersed, which will verify that the lint rule preserves comments when reordering.
crates/biome_js_analyze/tests/specs/nursery/useVueDefineMacrosOrder/invalid-c.vue (1)
2-4: Missingrefimport is intentional in invalid-c.vue nursery fixture. These test cases deliberately omit imports to simulate invalid code paths.crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs (2)
1-108: LGTM on rule metadata and documentation.The rule declaration, metadata, and examples are clear and well-structured. Good handling of the
withDefaultswrapper and flexibleorderconfiguration.
304-361: LGTM on helper functions.The helper functions correctly extract macro names from call expressions and declarators, handle
withDefaults(defineProps(...)), and identify skippable pre-macro content (imports, types, debugger statements, etc.).
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs
Outdated
Show resolved
Hide resolved
crates/biome_js_analyze/src/lint/nursery/use_vue_define_macros_order.rs
Outdated
Show resolved
Hide resolved
1262b48 to
1612ad0
Compare
Macros themselves inside multi script file are rare, but I was more referring into the fact that multi script handling itself is broken. When I tried to add snapshots, only first script would be outputted, second one discarded. In my own codebases I use them a lot, so no idea how this rule would behave. In any case this is temporary issue. |
Summary
Closes #7345
Multi script handling is broken, would need for setup attribute information to land first, but since this is nursery rule for now I left it as is.
Changes from Eslint rule:
defineExposeLastoption.defineModelwhile original rule does not.defineModelwas added to the Vue much later than the rule itself, but given how common it is and because it combines functionality ofdefineProps/defineEmitsseems like a good default to have.I almost wonder if
defineOptionsanddefineSlotsshould be included as well.Test Plan
Added snapshots