Skip to content

Feature/mvp#1

Merged
suranig merged 42 commits intomasterfrom
feature/mvp
Jun 4, 2025
Merged

Feature/mvp#1
suranig merged 42 commits intomasterfrom
feature/mvp

Conversation

@suranig
Copy link
Copy Markdown
Owner

@suranig suranig commented May 26, 2025

No description provided.

@suranig suranig self-assigned this May 26, 2025
@suranig suranig added the enhancement New feature or request label May 26, 2025
suranig added 14 commits June 3, 2025 01:20
- Remove any type from Redis lock method
- Fix Redis SET command syntax to use proper parameters
- Improve error handling in lock/unlock operations
- Remove unused fetch import
- Fix global declaration
- Improve code organization and documentation
- Rename error variable to unlockError in catch block for clarity
- Improve error logging in lock/unlock operations
- Add better type safety for error handling
- Add concrete types instead of any
- Fix MemoryBackend generic type
- Add proper type for logEvents
- Remove unused vi import
- Add proper generic types to MemoryBackend
- Fix CacheHandler type parameters
- Add CacheLogEvent type for logEvents
- Add union type for MemoryBackend generic
- Fix type casting in test cases
- Improve error handling types
- Add return type to register function
- Remove console.log statements
- Improve code organization
- Add detailed JSDoc @param tags to all interface methods
- Clarify parameter usage in documentation
- Fix ESLint unused parameter warnings
@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 54.65995% with 180 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/backends/redis.ts 12.14% 94 Missing ⚠️
src/cache/createCacheHandler.ts 71.01% 40 Missing ⚠️
.eslintrc.cjs 0.00% 24 Missing ⚠️
vitest.config.ts 0.00% 13 Missing ⚠️
src/index.ts 0.00% 5 Missing ⚠️
src/cache/fetchWithCache.ts 86.36% 3 Missing ⚠️
src/instrumentation/index.ts 96.00% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@suranig suranig merged commit dc44d91 into master Jun 4, 2025
3 checks passed
@suranig suranig deleted the feature/mvp branch June 6, 2025 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants