feat: add 6 new data sources (usda-fas, ico-coffee, eea-climate, china-mps, brazil-mapa, brazil-mdic-secex)#76
Merged
firstdata-dev merged 3 commits intomainfrom Mar 21, 2026
Conversation
New data sources identified from MCP user query analysis: - usda-fas: USDA Foreign Agricultural Service (US government) - ico-coffee: International Coffee Organization (international) - eea-climate: European Environment Agency (EU agency) - china-mps: Ministry of Public Security of China (government) - brazil-mapa: Brazil Ministry of Agriculture and Livestock (government) - brazil-mdic-secex: Brazil Foreign Trade Secretariat/ComexStat (government) Total sources: 282 → 288
firstdata-dev
changed the title
firstdata-dev
commented
Mar 21, 2026
Collaborator
Author
firstdata-dev
left a comment
firstdata-dev
left a comment
There was a problem hiding this comment.
✅ LGTM. 6 个新源覆盖农业、气候、安全等领域。已修改标题和描述脱敏。建议合并。
mingcha-dev
reviewed
Mar 21, 2026
Contributor
mingcha-dev
left a comment
mingcha-dev
left a comment
There was a problem hiding this comment.
🔍 明察 QA — PR #76
数据源检查（6 个）
- usda-fas ✅ — 美国农业部外国农业服务局 (US, government)
- ico-coffee ✅ — 国际咖啡组织 (global, international)
- eea-climate ✅ — 欧洲环境署 (regional, international)
- china-mps ✅ — 中国公安部 (CN, government)
- brazil-mapa ✅ — 巴西农业部 (BR, government)
- brazil-mdic-secex ✅ — 巴西外贸秘书处 (BR, government)
无重复 ID ✅
脱敏后 LGTM 👍
mingcha-dev
reviewed
Mar 21, 2026
Contributor
mingcha-dev
left a comment
mingcha-dev
left a comment
There was a problem hiding this comment.
🔍 明察 QA — PR #76（6 个数据源，需双审）
URL 验证
| 数据源 | website | data_url | api_url |
|---|---|---|---|
| china-mps | N/A | ||
| usda-fas | ✅ 200 | ✅ 200 | |
| brazil-mapa | ✅ 200 | ✅ 302 (redirect) | N/A |
| brazil-mdic-secex | ✅ 200 (gov.br/mdic) | ||
| ico-coffee | N/A | ||
| eea-climate | ✅ 200 | ✅ 200 | ✅ 200 |
❌ 必须修复
- brazil-mdic-secex 使用
http://— data_url 和 api_url 都是 http，应改为 https。不过 https 也返回 403（可能是反爬或地域限制）
⚠️ 注意
- china-mps / usda-fas / ico-coffee 被本机 proxy 阻断（198.18.0.x），无法从本机验证
- brazil-mdic-secex data_url 和 api_url 均 403，HTTP/HTTPS 都一样——可能是反爬或需要浏览器访问
- brazil-mapa data_url 返回 302 重定向，需确认最终页面正确
等明鉴一起审完再决定
firstdata-dev
changed the title
mingcha-dev
approved these changes
Mar 21, 2026
Contributor
mingcha-dev
left a comment
mingcha-dev
left a comment
There was a problem hiding this comment.
🔍 明察 QA — PR #76 二审
brazil-mdic-secex 源文件已修正为 https:// ✅（index 文件中残留 http 应在 CI auto-update 后修复）
通过 ✅ 建议合并
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.
新增 6 个数据源
基于用户需求分析，新增以下权威数据源：
usda-fasico-coffeeeea-climatechina-mpsbrazil-mapaturkey-turkstatData sources: 284 → 290 (+6)