Skip to content

Conversation

@adubatl
Copy link
Owner

@adubatl adubatl commented Feb 21, 2025

📥 Pull Request

📘 Description
Briefly describe the changes you've made.

🧪 Testing
Describe the tests you performed to validate your changes.

@adubatl adubatl changed the title Adubatl/model fetching po c Adubatl/model fetching poc Feb 22, 2025
…/wiz flows, but also for searchable models)

use_shortcuts is a bummer with search, so bye

make the filters behave

use a real list of the providers

we dont need preferred models honestly, it all lives in the lite llm package anyway

cleanup linting/comment changes, move all models list to the other option

cleanup cli/wizard implementation and move a util to utils

get tests passing and do a little more pr noise cleanup

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

format manually something is weird.

missing double-quote

get tests passing and do a little more pr noise cleanup

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

formatting noise

format manually something is weird.
@adubatl adubatl force-pushed the adubatl/model_fetching_PoC branch from ee1d946 to d1cb209 Compare February 22, 2025 17:55
assert not is_snake_case("hello-world")
assert not is_snake_case("hello world")

def test_validator_not_empty(self):
Copy link
Owner Author

Choose a reason for hiding this comment

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

Moved to test_cli_validation because of the new validation handling from questionary

license = { text = "MIT" }
readme = "README.md"
requires-python = ">=3.10"
requires-python = "<3.13,>=3.10"
Copy link
Owner Author

Choose a reason for hiding this comment

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

appeases questionary, and prevents first-time setup error if you are on a super new python.

]


def get_validated_input(
Copy link
Owner Author

Choose a reason for hiding this comment

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

questionary validation fires on input and shows the message clearly without needing to interrupt the execution. I moved all the validation into this single command.

Copy link
Owner Author

Choose a reason for hiding this comment

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

this is from swapping inquirer for questionary.

desi003 and others added 30 commits March 18, 2025 14:31
…agentql

Set X-TF-Request-Origin header in AgentQL tool
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.

8 participants