Run ruff across codebase, add lint CI, fix gitignore

- Fix pyproject.toml: remove build-system (not a package), fix
  dependencies placement, add SIM108/N812 to ignore list
- Run ruff check --fix and ruff format across all 22 generators
- Fix SIM103 (return condition directly) in 3 files
- Fix SIM105 (contextlib.suppress) in anthropic_news_blog.py
- Add .github/workflows/lint.yml: ruff check on PRs and pushes
  to main (only triggers on .py and pyproject.toml changes)
- Add feeds/.gitkeep to preserve feeds directory
- Fix stray G character in .gitignore line 2
- Update uv.lock

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: OliverB

.github/workflows/lint.yml

@@ -0,0 +1,37 @@
+name: Lint
+
+on:
+  pull_request:
+    paths:
+      - "**.py"
+      - "pyproject.toml"
+  push:
+    branches: [main]
+    paths:
+      - "**.py"
+      - "pyproject.toml"
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    timeout-minutes: 5
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v6
+
+      - name: Set up Python
+        uses: actions/setup-python@v6
+        with:
+          python-version: "3.11"
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v8.0.0
+        with:
+          enable-cache: true
+
+      - name: Lint and format check
+        run: |
+          uv sync --group dev
+          uv run ruff check .
+          uv run ruff format --check .

.github/workflows/run_feeds.yml

@@ -32,8 +32,7 @@ jobs:
         timeout-minutes: 20
         run: |
           set -e
-          uv venv
-          uv pip install -r requirements.txt
+          uv sync
           uv run feed_generators/run_all_feeds.py --skip-selenium
 
       - name: Commit and push feeds

.github/workflows/run_selenium_feeds.yml

@@ -38,8 +38,7 @@ jobs:
         timeout-minutes: 45
         run: |
           set -e
-          uv venv
-          uv pip install -r requirements.txt
+          uv sync
           uv run feed_generators/run_all_feeds.py --selenium-only
 
       - name: Commit and push feeds

.github/workflows/test_feed.yml

@@ -25,8 +25,7 @@ jobs:
       - name: Install dependencies and run test feed
         run: |
           set -e
-          uv venv
-          uv pip install -r requirements.txt
+          uv sync
           uv run feed_generators/ollama_blog.py
 
       - name: Upload test feed artifact

.gitignore

@@ -1,5 +1,4 @@
 # Byte-compiled / optimized / DLL files
-G
 __pycache__/
 *.py[cod]
 *$py.class

AGENTS.md

@@ -36,7 +36,9 @@ make feeds_generate_all   # Run all feed generators
 make feeds_<name>         # Run specific feed (e.g., feeds_ollama, feeds_anthropic_news)
 
 # Development
-make dev_format           # Format code with black and isort
+make dev_lint             # Check code with ruff
+make dev_lint_fix         # Auto-fix and format with ruff
+make dev_format           # Alias for dev_lint_fix
 make dev_test_feed        # Run test feed generator
 
 # Run single generator directly

feed_generators/anthropic_changelog_claude_code.py

@@ -3,6 +3,7 @@
 
 import pytz
 from feedgen.feed import FeedGenerator
+
 from utils import fetch_page, save_rss_feed, setup_feed_links, setup_logging
 
 logger = setup_logging()
