Skip to content

Add memorizer ability to updated preferences file#240

Closed
anish-jayant wants to merge 3 commits intoopenhome-dev:devfrom
anish-jayant:memorizer
Closed

Add memorizer ability to updated preferences file#240
anish-jayant wants to merge 3 commits intoopenhome-dev:devfrom
anish-jayant:memorizer

Conversation

@anish-jayant
Copy link
Copy Markdown

@anish-jayant anish-jayant commented Apr 3, 2026

What does this Ability do?

Suggested Trigger Words

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 (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)
  • 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 — using placeholders
  • 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?

@anish-jayant anish-jayant requested a review from a team as a code owner April 3, 2026 07:59
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

🔀 Branch Merge Check

PR direction: memorizerdev

Passedmemorizerdev is a valid merge direction

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 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 Apr 3, 2026

❌ Ability Validation Failed

📋 Validating: community/memorizer
  ❌ Must declare 'worker: AgentWorker = None' as a class attribute
  ❌ Must declare 'capability_worker: CapabilityWorker = None' as a class attribute
  ❌ 2 error(s) found

📚 How to fix

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

github-actions bot commented Apr 3, 2026

🔍 Lint Results

__init__.py — Empty as expected

Files linted: community/memorizer/main.py

✅ Flake8 — Passed

✅ All checks passed!

Signed-off-by: Uzair Ullah <uzairullahmail@gmail.com>
@uzair401 uzair401 closed this Apr 3, 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