Merge pull request #120 from fmacpro/codex/fix-mobile-screenshot-functionality-and-add-tests-p6b8nr

Enable screenshot capture in parseArticle

Author: fmacpro

index.js

@@ -549,7 +549,7 @@ const articleParser = async function (browser, options, socket) {
   if (options.enabled.includes('screenshot') && timeLeft() > 300) {
     log('analyze', 'Capturing screenshot')
     try {
-      article.mobile = await page.screenshot({ encoding: 'base64', type: 'jpeg', quality: 60 })
+      article.screenshot = await page.screenshot({ encoding: 'base64', type: 'jpeg', quality: 60 })
     } catch { /* ignore screenshot failures (e.g., page closed on timeout) */ }
   }
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "horseman-article-parser",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Web Page Inspection Tool. Sentiment Analysis, Keyword Extraction, Named Entity Recognition & Spell Check",
   "type": "module",
   "main": "index.js",

scripts/single-sample-run.js

@@ -108,7 +108,7 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url))
       metadescription: article.meta.description.text,
       url: article.url,
       siteicon: article.siteicon,
-      mobile: article.mobile,
+      screenshot: article.screenshot,
       sentiment: { score: article.sentiment.score, comparative: article.sentiment.comparative },
       keyphrases: article.processed.keyphrases,
       keywords: article.processed.keywords,

tests/parseArticle.test.js

@@ -73,8 +73,8 @@ test('parseArticle captures a screenshot when enabled', { timeout: TEST_TIMEOUT
     t.skip('puppeteer unavailable: ' + err.message)
     return
   }
-  assert.equal(typeof article.mobile, 'string')
-  assert.ok(Buffer.from(article.mobile, 'base64').length > 1000)
+  assert.equal(typeof article.screenshot, 'string')
+  assert.ok(Buffer.from(article.screenshot, 'base64').length > 1000)
 })
 
 test('parseArticle uses rules overrides for title and content', { timeout: TEST_TIMEOUT }, async (t) => {