Take feed discovery, OPML parsing and the submit_feed tool from @profullstack/submit-feed - #190
Merged
Conversation
…ullstack/submit-feed The submit path was extracted into a package so p0dcasters.com and any later directory take feeds the same way, and this is the site switching to it. discover.js is now a seam over the package: the same four names, so nothing in the workspace changed an import, plus the one thing this directory does that the package does not, admitting a playlist as a feed. parseOpml is the package's; the OPML writers stay here because they use this site's column names and stream. The MCP submit_feed tool takes its name, description, schema and annotations from the shared definition and keeps its own run. Behaviour is unchanged where it matters and tightened in one place: an application/xml or text/xml body is a feed only if it has a feed root, so a sitemap or an OPML file served as XML is no longer handed to the parser. Every suite is where it was: feed 344, web 427, ingest 159, cli 25. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014JGnonZiv2TP3aSiu4LLWZ
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The submit path was extracted into @profullstack/submit-feed so p0dcasters.com (profullstack/p0dcasters#21) and any later directory take feeds the same way. This switches rssamplifier to it where the code is identical:
packages/feed/src/discover.jsis a seam over the package:normalizeUrl,findFeedLinks,guessFeedUrlsre-exported,looksLikeFeedwraps the package's sniff with this site's playlist admission. No import in the workspace changed.parseOpmlcomes from the package;buildOpml/opmlHead/opmlOutline/opmlFootstay (site column names, streamed export).submit_feedtool takes its name, description, input schema and annotations fromsubmitFeedTool(), keeping its ownrun(ledger, express lane, status page).One deliberate tightening: an
application/xml/text/xmlbody counts as a feed only with an<rss>,<feed>or<rdf:RDF>root, so a sitemap or OPML served as XML is no longer handed to the parser.Suites unchanged from baseline: feed 344, web 427, ingest 159, cli 25.
🤖 Generated with Claude Code
https://claude.ai/code/session_014JGnonZiv2TP3aSiu4LLWZ