Skip to content

chore: add tsgo type checker alongside tsc#354

Open
casamitjana wants to merge 1 commit intomainfrom
chore/add-tsgo-type-checker
Open

chore: add tsgo type checker alongside tsc#354
casamitjana wants to merge 1 commit intomainfrom
chore/add-tsgo-type-checker

Conversation

@casamitjana
Copy link
Copy Markdown

@casamitjana casamitjana commented Jan 22, 2026

Summary

  • Add @typescript/native-preview dev dependency for faster type checking
  • Add check-types script (tsgo --noEmit)
  • Add check-types:tsc script (tsc --noEmit)
  • lint script remains on tsc since tsgo is preview software

Measurements

Command Time
npm run check-types (tsgo) 0.82s
npm run check-types:tsc (tsc) 1.61s

tsgo is ~49% faster on this codebase.

Test plan

  • npm install succeeds
  • npm run check-types passes
  • npm run check-types:tsc passes
  • Both produce equivalent results

Summary by CodeRabbit

  • Chores
    • Added development dependencies for enhanced development tooling.
    • Added new development scripts for type validation.

✏️ Tip: You can customize this high-level summary in your review settings.

- Add `@typescript/native-preview` dev dependency
- Add `check-types` script (tsgo --noEmit)
- Add `check-types:tsc` script (tsc --noEmit)
- `lint` script remains on tsc since tsgo is preview software

| Command | Time |
|---------|------|
| `npm run check-types` (tsgo) | 0.82s |
| `npm run check-types:tsc` (tsc) | 1.61s |

tsgo is ~49% faster on this codebase.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 22, 2026

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'tools'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

Added two new npm scripts for TypeScript type checking (check-types and check-types:tsc) and introduced the @typescript/native-preview dependency to both dependencies and devDependencies in package.json.

Changes

Cohort / File(s) Summary
Package Configuration
package.json
Added scripts: "check-types": "tsgo --noEmit" and "check-types:tsc": "tsc --noEmit". Added @typescript/native-preview (^7.0.0-dev.20260122.3) to both dependencies and devDependencies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides detailed context including summary, performance measurements, and test plan, but does not follow the required template structure with Changes section and Checklist items. Reorganize the description to match the required template: add a 'Changes' section describing modifications, and complete the Checklist with release label and documentation/test verification items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding tsgo type checker alongside tsc for faster type checking.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

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