Skip to content

fix: id field not being mapped#20

Merged
sanny-io merged 2 commits intodevfrom
fix/auth-id-map
Feb 3, 2026
Merged

fix: id field not being mapped#20
sanny-io merged 2 commits intodevfrom
fix/auth-id-map

Conversation

@sanny-io
Copy link
Member

@sanny-io sanny-io commented Feb 3, 2026

Summary by CodeRabbit

  • Chores

    • Version bumped to 1.0.6
  • Refactor

    • Improved authentication identifier generation logic for enhanced cache handling

@sanny-io sanny-io merged commit 024ae1a into dev Feb 3, 2026
1 check was pending
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Version bump to 1.0.6. Modified authId construction in the plugin from concatenating joined keys to mapping field values from client.$auth into an array, changing the hash input for cache key generation.

Changes

Cohort / File(s) Summary
Version Update
package.json
Version bumped from 1.0.5 to 1.0.6.
Auth ID Generation
src/plugin.ts
Changed authId construction from a concatenated string of joined keys to an array of values extracted from client.$auth based on idFields list; authId remains undefined if client.$auth is absent.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • visualbravo/zenstack-cache#15: Modifies cache key/hash generation in src/plugin.ts by altering how client.$auth is incorporated for authId construction.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/auth-id-map

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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