Skip to content

Private Notes Ability#234

Closed
VaughnHays wants to merge 6 commits intoopenhome-dev:devfrom
VaughnHays:private_notes
Closed

Private Notes Ability#234
VaughnHays wants to merge 6 commits intoopenhome-dev:devfrom
VaughnHays:private_notes

Conversation

@VaughnHays
Copy link
Copy Markdown

What does this Ability do?

A note-taking Ability for OpenHome that stays private by design. Your OpenHome Personality will never surface them unprompted.

Suggested Trigger Words

note, notes, take a note, note this down, read my notes,
delete my notes, my notes, edit my note, update my note,
change my note, fix my note

Type

  • [ x] 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 (said "stop", "exit", etc.)
  • Error scenarios tested (API down, bad input, etc.)

Checklist

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

@VaughnHays VaughnHays requested a review from a team as a code owner March 30, 2026 04:48
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 30, 2026

🔀 Branch Merge Check

PR direction: private_notesdev

Passedprivate_notesdev is a valid merge direction

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

github-actions bot commented Mar 30, 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 30, 2026

✅ Ability Validation Passed

📋 Validating: community/private-notes
  ✅ All checks passed!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 30, 2026

🔍 Lint Results

__init__.py — Empty as expected

Files linted: community/private-notes/main.py

✅ Flake8 — Passed

✅ All checks passed!

Signed-off-by: Uzair Ullah <uzairullahmail@gmail.com>
… method

Refactor capability_worker initialization and remove unused method for getting last user message.

Signed-off-by: Uzair Ullah <uzairullahmail@gmail.com>
Updated trigger words for note-taking functionality and clarified note retrieval options.

Signed-off-by: Uzair Ullah <uzairullahmail@gmail.com>
@uzair401
Copy link
Copy Markdown
Contributor

Hi @VaughnHays, great work on this ability. I’ve made some updates, but a few additional changes are still needed to make it fully compliant with the latest SDK methods. Please review the guides below, update the code accordingly, and reopen the PR.

https://docs.openhome.com/
https://github.com/openhome-dev/abilities/blob/dev/docs/OpenHome_SDK_Reference.md

@uzair401 uzair401 closed this Mar 30, 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.

2 participants