chore: oxc.requireConfig search for .oxlintrc.jsonc too

[Sysix]

Upstream: oxc-project/oxc#19870

[Sysix]

• chore: oxc.requireConfig search for .oxlintrc.jsonc too #110 👈 (View in Graphite)
• main

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

[Copilot]

Pull request overview

Updates the VS Code extension’s oxc.requireConfig gating to recognize .oxlintrc.jsonc in addition to .oxlintrc.json, aligning behavior and documentation with upstream oxc config discovery.

Changes:

• Extend config file globbing to include .oxlintrc.jsonc for startup gating and file watching.
• Update user-facing descriptions/documentation to mention .oxlintrc.json(c).
• Tighten repo linting by enabling typescript/no-floating-promises in oxlint.config.json.

Reviewed changes

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

File	Description
package.json	Updates oxc.requireConfig setting description to mention .jsonc.
oxlint.config.json	Enables typescript/no-floating-promises for stricter promise handling.
client/tools/linter.ts	Adds .oxlintrc.jsonc to config detection glob and reuses it in several places.
README.md	Updates documentation text for oxc.requireConfig.

Comments suppressed due to low confidence (1)

client/tools/linter.ts:44

• The comment above allowedToStartServer still says requireConfig checks for a .oxlintrc.json file only, but the logic now also considers .oxlintrc.jsonc and oxlint.config.ts. Update the comment so it matches the current behavior.

const oxlintConfigDefaultFilePattern = `**/{.oxlintrc.json,.oxlintrc.jsonc,oxlint.config.ts}`;

export default class LinterTool implements ToolInterface {
  // Global flag to check if the user allows us to start the server.
  // When `oxc.requireConfig` is `true`, make sure one `.oxlintrc.json` file is present.
  private allowedToStartServer: boolean = false;

---

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

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (1)

client/tools/linter.ts:44

• The comment above allowedToStartServer still says we only check for .oxlintrc.json, but the logic now also considers .oxlintrc.jsonc and oxlint.config.ts. Please update the comment so it matches the actual activation conditions (and stays correct if the pattern changes again).

  // Global flag to check if the user allows us to start the server.
  // When `oxc.requireConfig` is `true`, make sure one `.oxlintrc.json` file is present.
  private allowedToStartServer: boolean = false;

---

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