Skip to content

feat(access): build the basic SSO types - #747

Draft
rybit wants to merge 1 commit into
mainfrom
rybit/add-access-types
Draft

feat(access): build the basic SSO types#747
rybit wants to merge 1 commit into
mainfrom
rybit/add-access-types

Conversation

@rybit

@rybit rybit commented Aug 13, 2026

Copy link
Copy Markdown
Member

Creates the types for SSO users.

@rybit
rybit requested a review from a team as a code owner August 13, 2026 22:50
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ac8e9b0d-0b1a-456e-a6ee-17cc2b41d091

📥 Commits

Reviewing files that changed from the base of the PR and between cdf2d48 and 7b17bbf.

📒 Files selected for processing (3)
  • packages/types/src/lib/context/context.ts
  • packages/types/src/lib/context/netlify-user.ts
  • packages/types/src/main.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • netlify/blueprints (manual)

📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Added support for accessing authenticated user details in the request context after SSO site protection.
    • Exposed user identification, optional email, and session expiration information through the public types package.

Walkthrough

The types package adds the exported NetlifyUser interface with id, optional email, and expiresAt fields. The Context interface now includes an optional user property typed as NetlifyUser. The package entry point exports NetlifyUser.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Mergeability Score: ⚪ Minimal · up to 7b17b

This PR adds basic SSO user types and context exports without any identified merge-blocking risk; it is merge-ready after normal checks and review.

Suggested reviewers: eduardoboucas

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the addition of basic SSO user types, which is the main change in the pull request.
Description check ✅ Passed The description directly states that the pull request adds types for SSO users.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rybit/add-access-types

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed: dependency version conflict. Check your lock file or package.json.


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.

@rybit
rybit marked this pull request as draft August 13, 2026 22:54
@pkg-pr-new

pkg-pr-new Bot commented Aug 13, 2026

Copy link
Copy Markdown

Open in StackBlitz

@netlify/ai

npm i https://pkg.pr.new/@netlify/ai@747

@netlify/aws-lambda-compat

npm i https://pkg.pr.new/@netlify/aws-lambda-compat@747

@netlify/blobs

npm i https://pkg.pr.new/@netlify/blobs@747

@netlify/cache

npm i https://pkg.pr.new/@netlify/cache@747

@netlify/dev

npm i https://pkg.pr.new/@netlify/dev@747

@netlify/dev-utils

npm i https://pkg.pr.new/@netlify/dev-utils@747

@netlify/headers

npm i https://pkg.pr.new/@netlify/headers@747

@netlify/images

npm i https://pkg.pr.new/@netlify/images@747

@netlify/otel

npm i https://pkg.pr.new/@netlify/otel@747

@netlify/redirects

npm i https://pkg.pr.new/@netlify/redirects@747

@netlify/runtime

npm i https://pkg.pr.new/@netlify/runtime@747

@netlify/runtime-utils

npm i https://pkg.pr.new/@netlify/runtime-utils@747

@netlify/static

npm i https://pkg.pr.new/@netlify/static@747

@netlify/types

npm i https://pkg.pr.new/@netlify/types@747

@netlify/database-dev

npm i https://pkg.pr.new/@netlify/database-dev@747

@netlify/database

npm i https://pkg.pr.new/@netlify/database@747

@netlify/database-proxy

npm i https://pkg.pr.new/@netlify/database-proxy@747

@netlify/edge-functions-dev

npm i https://pkg.pr.new/@netlify/edge-functions-dev@747

@netlify/edge-functions

npm i https://pkg.pr.new/@netlify/edge-functions@747

@netlify/functions-dev

npm i https://pkg.pr.new/@netlify/functions-dev@747

@netlify/functions

npm i https://pkg.pr.new/@netlify/functions@747

@netlify/identity

npm i https://pkg.pr.new/@netlify/identity@747

commit: 7b17bbf

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