'` over backtick templates. ### closeSendModal() nulls currentImageId If a function calls `closeSendModal()` (which sets `currentImageId = null`) before using `currentImageId` in a `fetch()` URL, the URL becomes `/api/report-image/null/send` → 404 HTML → `r.json()` throws "is not valid json". **Always save to local var first**: `var id = currentImageId; closeSendModal(); fetch('/path/' + id)`. ### body { overflow: hidden } breaks scrollable pages base.html sets `body { overflow: hidden; height: 100vh }` for the sidebar layout. Pages that need scrolling (articles, reports, report-images) must override: `` in the template body block. ### NEVER blame browser cache first When user reports "page not working", "spinning forever", or "404": verify backend FIRST. Check `curl` the API, `python3 -c "import app"` for syntax errors, `grep` rendered HTML. Only after server-side passes should client caching be considered. User was frustrated by repeated cache-blame for real code bugs. ### Article dedup race condition When two summarize/learn requests for the same article arrive simultaneously, both pass the dedup check before either hermes finishes and writes to `daily_reports`. This means the second request spawns a redundant hermes subprocess. The dedup only prevents re-processing after the first result is fully persisted. Acceptable for now — practical dedup savings come from cross-session reuse (group digests, next-day page loads), not from same-second double-clicks. ### Server management - `DEBUG = False` (auto-reload creates notification storms from killed processes) - Template changes require `restart.sh` - `restart.sh` flow: `pkill -f 'python3 app.py'` → `fuser -k 5000/tcp` → `python3 app.py` ### CRITICAL: Never delete large blocks of app.py without verifying boundaries When deleting code blocks from app.py, ALWAYS read the deletion range first with `read_file(offset, limit)` to confirm exactly what will be removed. A single misaligned deletion can remove hundreds of lines including ALL main routes (/reports, /articles, /api/sessions, etc.). If file becomes corrupted, check: `python3 -c "import app; print(sum(1 for _ in app.app.url_map.iter_rules()))"` — a healthy app.py has 29-30 routes. Anything less than 20 means routes were lost. ## DB Tables ```sql -- daily_reports: generated Markdown reports + article summaries/learn daily_reports ( id, chatroom_id, chatroom_name, report_date, content, msg_count, image_count, article_count, dedup_key TEXT, -- 2026-07-08: biz|mid|idx|sn for article dedup article_url TEXT, -- 2026-07-08: original URL for reference created_at ) -- Index: idx_daily_reports_dedup ON (dedup_key, report_date) -- report_images: generated WSJ-style infographics report_images (id, report_id, image_path, style, created_at) -- existing tables article_queue, image_analysis, chat_summaries, processed_messages ``` ## Hermes CLI Subprocess Pattern See `references/hermes-cli-subprocess-pattern.md` for the complete reference on spawning Hermes from Flask. Key flags: | flag | purpose | |------|---------| | `-q "prompt"` | Non-interactive single query | | `-s skill` | Preload skill(s) | | `--yolo` | Skip confirmations | | `-t groups` | Enable tool groups (terminal,file,web) | | `capture_output=True` | Capture stdout/stderr | | `timeout=300` | 5-min timeout for long operations | Do NOT use `--quiet` — it suppresses output generation. Use stdout as fallback when result files aren't written. ## References - `references/hermes-cli-subprocess-pattern.md` — Complete guide for spawning Hermes CLI subprocesses from Flask - `references/message-xml-patterns.md` — WeChat message XML formats and parsing - `references/hermes-subprocess-pitfalls.md` — Common failures and fixes - `references/frontend-js-pitfalls.md` — JS bugs: closeSendModal before fetch, template literal escaping, image modals, route ordering - `references/article-test-script.md` — Automated test: URL validity → summarize → learn-build - `references/report-list-ui-patterns.md` — Three report types, filtering, mobile button layout - `scripts/test_articles.py` — Rerunnable test script for article links + summarize + learn pipeline - `templates/restart.sh` — Server restart script ## LAN Access Set `HOST = "0.0.0.0"` in config.py to allow Mac/other devices on same WiFi to access. Phone IP: `ifconfig | grep wlan0 -A1 | grep inet`. Access from Mac: `http://:5000`。 - `references/wechat-image-cdn-investigation.md` — Full-resolution image CDN investigation (2026-07-08): why only thumbnails are local, hex-encoded CDN params, viable download approaches - `references/article-test-script.md` — Automated test script: URL validity → summarize → learn-build