Skip to content

Adubatl/model fetching poc - #2

Draft
adubatl wants to merge 58 commits into
mainfrom
adubatl/model_fetching_PoC
Draft

Adubatl/model fetching poc#2
adubatl wants to merge 58 commits into
mainfrom
adubatl/model_fetching_PoC

Conversation

@adubatl

Copy link
Copy Markdown
Owner

📥 Pull Request

📘 Description
Briefly describe the changes you've made.

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

@adubatladubatl changed the title Adubatl/model fetching po cAdubatl/model fetching pocFeb 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
adubatlforce-pushed the adubatl/model_fetching_PoC branch from ee1d946 to d1cb209CompareFebruary 22, 2025 17:55
Comment threadtests/test_utils.py
assert not is_snake_case("hello-world")
assert not is_snake_case("hello world")

def test_validator_not_empty(self):

Copy link
Copy Markdown
OwnerAuthor

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

Comment threadpyproject.toml
license = { text = "MIT" }
readme = "README.md"
requires-python = ">=3.10"
requires-python = "<3.13,>=3.10"

Copy link
Copy Markdown
OwnerAuthor

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.

Comment threadagentstack/cli/cli.py
]


def get_validated_input(

Copy link
Copy Markdown
OwnerAuthor

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.

Comment threadpoetry.lock

Copy link
Copy Markdown
OwnerAuthor

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.

Comment threadagentstack/providers.py
desi003and others added 30 commits March 18, 2025 14:31
…agentql
Set X-TF-Request-Origin header in AgentQL tool
Sign up for freeto 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.

7 participants

@adubatl@samsterpiece@desi003@braelyn-ai@tcdent@Haakam21@amaan-ai20