Skip to content

Remove commitizen#1150

Merged
andrejak merged 3 commits into
mainfrom
andreja/tsp-1310-resolve-security-vulnerabilities-on-public-sdk-repo-3
Apr 20, 2026
Merged

Remove commitizen#1150
andrejak merged 3 commits into
mainfrom
andreja/tsp-1310-resolve-security-vulnerabilities-on-public-sdk-repo-3

Conversation

@andrejak

@andrejak andrejak commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Why drop commitizen / @commitlint/cz-commitlint

  • to get rid of some vulnerabilities:commitizen is the only remaining source of both tmp@0.0.33 ([Un-nest debug options for the client #48]) and lodash@4.17.21 ([[FLAPI-1301] Export order change related functions #118] / [build(deps-dev): bump @typescript-eslint/parser from 4.27.0 to 4.28.0 #117]) in the tree, and it has no upstream fix path (its inquirer@8 → external-editor@3 chain is frozen). Removing it clears tmp entirely and downgrades the remaining lodash pins to the safer 4.17.23 line.
  • Kept @commitlint/cli + husky commit-msg hook — commit-message enforcement is unchanged. Only the interactive yarn commit flow is gone, which nobody strictly needs; contributors can still write a conventional commit message by hand and commitlint will validate it.
  • Also added tslib@2.8.1 as an explicit devDependency — it's a peer dep of @rollup/plugin-typescript that was previously being satisfied transitively through commitizen, so removing commitizen broke the build until we added it directly.

@andrejak andrejak marked this pull request as ready for review April 20, 2026 09:31
@andrejak andrejak requested a review from a team as a code owner April 20, 2026 09:31
@andrejak andrejak merged commit 5cab32e into main Apr 20, 2026
5 checks passed
@andrejak andrejak deleted the andreja/tsp-1310-resolve-security-vulnerabilities-on-public-sdk-repo-3 branch April 20, 2026 09:52
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