Conversation
🔀 Branch Merge CheckPR direction: ✅ Passed — |
✅ Community PR Path Check — PassedAll changed files are inside the |
✅ Ability Validation Passed |
🔍 Lint Results✅
|
|
Hey @FHLiang221, great work on the structure here — the LLM-routed intent classification is the right approach and the background watcher pattern is solid. Found a few things that need to be fixed before this can be approved though:
|
uzair401
left a comment
There was a problem hiding this comment.
Please address the issues explained in the comments above, and once resolved I'll take another look!
Signed-off-by: Uzair Ullah <uzairullahmail@gmail.com>
Updated quit and exit words for better user interaction. Signed-off-by: Uzair Ullah <uzairullahmail@gmail.com>
uzair401
left a comment
There was a problem hiding this comment.
Approved for community.
We will proceed with adopting this ability for a 30-day evaluation period to monitor performance, identify any bugs, and implement any necessary fixes or optimizations. If the ability proves stable and meets our quality standards during this period, we will move forward with publishing it to the marketplace, ensuring full credit is given to the original author for their work and contribution.
We truly appreciate your effort and encourage you to continue developing and submitting new abilities. Contributions like yours help strengthen the ecosystem, and we look forward to seeing more of your work.
Changes made before merge: Removed hardcoded delete, list, and bare trigger fast-paths. these silently failed on natural/native speech like "get rid of my alarms". Everything now routes through the LLM. Expanded quit/exit/affirmative word lists — critically "stop" was missing. Hardened the LLM prompt to enforce plain spoken English on QUESTION responses. Cleaned up spoken strings and capped list output at 3 items.
What does this Ability do?
A combined interactive + background daemon ability that handles timers, alarms, and reminders through natural voice interaction. Users can set, list, cancel, and delete events. A background watcher fires events when due — timers get a spoken notification, alarms play a sound, and reminders speak the message.
Suggested Trigger Words
Type
External APIs
Testing
Checklist
community/timers-alarms-reminders/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,connection_manager,user_config)asyncio.sleep()orasyncio.create_task()— usingsession_tasksAnything else?
Includes a
background.pybackground daemon that pollsscheduled_events.jsonevery 5 seconds and fires due events. Also writesupcoming_schedule.mdso the Personality stays aware of what's scheduled. Includesalarm.mp3for alarm sounds.