google-news-trends-mcp 0.2.0__py3-none-any.whl → 0.2.1__py3-none-any.whl

google_news_trends_mcp/news.py

@@ -48,9 +48,6 @@ google_news = GNews(
     # exclude_websites=[],
 )
 
-playwright: Optional[Playwright] = None
-browser: Optional[Browser] = None
-
 ProgressCallback = Callable[[float, Optional[float]], Awaitable[None]]
 
 
@@ -64,14 +61,15 @@ class BrowserManager(AsyncContextDecorator):
         if cls.browser is None:
             async with cls._lock:
                 if cls.browser is None:
-                    await cls._startup()
+                    logger.info("Starting browser...")
+                    try:
+                        cls.playwright = await async_playwright().start()
+                        cls.browser = await cls.playwright.chromium.launch(headless=True)
+                    except Exception as e:
+                        logger.critical("Browser startup failed", exc_info=e)
+                        raise SystemExit(1)
         return cast(Browser, cls.browser)
 
-    @classmethod
-    async def _startup(cls):
-        logger.info("Starting browser...")
-        cls.playwright = await async_playwright().start()
-        cls.browser = await cls.playwright.chromium.launch(headless=True)
 
     @classmethod
     async def _shutdown(cls):
@@ -89,11 +87,11 @@ class BrowserManager(AsyncContextDecorator):
         async def _browser_context_cm():
             browser_inst = await cls._get_browser()
             context = await browser_inst.new_context()
-            logging.debug("Created browser context...")
+            logger.debug("Created browser context...")
             try:
                 yield context
             finally:
-                logging.debug("Closing browser context...")
+                logger.debug("Closing browser context...")
                 await context.close()
 
         return _browser_context_cm()
@@ -119,7 +117,7 @@ async def download_article_with_playwright(url) -> newspaper.Article | None:
             article = newspaper.article(url, input_html=content)
             return article
     except Exception as e:
-        logging.warning(f"Error downloading article with Playwright from {url}\n {e.args}")
+        logger.warning(f"Error downloading article with Playwright from {url}\n {e.args}")
         return None
 
 
@@ -128,18 +126,18 @@ def download_article_with_scraper(url) -> newspaper.Article | None:
     try:
         article = newspaper.article(url)
     except Exception as e:
-        logging.debug(f"Error downloading article with newspaper from {url}\n {e.args}")
+        logger.debug(f"Error downloading article with newspaper from {url}\n {e.args}")
         try:
             # Retry with cloudscraper
             response = scraper.get(url)
             if response.status_code < 400:
                 article = newspaper.article(url, input_html=response.text)
             else:
