Skip to content

chore: remove unused tsconfig.json - #261

Merged
ilteoood merged 1 commit into
mainfrom
chore/remove-unused-tsconfig
Oct 19, 2026
Merged

ilteoood merged 1 commit into
mainfrom
chore/remove-unused-tsconfig

Conversation

@Tony133

@Tony133 Tony133 commented Oct 19, 2026

Copy link
Copy Markdown
Member

Proposal:

Removes tsconfig.json (and its entry in .npmignore), which is no longer used by anything in the repo.

Why

  • The plugin is plain JavaScript (index.js) with hand-written types in types/index.d.ts: there is no build step, no tsc invocation and no checkJs.
  • The only consumer of the TypeScript compiler is tstyche (test:typescript). Since tstyche v4, when a file is not covered by a tsconfig.json it applies its own strict default compiler options (strict, exactOptionalPropertyTypes, noUncheckedIndexedAccess, module/moduleResolution: nodenext), which are at least as strict as the config in this repo.
  • Most options in the file are emit-related (outDir, declaration, sourceMap, importHelpers, ...) and have no effect, and include references a non-existent ./index.d.ts.
  • The file is already excluded from the published package via .npmignore.

Verification

npm test passes unchanged with the file removed:

@Tony133
Tony133 marked this pull request as ready for review October 19, 2026 10:49
@Tony133
Tony133 force-pushed the chore/remove-unused-tsconfig branch from bb6941b to c872f4d Compare October 19, 2026 10:50
@Tony133
Tony133 requested a review from a team October 19, 2026 10:51
@ilteoood
ilteoood merged commit 8473ed0 into main Oct 19, 2026
20 checks passed
@ilteoood
ilteoood deleted the chore/remove-unused-tsconfig branch October 19, 2026 11:36
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