Develop by Zaiidmo · Pull Request #19 · CISCODE-MA/CacheKit

Zaiidmo · 2026-04-07T11:00:36Z

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?

…ckages - Replace git tag --list strategy with package.json-driven tag validation in all 16 publish workflows; use git rev-parse to verify the exact tag exists rather than guessing the latest repo-wide tag - Update error guidance to reflect feat/** → develop → master flow - Standardize dependabot to npm-only, grouped, monthly cadence across all 16 packages; remove github-actions ecosystem updates - Add missing dependabot.yml to AuthKit-UI, ChartKit-UI, HealthKit, HooksKit, paymentkit, StorageKit

* feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types

@CacheEvict

* feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * feat(COMPT-57): add @Cacheable and @CacheEvict decorators with key interpolation

@CacheEvict

* feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * feat(COMPT-57): add @Cacheable and @CacheEvict decorators with key interpolation * test(COMPT-58): add full test suite with 95%+ coverage across all adapters, service, and decorators * fix(lint): fix import/order and no-require-imports violations in spec files

…#5) * docs(COMPT-59): add README, update peer deps, create v0.1.0 changeset * style: fix Prettier formatting across all files

…clean up index exports (#6)

…st/ (#8)

* fix(ci): fix SonarCloud coverage — use src/**/*.spec.ts instead of test/ * chore: set version to 0.0.1 for initial publish

@CacheEvict

