Skip to content

fix: align pdf_form_filler and evm_tx_handler manifest names with reg…#209

Merged
rosspeili merged 2 commits into
ARPAHLS:mainfrom
rosspeili:feat/issue-201-manifest-names
Jul 1, 2026
Merged

fix: align pdf_form_filler and evm_tx_handler manifest names with reg…#209
rosspeili merged 2 commits into
ARPAHLS:mainfrom
rosspeili:feat/issue-201-manifest-names

Conversation

@rosspeili

Copy link
Copy Markdown
Contributor

Summary

Aligns the last two legacy short manifest names with registry paths (#101 Option A):

  • office/pdf_form_filler — was pdf_form_filler
  • defi/evm_tx_handler — was evm_tx_handler

Gemini/Claude tool names now use the full ID (e.g. office/pdf_form_filler). OpenAI/DeepSeek use sanitized names (office_pdf_form_filler, defi_evm_tx_handler).

Fixes #201

Changes

  • Manifestsname set to category/skill_name for both skills
  • Examples — PDF scripts dispatch via bundle["manifest"]["name"] (evm examples already did)
  • Tests — bundle + maintainer manifest assertions; CLI fixtures use full IDs
  • Docs — skill catalog pages, CONTRIBUTING, cli.md, agent_loops.md, introduction.md
  • CHANGELOG[Unreleased] entry

Out of scope

  • Loader strict validation (#201 follow-up)
  • Category moves (#53)

Type of change

  • Bug Report Fix — manifest / tool-name identity drift
  • Doc Fix — naming and dispatch guidance

Checklist

  • Agent Code of Conduct
  • pytest skills/office/pdf_form_filler/test_skill.py skills/defi/evm_tx_handler/test_skill.py
  • pytest tests/test_cli.py tests/test_registry_docs.py tests/skills/office/test_pdf_form_filler.py
  • skillware list / skillware test for both skills
  • CHANGELOG.md updated

New or updated skill

Manifest metadata + docs/examples only — no new skills, no logic changes.

  • manifest.yaml name matches folder path
  • Examples use manifest-derived tool dispatch
  • test_skill.py + catalog docs updated
  • SkillLoader.load_skill(...) unchanged (path-based)

Related

Fixes #201 · Related #101, #178

rosspeili added 2 commits July 1, 2026 14:37
…istry paths

Set manifest name to category/skill_name per ARPAHLS#101 Option A, update examples to dispatch tools via manifest-derived names, and sync docs, tests, and CLI fixtures. Fixes ARPAHLS#201.
@rosspeili rosspeili merged commit e01ff33 into ARPAHLS:main Jul 1, 2026
5 checks passed
@rosspeili rosspeili deleted the feat/issue-201-manifest-names branch July 1, 2026 11:49
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]: Migrate legacy short manifest names to category/skill_name paths

1 participant