-                logging.debug(
+                logger.debug(
                     f"Failed to download article with cloudscraper from {url}, status code: {response.status_code}"
                 )
         except Exception as e:
-            logging.debug(f"Error downloading article with cloudscraper from {url}\n {e.args}")
+            logger.debug(f"Error downloading article with cloudscraper from {url}\n {e.args}")
     return article
 
 
@@ -150,10 +148,10 @@ def decode_url(url: str) -> str:
             if decoded_url.get("status"):
                 url = decoded_url["decoded_url"]
             else:
-                logging.debug("Failed to decode Google News RSS link:")
+                logger.debug("Failed to decode Google News RSS link:")
                 return ""
         except Exception as err:
-            logging.warning(f"Error while decoding url {url}\n {err.args}")
+            logger.warning(f"Error while decoding url {url}\n {err.args}")
     return url
 
 
@@ -184,7 +182,7 @@ async def process_gnews_articles(
     for idx, gnews_article in enumerate(gnews_articles):
         article = await download_article(gnews_article["url"])
         if article is None or not article.text:
-            logging.debug(f"Failed to download article from {gnews_article['url']}:\n{article}")
+            logger.debug(f"Failed to download article from {gnews_article['url']}:\n{article}")
             continue
         article.parse()
         if nlp:
@@ -209,7 +207,7 @@ async def get_news_by_keyword(
     google_news.max_results = max_results
     gnews_articles = google_news.get_news(keyword)
     if not gnews_articles:
-        logging.debug(f"No articles found for keyword '{keyword}' in the last {period} days.")
+        logger.debug(f"No articles found for keyword '{keyword}' in the last {period} days.")
         return []
     return await process_gnews_articles(gnews_articles, nlp=nlp, report_progress=report_progress)
 
@@ -227,7 +225,7 @@ async def get_top_news(
     google_news.max_results = max_results
     gnews_articles = google_news.get_top_news()
     if not gnews_articles:
-        logging.debug("No top news articles found.")
+        logger.debug("No top news articles found.")
         return []
     return await process_gnews_articles(gnews_articles, nlp=nlp, report_progress=report_progress)
 
@@ -244,7 +242,7 @@ async def get_news_by_location(
     google_news.max_results = max_results
     gnews_articles = google_news.get_news_by_location(location)
     if not gnews_articles:
-        logging.debug(f"No articles found for location '{location}' in the last {period} days.")
+        logger.debug(f"No articles found for location '{location}' in the last {period} days.")
         return []
     return await process_gnews_articles(gnews_articles, nlp=nlp, report_progress=report_progress)
 
@@ -271,7 +269,7 @@ async def get_news_by_topic(
     google_news.max_results = max_results
     gnews_articles = google_news.get_news_by_topic(topic)
     if not gnews_articles:
-        logging.debug(f"No articles found for topic '{topic}' in the last {period} days.")
+        logger.debug(f"No articles found for topic '{topic}' in the last {period} days.")
         return []
     return await process_gnews_articles(gnews_articles, nlp=nlp, report_progress=report_progress)
 
@@ -303,7 +301,7 @@ async def get_trending_terms(
             return [{"keyword": trend.keyword, "volume": trend.volume} for trend in trends]
         return trends
     except Exception as e:
-        logging.warning(f"Error fetching trending terms: {e}")
+        logger.warning(f"Error fetching trending terms: {e}")
         return []
 
 
@@ -351,4 +349,4 @@ def save_article_to_json(article: newspaper.Article, filename: str = "") -> None
             filename += ".json"
     with open(filename, "w") as f:
         json.dump(article_data, f, indent=4)
-    logging.debug(f"Article saved to {filename}")
+    logger.debug(f"Article saved to {filename}")

google_news_trends_mcp/server.py

@@ -154,6 +154,19 @@ async def summarize_article(article: Article, ctx: Context) -> None:
         article.summary = "No summary available."
 
 
+async def summarize_articles(articles: list[Article], ctx: Context) -> None:
+    total_articles = len(articles)
+    try:
+        for idx, article in enumerate(articles):
+            await summarize_article(article, ctx)
+            await ctx.report_progress(idx, total_articles)
+    except Exception as err:
+        await ctx.debug(f"Failed to use LLM sampling for article summary:\n{err.args}")
+        for idx, article in enumerate(articles):
+            article.nlp()
+            await ctx.report_progress(idx, total_articles)
+
+
 @mcp.tool(
     description=news.get_news_by_keyword.__doc__,
     tags={"news", "articles", "keyword"},
@@ -185,16 +198,7 @@ async def get_news_by_keyword(
         report_progress=ctx.report_progress,
     )
     if summarize:
-        total_articles = len(articles)
-        try:
-            for idx, article in enumerate(articles):
-                await summarize_article(article, ctx)
-                await ctx.report_progress(idx, total_articles)
-        except Exception as err:
-            await ctx.debug(f"Failed to use LLM sampling for article summary:\n{err.args}")
-            for idx, article in enumerate(articles):
-                article.nlp()
-                await ctx.report_progress(idx, total_articles)
+        await summarize_articles(articles, ctx)
     await ctx.report_progress(progress=len(articles), total=len(articles))
     return [ArticleOut(**a.to_json(False)) for a in articles]
 
@@ -230,16 +234,7 @@ async def get_news_by_location(
         report_progress=ctx.report_progress,
     )
     if summarize:
-        total_articles = len(articles)
-        try:
-            for idx, article in enumerate(articles):
-                await summarize_article(article, ctx)
-                await ctx.report_progress(idx, total_articles)
-        except Exception as err:
-            await ctx.debug(f"Failed to use LLM sampling for article summary:\n{err.args}")
-            for idx, article in enumerate(articles):
-                article.nlp()
-                await ctx.report_progress(idx, total_articles)
+        await summarize_articles(articles, ctx)
     await ctx.report_progress(progress=len(articles), total=len(articles))
     return [ArticleOut(**a.to_json(False)) for a in articles]
 
@@ -272,17 +267,7 @@ async def get_news_by_topic(
         report_progress=ctx.report_progress,
     )
     if summarize:
-        total_articles = len(articles)
-        try:
-            for idx, article in enumerate(articles):
-                await summarize_article(article, ctx)
-                await ctx.report_progress(idx, total_articles)
-        except Exception as err:
-            await ctx.debug(f"Failed to use LLM sampling for article summary:\n{err.args}")
-            for idx, article in enumerate(articles):
-                article.nlp()
-                await ctx.report_progress(idx, total_articles)
-
+        await summarize_articles(articles, ctx)
     await ctx.report_progress(progress=len(articles), total=len(articles))
     return [ArticleOut(**a.to_json(False)) for a in articles]
 
@@ -313,17 +298,7 @@ async def get_top_news(
         report_progress=ctx.report_progress,
     )
     if summarize:
-        total_articles = len(articles)
-        try:
-            for idx, article in enumerate(articles):
-                await summarize_article(article, ctx)
-                await ctx.report_progress(idx, total_articles)
-        except Exception as err:
-            await ctx.debug(f"Failed to use LLM sampling for article summary:\n{err.args}")
-            for idx, article in enumerate(articles):
-                article.nlp()
-                await ctx.report_progress(idx, total_articles)
-
+        await summarize_articles(articles, ctx)
     await ctx.report_progress(progress=len(articles), total=len(articles))
     return [ArticleOut(**a.to_json(False)) for a in articles]
 
@@ -340,7 +315,6 @@ async def get_trending_terms(
     if not full_data:
         trends = await news.get_trending_terms(geo=geo, full_data=False, max_results=max_results)
         return [TrendingTermOut(keyword=str(tt["keyword"]), volume=tt["volume"]) for tt in trends]
-
     trends = await news.get_trending_terms(geo=geo, full_data=True, max_results=max_results)
     return [TrendingTermOut(**tt.__dict__) for tt in trends]
 

{google_news_trends_mcp-0.2.0.dist-info → google_news_trends_mcp-0.2.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: google-news-trends-mcp
-Version: 0.2.0
+Version: 0.2.1
 Summary: An MCP server to access Google News and Google Trends.
 Author-email: Jesse Manek <jesse.manek@gmail.com>
 License-Expression: MIT
@@ -27,7 +27,7 @@ Dynamic: license-file
 
 # Google News Trends MCP
 
-An MCP server to access Google News and Google Trends.  Does not rely on any paid APIs.  
+An MCP server to access Google News and Google Trends.
 
 ## Features
 

google_news_trends_mcp-0.2.1.dist-info/RECORD

@@ -0,0 +1,11 @@
+google_news_trends_mcp/__init__.py,sha256=nDWNd6_TSf4vDQuHVBoAf4QfZCB3ZUFQ0M7XvifNJ-g,78
+google_news_trends_mcp/__main__.py,sha256=ysiAk_xpnnW3lrLlzdIQQa71tuGBRT8WocbecBsY2Fs,87
+google_news_trends_mcp/cli.py,sha256=3Z916898HXTigmQYEfvb7ybfbuUE7bjMC6yjT5-l6u0,4558
+google_news_trends_mcp/news.py,sha256=EhTIqRmFOQKAhyV8rLySaPoILX3uyKbrzO6jxo89FhA,12528
+google_news_trends_mcp/server.py,sha256=ZFZvCxSVLma7QlanFsXyu6gpCHZSS0y5YfnZl7HnXnc,13780
+google_news_trends_mcp-0.2.1.dist-info/licenses/LICENSE,sha256=5dsv2ZI5EZIer0a9MktVmILVrlp5vqH_0tPIe3bRLgE,1067
+google_news_trends_mcp-0.2.1.dist-info/METADATA,sha256=AdnEqmflIhFNUV052z9VHIOum2x0_TzI_e3hK84vdfM,4419
+google_news_trends_mcp-0.2.1.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+google_news_trends_mcp-0.2.1.dist-info/entry_points.txt,sha256=eVT3xd6YJQgsWAUBwhnffuwhXNF7yyt_uco6fjBy-1o,130
+google_news_trends_mcp-0.2.1.dist-info/top_level.txt,sha256=RFheDbzhNnEV_Y3iFNm7jhRhY1P1wQgfiYqVpXCTD_U,23
+google_news_trends_mcp-0.2.1.dist-info/RECORD,,

google_news_trends_mcp-0.2.0.dist-info/RECORD

@@ -1,11 +0,0 @@
-google_news_trends_mcp/__init__.py,sha256=nDWNd6_TSf4vDQuHVBoAf4QfZCB3ZUFQ0M7XvifNJ-g,78
-google_news_trends_mcp/__main__.py,sha256=ysiAk_xpnnW3lrLlzdIQQa71tuGBRT8WocbecBsY2Fs,87
-google_news_trends_mcp/cli.py,sha256=3Z916898HXTigmQYEfvb7ybfbuUE7bjMC6yjT5-l6u0,4558
-google_news_trends_mcp/news.py,sha256=MPNZlzI7KXkhQ2uj7233N2i9kFHGUgGMdRBCAbj-B44,12471
-google_news_trends_mcp/server.py,sha256=S-tlFY1wiFm9VPeb4NDnV0NGtczaQDmx20kIrZZQHto,15031
-google_news_trends_mcp-0.2.0.dist-info/licenses/LICENSE,sha256=5dsv2ZI5EZIer0a9MktVmILVrlp5vqH_0tPIe3bRLgE,1067
-google_news_trends_mcp-0.2.0.dist-info/METADATA,sha256=rok_3L-eDVXQJSuG6ze1Vuicnh-kpcWyJxaF2DbqZ1s,4454
-google_news_trends_mcp-0.2.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-google_news_trends_mcp-0.2.0.dist-info/entry_points.txt,sha256=eVT3xd6YJQgsWAUBwhnffuwhXNF7yyt_uco6fjBy-1o,130
-google_news_trends_mcp-0.2.0.dist-info/top_level.txt,sha256=RFheDbzhNnEV_Y3iFNm7jhRhY1P1wQgfiYqVpXCTD_U,23
-google_news_trends_mcp-0.2.0.dist-info/RECORD,,