Skip to content

[arenabuddy] Update CLI documentation and fix MTGA database file extension#323

Open
gazure wants to merge 1 commit intomainfrom
ga/fix-scraper
Open

[arenabuddy] Update CLI documentation and fix MTGA database file extension#323
gazure wants to merge 1 commit intomainfrom
ga/fix-scraper

Conversation

@gazure
Copy link
Owner

@gazure gazure commented Mar 1, 2026

This pull request updates the CLI documentation and fixes a critical bug in the MTGA database detection logic.

Documentation Updates:

  • Clarified that the CLI tool is named arenabuddyctl
  • Added comprehensive examples showing how to run commands using cargo run -p arenabuddy_cli
  • Documented the new scrape-mtga command with auto-detection of MTGA install paths and custom path options
  • Added examples for the REPL and event-log commands
  • Updated help command examples to use the correct cargo syntax

Bug Fix:

  • Fixed incorrect file extension filter in MTGA database detection - changed from .rs to .mtga to properly locate Raw_CardDatabase_*.mtga files

@gazure gazure force-pushed the ga/fix-mulligan branch from b128ef0 to 7c9fc52 Compare March 1, 2026 00:21
@gazure gazure force-pushed the ga/fix-scraper branch 2 times, most recently from cdcbff8 to 661148f Compare March 1, 2026 00:48
@gazure gazure force-pushed the ga/fix-mulligan branch from 7c9fc52 to 84a5540 Compare March 1, 2026 00:48
@gazure gazure force-pushed the ga/fix-mulligan branch from 84a5540 to 30c8539 Compare March 1, 2026 01:08
@gazure gazure force-pushed the ga/fix-mulligan branch from 30c8539 to 899e9af Compare March 1, 2026 01:12
@gazure gazure force-pushed the ga/fix-mulligan branch 2 times, most recently from 844c720 to 0836d29 Compare March 1, 2026 01:16
@gazure gazure changed the base branch from ga/fix-mulligan to graphite-base/323 March 1, 2026 01:18
@gazure gazure force-pushed the graphite-base/323 branch from 0836d29 to d8ffd73 Compare March 1, 2026 01:19
@gazure gazure changed the base branch from graphite-base/323 to main March 1, 2026 01:19
Copy link
Owner Author

gazure commented Mar 1, 2026

Merge activity

  • Mar 1, 1:19 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

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