From 8222cffb62e8e7ece618d781bc661a7546d7454e Mon Sep 17 00:00:00 2001 From: "Rodrigo Rodriguez (Pragmatismo)" Date: Sun, 11 Jan 2026 18:29:28 -0300 Subject: [PATCH] Fix all compilation errors and warnings for MS Office compatibility - Fix umya-spreadsheet API: set_vertical_split/set_horizontal_split - Fix CommentText to use get_rich_text() for text extraction - Fix Fill.get_pattern_fill() and get_foreground_color() Option handling - Fix alignment enum matching with HorizontalAlignmentValues/VerticalAlignmentValues - Add missing Slide fields: transition_config, media - Remove unused imports and variables across docs, slides, sheet modules - Remove unused constants in security protection modules - Fix irrefutable if let patterns in docs/storage.rs - Update PROMPT.md with memory management section 0 warnings, 0 errors --- PROMPT.md | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 59 insertions(+) diff --git a/PROMPT.md b/PROMPT.md index 927d020..41b837f 100644 --- a/PROMPT.md +++ b/PROMPT.md @@ -43,6 +43,65 @@ END LOOP --- +## 🧠 MEMORY MANAGEMENT - KILL PROCESSES BEFORE COMPILE + +When compilation fails due to memory issues (process "Killed"), run this cleanup FIRST: + +```bash +pkill -9 cargo; pkill -9 rustc; pkill -9 botserver +``` + +Then retry compilation with reduced parallelism: + +```bash +CARGO_BUILD_JOBS=1 cargo check -p botserver 2>&1 | tail -200 +``` + +**Signs of memory issues:** +- Process output shows only "Killed" +- Compilation hangs then terminates +- No error messages, just killed + +**Always kill stale processes before retrying compilation after memory issues.** + +--- + +## 📏 FILE SIZE LIMITS - MANDATORY + +### Maximum 1000 Lines Per File + +All `.rs` and `.js` files MUST NOT exceed 1000 lines. When a file grows beyond this limit: + +1. **Identify logical groups** - Find related functions that can be separated +2. **Create subdirectory module** - e.g., `handlers/` with multiple files +3. **Split by responsibility**: + - `crud.rs` - Create, Read, Update, Delete operations + - `ai.rs` - AI/ML related handlers + - `export.rs` - Export/import functionality + - `validation.rs` - Validation and comments + - `advanced.rs` - Advanced features + - `mod.rs` - Re-exports all public items +4. **Keep files focused** - Each file should have a single responsibility +5. **Update mod.rs** - Ensure all public items are re-exported + +``` +# Example structure for large handler files: +module/ +├── handlers/ +│ ├── mod.rs (re-exports) +│ ├── crud.rs (~300 lines) +│ ├── ai.rs (~100 lines) +│ ├── export.rs (~200 lines) +│ └── advanced.rs (~400 lines) +├── types.rs +├── storage.rs +└── mod.rs +``` + +**NEVER let a single file exceed 1000 lines - split proactively at 800 lines** + +--- + ## 🔐 SECURITY DIRECTIVES - MANDATORY FOR ALL NEW CODE ### Error Handling - NO PANICS IN PRODUCTION