Skip to content

Ensure distinct --batch and --checkpoint files#1931

Merged
rolandwalker merged 1 commit into
mainfrom
RW/ensure-batch-checkpoint-distinct-files
Jun 13, 2026
Merged

Ensure distinct --batch and --checkpoint files#1931
rolandwalker merged 1 commit into
mainfrom
RW/ensure-batch-checkpoint-distinct-files

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

Ensure distinct --batch and --checkpoint files, to avoid --checkpoint logging SQL back to the source --batch.

The --checkpoint argument is converted to a string, and the opened checkpoint is cached to avoid repeatedly seeking to the end of a large file.

The change to a string is visible in Click's generated helpdoc.

Incidentally improve the helpdoc for --resume, which looks like it may have been mangled by an agent during refactoring.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Jun 13, 2026
to avoid --checkpoint logging SQL back to the source --batch.

The --checkpoint argument is converted to a string, and the opened
checkpoint is cached to avoid repeatedly seeking to the end of a large
file.

The change to a string is visible in Click's generated helpdoc.

Incidentally improve the helpdoc for --resume, which looks like it may
have been mangled by an agent during refactoring.
@rolandwalker rolandwalker force-pushed the RW/ensure-batch-checkpoint-distinct-files branch from b55d091 to dc8c098 Compare June 13, 2026 19:30
@rolandwalker rolandwalker merged commit 5057282 into main Jun 13, 2026
11 checks passed
@rolandwalker rolandwalker deleted the RW/ensure-batch-checkpoint-distinct-files branch June 13, 2026 20:27
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