feat: add news-summarizer-agent using NewsAPI and ASI:One LLM#141
Conversation
|
@gautammanak1 this has been open for 16 days. Happy to make any requested changes. |
|
Please fix the CI issues and also add ChatProtocol support. At the moment, the workflow doesn't seem to be working correctly, which is likely why the PR has remained open. Once those changes are addressed, I can review it again. |
|
Hi @gautammanak1, I've updated the PR to address the CI issues and added Chat Protocol support using uAgents. The agent has been moved to contributors/news-summarizer-agent/ as required by the repository structure. Please let me know if anything else needs to be changed. |
|
Hi @gautammanak1, I've resolved the merge conflict in contributors/CHANGELOG.md. The PR should be ready for review now. |
|
Summary This PR adds a clean, beginner-friendly What I liked
Security
Correctness / logic
Performance
Code quality
Tests
Suggested fixes None blocking. If you want to polish further, here are optional enhancements:
Questions for the author
LGTM! ✅ This is a well-structured, beginner-friendly example that fills a clear gap in the repo. The code is safe, readable, and follows Fetch.ai agent patterns. Ship it! |
|
Hi @gautammanak1, I've pushed a fix removing a stray top-level demo.png file that was tripping the changelog-check (the diff had a leftover file outside contributors/, which made the workflow look for a root CHANGELOG.md update instead of contributors/CHANGELOG.md). Could you approve the pending workflow run so the checks execute on the latest commit? Also, if you're able to formally approve via Files changed → Review changes → Approve, that should satisfy the review-required check too. Thanks for your patience on this! |
|
Hi @gautammanak1, thank you for reviewing and approving my PR! 🎉 I noticed that all the checks have passed and the PR is now ready to be merged. When you have a chance, could you please merge it so it can be counted for GSSoC? |
Closes #126
Summary
Adds a new beginner-friendly
news-summarizer-agentexample that fetches thetop 5 news headlines for any topic using the free NewsAPI (no credit card
required) and summarizes them using ASI:One LLM. This fills the gap for a
beginner example combining real-time data fetching with LLM summarization.
Type of Change
Checklist
ruff check ..ruff format ..README.mdfor changed example(s)..env.exampleif environment variables are required.Related Issue
Closes #126
Notes for Reviewers
all GSSoC contributors
asi1-minimodel via the standard chat completions endpoint.envis gitignored, only.env.exampleincludedruff checkandruff formatwith no issues