* fix(ci): fix SonarCloud coverage — use src/**/*.spec.ts instead of test/ * Develop (#7) * ops: UPDATED publish workflow and dependabot PR limits * ops (ci): standardize publish validation and dependabot across all packages - Replace git tag --list strategy with package.json-driven tag validation in all 16 publish workflows; use git rev-parse to verify the exact tag exists rather than guessing the latest repo-wide tag - Update error guidance to reflect feat/** → develop → master flow - Standardize dependabot to npm-only, grouped, monthly cadence across all 16 packages; remove github-actions ecosystem updates - Add missing dependabot.yml to AuthKit-UI, ChartKit-UI, HealthKit, HooksKit, paymentkit, StorageKit * security: added CODEOWNER file for branches security * ops: updated relese check workflow# * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters (#1) * Feature/compt 56 cache module service (#2) * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * Feature/compt 57 cacheable cacheevict decorators (#3) * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * feat(COMPT-57): add @Cacheable and @CacheEvict decorators with key interpolation * Feature/compt 58 test suite (#4) * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * feat(COMPT-57): add @Cacheable and @CacheEvict decorators with key interpolation * test(COMPT-58): add full test suite with 95%+ coverage across all adapters, service, and decorators * fix(lint): fix import/order and no-require-imports violations in spec files * docs(COMPT-59): add README, update peer deps, create v0.1.0 changeset (#5) * docs(COMPT-59): add README, update peer deps, create v0.1.0 changeset * style: fix Prettier formatting across all files * improvement: replace KEYS with SCAN, fix @Cacheable null-return bug, clean up index exports (#6) * ci: update release check workflow * fix(ci): fix SonarCloud coverage — use src/**/*.spec.ts instead of test/ (#8) * ops: updated release check jobs ] --------- Co-authored-by: Zaiidmo <zaiidmoumnii@gmail.com> * chore: set version to 0.0.1 for initial publish --------- Co-authored-by: Zaiidmo <zaiidmoumnii@gmail.com>

@CacheEvict

* fix(ci): fix SonarCloud coverage — use src/**/*.spec.ts instead of test/ * Develop (#7) * ops: UPDATED publish workflow and dependabot PR limits * ops (ci): standardize publish validation and dependabot across all packages - Replace git tag --list strategy with package.json-driven tag validation in all 16 publish workflows; use git rev-parse to verify the exact tag exists rather than guessing the latest repo-wide tag - Update error guidance to reflect feat/** → develop → master flow - Standardize dependabot to npm-only, grouped, monthly cadence across all 16 packages; remove github-actions ecosystem updates - Add missing dependabot.yml to AuthKit-UI, ChartKit-UI, HealthKit, HooksKit, paymentkit, StorageKit * security: added CODEOWNER file for branches security * ops: updated relese check workflow# * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters (#1) * Feature/compt 56 cache module service (#2) * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * Feature/compt 57 cacheable cacheevict decorators (#3) * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * feat(COMPT-57): add @Cacheable and @CacheEvict decorators with key interpolation * Feature/compt 58 test suite (#4) * feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters * feat(COMPT-56): add CacheModule, CacheService, and DI tokens * style: fix Prettier formatting across all files * style: fix Prettier formatting after develop merge * fix(lint): fix import order and replace any types with proper NestJS types * feat(COMPT-57): add @Cacheable and @CacheEvict decorators with key interpolation * test(COMPT-58): add full test suite with 95%+ coverage across all adapters, service, and decorators * fix(lint): fix import/order and no-require-imports violations in spec files * docs(COMPT-59): add README, update peer deps, create v0.1.0 changeset (#5) * docs(COMPT-59): add README, update peer deps, create v0.1.0 changeset * style: fix Prettier formatting across all files * improvement: replace KEYS with SCAN, fix @Cacheable null-return bug, clean up index exports (#6) * ci: update release check workflow * fix(ci): fix SonarCloud coverage — use src/**/*.spec.ts instead of test/ (#8) * ops: updated release check jobs ] --------- Co-authored-by: Zaiidmo <zaiidmoumnii@gmail.com> * chore: set version to 0.0.1 for initial publish --------- Co-authored-by: Zaiidmo <zaiidmoumnii@gmail.com>

* chore: align version to 1.0.0 to match master * chore: revert version to 0.0.1 as required

Copilot

Pull request overview

This PR updates the package’s published version/lockfile metadata and adjusts CI SonarCloud configuration for the CacheKit NestJS module library.

Changes:

Updates the package version and corresponding lockfile version metadata.
Adjusts dependency/peer-dependency information in package-lock.json (notably around optional peers like ioredis).
Modifies SonarCloud scan arguments in the release-check workflow.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File	Description
package.json	Changes package version metadata (currently creates a version mismatch vs lockfile).
package-lock.json	Updates lockfile version and dependency/peer-dependency resolution metadata.
.github/workflows/release-check.yml	Changes SonarCloud configuration by removing explicit `sonar.tests` directory.

Copilot · 2026-04-07T11:05:01Z

package.json

-  "name": "@ciscode/cachekit",
-  "version": "1.0.0",
+  "name": "@ciscode/cache-kit",
+  "version": "0.0.1",


The package version is being manually changed and appears to be moving backwards (from 1.x to 0.0.x). This conflicts with the documented Changesets-based release flow (versions should be updated via Changesets / the automated “Version Packages” PR), and can also block publishing if a higher version was already released. Revert the manual version edit and use a changeset / version-packages workflow instead.

Suggested change

"version": "0.0.1",

"version": "1.0.0",

Copilot · 2026-04-07T11:05:01Z

package.json

+  "name": "@ciscode/cache-kit",
+  "version": "0.0.1",
  "description": "CacheKit backend module for reusable caching utilities and integrations.",


package.json and package-lock.json versions are inconsistent (package.json shows 0.0.0 while package-lock.json is 0.0.1). This will cause confusing installs and CI diffs. Align the versions (regenerate the lockfile after fixing package.json).

Copilot · 2026-04-07T11:05:01Z

.github/workflows/release-check.yml

          args: >
            -Dsonar.organization=${{ env.SONAR_ORGANIZATION }}
            -Dsonar.projectKey=${{ env.SONAR_PROJECT_KEY }}
            -Dsonar.sources=src


Removing -Dsonar.tests=test means the test/ directory (e.g., test/smoke.test.ts) is no longer part of the analysis scope because sonar.sources is limited to src. With this change, sonar.test.inclusions=**/*.test.ts won’t pick up tests under test/. Re-add sonar.tests=test (or expand sonar.sources/set sonar.tests appropriately) to keep SonarCloud test classification consistent.

Suggested change

-Dsonar.sources=src

-Dsonar.sources=src

-Dsonar.tests=test

sonarqubecloud · 2026-04-07T11:06:26Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Zaiidmo and others added 22 commits March 12, 2026 10:18

ops: UPDATED publish workflow and dependabot PR limits

1be7028

security: added CODEOWNER file for branches security

d076d49

ops: updated relese check workflow#

c1047cf

feat(COMPT-55): add ICacheStore port and Redis/InMemory adapters (#1)

7179644

docs(COMPT-59): add README, update peer deps, create v0.1.0 changeset (…

d4b3570

…#5) * docs(COMPT-59): add README, update peer deps, create v0.1.0 changeset * style: fix Prettier formatting across all files

improvement: replace KEYS with SCAN, fix @Cacheable null-return bug, …

3eb5fe9

…clean up index exports (#6)

ci: update release check workflow

4dfbd8e

fix(ci): fix SonarCloud coverage — use src/**/*.spec.ts instead of te…

bf3a3dd

…st/ (#8)

ops: updated release check jobs ]

878d4ac

ops: updated release check jobs ]

8ac1070

install dep

3b194a9

Fix/sonar test inclusions (#10)

847d208

* fix(ci): fix SonarCloud coverage — use src/**/*.spec.ts instead of test/ * chore: set version to 0.0.1 for initial publish

Fix/align version 1.0.0 (#17)

e375c78

* chore: align version to 1.0.0 to match master * chore: revert version to 0.0.1 as required

Merge branch 'develop' of github.com:CISCODE-MA/CacheKit into develop

1709e57

v0.0.1

43452a0

fix(chore): reverted versions tags and fixed merge conflits

3a5e068

Zaiidmo requested review from a team and Copilot April 7, 2026 11:00

0.0.1

c5f4f64

Copilot started reviewing on behalf of Zaiidmo April 7, 2026 11:01 View session

Zaiidmo added 2 commits April 7, 2026 12:03

chore: dump version 1

dbf275d

fix(merge): package.json versions conflicts

9a0aae8

Copilot AI reviewed Apr 7, 2026

View reviewed changes

Zaiidmo merged commit 06eb991 into master Apr 7, 2026
6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Develop#19

Develop#19
Zaiidmo merged 25 commits intomasterfrom
develop

Zaiidmo commented Apr 7, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 7, 2026

Uh oh!

Copilot AI Apr 7, 2026

Uh oh!

Copilot AI Apr 7, 2026

Uh oh!

sonarqubecloud bot commented Apr 7, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Zaiidmo commented Apr 7, 2026

Summary

Why

Checklist

Notes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 7, 2026

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Apr 7, 2026

Quality Gate passed

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants