Skip to content

Conversation

@bokelley
Copy link
Contributor

@bokelley bokelley commented Jan 4, 2026

Summary

  • Add URL normalization for feed deduplication (handles trailing slashes, www prefix, query params)
  • Add findSimilarFeeds() to detect duplicate feeds before adding
  • Add propose_news_source tool for community members to suggest news sources
  • Add admin tools to review/approve/reject proposals (list_feed_proposals, approve_feed_proposal, reject_feed_proposal)
  • Add feed_proposals database table with migration 117

Context

Addie was hallucinating feed lists instead of querying the database. The admin UI showed 38 feeds while Addie claimed only 8. This PR:

  1. Improves deduplication to catch URL variations (with/without trailing slash, www prefix, etc.)
  2. Adds a system for community members to propose news sources for admin review

Test plan

  • Build passes
  • All 161 tests pass
  • Tested admin feeds UI in browser - loads correctly and shows feed count
  • Add Feed modal works
  • Code review addressed critical issues (input validation, URL normalization consistency)

🤖 Generated with Claude Code

bokelley and others added 2 commits January 3, 2026 22:14
- Add URL normalization for feed deduplication (handles trailing slashes,
  www prefix, query params)
- Add findSimilarFeeds() to detect duplicate feeds before adding
- Add propose_news_source tool for community members to suggest feeds
- Add admin tools to review/approve/reject proposals (list_feed_proposals,
  approve_feed_proposal, reject_feed_proposal)
- Add feed_proposals database table with migration

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
117_personal_domains.sql was added to main, so renaming our migration.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@bokelley bokelley merged commit deb0b25 into main Jan 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants