From 86b821739aa151f5cd156c2d5fa410b1f05e93c0 Mon Sep 17 00:00:00 2001 From: TekH Date: Sat, 6 Jun 2026 13:47:13 +0200 Subject: [PATCH] Add "Session" column and new entries to Mistakes History Enhanced the "Mistakes History" table in `COPILOT_CONTEXT_EN.md`: - Added a "Session" column to track when mistakes occurred. - Updated table with session numbers for existing mistakes. - Added new entries documenting recurring issues like over-engineering, ignoring revert instructions, and user feedback. - Highlighted the importance of configurability and simplicity in design. - Documented specific mistakes related to DevExpress and PDF.js. These changes improve traceability, accountability, and alignment with user preferences. --- COPILOT_CONTEXT_EN.md | 24 ++++++++++++++---------- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/COPILOT_CONTEXT_EN.md b/COPILOT_CONTEXT_EN.md index fbc33bd4..86e77b76 100644 --- a/COPILOT_CONTEXT_EN.md +++ b/COPILOT_CONTEXT_EN.md @@ -334,16 +334,20 @@ return report; ## Mistakes History — Do NOT Repeat -| Mistake | Why Wrong | -|---|---| -| `BottomMarginBand` for per-page signatures | Repeats on every page; Y offset wrong | -| `imageY = (page-1) * 1169 + ann.Y` | Inflates DetailBand; 35 pages ? 140 pages | -| `e.Graph?.PrintingSystem` in BeforePrint | `Graph` not on `CancelEventArgs` | -| `ctrl.Report?.PrintingSystem` | `PrintingSystem` not on `XtraReportBase` in WASM | -| Adding stamp endpoint to `DocumentController` | Not needed; stamping is done client-side in ReceiverUI | -| iText7 via API (server-side) | Unnecessary; iText7 runs fine in WASM directly | -| **PDF.js: Hardcoded quality values** | **Use appsettings.json for configurability** | -| **PDF.js: Hardcoded zoom step (1%)** | **Too granular; use configurable percentage** | +| Mistake | Why Wrong | Session | +|---|---|---| +| `BottomMarginBand` for per-page signatures | Repeats on every page; Y offset wrong | 4 | +| `imageY = (page-1) * 1169 + ann.Y` | Inflates DetailBand; 35 pages ? 140 pages | 8 | +| `e.Graph?.PrintingSystem` in BeforePrint | `Graph` not on `CancelEventArgs` | 5 | +| `ctrl.Report?.PrintingSystem` | `PrintingSystem` not on `XtraReportBase` in WASM | — | +| Adding stamp endpoint to `DocumentController` | Not needed; stamping is done client-side in ReceiverUI | — | +| iText7 via API (server-side) | Unnecessary; iText7 runs fine in WASM directly | 10 | +| **PDF.js: Hardcoded quality values** | **Use appsettings.json for configurability** | **11** | +| **PDF.js: Hardcoded zoom step (1%)** | **Too granular; use configurable percentage** | **11** | +| **Toolbar: Complex left/center/right layout** | **User wants simple horizontal layout; failed multiple times to implement** | **11** | +| **Zoom label: Badge style (gradient/border/padding)** | **Over-designed; user prefers simple text label** | **11** | +| **Attempting to "improve" simple designs** | **User requests simplicity; AI keeps over-engineering** | **11** | +| **Ignoring explicit "revert" instructions** | **User said revert toolbar, AI tried to fix CSS instead of reverting HTML structure** | **11** | ---