b310-digital / b310-digital/teammapper

Remove uuid package

Open
#1,105 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
TypeScript
Stars
496
Forks
41
Avg merge
2h 16m
Merged PRs (30d)
11

Description

**Describe the bug**
The uuid package does not support commonJS anymore, therefore the version is currently locked to v11. In addition, the whole package is currently only used for validating the format of uuid's. Its probably better to implement a simple check ourselves and to get rid of the dependency.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate every uuid usage and its dependency declaration, then read the validation call sites to understand the current format checks. Confirm the replacement preserves the required UUID validation, remove the dependency, and run the relevant existing test suite or validation tests if available.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.