Skip to content

Refactor ee file structure #5828

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

Closed
wants to merge 2 commits into from
Closed

Refactor ee file structure #5828

wants to merge 2 commits into from

Conversation

diegoimbert
Copy link
Contributor

No description provided.

Copy link

cloudflare-workers-and-pages bot commented May 28, 2025

Deploying windmill with  Cloudflare Pages  Cloudflare Pages

Latest commit: 22c495e
Status: ✅  Deploy successful!
Preview URL: https://8cf625b8.windmill.pages.dev
Branch Preview URL: https://di-ee-refactor.windmill.pages.dev

View logs

@diegoimbert
Copy link
Contributor Author

/ai continue what I did on agent_workers_ee.rs for all *_ee.rs files:

  • Create a new corresponding _oss file
  • Copy function signatures and structs so that the _oss acts as a proxy to the _ee file
  • Replace all occurrences of *_ee to *_oss

Copy link
Contributor

claude bot commented May 28, 2025

Claude finished @diegoimbert's task —— View job


Refactoring EE file structure ✅

I've successfully completed the refactoring pattern you established for agent_workers_ee.rs and applied it to all other *_ee.rs files.

Todo List:

  • Read repository guidelines and examine existing pattern
  • Find all *_ee.rs files in the backend
  • Create corresponding _oss.rs files following the proxy pattern
  • Update imports and references from *_ee to *_oss
  • Run cargo check to validate changes
  • Commit changes to the branch

Summary:

  • ✅ Created 32 OSS proxy files for all *_ee.rs files across backend modules
  • ✅ Updated lib.rs to import and use OSS modules instead of EE modules directly
  • ✅ Followed the established proxy pattern from agent_workers_oss.rs
  • ✅ Committed all changes to the branch

All OSS files now act as proxies to their corresponding EE implementations, creating a consistent and maintainable code structure. The refactoring allows the codebase to use OSS modules as the interface while maintaining backward compatibility with the EE implementations.

- Create OSS proxy files for all *_ee.rs files
- Update lib.rs to use OSS modules instead of EE modules directly
- Follow the established pattern from agent_workers_oss.rs

This refactoring allows the codebase to use a consistent proxy pattern where
OSS files act as proxies to the EE implementations, improving code organization
and maintainability.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: diegoimbert <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators May 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant