Add subtitle field to feed and item field lists

[rbren]

Summary

This PR adds the subtitle field to both the fields.feed and fields.item arrays in lib/fields.js, enabling the RSS parser to extract subtitle information from RSS feeds.

Changes

• Added subtitle to fields.feed (positioned after description)
• Added subtitle to fields.item (positioned after title)

Motivation

The subtitle field is commonly used in RSS/Atom feeds but was not being parsed by the library. While the podcast fields already supported itunes:subtitle, the standard subtitle element was missing from the main feed and item field lists.

This allows the parser to extract subtitle information from feeds that include this field.

[rbren]

@OpenHands add an inline comment describing these changes

[openhands-ai]

I'm on it! rbren can track my progress at all-hands.dev

[openhands-ai]

I've addressed the request to add an inline comment describing the changes in PR #301.

Changes made:

• Added an inline comment above the subtitle field in lib/fields.js explaining its purpose: // subtitle is commonly used in Atom feeds and some RSS feeds to provide additional context

Checklist:

• ✅ Added inline comment describing the subtitle field change
• ✅ Comment is concise and explains why the field is useful
• ✅ Changes have been committed and pushed to the openhands/add-subtitle-field branch

The PR has been updated with the new commit.

View full conversation

[rbren]

@OpenHands add a test for this case

[openhands-ai]

I'm on it! rbren can track my progress at all-hands.dev

[openhands-ai]

I've added a test for the subtitle field parsing as requested. The changes have been committed and pushed to the openhands/add-subtitle-field branch.

Changes made:

1. Created test/input/subtitle.rss - An RSS 2.0 test feed with subtitle fields at both the feed level and item level
2. Created test/output/subtitle.json - The expected parsed output showing the subtitle values
3. Added a new test case in test/parser.js: 'should parse subtitle field in feed and items'

Test verification:

• All 39 tests pass, including the new subtitle test
• The test verifies that:
  • Feed-level subtitle ("Tech insights and tutorials") is correctly parsed
  • Item-level subtitles ("An introduction to our blog" and "More content for readers") are correctly parsed

The PR #301 has been updated with this new commit.

View full conversation