Skip to content

Add mta-trigger-test community ability#228

Closed
chadnewbry wants to merge 2 commits intoopenhome-dev:devfrom
chadnewbry:add-mta-trigger-test
Closed

Add mta-trigger-test community ability#228
chadnewbry wants to merge 2 commits intoopenhome-dev:devfrom
chadnewbry:add-mta-trigger-test

Conversation

@chadnewbry
Copy link
Copy Markdown

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 test
  • test mta ability
  • nyc mta test

Type

  • New community Ability
  • Improvement to existing Ability
  • Bug fix
  • Documentation update

External APIs

  • No external APIs
  • Uses external API(s):

Testing

  • Tested in OpenHome Live Editor
  • All exit paths tested
  • Error scenarios tested

Local Verification

  • python3 validate_ability.py community/mta-trigger-test

Expected output when triggered:

MTA ability triggered successfully.

Checklist

  • Files are in community/mta-trigger-test/
  • main.py follows SDK pattern (extends MatchingCapability, has register_capability + call)
  • README.md included with description, suggested triggers, and setup
  • resume_normal_flow() called on every exit path
  • No print() — using editor_logging_handler
  • No hardcoded API keys
  • No blocked imports (redis, user_config)
  • No asyncio.sleep() or asyncio.create_task() — using session_tasks
  • Error handling on all external calls
  • Tested in OpenHome Live Editor

Anything 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.

@chadnewbry chadnewbry requested a review from a team as a code owner March 27, 2026 22:26
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

✅ Community PR Path Check — Passed

All changed files are inside the community/ folder. Looks good!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

🔀 Branch Merge Check

PR direction: add-mta-trigger-testdev

Passedadd-mta-trigger-testdev is a valid merge direction

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

❌ Ability Validation Failed

📋 Validating: community/mta-trigger-test
  ❌ Missing required file: README.md
  ❌ Missing required file: __init__.py
  ❌ 2 error(s) found

📚 How to fix

@github-actions github-actions bot added the community-ability Community-contributed ability label Mar 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

🔍 Lint Results

🔧 Auto-formatted

Some files were automatically cleaned and formatted with autoflake + autopep8 and committed.

  • Unused imports removed (autoflake)
  • Unused variables removed (autoflake)
  • PEP8 formatting applied (autopep8)

Files linted: community/mta-trigger-test/main.py

✅ Flake8 — Passed

✅ All checks passed!

@chadnewbry chadnewbry force-pushed the add-mta-trigger-test branch from 8fca3b6 to 68ef33f Compare March 27, 2026 22:30
@chadnewbry
Copy link
Copy Markdown
Author

Using this to debug issues I was having, getting my ability to work; got that solved.

@chadnewbry chadnewbry closed this Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-ability Community-contributed ability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant