Skip to content

bookmyshow cli#22

Draft
Mr-Ashish wants to merge 2 commits into
agentrhq:mainfrom
Mr-Ashish:main
Draft

bookmyshow cli#22
Mr-Ashish wants to merge 2 commits into
agentrhq:mainfrom
Mr-Ashish:main

Conversation

@Mr-Ashish

Copy link
Copy Markdown

Description

Related issue:

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🌐 New site adapter
  • 📝 Documentation
  • ♻️ Refactor
  • 🔧 CI / build / tooling

Checklist

  • I ran the checks relevant to this PR
  • I updated tests or docs if needed
  • I included output or screenshots when useful

Adapter Notes

  • Updated generated or lean docs when command discoverability changed
  • Used positional args for the command's primary subject unless a named flag is clearly better
  • Normalized expected adapter failures to CliError subclasses instead of raw Error

Screenshots / Output

Mr-Ashish added 2 commits July 4, 2026 14:54
Add a full BookMyShow adapter with 6 commands:
- bookmyshow cities — list Indian cities/regions
- bookmyshow movies <city> — now-showing movies via SSR extraction
- bookmyshow upcoming <city> — upcoming movies via SSR extraction
- bookmyshow events <city> — events listing
- bookmyshow search <query> — search across movies/events
- bookmyshow movie <code> — movie detail view

Key design decisions:
- Uses browser bridge (Strategy.COOKIE) to bypass Cloudflare JS challenge
- Movies/upcoming extract data from SSR __INITIAL_STATE__ since API
  endpoints were removed
- Cities uses new /api/explore/v1/discover/regions endpoint
- Shared utils: bmsFetch, bmsDiscoverPage, extractMovieCards,
  field accessors, response validators, retry with backoff
- makeMovieListingCommand factory eliminates duplication between
  movies.js and upcoming.js
- 64 adapter-specific tests covering all commands, edge cases,
  and error paths
@Mr-Ashish Mr-Ashish marked this pull request as draft July 4, 2026 09:31
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.

1 participant