- 
                Notifications
    
You must be signed in to change notification settings  - Fork 402
 
Refactor signal architecture for scalability #7098
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
base: cursor/implement-waitlist-hook-with-signal-primitives-3ad1
Are you sure you want to change the base?
Refactor signal architecture for scalability #7098
Conversation
Introduces a factory function for creating resource signals and refactors the state proxy to use this factory. This reduces code duplication and improves maintainability. Co-authored-by: bryce <[email protected]>
| 
           Cursor Agent can help with this pull request. Just   | 
    
          
 | 
    
| 
           The latest updates on your projects. Learn more about Vercel for GitHub. 
  | 
    
| 
          
 Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the  You can disable this status message by setting the  ✨ Finishing touches🧪 Generate unit tests (beta)
 Comment   | 
    
Introduce a registry for resource signals and a helper function to retrieve them. This simplifies signal management and makes it easier to add new resources. Co-authored-by: bryce <[email protected]>
This change introduces a map to store and retrieve signal sets by resource name, simplifying signal access and removing redundant signal getters from the State class. Co-authored-by: bryce <[email protected]>
This pull request contains changes generated by a Cursor Cloud Agent