Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

@waleedlatif1 waleedlatif1 commented Jan 22, 2026

waleedlatif1 and others added 9 commits January 21, 2026 09:47
#2924)

* fix(auth): improve reset password flow and consolidate brand detection

* fix(auth): set errorHandled for EMAIL_NOT_VERIFIED to prevent duplicate error

* fix(auth): clear success message on login errors

* chore(auth): fix import order per lint
* fix(action-bar): duplicate subflows with children

* fix(action-bar): add validateTriggerPaste for subflow duplicate
* fix(resolvers): agent response format, input formats, root level

* fix response block initial seeding

* fix tests
#2926)

* fix(messages-input): fix cursor alignment and auto-resize with overlay

* fixed remaining zustand warnings
* fix(stores): remove dead code causing log spam on startup

* fix(stores): replace custom tools zustand store with react query cache
@vercel
Copy link

vercel bot commented Jan 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
docs Skipped Skipped Jan 22, 2026 4:03am

Request Review

@waleedlatif1 waleedlatif1 changed the title v0.5.67: loading, password reset, ui improvements v0.5.67: loading, password reset, ui improvements, helm updates Jan 22, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 22, 2026

Greptile Summary

  • Comprehensive refactoring to modernize Zustand store patterns by migrating from deprecated createWithEqualityFn to useShallow and useStoreWithEqualityFn APIs across the workflow editor components
  • Cleanup of dead code and import consolidation including removal of unused custom tools store, movement of normalizeName utility from stores to executor constants, and elimination of excessive logging that caused startup spam
  • UI and UX improvements including fixing notification text overflow, cursor alignment issues in messages input, password reset flow enhancements with brand detection consolidation, and better error handling

Important Files Changed

Filename Overview
packages/logger/src/index.ts Fixed webpack bundling issue by replacing dynamic environment variable access with direct property access to enable proper variable inlining
apps/sim/stores/custom-tools/store.ts Completely removed custom tools Zustand store as part of migration to React Query cache management
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/messages-input/messages-input.tsx Major fix for cursor alignment and auto-resize functionality by improving textarea-overlay synchronization system
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/action-bar/action-bar.tsx Refactored duplicate block functionality to use centralized copy/paste workflow pattern with proper validation for trigger blocks
apps/sim/app/api/function/execute/route.ts Significant consolidation of code resolution logic with improved variable resolution and error handling patterns

Confidence score: 4/5

  • This PR contains mostly safe refactoring and modernization changes with some complex UI fixes that require careful review
  • Score reflects extensive but well-structured changes across many files, with most being straightforward API migrations and cleanup
  • Pay close attention to the messages-input component changes and function execution route refactoring as these involve complex logic modifications

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

47 files reviewed, 4 comments

Edit Code Review Agent Settings | Greptile

Copy link

@cursor cursor bot left a 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.

- Refactor auth forms to use BrandedButton component
- Add BrandedLink component for changelog page
- Reduce code duplication in login, signup, reset-password forms
- Update star count default value
* fix(custom-tools): remove unsafe title fallback in getCustomTool

* fix(custom-tools): restore title fallback in getCustomTool lookup

Custom tools are referenced by title (custom_${title}), not database ID.
The title fallback is required for client-side tool resolution to work.
* fix(null-statuses): empty bodies handling

* address bugbot comment
* fix(microsoft): proactive refresh needed

* fix(x): missing token refresh flag

* notion and linear missing flag too

* address bugbot comment
* fix(auth): handle EMAIL_NOT_VERIFIED in onError callback

* refactor(auth): extract redirectToVerify helper to reduce duplication
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.

4 participants