Skip to content

Add DeepSeek tool adapter and usage guide.#70

Merged
rosspeili merged 1 commit into
ARPAHLS:mainfrom
rosspeili:feat/issue-66-deepseek-adapter-and-docs
May 17, 2026
Merged

Add DeepSeek tool adapter and usage guide.#70
rosspeili merged 1 commit into
ARPAHLS:mainfrom
rosspeili:feat/issue-66-deepseek-adapter-and-docs

Conversation

@rosspeili
Copy link
Copy Markdown
Contributor

Description

Adds DeepSeek as a first-class provider alongside OpenAI, not a replacement.

  • SkillLoader.to_deepseek_tool() — Separate public adapter (does not call to_openai_tool()).
  • _sanitize_function_tool_name() — Shared internal helper; OpenAI and DeepSeek each keep their own public sanitizer wrappers.
  • Teststest_to_deepseek_tool, test_sanitize_deepseek_tool_name; existing OpenAI tests unchanged.
  • docs/usage/deepseek.md + examples/deepseek_tos_evaluator.py — Full loop with DEEPSEEK_API_KEY and base_url=https://api.deepseek.com.
  • Links from README, introduction, api_keys, openai.md (related), and tos_evaluator skill doc.

Type of Change

  • Framework Feature / RFC Updates (loader.py)
  • Doc Fix (usage guide + example)
  • Skill Proposal / Bug Fix

Checklist (all PRs)

  • My code follows the Agent Code of Conduct.
  • pytest tests/test_loader.py and flake8 on changed Python files pass.

New or updated skill

Not applicable (skills/ unchanged). tos_evaluator.md adds a DeepSeek example link only.

Constitution & Safety

Adapter-only; no skill behavior change. Use sanitized tool names from to_deepseek_tool() in the agent loop.

Related Issues

Fixes #66. Builds on #1 / #20 (OpenAI adapter remains separate).

Implement to_deepseek_tool as a separate public adapter from OpenAI, with shared internal name sanitization, unit tests, docs/usage/deepseek.md, and examples/deepseek_tos_evaluator.py. Link from README, introduction, api_keys, and tos_evaluator docs.

Fixes ARPAHLS#66
@rosspeili rosspeili merged commit 1ba0a98 into ARPAHLS:main May 17, 2026
5 checks passed
@rosspeili rosspeili deleted the feat/issue-66-deepseek-adapter-and-docs branch May 17, 2026 10:26
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.

[Feat]: Add DeepSeek model adapter and usage guide

1 participant