Skip to content

feat(js): add new User.sessions scope#1080

Merged
simeng-li merged 2 commits intomasterfrom
simeng-update-user-scope
Apr 8, 2026
Merged

feat(js): add new User.sessions scope#1080
simeng-li merged 2 commits intomasterfrom
simeng-update-user-scope

Conversation

@simeng-li
Copy link
Copy Markdown
Contributor

Summary

Add new User.sessions scope

Testing

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link
Copy Markdown
Contributor

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

Adds a new JS SDK user scope (urn:logto:scope:sessions) intended for session-management authorization (Account API), without introducing any new ID token or userinfo claims.

Changes:

  • Add UserScope.Sessions to the UserScope enum.
  • Ensure the new scope maps to no claims in idTokenClaims / userinfoClaims.
  • Add a changeset entry for releasing the new scope.

Reviewed changes

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

File Description
packages/js/src/consts/openid.ts Introduces UserScope.Sessions and wires it into the scope→claims maps as an empty-claims scope.
.changeset/cold-mayflies-crash.md Adds release note for the new sessions scope.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/js/src/consts/openid.ts
Comment thread .changeset/cold-mayflies-crash.md Outdated
Comment thread .changeset/cold-mayflies-crash.md Outdated
add new User.sessions scope
@simeng-li simeng-li force-pushed the simeng-update-user-scope branch from b157759 to e4cb17f Compare April 8, 2026 03:07
@simeng-li simeng-li merged commit 1a61d7a into master Apr 8, 2026
20 checks passed
@simeng-li simeng-li deleted the simeng-update-user-scope branch April 8, 2026 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants