Skip to content

Improve import review recovery and navigation performance - #157

Merged
DeusExTaco merged 14 commits into
developfrom
feature/utility-queue-test-isolation
Sep 27, 2026
Merged

DeusExTaco merged 14 commits into
developfrom
feature/utility-queue-test-isolation

Conversation

@DeusExTaco

Copy link
Copy Markdown
Contributor

Summary

  • Simplifies Step 3 import review while preserving the recovery, safety, mixed-folder, and story-arc workflows proven against large real-world libraries.
  • Strengthens deferred-file recovery, source rechecks, issue assignment, ComicInfo/XML handling, and catalog download progress reporting.
  • Adds detailed navigation timing, actionable health diagnostics, set-based Library summaries, catalog path reuse, and disposable Ethan/BigRedOne scale benchmarks.

Why

Large and messy Mylar libraries need a clearer review surface without losing the recovery safeguards built for corrupted, misplaced, duplicated, stale, and oversized files. Large collections also need measurable navigation diagnostics and fewer avoidable Library queries.

Validation

  • make ci-full
  • Main suite: 12,124 passed, 13 skipped, 1 xfailed; 91.20% coverage
  • Accessibility: 25 passed
  • Chromium E2E: 578 passed, 2 skipped
  • Firefox E2E: 578 passed, 2 skipped
  • Docker build, container security runtime, Grype gate, health check, and 7 smoke tests passed
  • git diff --check

Notes

  • Source-file safety and in-place import ownership rules remain intact.
  • The branch intentionally retains detailed follow-up handling rather than hiding unresolved import work.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 27, 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-27T10:32:27.305263Z 2633446 Manual request
ℹ️ 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: 9f9d2b6438

ℹ️ 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/services/import_review_source_actions.py Outdated
Comment thread src/pullbox/services/import_review_source_actions.py
@DeusExTaco

Copy link
Copy Markdown
Contributor Author

@codex review

@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: 26334461ef

ℹ️ 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/services/import_review_actions.py
Comment thread src/pullbox/services/import_story_arc_review.py
Comment thread src/pullbox/providers/download/nzbget.py Outdated
@DeusExTaco DeusExTaco added the ci:full Run the full CircleCI PR gate label Sep 27, 2026
@DeusExTaco
DeusExTaco merged commit a513fc2 into develop Sep 27, 2026
33 of 56 checks passed
@DeusExTaco
DeusExTaco deleted the feature/utility-queue-test-isolation branch September 27, 2026 11:05
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