Auto: fix/youtube-cleanup-remnants #11

Merged

claude-bot merged 2 commits from fix/youtube-cleanup-remnants into main

2026-03-01 12:58:46 -07:00

claude-bot commented

2026-03-01 12:51:26 -07:00

Collaborator

Automated PR for branch fix/youtube-cleanup-remnants.

Automated PR for branch `fix/youtube-cleanup-remnants`.

claude-bot added 1 commit

2026-03-01 12:51:26 -07:00

Clean up remaining YouTube remnants from docs and dead code

Auto PR Review / review (push) Failing after 1m40s

Details

3e74e27cb5

Update README.md and DEPLOYMENT.md to remove YouTube references,
SUPADATA_API_KEY, and YouTube tier limits. Remove dead parent_request_id
column from database schema and log function. Remove unused retry_async
and retry_sync decorators from utils.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude-bot commented

2026-03-01 12:52:28 -07:00

Author

Collaborator

Changes Requested

Clean removal of YouTube summarization remnants from docs, database schema, and utility code. One issue found: httpx was removed from the README's dependency list but is still actively used throughout the codebase (webhook.py, feedback handler, tests) and remains in requirements.txt.

Warnings

README.md: The README diff removes httpx from the Tech Stack section, but httpx is still an active dependency — it's in requirements.txt and used in webhook.py, handlers/feedback.py, and multiple test files. This makes the README inaccurate. Either keep httpx listed under Tech Stack, or if the intent is to remove httpx entirely, that's a much larger change.

Suggestions

README.md: Minor formatting nit: the removal of the httpx line also removed the blank line before the '## Development' heading, merging the Tech Stack list directly into the next section. Consider keeping a blank line for readability.

## Changes Requested Clean removal of YouTube summarization remnants from docs, database schema, and utility code. One issue found: `httpx` was removed from the README's dependency list but is still actively used throughout the codebase (webhook.py, feedback handler, tests) and remains in requirements.txt. ### Warnings - **README.md**: The README diff removes httpx from the Tech Stack section, but httpx is still an active dependency — it's in requirements.txt and used in webhook.py, handlers/feedback.py, and multiple test files. This makes the README inaccurate. Either keep httpx listed under Tech Stack, or if the intent is to remove httpx entirely, that's a much larger change. ### Suggestions - **README.md**: Minor formatting nit: the removal of the httpx line also removed the blank line before the '## Development' heading, merging the Tech Stack list directly into the next section. Consider keeping a blank line for readability.

bittabola added 1 commit

2026-03-01 12:56:45 -07:00

fix: restore httpx in Tech Stack and add missing blank line

Auto PR Review / review (push) Successful in 2m0s

Details

564d2688c0

httpx is still an active dependency used by webhook.py, feedback handler,
and tests. Also restores missing blank line before Development section.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude-bot commented

2026-03-01 12:58:45 -07:00

Author

Collaborator

Approved

Clean removal of YouTube summarization feature remnants. All references to parent_request_id, retry_async/retry_sync, SUPADATA_API_KEY, and YouTube-related docs/code have been properly cleaned up with no remaining dependencies. The only YouTube mention left is a benign comment in admin_dashboard.py:423 noting the feature was removed, which is fine as historical context.

Suggestions

admin_dashboard.py: Consider removing or simplifying the comment '# Calculate amortized revenue (translation only; YouTube feature removed)' at line 423 — the parenthetical about YouTube removal is transitional context that will become stale. A simpler comment like '# Calculate amortized revenue' would age better.

## Approved Clean removal of YouTube summarization feature remnants. All references to `parent_request_id`, `retry_async`/`retry_sync`, `SUPADATA_API_KEY`, and YouTube-related docs/code have been properly cleaned up with no remaining dependencies. The only YouTube mention left is a benign comment in admin_dashboard.py:423 noting the feature was removed, which is fine as historical context. ### Suggestions - **admin_dashboard.py**: Consider removing or simplifying the comment '# Calculate amortized revenue (translation only; YouTube feature removed)' at line 423 — the parenthetical about YouTube removal is transitional context that will become stale. A simpler comment like '# Calculate amortized revenue' would age better.

claude-bot merged commit 89775ef919 into main

2026-03-01 12:58:46 -07:00

claude-bot deleted branch fix/youtube-cleanup-remnants

2026-03-01 12:58:46 -07:00

claude-bot referenced this pull request from a commit

2026-03-01 12:58:47 -07:00

Auto: fix/youtube-cleanup-remnants (#11)