Fix 3 root cause bugs blocking all 3 export docs

1. export_core.py: fix broken regex r'|\s+|' → r'\|\s+\|'
- unescaped pipes in regex = OR operator → matched every char position
- ALL content rendered as vertical pipe-table (|E|r|r|o|r|e|...)
- one-char fix eliminates the vertical text bug across all docs

2. export_html.py: sidebar-header + cover use accent not primary
- when club primary = white (#FFFFFF), sidebar was white-on-white invisible
- sidebar-header: var(--primary) → var(--accent) (already dark)
- cover gradient: start from accent (dark) not primary (white)
- _process_section_content: validate content before rendering
shows orange warning box instead of broken content on API error

3. agents.py: on API failure return empty content not error message
- was: return {'content': wrapped.user_message}
- error text 'Errore di connessione.' stored as plan section content
- when exported it caused the vertical text bug (+ fix 1 above)
- now: return {'content': ''} with error info in metadata only

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

agents.py

@@ -1034,7 +1034,7 @@ e soggette a revisione post-allineamento.
                 except Exception as e:
                     wrapped = handle_exception(e, context=f"agent_{self.spec.name}_ollama")
                     log_exception(wrapped, context=f"agent_{self.spec.name}")
-                    return {'content': wrapped.user_message, 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': wrapped.error_id}}
+                    return {'content': '', 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': wrapped.error_id, 'error_msg': wrapped.user_message}}
 
             # === OPENROUTER PATH ===
             elif self._provider == "openrouter" and self._openrouter_client:
@@ -1050,7 +1050,7 @@ e soggette a revisione post-allineamento.
                 except Exception as e:
                     wrapped = handle_exception(e, context=f"agent_{self.spec.name}_openrouter")
                     log_exception(wrapped, context=f"agent_{self.spec.name}")
-                    return {'content': wrapped.user_message, 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': wrapped.error_id}}
+                    return {'content': '', 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': wrapped.error_id, 'error_msg': wrapped.user_message}}
 
             # === GEMINI PATH (default) ===
             else:
@@ -1094,20 +1094,20 @@ e soggette a revisione post-allineamento.
                             details={"agent": self.spec.name, "fallback_error": str(fallback_e)}
                         )
                         log_exception(err, context=f"agent_{self.spec.name}")
-                        return {'content': err.user_message, 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': err.error_id}}
+                        return {'content': '', 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': err.error_id, 'error_msg': err.user_message}}
                 else:
                     err = AgentError(
                         message=f"Agent {self.spec.name} InvalidArgument: {e}",
                         details={"agent": self.spec.name, "error_type": "InvalidArgument"}
                     )
                     log_exception(err, context=f"agent_{self.spec.name}")
-                    return {'content': err.user_message, 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': err.error_id}}
+                    return {'content': '', 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': err.error_id, 'error_msg': err.user_message}}
 
               except Exception as e:
                 # Map to appropriate error type
                 wrapped = handle_exception(e, context=f"agent_{self.spec.name}")
                 log_exception(wrapped, context=f"agent_{self.spec.name}")
-                return {'content': wrapped.user_message, 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': wrapped.error_id}}
+                return {'content': '', 'sources': [], 'unverified_claims': [], 'metadata': {'error_id': wrapped.error_id, 'error_msg': wrapped.user_message}}
 
         cleaned = self._post_process(raw_content)
         content, sources, unverified = self.sourcer.process_content(cleaned, context=club_data.get('club_name', ''))

export_core.py

@@ -171,7 +171,8 @@ class BaseExporter(ABC):
         content = content.replace('\r\n', '\n')
 
         # TABLES: Separate markdown table rows that are on a single line
-        content = re.sub(r'|\s+|', '|\n|', content)
+        # NOTE: pipes must be escaped with \| — unescaped | in regex means OR
+        content = re.sub(r'\|\s+\|', '|\n|', content)
 
         # Pattern: ### 1. TITLE -> converts to header h3 (removes the number)
         content = re.sub(r'###\s+\d+[\.\)]\s*', '\n\n### ', content)

export_html.py

@@ -127,9 +127,19 @@ class ChunkedHTMLExporter(BaseExporter):
 
     def _process_section_content(self, content: str) -> str:
         """Converte contenuto markdown-like in HTML usando BaseExporter logic"""
-        if content is None:
-            return "<p><em>Contenuto non disponibile</em></p>"
-        
+        if not content:
+            return "<p><em>Contenuto non disponibile per questa sezione.</em></p>"
+
+        # Guard: content troppo corto o messaggio d'errore → non renderizzare
+        _error_markers = ["errore di connessione", "verifica la connessione", "si è verificato un errore"]
+        if len(content.strip()) < 80 or any(m in content.lower() for m in _error_markers):
+            return (
+                '<div style="border-left:4px solid #F57C00; background:#FFF8E1; padding:14px 18px; '
+                'border-radius:0 6px 6px 0; color:#555; font-style:italic;">'
+                '⚠ Sezione non generata correttamente. Rigenera il piano per ottenere i contenuti completi.'
+                '</div>'
+            )
+
         normalized = self._normalize_markdown(content)
         return self._markdown_to_html(normalized)
 
@@ -326,7 +336,7 @@ class ChunkedHTMLExporter(BaseExporter):
             display: flex; flex-direction: column; z-index: 1000;
         }}
 
-        .sidebar-header {{ padding: 24px 20px; background: var(--primary); color: white; text-align: center; border-bottom: 3px solid var(--accent); }}
+        .sidebar-header {{ padding: 24px 20px; background: var(--accent); color: var(--text-on-primary); text-align: center; border-bottom: 3px solid var(--primary); }}
         .nav {{ flex: 1; overflow-y: auto; padding: 16px 10px; }}
         .nav-item {{ display: block; padding: 10px 14px; margin-bottom: 3px; color: #4a5568; text-decoration: none; border-radius: 6px; font-weight: 600; font-size: 0.85rem; border-left: 3px solid transparent; }}
         .nav-item:hover {{ background: var(--bg); color: var(--accent); border-left-color: var(--accent); }}
@@ -335,7 +345,7 @@ class ChunkedHTMLExporter(BaseExporter):
         .container {{ max-width: 900px; margin: 0 auto; padding: 60px 40px; }}
 
         .cover {{
-            height: 55vh; background: linear-gradient(150deg, var(--primary) 0%, var(--accent) 100%);
+            height: 55vh; background: linear-gradient(150deg, var(--accent) 0%, var(--primary) 100%);
             color: white; display: flex; flex-direction: column;
             justify-content: center; align-items: center; text-align: center;
             position: relative; overflow: hidden;