Skip to content

Remove twitter-digest browser collection#64

Merged
Will-Guan merged 1 commit into
mainfrom
twitter-digest-api-only
Jul 8, 2026
Merged

Remove twitter-digest browser collection#64
Will-Guan merged 1 commit into
mainfrom
twitter-digest-api-only

Conversation

@boboliu-1010

Copy link
Copy Markdown
Collaborator

Summary

  • make twitter-digest API-only: auto/api both use API and browser source exits with a clear error
  • remove browser collection implementation files, including CDP, DOM scripts, DM scraper, and public scraper
  • update install/docs/agent prompt so the skill no longer checks for browser prerequisites or suggests browser fallback

Validation

  • python3 -m compileall -q twitter-digest/scripts
  • sh -n twitter-digest/install.sh && sh -n twitter-digest/uninstall.sh
  • python3 twitter-digest/scripts/install.py --client codex --dry-run
  • python3 twitter-digest/scripts/run_daily_digest.py --help
  • python3 twitter-digest/scripts/run_daily_digest.py --source browser --out /tmp/twitter-digest-api-only-test
  • TWITTER_DIGEST_NO_INSTALL_REDIRECT=1 X_BEARER_TOKEN=invalid python3 twitter-digest/scripts/run_daily_digest.py --source auto --out /tmp/twitter-digest-api-only-test --max-public-items 1 --dm-max-events 1

@boboliu-1010 boboliu-1010 force-pushed the twitter-digest-api-only branch 7 times, most recently from 17841ff to 5fe6111 Compare July 7, 2026 08:05
@boboliu-1010 boboliu-1010 marked this pull request as ready for review July 7, 2026 08:06
@boboliu-1010 boboliu-1010 force-pushed the twitter-digest-api-only branch from 5fe6111 to 43ad54e Compare July 8, 2026 02:39
@Will-Guan Will-Guan merged commit e848456 into main Jul 8, 2026
1 of 2 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