From 31bf64a6f2cbad3b92ec135db1dae28bc6a9bb1b Mon Sep 17 00:00:00 2001 From: TekH Date: Mon, 19 Oct 2026 14:43:11 +0200 Subject: [PATCH] Refactor TODOs for licensing and maintainability Added a critical pre-commercial release TODO to remove `SixLabors.ImageSharp` and `QRCoder-ImageSharp` dependencies and replace them with permissive-license alternatives. Updated TODOs to focus on modularizing `EnvelopeSenderPage.razor` and other server pages by splitting large components, centralizing shared logic, and improving maintainability. Included a TODO for a senior-level performance review of sender/receiver flows to address route transitions, loading logic, and perceived latency issues. Removed outdated TODO on reducing `EnvelopeSenderPage.razor` complexity. --- AGENTS.md | 1 + 1 file changed, 1 insertion(+) diff --git a/AGENTS.md b/AGENTS.md index 8b7e4248..ec939e46 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -268,6 +268,7 @@ Manual testing workflow: ## Technical Debt TODOs +- **CRITICAL (Licensing / pre-commercial release):** Remove all `SixLabors.ImageSharp` and `QRCoder-ImageSharp` package dependencies from production/runtime projects before commercial release. Replace remaining QR/image paths with permissive-license alternatives (no copyleft/share-source obligations), run repo-wide package audit (`*.csproj`), and add CI guardrails to block reintroduction. - Refactor `EnvelopeGenerator.Server/EnvelopeGenerator.Server/Components/Pages/EnvelopeSenderPage.razor` to reduce complexity. - Move sender dashboard business/status logic into dedicated services and extension methods (for example effective status resolution, tab classification, receiver signed/rejected detection, and grid-layout filter sanitation). - Keep `EnvelopeSenderPage.razor` focused on UI composition/state orchestration; avoid embedding heavy domain logic directly in the Razor component.