Deckbuilder MCP Server

# Task 4.1 Analysis: Table Markdown Parsing Code Identification ## Summary Comprehensive analysis of all table markdown parsing code in the existing codebase that needs to be removed/simplified as part of the refactor to plain text table processing. ## Key Findings ### 🎯 **Core Issue**: Complex markdown parsing in table cells adds unnecessary overhead - **GOAL**: Remove all markdown parsing within table cells, treat as plain text - **BENEFIT**: 50%+ performance improvement + reduced complexity --- ## 📋 **Files Containing Table Markdown Processing** ### 1. **Core Module Files** #### `/src/deckbuilder/core/slide_builder_legacy.py` (1,352 lines - TO BE REFACTORED) **Methods with table markdown parsing:** - Line 705: `_handle_table_placeholder()` - handles table markdown in placeholders - Line 759: `_process_table_data()` - **KEY METHOD** - calls content_formatter markdown parsing - Line 788: Calls `content_formatter.parse_table_markdown_with_formatting(table_content["markdown"])` - Line 826: `_clear_table_content_from_placeholders()` - clears table markdown from placeholders - Line 863: `_is_table_markdown()` - **ENHANCED in TableHandler** ✅ - Line 893: `_detect_existing_tables()` - **ENHANCED in TableHandler** ✅ - Line 952: `_find_table_content_in_slide_data()` - **ENHANCED in TableHandler** ✅ - Line 1034: `_contains_table_content()` - **DUPLICATE - DELETE** ✅ **Status**: ✅ TableHandler created to replace these methods #### `/src/deckbuilder/core/field_processor.py` - Line 293: Comment about parsing markdown for TABLE placeholders - Line 306: `Create a real PowerPoint table from markdown table data.` - Line 308: Comment `# Parse markdown table` **Action**: Update to use plain text TableHandler #### `/src/deckbuilder/core/validation.py` - Line 399-401: Special handling for table markdown content in validation - Line 492-500: `_is_table_markdown()` method for validation **Action**: Update to use TableHandler.detect_table_content() ### 2. **Content Processing Files (HEAVY MARKDOWN PARSING)** #### `/src/deckbuilder/content/table_parser.py` - **MAJOR MARKDOWN PROCESSING** **Methods to be simplified:** - Line 13: `is_table_content()` - basic table detection (keep structure logic) - Line 44: `extract_table_markdown()` - extracts table markdown from mixed content - Line 77: `parse_markdown_table()` - **COMPLEX MARKDOWN PARSING** (remove formatting parsing) **Action**: Simplify to plain text parsing only #### `/src/deckbuilder/content/formatter.py` - **COMPLEX FORMATTING PRESERVATION** - Line 504: `parse_table_markdown_with_formatting()` - **MAJOR COMPLEXITY** - Preserves inline formatting (bold, italic, underline) - Creates formatted table data - Complex cell-by-cell markdown parsing - Lines 520-560: Complex markdown parsing logic **Action**: Replace with plain text parsing or remove entirely #### `/src/deckbuilder/content/table_integration.py` - Line 11: Imports `parse_markdown_table` from table_parser - Line 28-29: Parses table markdown - Line 187: `process_markdown_content` function - Line 194-196: Complex markdown table processing **Action**: Update to use plain text TableHandler #### `/src/deckbuilder/content/frontmatter_to_json_converter.py` - Line 11: Imports table markdown parsing functions - Line 27: `Check if content contains table markdown syntax` - Line 30-32: `_extract_table_markdown_from_content()` - Line 52-55: `_parse_markdown_table()` method - Line 94-102: `process_markdown_content()` function - Lines 196-199: Content field markdown parsing - Line 258: Comment about proper formatting vs raw markdown **Action**: Update to use plain text processing --- ## 🔍 **Detailed Analysis by Complexity Level** ### **Level 1: Simple Detection (KEEP, ENHANCE)** These methods do basic table structure detection - keep the logic but simplify: - `is_table_content()` in table_parser.py ✅ **Enhanced in TableHandler** - `_is_table_markdown()` in slide_builder_legacy.py ✅ **Enhanced in TableHandler** ### **Level 2: Complex Markdown Parsing (REMOVE/SIMPLIFY)** These methods parse markdown formatting within cells - REMOVE: - `parse_table_markdown_with_formatting()` in formatter.py - **504+ lines of complexity** - `parse_markdown_table()` in table_parser.py - **Complex cell formatting** - `extract_table_markdown()` in table_parser.py - **Markdown extraction** ### **Level 3: Integration Methods (UPDATE)** These methods integrate markdown parsing with slide creation - UPDATE: - `_process_table_data()` in slide_builder_legacy.py - **UPDATE to use TableHandler** - Methods in frontmatter_to_json_converter.py - **UPDATE to plain text** - Methods in table_integration.py - **UPDATE to use TableHandler** --- ## 🎯 **Refactor Strategy** ### **Phase 4.1 ✅ COMPLETED**: Identification - Identified 8+ files with table markdown processing - Catalogued 15+ methods requiring changes - Analyzed complexity levels ### **Phase 4.2**: Remove Complex Markdown Parsing 1. **PRIORITY 1**: Remove `parse_table_markdown_with_formatting()` from formatter.py 2. **PRIORITY 2**: Simplify `parse_markdown_table()` in table_parser.py to plain text 3. **PRIORITY 3**: Remove markdown extraction methods ### **Phase 4.3**: Update Integration Points 1. Update `field_processor.py` to use TableHandler 2. Update `validation.py` to use TableHandler.detect_table_content() 3. Update converter methods to use plain text processing ### **Phase 4.4**: Performance Validation 1. Run benchmarks comparing old vs new table processing 2. Verify 50%+ performance improvement 3. Test with large tables from test files --- ## 📊 **Expected Impact** ### **Code Reduction**: - **formatter.py**: Remove ~60 lines of complex markdown parsing - **table_parser.py**: Simplify ~100 lines to plain text processing - **slide_builder_legacy.py**: Remove duplicate methods, simplify table processing - **Total**: ~200+ lines of complex parsing logic removed ### **Performance Improvement**: - **Table Detection**: Same speed (simple structure checking) - **Table Parsing**: 50%+ faster (no markdown processing per cell) - **Table Creation**: Same speed (PowerPoint table creation unchanged) - **Memory Usage**: Reduced (no formatting objects per cell) ### **Complexity Reduction**: - **Cyclomatic Complexity**: Reduced significantly - **Maintenance**: Single table processing path - **Testing**: Simpler test cases (no markdown formatting edge cases) --- ## ✅ **Next Steps** 1. **Task 4.2**: Remove `parse_table_markdown_with_formatting()` 2. **Task 4.3**: Simplify table_parser.py to plain text only 3. **Task 4.4**: Update integration points to use TableHandler 4. **Task 4.5**: Performance validation and testing --- ## 🎯 **Success Criteria** - [ ] All markdown parsing within table cells removed - [ ] Table structure detection preserved (rows/columns) - [ ] All integration points updated to use TableHandler - [ ] Performance improvement of 50%+ validated - [ ] All existing tests pass with plain text table processing - [ ] Zero functionality regression except markdown→plain text in cells --- **STATUS**: Task 4.1 COMPLETE ✅ **READY FOR**: Task 4.2 - Remove complex markdown parsing methods