feat(linter): allow configuring tsgolint rules

[camchenry]

This depends on the next version of tsgolint (after 0.6.0) to be released in order to work, otherwise not all options will not be accepted.

This implements rule configuration for rules that run in tsgolint. This is achieved by serializing the rule configuration as JSON and sending it as part of the payload, which is now supported by tsgolint. This is implemented by adding a new to_configuration method, which does the opposite of from_configuration. This is only implemented and used for tsgolint rules, so that there's no additional performance cost unless you are using type-aware linting.

Unfortunately, this does mean that we sort of have to maintain config parsing in two separate codebases, but I think we can improve this in the future with more tooling.

---

Generated rule documentation:

## Configuration

This rule accepts a configuration object with the following properties:

### allowForKnownSafeCalls

type: `array`

default: `[]`

Allows specific calls to be ignored, specified as type or value specifiers.


#### allowForKnownSafeCalls[n]



Type or value specifier for matching specific declarations

Supports four types of specifiers:

1. **String specifier** (deprecated): Universal match by name
```json
"Promise"
```

2. **File specifier**: Match types/values declared in local files
```json
{ "from": "file", "name": "MyType" }
{ "from": "file", "name": ["Type1", "Type2"] }
{ "from": "file", "name": "MyType", "path": "./types.ts" }
```

3. **Lib specifier**: Match TypeScript built-in lib types
```json
{ "from": "lib", "name": "Promise" }
{ "from": "lib", "name": ["Promise", "PromiseLike"] }
```

4. **Package specifier**: Match types/values from npm packages
```json
{ "from": "package", "name": "Observable", "package": "rxjs" }
{ "from": "package", "name": ["Observable", "Subject"], "package": "rxjs" }
```


### allowForKnownSafePromises

type: `array`

default: `[]`

Allows specific Promise types to be ignored, specified as type or value specifiers.


#### allowForKnownSafePromises[n]



Type or value specifier for matching specific declarations

Supports four types of specifiers:

1. **String specifier** (deprecated): Universal match by name
```json
"Promise"
```

2. **File specifier**: Match types/values declared in local files
```json
{ "from": "file", "name": "MyType" }
{ "from": "file", "name": ["Type1", "Type2"] }
{ "from": "file", "name": "MyType", "path": "./types.ts" }
```

3. **Lib specifier**: Match TypeScript built-in lib types
```json
{ "from": "lib", "name": "Promise" }
{ "from": "lib", "name": ["Promise", "PromiseLike"] }
```

4. **Package specifier**: Match types/values from npm packages
```json
{ "from": "package", "name": "Observable", "package": "rxjs" }
{ "from": "package", "name": ["Observable", "Subject"], "package": "rxjs" }
```


### checkThenables

type: `boolean`

default: `false`

Check for thenable objects that are not necessarily Promises.


### ignoreIIFE

type: `boolean`

default: `false`

Ignore immediately invoked function expressions (IIFEs).


### ignoreVoid

type: `boolean`

default: `true`

Ignore Promises that are void expressions.

[camchenry]

• test(linter): add e2e test exercising many config options #15660
• feat(linter): implement config for all tsgolint rules supporting options #15659
• feat(linter): allow configuring tsgolint rules #15286 👈 (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 hot fixes, skip the queue and merge this PR next

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.

[codspeed-hq]

CodSpeed Performance Report

Merging #15286 will not alter performance

_{Comparing 11-04-feat_linter_allow_configuring_tsgolint_rules (5d935a9) with main (ba76081)¹}

Summary

✅ 4 untouched
⏩ 33 skipped²

Footnotes

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

2. 33 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. ↩

[Copilot]

Pull Request Overview

This PR implements rule configuration support for tsgolint by adding a to_configuration method that serializes rule configurations to JSON. This enables oxlint to pass rule-specific options to the tsgolint type-aware linter.

• Adds to_configuration() trait method to serialize rule configurations
• Implements configuration structs and serialization for no-floating-promises rule
• Updates tsgolint integration to include rule options in the payload (version 2 format)

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
crates/oxc_macros/src/declare_all_lint_rules.rs	Adds generated to_configuration method that delegates to individual rule implementations
crates/oxc_linter/src/rule.rs	Defines the to_configuration trait method with clear documentation about its usage for tsgolint rules
crates/oxc_linter/src/tsgolint.rs	Extends Rule struct with options field and implements Ord/PartialOrd; updates payload construction to include serialized rule configurations
crates/oxc_linter/src/rules/typescript/no_floating_promises.rs	Implements complete configuration support including config structs, type specifiers, serialization, and comprehensive tests

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[camc314]

Merge activity

• Nov 14, 10:22 PM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Nov 14, 10:22 PM UTC: camc314 added this pull request to the Graphite merge queue.
• Nov 14, 10:28 PM UTC: Merged by the Graphite merge queue.