Skip to content
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

[Feature]: Refactor Parameter Validations in Resolvers' main() Functions #7571

Open
openhands-agent opened this issue Mar 28, 2025 · 0 comments
Labels
enhancement New feature or request resolver Related to OpenHands Resolver

Comments

@openhands-agent
Copy link
Contributor

What problem or use case are you trying to solve?
Parameter validations in each resolver's main() function should be refactored to use a shared code from the utils library to promote code reusability and maintainability. The current implementation across multiple resolver files lacks consistency.

Desired UX of the solution
Refactored validations within the resolver functions would use a shared utility from the utils module. The reusable utility should ensure consistent parameter validation and reduced code duplication.

Technical implementation
The shared utility function can be added to the utils module. Each resolver's main() function should then replace the current ad-hoc validation logic with the utility function.

Alternatives considered
<N/A>

Additional context
<N/A>

Issue Created By: tawago on Slack
Link to Slack Thread: https://openhands-ai.slack.com/archives/C078L0FUGUX/p1743155016537279?thread_ts=1743155016.537279&cid=C078L0FUGUX

@mamoodi mamoodi added enhancement New feature or request resolver Related to OpenHands Resolver labels Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request resolver Related to OpenHands Resolver
Projects
None yet
Development

No branches or pull requests

2 participants