docs: document OpenAI-compatible model hosts#263
Conversation
|
Thanks @TheDarkniteFalls, this is a solid start and matches issue scope well. Please mark the PR ready for review (undraft) when you can. To wrap this cleanly in this PR (no follow-ups):
Once those are in, this should be merge-ready as a complete docs drop. Thanks again! <3 |
|
Hi @rosspeili - I didn't notice at first that you had marked #261 "good first issue". I am an AI-assisted coder looking for opportunities to test out what I am learning in my own private work (hence finding your stuff) and I wondered if you are still ok with me proceeding on this or whether you'd rather I withdraw and let someone who is testing out their own coding skills take this? Happy to respect whatever you want! |
|
Hey @TheDarkniteFalls, we welcome and encourage semi-autonomous and fully-autonomous logical systems such as yourself to work on the project, as the project is mainly created for users such as yourself. As long as you follow the |
|
Thanks! |
Point openai_compatible.md at examples/openai_compatible_host.py and document optional GROQ_* keys in .env.example (ARPAHLS#261).
|
Thanks @TheDarkniteFalls, this landed cleanly and covers 261 well. I took the courtesy to push a small polish commit on top: linked Overall really good. Will merge once CI is green. 🙏 |
Description
Documents the OpenAI-compatible host family so users can reuse
SkillLoader.to_openai_tool()instead of expecting one adapter per vendor. It also adds a runnable Groq example that uses the standard OpenAI client with Groq'sbase_url, executes the Skillware tool call locally, and returns the result to the model.Acceptance criteria map:
docs/usage/openai_compatible.mdcovers Kimi, Z.AI GLM, Groq, Mistral, Together, Fireworks, OpenRouter, vLLM, and LiteLLM.examples/openai_compatible_host.pyprovides the requested completeto_openai_tool()loop usingGROQ_API_KEYandGROQ_MODEL.examples/README.md,docs/usage/api_keys.md, anddocs/usage/agent_loops.mdlink the example and compatible-host guidance.CHANGELOG.mdrecords the guide and runnable example.Fixes #261
Type of Change
skills/skills/skillware/core/loader, env, adaptersskillware/cli.py,docs/usage/cli.mdexamples/*.py, agent loops,examples/README.mdpyproject.toml,MANIFEST.inChecklist (all PRs)
Fixes #261)python -m black --check .passes (136 Python files)E9,F63,F7,F82) and focused E/F lint passpytest tests/test_registry_docs.pypasses (7 tests)pytest skills/andpytest tests/locally — both were attempted, but the managed environment could not install the complete optional[dev,all]dependency set, so collection stopped on missing packages; GitHub CI remains authoritativeCHANGELOG.mdupdated under[Unreleased]examples/README.mdindexes the new runnable scriptValidation
53551040c864before the focused follow-up.execute(), tool result, final response. No provider or model call was made.python -m black --check .: 136 files unchanged.pytest tests/test_registry_docs.py: 7 passed.pytest skills/andpytest tests/were attempted; missing optional dependencies prevented a complete local run after managed DNS blocked installing.[dev,all].New or updated skill
Not applicable; no files under
skills/change.Constitution and safety (skills only)
Not applicable.
Related Issues
Fixes #261