rss-feeds: Skill for Claude Code

RSS Feed Review

Review RSS feed generators and their output XML for correctness, robustness, and adherence to project conventions.

Instructions

1. Determine scope — review all feed generators by default, or a specific one if the user specifies.
2. Read the target generator(s) and their corresponding feeds/feed_*.xml output files.
3. Read feed_generators/utils.py to understand shared helpers.
4. Evaluate against the checklists below. For every finding, cite file_path:line_number.
5. If everything looks good, say so briefly.

Generator Code Review

Selectors & Parsing

• Are CSS selectors specific enough to survive minor site redesigns?
• Are selectors targeting semantic elements (article, h2) over generated class names?
• Is there fallback logic if a selector returns no results?

Error Handling

• Does fetch_* use timeout= on requests?
• Are HTTP errors handled (response.raise_for_status() or status check)?
• Are Selenium waits using explicit waits (WebDriverWait) rather than time.sleep()?
• Is the Selenium driver properly closed in a finally block?

Feed Link Setup

Critical convention (from AGENTS.md):

from utils import setup_feed_links
setup_feed_links(fg, blog_url="https://...", feed_name="...")

• The main <link> must point to the original blog URL, NOT the feed URL
• rel="self" must be set first, rel="alternate" must be set last
• Generators should use the setup_feed_links() helper from utils.py
• Flag any generator that sets links manually instead of using the helper

Cache Logic (Pagination & Selenium patterns only)

• Is cache loaded before fetching new articles?
• Are articles deduped by URL before saving?
• Is the cache sorted by date descending?
• Does the --full flag correctly bypass incremental logic?

Pattern Compliance

• Simple Static: No cache needed, fetches all posts each run
• Pagination + Caching: URL-based pagination with JSON cache in cache/
• Selenium + Click: Uses undetected-chromedriver, clicks load-more buttons, caches results
• Is the generator using the right pattern for how the target site loads content?