feat: add rectangle/page UI, backend full-page tighten fix, and expanded docs#67
Merged
theodubus merged 1 commit intoFeb 23, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Description
README.mdto explain redaction theory, feature set, limits (including flattened/scanned PDFs), media placeholders and roadmap, addCONTRIBUTING.md, and adddocs/DEPLOYMENT.mdwith Docker and packaging guidance, and clarifydocs/SECURITY.mdinvariants and recommendations._tighten_rect_verticalinbackend/app/redaction.pyto skip aggressive vertical tightening for large rects (h > 24.0) so full-page/very-tall rectangles are preserved.backend/tests/test_redact_rectangles.py::test_redact_rectangles_full_page_rect_redacts_targetthat posts a full-page rectangle and asserts the auditpassand that the secret is removed.frontend/src/App.tsx,frontend/src/components/PdfViewer.tsx,frontend/src/components/Rules/*,frontend/src/types/uiRules.ts,frontend/src/api.ts, locales, and styles; the UI now supports drawing rectangles, adding page rules, preview badges, tracking current page and page sizes, and sendingapply_images,apply_graphics,sanitize_metadata,remove_annotations, andremove_attachmentsin the/api/redact/applypayload when applicable.Testing
git diff --checkwhich passed with no whitespace/patch issues.rg) and manual doc sanity checks on the changed docs which passed.test_redact_rectangles_full_page_rect_redacts_targetwas added but not run here.Codex Task