Skip to content

Comments

feat: incremental extraction#55

Merged
Crauzer merged 1 commit intomainfrom
incremental-extraction
Feb 1, 2026
Merged

feat: incremental extraction#55
Crauzer merged 1 commit intomainfrom
incremental-extraction

Conversation

@Crauzer
Copy link
Member

@Crauzer Crauzer commented Feb 1, 2026

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds incremental extraction support to wadtools extract by skipping already-extracted outputs by default, with an opt-in flag to overwrite.

Changes:

  • Added --overwrite flag to the extract subcommand and plumbed it through the CLI argument layer.
  • Updated the extractor to skip writing when the destination file already exists (unless overwriting), and to report skipped-existing counts.
  • Updated README usage/docs to describe the new behavior and flag.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
crates/wadtools/src/main.rs Adds the --overwrite CLI flag and passes it into extraction args.
crates/wadtools/src/extractor.rs Implements skip-existing/overwrite behavior during chunk writes and returns extracted + skipped counts.
crates/wadtools/src/commands/extract.rs Adapts to the new extractor return type and logs skipped-existing counts.
README.md Documents --overwrite and provides examples for re-extraction behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Crauzer Crauzer force-pushed the incremental-extraction branch from 9412e06 to e92e0ae Compare February 1, 2026 12:55
@Crauzer Crauzer merged commit 6bd57a8 into main Feb 1, 2026
3 checks passed
@Crauzer Crauzer deleted the incremental-extraction branch February 1, 2026 12:56
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