Skip to content

feat: Add widgets:pipes:manage to WidgetScope type#1609

Open
thompsongl wants to merge 1 commit into
mainfrom
devin/1780667455-add-pipes-manage-scope
Open

feat: Add widgets:pipes:manage to WidgetScope type#1609
thompsongl wants to merge 1 commit into
mainfrom
devin/1780667455-add-pipes-manage-scope

Conversation

@thompsongl
Copy link
Copy Markdown

Description

Add widgets:pipes:manage to the WidgetScope type to support the Pipes management widget scope.

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

Link to Devin session: https://app.devin.ai/sessions/1aa6c55dd1334c63baef303ea0ebd7de
Requested by: @thompsongl

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@thompsongl thompsongl requested review from a team as code owners June 5, 2026 13:51
@thompsongl thompsongl requested a review from alisherry June 5, 2026 13:51
@devin-ai-integration
Copy link
Copy Markdown
Contributor

Original prompt from greg.thompson

We need widgets:pipes:manage to be included in the WidgetScope type for the node sdk

@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot changed the title Add widgets:pipes:manage to WidgetScope type feat: Add widgets:pipes:manage to WidgetScope type Jun 5, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 5, 2026

Greptile Summary

This PR adds 'widgets:pipes:manage' to the WidgetScope union type in src/widgets/interfaces/get-token.ts, enabling the Pipes management widget scope to be passed as a valid value when requesting a widget token.

  • The new literal follows the same widgets:<resource>:manage naming convention as all existing scopes and requires no changes to serialization or deserialization logic.

Confidence Score: 5/5

Safe to merge — adds a single string literal to a union type with no runtime logic changes.

The change is one line: a new string literal appended to an existing union type. It follows the established naming convention exactly, touches no serialization/deserialization logic, and introduces no new behaviour that could regress existing consumers.

No files require special attention.

Important Files Changed

Filename Overview
src/widgets/interfaces/get-token.ts Adds 'widgets:pipes:manage' as a new member of the WidgetScope union type, following the exact same pattern as all existing scope literals.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["GetTokenOptions\n{ organizationId, userId, scopes }"] --> B["WidgetScope union type"]
    B --> C["widgets:users-table:manage"]
    B --> D["widgets:sso:manage"]
    B --> E["widgets:domain-verification:manage"]
    B --> F["widgets:dsync:manage"]
    B --> G["widgets:api-keys:manage"]
    B --> H["widgets:audit-log-streaming:manage"]
    B --> I["widgets:pipes:manage NEW"]
    A --> J["serializeGetTokenOptions()"]
    J --> K["SerializedGetTokenOptions\n{ organization_id, user_id, scopes }"]
Loading

Reviews (1): Last reviewed commit: "Add widgets:pipes:manage to WidgetScope ..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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.

1 participant