Skip to content

chore(oxlint-config): move to tsrs 0.20.1 - #262

Merged
jan-kubica merged 3 commits into
mainfrom
chore/tsrs-0-20
Nov 5, 2026
Merged

jan-kubica merged 3 commits into
mainfrom
chore/tsrs-0-20

Conversation

@jan-kubica

@jan-kubica jan-kubica commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Moves @stll/oxlint-config from tsrs 0.18.0-ts7.1.0-dev.20260929 to 0.20.1-ts7.1.0-beta.

  • tsrs 0.20 publishes @tsrs/darwin-x64 and @tsrs/win32-x64, so the package's checker and tsconfig discovery install on Intel macOS and Windows x64.
  • tsrs 0.20 moved its Node API from tsrs/unstable/* to tsrs/*; configuration discovery and the node script types test import tsrs/async.
  • README documents the new version and import path.
  • Separate commit: removes the bun-types quarantine exclusion that expired on 2026-10-27 (bun-types 1.4.4 is past the release-age window); the shared toolchain policy test rejects expired exclusions.

tsrs 0.20 publishes Intel macOS and Windows x64 binaries and moves its
Node API from tsrs/unstable/* to tsrs/*.
@jan-kubica
jan-kubica enabled auto-merge November 5, 2026 19:56
@coderabbitai

coderabbitai Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Explain Change →

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 52 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 303c807 and 985c868.

Files selected for processing: 2

  • bunfig.toml
  • packages/oxlint-config/src/toolchain-node-projects.ts

No actionable comments were generated in the recent review.

Walkthrough

The @stll/oxlint-config package now uses tsrs 0.20.1 and resolves its Node API through tsrs/async. The test import, README guidance, and patch changeset reflect these updates.

Changes: tsrs Upgrade

Layer / File(s) Summary
Dependency and API path update
packages/oxlint-config/package.json, packages/oxlint-config/src/toolchain-node-projects.ts, packages/oxlint-config/src/node-script-types.test.ts, README.md, .changeset/tsrs-0-20.md
The package dependency now uses tsrs 0.20.1. API resolution, the test import, and README guidance use tsrs/async instead of tsrs/unstable/async. The changeset describes the upgrade and the published Intel macOS and Windows x64 binaries.

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Possibly related PRs

  • stella/tooling#221: This PR introduced tsrs 0.18.0 and changed configuration discovery and tests to use tsrs/unstable/async; this PR updates the dependency and API path.
Merge Risk: Minimal · previous review 303c8

The dependency upgrade and API-path migration are consistent across implementation and documentation. No actionable issue is identified; the change is mergeable subject to normal checks.

Pre-merge checks: Passed 5

Check name Status Explanation
Title check Passed The title names the package (oxlint-config) and the change (move to tsrs 0.20.1), which matches the changeset. It is short, specific, and follows the conventional-commit style. The title covers the ma…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description directly matches the changeset. It documents the tsrs version update, platform binary support, Node API import path changes, README update, and related dependency policy change.
Full details: Title check

Explanation

The title names the package (oxlint-config) and the change (move to tsrs 0.20.1), which matches the changeset. It is short, specific, and follows the conventional-commit style. The title covers the main change, which is the dependency bump with the matching Node API import path change.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 2 files. (3 skipped: 3 unsupported.)

Finishing Touches

Generate docstrings

  • Commit to this branch
  • Create a new PR
Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between b900b9d and 303c807.

Files ignored due to path filters: 1

  • bun.lock is excluded by !**/*.lock
Files selected for processing: 5

  • .changeset/tsrs-0-20.md
  • README.md
  • packages/oxlint-config/package.json
  • packages/oxlint-config/src/node-script-types.test.ts
  • packages/oxlint-config/src/toolchain-node-projects.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Recent review details

Context from checks skipped due to timeout: 9

  • GitHub Check: cla
  • GitHub Check: cla / cla
  • GitHub Check: Enforce package quarantine / Enforce package quarantine
  • GitHub Check: rust-lints
  • GitHub Check: Shared tooling checks
  • GitHub Check: Hawk compiler compatibility
  • GitHub Check: Changeset present for published package changes
  • GitHub Check: Generated setup with PostgreSQL and Valkey
  • GitHub Check: Check generated AI instructions
Additional context used

Code guidelines: 1

AGENTS.md — configured
Path-based instructions: 1

Source excerpt: If TypeScript can make a class of bug structurally impossible (branded types, discriminated unions, exhaustive checks), prefer that over runtime validation or manual discipline Source excerpt: No enums: use `as const` object...

CodeRabbit inference engine (AGENTS.md)

Files:

  • packages/oxlint-config/src/node-script-types.test.ts
  • packages/oxlint-config/src/toolchain-node-projects.ts

bun-types 1.4.4 has passed the release-age window, so the temporary
exclusion that expired on 2026-10-27 is no longer needed.
@jan-kubica
jan-kubica added this pull request to the merge queue Nov 5, 2026
Merged via the queue into main with commit 547b98a Nov 5, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant