feat(linter/jsdoc): implement no-blank-blocks rule

[baevm]

this PR implements jsdoc/no-blank-blocks rule, issue #1170

[baevm]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 18e0b2f42b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[codspeed-hq]

Merging this PR will not alter performance

✅ 5 untouched benchmarks
⏩ 71 skipped benchmarks¹

---

_{Comparing baevm:jsdoc/no-blank-blocks (f1d1b01) with main (9a423f2)²}

Footnotes

1. 71 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 (9c8c5e5) during the generation of this report, so 9a423f2 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

[Copilot]

Pull request overview

This PR adds the jsdoc/no-blank-blocks rule to Oxc’s JSDoc linter plugin, including its configuration schema and generated bindings so the rule can be enabled/configured from oxlint.

Changes:

• Implement jsdoc/no-blank-blocks as a run_once rule with an optional auto-fixer (enableFixer).
• Add/update rule registration and generated metadata (rules enum, runner impls, timing snapshot).
• Extend oxlint configuration schema + generated TypeScript config types for the new rule.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tasks/track_linter_timings/linter_timings.snap	Adds timing entry for the new JSDoc rule.
npm/oxlint/configuration_schema.json	Adds jsdoc/no-blank-blocks rule schema and NoBlankBlocks config definition.
crates/oxc_linter/src/snapshots/jsdoc_no_blank_blocks.snap	Adds snapshot output for the new rule’s tests.
crates/oxc_linter/src/rules/jsdoc/no_blank_blocks.rs	Implements the rule logic, fixer span expansion, and inline tests.
crates/oxc_linter/src/rules.rs	Exposes the new jsdoc::no_blank_blocks module.
crates/oxc_linter/src/generated/rules_enum.rs	Registers the rule in the generated rules enum + metadata hooks.
crates/oxc_linter/src/generated/rule_runner_impls.rs	Adds the generated runner implementation (RunOnce).
apps/oxlint/src-js/package/config.generated.ts	Adds generated TS config typing for jsdoc/no-blank-blocks.

Suppressed comments (1)

crates/oxc_linter/src/rules/jsdoc/no_blank_blocks.rs:122

• The fixer expansion removes the next whitespace character after the comment, but is_whitespace() includes newlines. If a blank JSDoc appears inline before a line break (e.g. foo();/** */\nbar();), this will delete the newline and join statements/tokens, potentially changing semantics.

    if let Some(next) = source_text[span.end as usize..].chars().next()
        && next.is_whitespace()
    {
        span.end += u32::try_from(next.len_utf8()).unwrap_or_default();
    }