@@ -17,7 +18,7 @@ def fetch_changelog_content(
     try:
         return fetch_page(url)
     except Exception as e:
-        logger.error(f"Error fetching changelog content: {str(e)}")
+        logger.error(f"Error fetching changelog content: {e!s}")
         raise
 
 
@@ -40,11 +41,7 @@ def parse_changelog_markdown(markdown_content, max_versions=50):
                 if current_version and current_changes:
                     version_anchor = current_version.replace(".", "")
                     # Create HTML list for description
-                    description_html = (
-                        "<ul>"
-                        + "".join(f"<li>{change}</li>" for change in current_changes)
-                        + "</ul>"
-                    )
+                    description_html = "<ul>" + "".join(f"<li>{change}</li>" for change in current_changes) + "</ul>"
                     item = {
                         "title": f"v{current_version}",
                         "link": f"https://github.com/anthropics/claude-code/blob/main/CHANGELOG.md#{version_anchor}",
@@ -63,13 +60,9 @@ def parse_changelog_markdown(markdown_content, max_versions=50):
                 date_str = version_match.group(2)
                 if date_str:
                     try:
-                        current_date = datetime.strptime(
-                            date_str.strip(), "%Y-%m-%d"
-                        ).replace(tzinfo=pytz.UTC)
+                        current_date = datetime.strptime(date_str.strip(), "%Y-%m-%d").replace(tzinfo=pytz.UTC)
                     except ValueError:
-                        logger.warning(
-                            f"Could not parse date '{date_str}' for version {current_version}"
-                        )
+                        logger.warning(f"Could not parse date '{date_str}' for version {current_version}")
                 current_changes = []
                 continue
 
@@ -82,11 +75,7 @@ def parse_changelog_markdown(markdown_content, max_versions=50):
         # Don't forget the last version (if we haven't hit the limit)
         if current_version and current_changes and len(items) < max_versions:
             version_anchor = current_version.replace(".", "")
-            description_html = (
-                "<ul>"
-                + "".join(f"<li>{change}</li>" for change in current_changes)
-                + "</ul>"
-            )
+            description_html = "<ul>" + "".join(f"<li>{change}</li>" for change in current_changes) + "</ul>"
             item = {
                 "title": f"v{current_version}",
                 "link": f"https://github.com/anthropics/claude-code/blob/main/CHANGELOG.md#{version_anchor}",
@@ -101,7 +90,7 @@ def parse_changelog_markdown(markdown_content, max_versions=50):
         return items
 
     except Exception as e:
-        logger.error(f"Error parsing markdown content: {str(e)}")
+        logger.error(f"Error parsing markdown content: {e!s}")
         raise
 
 
@@ -132,7 +121,7 @@ def generate_rss_feed(items, feed_name=FEED_NAME):
         return fg
 
     except Exception as e:
-        logger.error(f"Error generating RSS feed: {str(e)}")
+        logger.error(f"Error generating RSS feed: {e!s}")
         raise
 
 
@@ -152,7 +141,7 @@ def main(feed_name=FEED_NAME):
         return True
 
     except Exception as e:
-        logger.error(f"Failed to generate RSS feed: {str(e)}")
+        logger.error(f"Failed to generate RSS feed: {e!s}")
         return False
 
 

feed_generators/anthropic_eng_blog.py

@@ -4,8 +4,8 @@
 import pytz
 from bs4 import BeautifulSoup
 from feedgen.feed import FeedGenerator
-from utils import (fetch_page, save_rss_feed, setup_feed_links, setup_logging,
-                   sort_posts_for_feed)
+
+from utils import fetch_page, save_rss_feed, setup_feed_links, setup_logging, sort_posts_for_feed
 
 logger = setup_logging()
 
@@ -18,7 +18,7 @@ def fetch_engineering_content(url=BLOG_URL):
     try:
         return fetch_page(url)
     except Exception as e:
-        logger.error(f"Error fetching engineering content: {str(e)}")
+        logger.error(f"Error fetching engineering content: {e!s}")
         raise
 
 
@@ -28,9 +28,7 @@ def validate_article(article):
         return False
     if not article.get("link") or not article["link"].startswith("http"):
         return False
-    if not article.get("date"):
-        return False
-    return True
+    return bool(article.get("date"))
 
 
 def parse_engineering_html(html_content):
@@ -42,18 +40,12 @@ def parse_engineering_html(html_content):
         # Find the Next.js script tag containing article data
         script_tag = None
         for script in soup.find_all("script"):
-            if (
-                script.string
-                and "publishedOn" in script.string
-                and "engineeringArticle" in script.string
-            ):
+            if script.string and "publishedOn" in script.string and "engineeringArticle" in script.string:
                 script_tag = script
                 break
 
         if not script_tag:
-            logger.error(
-                "Could not find Next.js data script containing article information"
-            )
+            logger.error("Could not find Next.js data script containing article information")
             return []
 
         script_content = script_tag.string
@@ -82,26 +74,16 @@ def parse_engineering_html(html_content):
 
                 # Extract title and summary (they appear AFTER the slug in the data)
                 # Use negative lookbehind to handle escaped quotes correctly
-                title_match = re.search(
-                    r'\\"title\\":\\"(.*?)(?<!\\)\\"', search_section
-                )
-                title = (
-                    title_match.group(1)
-                    if title_match
-                    else slug.replace("-", " ").title()
-                )
+                title_match = re.search(r'\\"title\\":\\"(.*?)(?<!\\)\\"', search_section)
+                title = title_match.group(1) if title_match else slug.replace("-", " ").title()
                 # Unescape the title using re.sub to handle all escaped characters
                 title = re.sub(r"\\(.)", r"\1", title) if title else title
 
                 # Extract summary/description
-                summary_match = re.search(
-                    r'\\"summary\\":\\"(.*?)(?<!\\)\\"', search_section
-                )
+                summary_match = re.search(r'\\"summary\\":\\"(.*?)(?<!\\)\\"', search_section)
                 description = summary_match.group(1) if summary_match else title
                 # Unescape the description
-                description = (
-                    re.sub(r"\\(.)", r"\1", description) if description else description
-                )
+                description = re.sub(r"\\(.)", r"\1", description) if description else description
 
                 # Parse the date
                 date = datetime.strptime(published_date, "%Y-%m-%d")
@@ -120,14 +102,14 @@ def parse_engineering_html(html_content):
                     logger.info(f"Found article: {title} ({published_date})")
 
             except Exception as e:
-                logger.warning(f"Error parsing article {slug}: {str(e)}")
+                logger.warning(f"Error parsing article {slug}: {e!s}")
                 continue
 
         logger.info(f"Successfully parsed {len(articles)} articles from JSON data")
         return articles
 
     except Exception as e:
-        logger.error(f"Error parsing HTML content: {str(e)}")
+        logger.error(f"Error parsing HTML content: {e!s}")
         raise
 
 
@@ -136,9 +118,7 @@ def generate_rss_feed(articles, feed_name=FEED_NAME):
     try:
         fg = FeedGenerator()
         fg.title("Anthropic Engineering Blog")
-        fg.description(
-            "Latest engineering articles and insights from Anthropic's engineering team"
-        )
+        fg.description("Latest engineering articles and insights from Anthropic's engineering team")
         setup_feed_links(fg, BLOG_URL, feed_name)
         fg.language("en")
 
@@ -164,7 +144,7 @@ def generate_rss_feed(articles, feed_name=FEED_NAME):
         return fg
 
     except Exception as e:
-        logger.error(f"Error generating RSS feed: {str(e)}")
+        logger.error(f"Error generating RSS feed: {e!s}")
         raise
 
 
@@ -191,7 +171,7 @@ def main(feed_name=FEED_NAME):
         return True
 
     except Exception as e:
-        logger.error(f"Failed to generate RSS feed: {str(e)}")
+        logger.error(f"Failed to generate RSS feed: {e!s}")
         return False