Skip to content

Feature/compt 56 cache module service#2

Merged
y-aithnini merged 6 commits intodevelopfrom
feature/COMPT-56-cache-module-service
Apr 1, 2026
Merged

Feature/compt 56 cache module service#2
y-aithnini merged 6 commits intodevelopfrom
feature/COMPT-56-cache-module-service

Conversation

@y-aithnini
Copy link
Copy Markdown

Summary

  • What does this PR change?

Why

  • Why is this change needed?

Checklist

  • Added/updated tests (if behavior changed)
  • npm run lint passes
  • npm run typecheck passes
  • npm test passes
  • npm run build passes
  • Added a changeset (npx changeset) if this affects consumers

Notes

  • Anything reviewers should pay attention to?

@y-aithnini y-aithnini requested a review from a team as a code owner April 1, 2026 08:16
Copilot AI review requested due to automatic review settings April 1, 2026 08:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Introduces a new CacheKit caching module for NestJS, providing a CacheService backed by pluggable store adapters (Redis or in-memory) and wiring via a dynamic CacheModule.

Changes:

  • Added CacheModule (sync + async registration) with DI tokens and a CacheService API (get/set/delete/clear/has/wrap).
  • Introduced the ICacheStore port plus Redis and in-memory adapter implementations.
  • Updated TS/Jest path aliases and added ioredis dependency.

Reviewed changes

Copilot reviewed 9 out of 11 changed files in this pull request and generated 17 comments.

Show a summary per file
File Description
tsconfig.json Adds @ports/* and @adapters/* path aliases.
jest.config.ts Adds Jest moduleNameMapper entries for new aliases.
src/cache-kit.module.ts Implements the dynamic Nest module and async registration patterns.
src/services/cache.service.ts Adds the primary caching service API used by consumers.
src/ports/cache-store.port.ts Defines the adapter contract (ICacheStore).
src/adapters/redis-cache-store.adapter.ts Adds ioredis-backed cache store adapter.
src/adapters/in-memory-cache-store.adapter.ts Adds Map-backed cache store adapter with TTL.
src/constants.ts Defines DI tokens for store + module options.
src/index.ts Updates public API exports to expose the new cache module/service/tokens/types.
package.json Adds ioredis dependency.
package-lock.json Locks dependency graph changes for ioredis and transitive deps.

Copilot AI review requested due to automatic review settings April 1, 2026 09:08
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

@y-aithnini y-aithnini merged commit 56e9bc7 into develop Apr 1, 2026
2 checks passed
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