Skip to content

X news feed ability#242

Open
alimujtaba478 wants to merge 28 commits intoopenhome-dev:devfrom
alimujtaba478:x-news-feed-ability
Open

X news feed ability#242
alimujtaba478 wants to merge 28 commits intoopenhome-dev:devfrom
alimujtaba478:x-news-feed-ability

Conversation

@alimujtaba478
Copy link
Copy Markdown
Contributor

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?

alimujtaba478 and others added 27 commits February 16, 2026 03:50
Signed-off-by: Muhammad Rizwan <awanrizwan615@gmail.com>
…-feed-ability

# Conflicts:
#	community/x-news-feed/config.json
Signed-off-by: alimujtaba478 <70099666+alimujtaba478@users.noreply.github.com>
…ile, write_file, etc.) instead

 Missing register capability tag — add one of the following to your class:
    #{{register_capability}}
    # {{register_capability}}

resolved the above issue
Signed-off-by: Uzair Ullah <uzairullahmail@gmail.com>
Signed-off-by: alimujtaba478 <70099666+alimujtaba478@users.noreply.github.com>
@alimujtaba478 alimujtaba478 requested a review from a team as a code owner April 5, 2026 22:46
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 5, 2026

🔀 Branch Merge Check

PR direction: x-news-feed-abilitydev

Passedx-news-feed-abilitydev is a valid merge direction

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 5, 2026

✅ Community PR Path Check — Passed

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

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

github-actions bot commented Apr 5, 2026

🔍 Lint Results

Files linted: community/x-news-feed/main.py

✅ Flake8 — Passed

✅ All checks passed!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 5, 2026

✅ Ability Validation Passed

📋 Validating: community/x-news-feed
  ✅ All checks passed!

@alimujtaba478
Copy link
Copy Markdown
Contributor Author

Hi @uzair401 ,

Loom Demo:
https://www.loom.com/share/1eda9de6dcdd4a8787190c78f3dbe961

I also reviewed the code against the latest voice naturalness audit. The major issues (brittle string matching, menu-driven ping-pong, long responses, and voice formatting problems) have been resolved using LLM classifiers, expanded natural word lists, the shared VOICE_GUARDRAIL, strict response length limits, and a single open-ended prompt in quick mode.
The Ability is now much more natural for real spoken English usage on smart speakers.
Ready for review. Please let me know if any further adjustments are needed.
Thanks!

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.

3 participants