Skip to content

Conversation

@simonprovost
Copy link
Member

@simonprovost simonprovost commented Sep 18, 2025

Hi @roquelopez and team!

Hope you’re well! Happy to propose a complete restructure of the library to be more (i) modern Python, (ii) highly typed and checked during runtime with a O(1) overhead, (iii) with one and only one entrypoint, (iv) hard-coded data/prompts fully YAML-transformed for easier maintenance, and(v) fully-lightweight docstringed 🥳

Next step I believe would be to someone to create a workflow to make AutoDDG available on PyPI so we can focus on an MCP with MCPStack. What do you think? I hope it wasn’t too much to ask for such a significant change, but trust me necessary to modernise it and make it easier to build an MCP on top of it.

Here’s a video showing some visuals (e.g. Readme, the quick start running, etc.)

VideoPRCompressed.mp4

Caution

This PR should not be merge prior #3 please, let me rebase before whenever this #3 is merged!

Cheers

@simonprovost simonprovost force-pushed the feat/modern_pythonic_library_transformation branch from d90a560 to f8c3f35 Compare September 20, 2025 21:27
@simonprovost simonprovost force-pushed the feat/modern_pythonic_library_transformation branch from f8c3f35 to 6cd4df3 Compare September 20, 2025 21:29
@simonprovost
Copy link
Member Author

Hi @roquelopez just so you know, I've successfully rebased this current PR's branch with main now that you've accepted #2 and that we merged it. No rush please especially because it's the weekend 🫡

Take care

@simonprovost
Copy link
Member Author

simonprovost commented Sep 28, 2025

Hi @roquelopez hope all is well 🙏

Maybe we can try to pre-review the PR with someone else from the team? E.g. someone Juliana cc-ed in the email a couple of days ago, if you are too busy at the moment?

(Please do not answer over the weekend nothing very very urgent anyway ^^ I just thought about this PR in between two rides in London!)

Cheers

@simonprovost
Copy link
Member Author

UP Friendly Reminder 🫡 @roquelopez

@roquelopez
Copy link
Contributor

Thanks, @simonprovost, for implementing this, it looks great to me!
Please go ahead and merge it. Make sure to squash the commits when merging and reference the PR in the commit message, e.g.: chore: Remodernisation of AutoDDG (#4)

@simonprovost simonprovost merged commit d705443 into VIDA-NYU:main Oct 23, 2025
1 check passed
@simonprovost
Copy link
Member Author

Thanks, @simonprovost, for implementing this, it looks great to me! Please go ahead and merge it. Make sure to squash the commits when merging and reference the PR in the commit message, e.g.: chore: Remodernisation of AutoDDG (#4)

Thanks so much for your time Roque ! ✅

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.

2 participants