Skip to content

馃悵 Preserve Enhance errors and fix MIME-handler return navigation - #203

Open
JFWooten4 wants to merge 3 commits into
mainfrom
fix-enhancer-mime-and-errors
Open

JFWooten4 wants to merge 3 commits into
mainfrom
fix-enhancer-mime-and-errors

Conversation

@JFWooten4

@JFWooten4 JFWooten4 commented Oct 18, 2026 •

Copy link
Copy Markdown
Member

Problem

Enhance can stay at 0% then disappear without a persistent error because its native-host failures are shown only as short-lived toasts. Under Chrome's PDF MIME handler, successful enhancement may also append ?url=file://... to the SEC PDF address instead of reopening the extension viewer.

Changes

  • Keep a persistent enhancement error notice with the failed stage, Copy error and Dismiss controls.
  • Clear previous errors when retrying.
  • Use the extension viewer URL for enhanced files and retain the selected page.
  • Provision the Enhancer-owned .venv during explicit local native-host setup when the selected checkout has setup-python-env.sh; leave older forks compatible.
  • Prefer the same isolated Enhancer Python for native-host metadata stamping, with fallback if no project venv exists.
  • Add browser-route, upload failure and interpreter selection regression tests.

Related

The Enhancer-side Python venv setup, preferred interpreter and shell startup fixes are in PDF-magic/enhancer#12. Reported against https://www.sec.gov/files/rules/sro/dtc/2009/dtcresponse101509.pdf.

The exact failure on the user's local installation remains unverified; the viewer now exposes errors instead of hiding them.

JFWooten4 and others added 3 commits October 18, 2026 17:32
Keep native-host and OCR failure details visible with copy and dismiss controls, and navigate successful enhancement results to the extension viewer rather than appending a query to the source PDF URL. Add regression coverage for SEC MIME-handler tabs and failed uploads.

Co-authored-by: Codex Web <noreply@openai.com>
Include the setup close button and tools menu controls referenced by enhancer initialization so the regression fixture faithfully mounts the module.

Co-authored-by: Codex Web <noreply@openai.com>
Run the Enhancer's isolated Python setup when installing its native host and prefer that environment for metadata stamping. Add host regression coverage and explain the setup behavior.

Co-authored-by: Codex Web <noreply@openai.com>
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