Repository navigation
feat: qualify metadata editing and recovery for the v2 edge preview - #175
Conversation
Require explicit approval against server-owned issue/user route grants, keep incomplete and unsafe results closed, and prevent duplicate queue intents. Preserve replacement confirmation and normal import validation. TDD: 7 demonstrated RED failures; 115 focused API/unit regressions and 6 three-theme/mobile browser cases pass. No full CI or PR.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3d3b4b44eb
ℹ️ 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".
|
Release qualification update: make ci-full passed after the AirDC++ ownership fix, including the full test suite, accessibility, Chromium, Firefox, image security/runtime checks, and Docker smoke. SQLite/PostgreSQL ownership interleavings also passed in 78 focused tests. Commit 8e8a8c5 only makes the file-proof test wait for an actual filesystem ctime change on Linux; it changes no production code or safety policy. Full GitHub checks are rerunning on that exact head. Anna verification remains excluded. |
|
The remaining Linux browser failures were screenshot capture paths hard-coded to /private/tmp, after their UI assertions had passed. Commit 83d40a2 uses pytest per-test temporary directories; both affected modules pass 44 Chromium/Firefox cases locally. All Python versions passed on the preceding head. The post-review commits are test-only; no additional review pass is requested. |
|
Release qualification update:
|
|
Edge publication completed successfully from merged develop commit All four exact-commit manual validation workflows passed. Docker Release run 39667906190 built AMD64 and ARM64, verified platform runtime protections and signatures, and promoted both registries to the same signed digest:
Images: Release run: https://github.com/pullboxapp/pullbox/actions/runs/39667906190 Anna verification is excluded. No production deployment was performed. |
Summary
series.jsonediting, and explicit selected-field propagation into comic files. Keep paired XML output reconciled and preserve later edits during import rollback.Validation
make ci-fullpassed: main suite 16,477 passed, 1,772 skipped, one expected failure; 91.74% coverage.Release Boundaries
2.0.0-devedge preview, not the completed v2.0 general release. Existing feature-flag defaults remain unchanged.