Skip to content

fix: remove typescript from dependencies#10

Merged
Bekacru merged 1 commit intobetter-auth:mainfrom
benmccann:typescript-dev
Jul 7, 2025
Merged

fix: remove typescript from dependencies#10
Bekacru merged 1 commit intobetter-auth:mainfrom
benmccann:typescript-dev

Conversation

@benmccann
Copy link
Contributor

@benmccann benmccann commented May 23, 2025

typescript should be in devDependencies. the package distributes .mjs/.cjs files so doesn't require typescript in any way. typescript weighs over 20mb: https://pkg-size.dev/typescript

@jazzberry-ai
Copy link

jazzberry-ai bot commented May 23, 2025

Bug Report

Name Severity Example test case Description
Missing transitive dependency Medium Install the library as a dependency in a new TypeScript project and try to build it. Moving typescript from dependencies to devDependencies prevents it from being installed as a transitive dependency, potentially causing type checking issues in consuming projects.

Comments? Email us.

@Bekacru Bekacru merged commit 0e0ee6c into better-auth:main Jul 7, 2025
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