Skip to content

Add completed import recovery cleanup - #138

Merged
DeusExTaco merged 2 commits into
developfrom
feature/import-recovery-cleanup
Sep 5, 2026
Merged

Add completed import recovery cleanup#138
DeusExTaco merged 2 commits into
developfrom
feature/import-recovery-cleanup

Conversation

@DeusExTaco

Copy link
Copy Markdown
Contributor

Summary

  • add safe, previewed bulk recovery actions for completed collection imports
  • categorize and paginate large result sets instead of rendering tens of thousands of rows
  • add non-destructive archive and restore controls for completed import history
  • preserve source files and require explicit confirmation for every bulk mutation

BigRedOne validation

  • exercised the recovery scopes against a disposable migrated copy of the 184,190-file diagnostic database
  • dismissed 38,383 stale Mylar references, skipped 707 probable cover files, and skipped 42 unusable files with zero rows left in those scopes
  • verified 9,603 safe conflict groups, 97 overrideable oversized files, and 350 retryable source-inspection rows
  • verified archive and restore preserve all 184,190 import rows

Validation

  • make ci-full
  • 10,630 Python tests passed with coverage above the 90% gate
  • accessibility, Chromium, Firefox, Docker build, container security scan, and smoke tests passed

@DeusExTaco DeusExTaco added the ci:full Run the full CircleCI PR gate label Sep 5, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-05T01:00:09.054835Z 4d1210c PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4d1210c708

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/pullbox/api/v1/import_completed_cleanup.py Outdated
Comment thread src/pullbox/services/import_completed_cleanup.py
Comment thread src/pullbox/ui/import_results_context.py Outdated
@DeusExTaco
DeusExTaco merged commit caa8280 into develop Sep 5, 2026
50 of 54 checks passed
@DeusExTaco
DeusExTaco deleted the feature/import-recovery-cleanup branch September 5, 2026 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full Run the full CircleCI PR gate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant