From 86cd9098b18807a8553da05405e2111bc4a4b9ba Mon Sep 17 00:00:00 2001
From: Bart Veneman <bart.veneman@drukwerkdeal.nl>
Date: Thu, 25 Nov 2021 20:25:01 +0100
Subject: [PATCH 1/2] adds option to exclude inline styles

---
 src/index.js  | 25 ++++++++++++++-----------
 test/index.js | 10 ++++++++++
 2 files changed, 24 insertions(+), 11 deletions(-)

diff --git a/src/index.js b/src/index.js
index 23235de..1e014ef 100644
--- a/src/index.js
+++ b/src/index.js
@@ -14,7 +14,7 @@ InvalidUrlError.prototype = Error.prototype
  * @param {string} waitUntil https://github.com/puppeteer/puppeteer/blob/master/docs/api.md#pagegotourl-options
  * @returns {string} All CSS that was found
  */
-module.exports = async (url, {waitUntil = 'networkidle0', origins = 'exclude'} = {}) => {
+module.exports = async (url, {waitUntil = 'networkidle0', origins = 'exclude', inlineStyles = 'include'} = {}) => {
 	// Setup a browser instance
 	const browser = await puppeteer.launch()
 
@@ -82,15 +82,18 @@ module.exports = async (url, {waitUntil = 'networkidle0', origins = 'exclude'} =
 	// CSSRule:
 	//    [x-extract-css-inline-style] { color: red; }
 	//
-	const inlineCssRules = await page.evaluate(() => {
-		return [...document.querySelectorAll('[style]')]
-			.map(element => element.getAttribute('style'))
-			// Filter out empty style="" attributes
-			.filter(Boolean)
-	})
-	const inlineCss = inlineCssRules
-		.map(rule => `[x-extract-css-inline-style] { ${rule} }`)
-		.map(css => ({type: 'inline', href: url, css}))
+	let inlineCss = []
+	if (inlineStyles !== 'exclude') {
+		const inlineCssRules = await page.evaluate(() => {
+			return [...document.querySelectorAll('[style]')]
+				.map(element => element.getAttribute('style'))
+				// Filter out empty style="" attributes
+				.filter(Boolean)
+		})
+		inlineCss = inlineCssRules
+			.map(rule => `[x-extract-css-inline-style] { ${rule} }`)
+			.map(css => ({type: 'inline', href: url, css}))
+	}
 
 	const links = coverage
 		// Filter out the <style> tags that were found in the coverage
@@ -108,7 +111,7 @@ module.exports = async (url, {waitUntil = 'networkidle0', origins = 'exclude'} =
 
 	const css = links
 		.concat(styleSheetsApiCss)
-		.concat(inlineCss)
+		.concat(inlineStyles === 'exclude' ? [] : inlineCss)
 
 	// Return the complete structure ...
 	if (origins === 'include') {
diff --git a/test/index.js b/test/index.js
index 194a878..47ef57d 100644
--- a/test/index.js
+++ b/test/index.js
@@ -87,6 +87,16 @@ test('it finds inline styles - HTML', async t => {
 	t.snapshot(actual)
 })
 
+test('it ignores inline styles when `inlineStyles !== "include"`', async t => {
+	const actual = await extractCss(server.url + '/inline-style-html.html', {
+		inlineStyles: 'exclude'
+	})
+
+	t.false(actual.includes('[x-extract-css-inline-style] { color: red; font-size: 12px; }'))
+	t.false(actual.includes('[x-extract-css-inline-style] { color: blue }'))
+	t.is(actual, '')
+})
+
 test('it finds inline styles - JS', async t => {
 	const actual = await extractCss(server.url + '/inline-style-js.html')
 

From 752f6a6a1d00fb027811674e5bd6de0265d60a1b Mon Sep 17 00:00:00 2001
From: Bart Veneman <bart.veneman@drukwerkdeal.nl>
Date: Thu, 25 Nov 2021 20:26:41 +0100
Subject: [PATCH 2/2] update readme with docs for new option

---
 readme.md | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/readme.md b/readme.md
index 26c6419..fda4044 100644
--- a/readme.md
+++ b/readme.md
@@ -56,9 +56,9 @@ npm install extract-css-core
 yarn add extract-css-core
 ```
 
-## Problem, solution and shortcomings
+## Motivation, solution and shortcomings
 
-### Problem
+### Motivation
 
 Existing packages like
 [get-css](https://github.com/cssstats/cssstats/tree/master/packages/get-css)
@@ -95,6 +95,14 @@ Default: `exclude`
 
 Possible values: `exclude`, `include`
 
+#### inlineStyles
+
+Type: `String`
+
+Default: `include`
+
+Possible values: `exclude`, `include`
+
 #### waitUntil
 
 Type: `String`