Skip to content

chore: replacing Cache with PermissionCache, bump deps#584

Merged
petruki merged 1 commit intomasterfrom
staging
May 7, 2026
Merged

chore: replacing Cache with PermissionCache, bump deps#584
petruki merged 1 commit intomasterfrom
staging

Conversation

@petruki
Copy link
Copy Markdown
Member

@petruki petruki commented May 7, 2026

This pull request primarily updates dependencies and refactors the cache helper for permissions to improve clarity and maintainability. It upgrades several packages to their latest versions, and renames the generic cache.js to permission-cache.js throughout the codebase, updating all relevant imports and references. Additionally, test files and identifiers have been renamed to align with this change.

Dependency Upgrades:

  • Upgraded key dependencies in package.json and npm-shrinkwrap.json, including express-rate-limit (8.5.1), graphql (16.14.0), sinon (22.0.0), and several others for improved security and compatibility.

Permission Cache Refactor:

  • Renamed src/helpers/cache.js to src/helpers/permission-cache.js, and updated all imports in service, resolver, and test files to use the new module name.

  • Changed the exported class from Cache to PermissionCache in the new helper file for better specificity.

Testing Updates:

  • Renamed the test file tests/unit-test/cache.test.js to tests/unit-test/permission-cache.test.js and updated test identifiers to match the new naming convention.

These changes help clarify the purpose of the permission cache, improve code consistency, and keep dependencies up to date.

@petruki petruki self-assigned this May 7, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 7, 2026

@petruki petruki merged commit 4a8d46c into master May 7, 2026
5 checks passed
@petruki petruki deleted the staging branch May 7, 2026 02:36
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.

1 participant