vn6295337 Claude Opus 4.5 commited on
Commit
bd752bd
·
1 Parent(s): 698592a

fix: Remove broken configs import from news-basket MCP server

Browse files

Company name is already cleaned in app.py before reaching MCP servers,
so this import was redundant and causing ModuleNotFoundError.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

.obsidian/workspace.json CHANGED
@@ -13,12 +13,12 @@
13
  "state": {
14
  "type": "markdown",
15
  "state": {
16
- "file": "docs/metrics_schema_human_readable.md",
17
  "mode": "source",
18
  "source": false
19
  },
20
  "icon": "lucide-file",
21
- "title": "metrics_schema_human_readable"
22
  }
23
  }
24
  ]
@@ -185,40 +185,41 @@
185
  },
186
  "active": "369b0d31e1527f56",
187
  "lastOpenFiles": [
188
- "docs/mcp_test_report_KO.md",
189
- "docs/metrics_schema_human_readable.md",
190
- "docs/mcp_raw_visa.md",
191
- "app.py.tmp.1610.1768146282892",
192
- "mcp-servers/volatility-basket/server.py.tmp.1610.1768144883587",
193
- "__pycache__/mcp_client.cpython-311.pyc.140529827342128",
 
 
 
 
 
 
 
 
 
 
 
 
 
 
194
  "docs/metrics_schema_emitted.md",
195
- "docs/schema_normalization.md",
196
- "mcp_client.py.tmp.1610.1768140230378",
197
- "mcp_client.py.tmp.1610.1768140224277",
198
- "mcp_client.py.tmp.1610.1768140212665",
199
- "mcp_client.py.tmp.1610.1768140199252",
200
- "mcp_client.py.tmp.1610.1768140165633",
201
- "mcp_client.py.tmp.1610.1768140149514",
202
- "mcp_client.py.tmp.1610.1768140127411",
203
  "docs/alphavantage_data_schema.md",
204
- "docs/mcp_data_sources.md",
205
- "docs/mcp_normalized_schemas.md",
206
- "docs/universal_metrics_schema.md",
207
- "docs/mcp_test_report_BAC.md",
208
- "docs/mcp_output_visa.md",
209
- "docs/mcp_output_citigroup.md",
210
- "docs/data_flow.md",
211
- "docs/volatility_data_schema.md",
212
  "docs/SEC_EDGAR_data_schema.md",
213
- "docs/tavily_data_schema.md",
214
- "docs/vader_data_schema.md",
215
- "docs/reddit_data_schema.md",
216
- "docs/finnhub_data_schema.md",
217
  "docs/yahoo_options_data_schema.md",
218
- "docs/yahoo_data_schema.md",
219
- "docs/mcp_output_sample.md",
220
- "docs/fundamentals_data_schema.md",
221
- "docs/valuation_data_schema.md",
222
- "mcp-servers/fundamentals-basket/mcp_sec_edgar.md"
223
  ]
224
  }
 
13
  "state": {
14
  "type": "markdown",
15
  "state": {
16
+ "file": "docs/mcp_data_structure.md",
17
  "mode": "source",
18
  "source": false
19
  },
20
  "icon": "lucide-file",
21
+ "title": "mcp_data_structure"
22
  }
23
  }
24
  ]
 
185
  },
186
  "active": "369b0d31e1527f56",
187
  "lastOpenFiles": [
188
+ "app.py.tmp.30809.1768319312452",
189
+ "app.py.tmp.30809.1768319301570",
190
+ "configs/__pycache__/output_schemas.cpython-311.pyc",
191
+ "mcp-servers/sentiment-basket/server.py.tmp.30809.1768316033327",
192
+ "docs/mcp_output_schemas.md",
193
+ "docs/mcp_data_structure.md",
194
+ "docs/sec_edgar_data_schema.md",
195
+ "mcp-servers/sentiment-basket/server.py.tmp.30809.1768315145571",
196
+ "mcp-servers/news-basket/server.py.tmp.30809.1768314706248",
197
+ "mcp-servers/macro-basket/server.py.tmp.30809.1768314005794",
198
+ "mcp-servers/fundamentals-basket/services/__pycache__/orchestrator.cpython-311.pyc.139915998807856",
199
+ "docs/nvidia_mcp_output_sample.md",
200
+ "mcp-servers/news-basket/config/__init__.py.tmp.31532.1768304362133",
201
+ "docs/yahoo_data_schema.md",
202
+ "docs/nyt_data_schema.md",
203
+ "docs/fred_data_schema.md",
204
+ "docs/tavily_data_schema.md",
205
+ "docs/newsapi_data_schema.md",
206
+ "docs/finnhub_data_schema.md",
207
+ "docs/reddit_data_schema.md",
208
  "docs/metrics_schema_emitted.md",
209
+ "docs/mcp_test_report_V.md",
 
 
 
 
 
 
 
210
  "docs/alphavantage_data_schema.md",
211
+ "__pycache__/mcp_client.cpython-311.pyc.140325135886896",
212
+ "mcp-servers/volatility-basket/__pycache__/server.cpython-311.pyc.140157304405200",
213
+ "docs/bls_data_schema.md",
214
+ "docs/bea_data_schema.md",
215
+ "docs/mcp_test_report_CVX.md",
216
+ "docs/metrics_schema_human_readable.md",
 
 
217
  "docs/SEC_EDGAR_data_schema.md",
 
 
 
 
218
  "docs/yahoo_options_data_schema.md",
219
+ "docs/macro_data_schema.md",
220
+ "docs/mcp_test_report_D.md",
221
+ "docs/volatility_data_schema.md",
222
+ "docs/mcp_data_sources.md",
223
+ "docs/mcp_output_sample.md"
224
  ]
225
  }
mcp-servers/news-basket/server.py CHANGED
@@ -35,8 +35,8 @@ def normalize_date(date_str: str) -> str:
35
  from pathlib import Path
36
  from typing import Optional
37
 
38
- # Import company name normalization from shared configs, domain filters from local config
39
- from configs.company_name_filters import clean_company_name
40
  from config.domain_filters import NEWS_DOMAINS, NYT_NEWS_DESKS, NEWSAPI_DOMAINS
41
 
42
  # Load environment variables (later files override earlier ones)
@@ -363,8 +363,8 @@ async def get_all_sources_news(ticker: str, company_name: str = None) -> dict:
363
  # Tavily query - general search
364
  tavily_query = base_query
365
 
366
- # NYT query - cleaned company name + "stock" for disambiguation (e.g., Apple vs apple fruit)
367
- nyt_query = f"{clean_company_name(company_name or ticker)} stock"
368
 
369
  # NewsAPI query - ticker symbol helps filter
370
  newsapi_query = f"{company_name or ticker} {ticker} stock"
 
35
  from pathlib import Path
36
  from typing import Optional
37
 
38
+ # Import domain filters from local config
39
+ # Note: company_name is already cleaned by app.py before reaching MCP servers
40
  from config.domain_filters import NEWS_DOMAINS, NYT_NEWS_DESKS, NEWSAPI_DOMAINS
41
 
42
  # Load environment variables (later files override earlier ones)
 
363
  # Tavily query - general search
364
  tavily_query = base_query
365
 
366
+ # NYT query - company name + "stock" for disambiguation (e.g., Apple vs apple fruit)
367
+ nyt_query = f"{company_name or ticker} stock"
368
 
369
  # NewsAPI query - ticker symbol helps filter
370
  newsapi_query = f"{company_name or ticker} {ticker} stock"