Skip to content

refactor(types): migrate from tsd to tstyche - #137

Open
Puppo wants to merge 4 commits into
mercurius-js:mainfrom
Puppo:refactor/migrate-tsd-to-tstyche
Open

refactor(types): migrate from tsd to tstyche#137
Puppo wants to merge 4 commits into
mercurius-js:mainfrom
Puppo:refactor/migrate-tsd-to-tstyche

Conversation

@Puppo

@Puppo Puppo commented May 18, 2026

Copy link
Copy Markdown
Contributor

Description

Migrates type tests from tsd to tstyche.

Changes

  • Replace tsd dependency with tstyche ^3.3.0
  • Update test:types script to use tstyche
  • Remove tsd config block from package.json
  • Move index.d.tstypes/index.d.ts and update types / files fields accordingly
  • Rename test/types/index.tstypes/index.tst.ts (tstyche convention)
  • Update lint:typescript scripts to point at types/*.tst.ts
  • Convert all assertions:
    • expectAssignable<T>(x)expect(x).type.toBeAssignableTo<T>()
    • expectError(() => {...})expect(() => {...}).type.not.toBeCallableWith()

Checklist

  • run npm run test && npm run benchmark --if-present
  • Tests and/or benchmarks included
  • Documentation changed or added
  • Commit follows Developer's Certificate of Origin

Puppo and others added 4 commits May 18, 2026 21:50
- Replace tsd with tstyche in devDependencies
- Update test:types script to use tstyche
- Remove tsd config block from package.json
- Rename test/types/index.ts to test/types/index.tst.ts
- Convert expectAssignable<T>(x) to expect(x).type.toBeAssignableTo<T>()
- Convert expectError(() => {...}) to expect(() => {...}).type.toRaiseError()

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The types declaration file was moved from index.d.ts to types/index.d.ts
but the files array in package.json was not updated, which would cause
the published npm package to ship without type declarations.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Puppo
Puppo marked this pull request as ready for review May 18, 2026 20:20
Comment thread package.json
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.

2 participants