Add mta-trigger-test community ability#228
Closed
chadnewbry wants to merge 2 commits intoopenhome-dev:devfrom
Closed
Add mta-trigger-test community ability#228chadnewbry wants to merge 2 commits intoopenhome-dev:devfrom
chadnewbry wants to merge 2 commits intoopenhome-dev:devfrom
Conversation
Contributor
✅ Community PR Path Check — PassedAll changed files are inside the |
Contributor
🔀 Branch Merge CheckPR direction: ✅ Passed — |
Contributor
❌ Ability Validation Failed📚 How to fix
|
Contributor
🔍 Lint Results🔧 Auto-formattedSome files were automatically cleaned and formatted with
Files linted: ✅ Flake8 — Passed
|
8fca3b6 to
68ef33f
Compare
Author
|
Using this to debug issues I was having, getting my ability to work; got that solved. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this Ability do?
🚦 Adds
mta-trigger-test, a minimal trigger-routing debug ability for OpenHome. It speaks one unmistakable line so developers can verify whether custom-ability routing is firing at all before debugging larger abilities.Suggested Trigger Words
mta trigger testtest mta abilitynyc mta testType
External APIs
Testing
Local Verification
python3 validate_ability.py community/mta-trigger-testExpected output when triggered:
Checklist
community/mta-trigger-test/main.pyfollows SDK pattern (extendsMatchingCapability, hasregister_capability+call)README.mdincluded with description, suggested triggers, and setupresume_normal_flow()called on every exit pathprint()— usingeditor_logging_handlerredis,user_config)asyncio.sleep()orasyncio.create_task()— usingsession_tasksAnything else?
🧪 This is intentionally tiny. The goal is to isolate whether trigger words route into custom abilities at all, independent of API calls, helper modules, or multi-turn logic.