Auto PR: Scheduled addition of new sources - #71
Conversation
📝 WalkthroughWalkthroughTwo new YAML source definition files are added: one for Google News and one for The Times of India. Each file defines the source's name, summary, tags, and URI. ChangesNew Source Definitions
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@sources/The-Times-of-India.yaml`:
- Around line 1-4: The source metadata is using the wrong website for The Times
of India entry. Update the uri field in the The Times of India YAML record to
the Times of India domain instead of the generic indiatimes portal, and keep the
change confined to the source definition identified by the name value.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 96be8805-3a43-401a-ba4c-fa1c55deb307
📒 Files selected for processing (2)
sources/Google-News.yamlsources/The-Times-of-India.yaml
| name: The Times of India | ||
| summary: English-language daily newspaper and digital news platform based in India. | ||
| tags: news,media,journalism,India | ||
| uri: https://www.indiatimes.com |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '\n== Relevant files ==\n'
git ls-files 'oapi.yaml' 'sources/The-Times-of-India.yaml' 'sources/*.yaml' | sed 's#^`#-` #'
printf '\n== Schema references for SourceInput ==\n'
rg -n --no-heading -A3 -B3 'SourceInput|uri:|https://' oapi.yaml sources/The-Times-of-India.yaml sources -g 'oapi.yaml' -g 'sources/*.yaml' || true
printf '\n== Full contents of the target source file ==\n'
cat -n sources/The-Times-of-India.yaml
printf '\n== Nearby source entries using Times of India domains ==\n'
rg -n --no-heading 'timesofindia\.indiatimes\.com|www\.timesofindia\.com|indiatimes\.com' sources -g '*.yaml' || trueRepository: SatyaLens/sources
Length of output: 10482
Use the Times of India domain here. https://www.indiatimes.com points to a different Times Internet portal; this source should use the Times of India site instead, e.g. https://timesofindia.indiatimes.com.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@sources/The-Times-of-India.yaml` around lines 1 - 4, The source metadata is
using the wrong website for The Times of India entry. Update the uri field in
the The Times of India YAML record to the Times of India domain instead of the
generic indiatimes portal, and keep the change confined to the source definition
identified by the name value.
Source: Coding guidelines
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
2 issues found across 2 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="sources/The-Times-of-India.yaml">
<violation number="1" location="sources/The-Times-of-India.yaml:1">
P0: Duplicate source: `sources/The-Times-of-India.yaml` represents the same entity as the existing `sources/India-Times.yaml`, whose summary explicitly describes "The Times of India". This will cause a duplicate POST or a conflict (URI unique index on `idx_sources_uri`).</violation>
<violation number="2" location="sources/The-Times-of-India.yaml:4">
P1: Wrong URI: `https://www.indiatimes.com` is not the canonical site for The Times of India. The official URI is `https://timesofindia.indiatimes.com`. This also conflicts with the existing `India-Times.yaml` entry which already uses the correct URI.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
| @@ -0,0 +1,4 @@ | |||
| name: The Times of India | |||
There was a problem hiding this comment.
P0: Duplicate source: sources/The-Times-of-India.yaml represents the same entity as the existing sources/India-Times.yaml, whose summary explicitly describes "The Times of India". This will cause a duplicate POST or a conflict (URI unique index on idx_sources_uri).
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At sources/The-Times-of-India.yaml, line 1:
<comment>Duplicate source: `sources/The-Times-of-India.yaml` represents the same entity as the existing `sources/India-Times.yaml`, whose summary explicitly describes "The Times of India". This will cause a duplicate POST or a conflict (URI unique index on `idx_sources_uri`).</comment>
<file context>
@@ -0,0 +1,4 @@
+name: The Times of India
+summary: English-language daily newspaper and digital news platform based in India.
+tags: news,media,journalism,India
</file context>
| name: The Times of India | ||
| summary: English-language daily newspaper and digital news platform based in India. | ||
| tags: news,media,journalism,India | ||
| uri: https://www.indiatimes.com |
There was a problem hiding this comment.
P1: Wrong URI: https://www.indiatimes.com is not the canonical site for The Times of India. The official URI is https://timesofindia.indiatimes.com. This also conflicts with the existing India-Times.yaml entry which already uses the correct URI.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At sources/The-Times-of-India.yaml, line 4:
<comment>Wrong URI: `https://www.indiatimes.com` is not the canonical site for The Times of India. The official URI is `https://timesofindia.indiatimes.com`. This also conflicts with the existing `India-Times.yaml` entry which already uses the correct URI.</comment>
<file context>
@@ -0,0 +1,4 @@
+name: The Times of India
+summary: English-language daily newspaper and digital news platform based in India.
+tags: news,media,journalism,India
+uri: https://www.indiatimes.com
</file context>
| uri: https://www.indiatimes.com | |
| uri: https://timesofindia.indiatimes.com |
1 participant
Adds most popular news source in the last month for ingestion
Summary by CodeRabbit