-
Notifications
You must be signed in to change notification settings - Fork 3.2k
v0.5.64: unsubscribe, search improvements, metrics, additional SSO configuration #2898
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
* fix(undo-redo): preserve subblock values during undo/redo cycles * added tests
…etrieval in docs to use RRF (#2889)
* Temp * Condition and router copilot syntax updates * Plan respond plan
* Superuser debug * Fix * update templates routes to use helper --------- Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
…igger types (#2895) * improvement(stats): should track mcp and a2a executions like other trigger types * update types
* feat(api): added workflows api route for dynamic discovery * added ability to edit parameter and workflow descriptions * added new rate limit category, ack PR comments * fix hasChanges logic * added whitespace trimming before hasChanges check
#2894) * improvement(kb): migrate manual fetches in kb module to use reactquery * converted remaining manual kb fetches * unwrap kb tags before API call, added more query invalidation for chunks * added resetMutation calls after modal closes
…& deregistration scripts (#2896) * fix(sso): updated registration & deregistration script for explicit support for Entra ID * cleanup * ack PR comment * ack PR comment * tested edge cases, ack'd PR comments * remove trailing slash
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
* feat(claude): added rules * fix(copilot): chat loading; refactor(copilot): components, utils, hooks * fix(copilot): options selection strikethrough * fix(copilot): options render inside thinking * fix(copilot): checkpoints, user-input; improvement(code): colors * fix(copilot): scrolling, tool-call truncation, thinking ui * fix(copilot): tool call spacing and shimmer/actions on previous messages * improvement(copilot): queue * addressed comments
Greptile SummaryThis PR combines multiple improvements across email management, search functionality, SSO configuration, metrics tracking, and workflow APIs. Major Changes:
Browser Use Update: Upgraded to v2 endpoints with documentation updates Confidence Score: 4/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant UI
participant API
participant DB
participant IdP as Identity Provider
Note over User,IdP: Workflow API Discovery Flow
User->>UI: Search for workflow
UI->>API: GET /api/v1/workflows?workspaceId=X
API->>DB: Query workflows with permissions
DB-->>API: Return workflow list
API-->>UI: Workflows with metadata
UI->>API: GET /api/v1/workflows/{id}
API->>DB: Fetch workflow details + blocks
DB-->>API: Workflow + blocks data
API->>API: Extract input fields from blocks
API-->>UI: Workflow with inputs schema
UI-->>User: Display workflow details
Note over User,IdP: SSO Registration Flow (Updated)
User->>API: Run register-sso-provider script
API->>API: Build config from env vars
API->>IdP: Fetch /.well-known/openid-configuration
IdP-->>API: Discovery document with endpoints
API->>API: Merge user-provided + discovered endpoints
API->>DB: Insert/update SSO provider
DB-->>API: Provider registered
API-->>User: SSO provider configured
Note over User,DB: Email Unsubscribe Flow
User->>API: Click unsubscribe link with token
API->>API: Verify token with email
API->>DB: Update email preferences
DB-->>API: Preferences saved
API-->>User: Unsubscribe confirmed
Note over User,DB: Undo/Redo Fix
User->>UI: Edit code in function block
User->>UI: Delete block
UI->>UI: Capture subblock values before delete
UI->>UI: Store in undo history with values
User->>UI: Undo delete
UI->>UI: Restore block with preserved values
UI-->>User: Code restored correctly
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additional Comments (3)
-
apps/sim/lib/copilot/tools/client/other/search-documentation.ts, line 71-73 (link)style: Setting success state twice is redundant. The second call on line 73 can be removed since the state is already set on line 71.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
-
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/user-input/utils.ts, line 176-178 (link)logic: The blocks context equality check uses
some()which returns true if ANY existing blockId matches ANY new blockId, but the function nameareContextsEqualsuggests exact equality. Consider if this should check for exact array equality instead. Should the blocks context comparison require exact match of all blockIds or is partial overlap the intended behavior? -
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-scroll-management.ts, line 34 (link)style:
messages: any[]should be properly typed. Consider using a specific message interface instead ofany[]Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Context Used: Context from
dashboard- TypeScript conventions and type safety (source)
169 files reviewed, 10 comments
...pace/[workspaceId]/w/components/sidebar/components/settings-modal/components/debug/debug.tsx
Show resolved
Hide resolved
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/terminal/terminal.tsx
Show resolved
Hide resolved
…d the ability to update if no changes made (#2897) * improvement(modal): fixed popover issue in custom tools modal, removed the ability to update if no changes made * improvement(modal): fixed popover issue in custom tools modal, removed the ability to update if no changes made * popover fixes, color picker keyboard nav, code simplification * color standardization * fix color picker * set discard alert state when closing modal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
apps/sim/app/workspace/[workspaceId]/knowledge/[id]/[documentId]/document.tsx
Show resolved
Hide resolved
* fix(kb): align bulk chunk operation with API response * fix(kb): skip local state update for failed chunks * fix(kb): correct errors type and refresh on partial failure
Uh oh!
There was an error while loading. Please reload this page.