Feat/report visual charts#16
Merged
Merged
Conversation
- Add deploy.sh: main deployment script with automatic SSL certificate fixes - Add scripts/fix_ssl_server.sh: server-side SSL certificate repair - Add scripts/diagnose.sh: diagnostic tool for deployment issues - Add scripts/README.md: detailed documentation and troubleshooting guide - Update CLAUDE.md: document GitHub Potential Users feature and public site Key features: - Automatic Python SSL certificate detection and repair (macOS/Linux) - Detailed error handling for 8 common deployment issues - Pre-flight checks and post-deployment verification - Support for Chinese error messages and fix instructions - Color-coded logging for better readability
Remove deploy.sh and scripts/* (BWG-targeted, port 8080 — conflict with current production topology where newyork:8081 is primary). Revert CLAUDE.md text describing GitHub-leads/marketing-site features that belong to a separate change set. Keep only the report-charts work.
- Restrict chart <img> rendering to /api/v1/report-assets/{32-hex}.svg
whitelist; non-matching markdown images fall back to escaped paragraph.
- Detect existing chart references by asset URL instead of literal heading
text (LLM produces variants that bypassed the old "## 数据图表速览" check
and caused duplicate chart sections).
- Delete generated SVG assets when human-report generation fails after the
fallback path, preventing orphan files in ~/.opencmo/report_assets/.
- Tests now exercise the asset endpoint via TestClient (real ASGI path),
lock the URL whitelist policy with rejection cases, cover the new
asset-URL dedup, and verify the cleanup helper.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Closed
study8677
added a commit
that referenced
this pull request
May 9, 2026
The Cursor showcase only had partial data (no competitors or keywords), which underplayed the product. Replace with two production scans that expose concrete signals: - Coze (AI agent platform): 50 keywords, 326 SERP snapshots, 153 GitHub leads, auto-discovered Dify/FastGPT/OpenAI GPTs, citability regression alert. - DigiGrow (UK SaaS): 672 SERP rank checks across 54 keywords, 34 actionable insights, real signal — keyword fell from #2 to #16 in 24h and was pushed as an insight without anyone polling the dashboard. Translated to all five locales. Drops the now-stale docs/showcase/cursor/ data dump.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verification
Checklist
noreplyemailCo-authored-by