diff --git a/README.md b/README.md
index 5a4d3f7..10a5111 100644
--- a/README.md
+++ b/README.md
@@ -6,7 +6,7 @@ PHP port of *Mozilla's* **[Readability.js](https://github.com/mozilla/readabilit
-Version 4.0 is a ground-up rewrite on PHP's native HTML parser ([Lexbor, included in PHP 8.4's DOM extension](https://blog.keyvan.net/p/parsing-html-with-php-84)), transcribed method-for-method from Readability.js v0.6.0. It parses HTML the way modern browsers do, needs no third-party parsing library, and is tested against Mozilla's own test corpus.
+Version 4.0 is a ground-up rewrite, produced using [Claude](https://claude.com/claude-code) (Anthropic's AI coding tool), to bring the code in line with the latest version of Readability.js (v0.6.0, transcribed method-for-method) and to take advantage of the new, faster native HTML parser introduced in PHP 8.4 ([Lexbor, included in the DOM extension](https://blog.keyvan.net/p/parsing-html-with-php-84)) and the new WHATWG URL parser introduced in PHP 8.5. It parses HTML the way modern browsers do, needs no third-party HTML parsing library, and is tested against Mozilla's own test corpus.
**Original Developer**: Andres Rey
@@ -82,16 +82,29 @@ foreach ($article->contentElement->querySelectorAll('img[src]') as $img) {
If you already have a `\Dom\HTMLDocument` (for example because you want to pre-process it), use `parseDocument()` instead of `parse()`. Note that the document is modified in place while the article is extracted.
-There is also a port of Mozilla's `isProbablyReaderable`, a quick check for whether it's worth running the full parse:
+### Checking if a page is readerable
+
+There is also a port of Mozilla's `isProbablyReaderable`: a quick-and-dirty way of figuring out if it's plausible that a page contains an article, without the cost of running the full parse. Like the original, it can produce both false positives and false negatives, but it's cheap enough to run on pages as they come in:
```php
use fivefilters\Readability\Readerable;
+// Only run the full parse if we suspect it will produce a meaningful result.
if (Readerable::isProbablyReaderable($html)) {
- // ...
+ $article = new Readability(new Configuration())->parse($html);
}
```
+It accepts an HTML string or a `\Dom\HTMLDocument`, and takes the same optional tuning parameters as Readability.js (same defaults):
+
+- **minContentLength**: default `140`, the minimum node content length used to decide if the document is readerable;
+- **minScore**: default `20`, the minimum cumulated 'score' used to determine if the document is readerable;
+- **visibilityChecker**: default `Readerable::isNodeVisible(...)`, the function used to determine if a node is visible.
+
+```php
+Readerable::isProbablyReaderable($html, minScore: 0, minContentLength: 120);
+```
+
## Options
Configuration is a readonly object; pass options as named constructor arguments (or as an array via `Configuration::fromArray()`):
diff --git a/UPGRADE.md b/UPGRADE.md
index e84b46f..7a6e1a9 100644
--- a/UPGRADE.md
+++ b/UPGRADE.md
@@ -181,7 +181,13 @@ Debug messages are sent to the logger independently of the `debug` flag (which o
## Behavior changes to be aware of
- **`parse()` never returns `false`/`bool`.** It returns an `Article` or throws `ParseException` — including for empty input, documents over `maxElemsToParse`, and pages where no article could be found (all cases where Readability.js returns `null`).
-- **The content is wrapped** in `…
`, exactly as Readability.js outputs. If you post-process the HTML, account for the wrapper.
+- **The content is wrapped** in `…
`, exactly as Readability.js outputs. 3.x serialized the extracted elements with no wrapper around them. If you post-process the HTML, account for the wrapper — or reproduce the unwrapped 3.x output with:
+
+ ```php
+ $html = $article->contentElement->firstElementChild->innerHTML;
+ ```
+
+ (`firstElementChild` is the wrapper `div` itself — always the only child of `contentElement` — and `innerHTML` serializes everything inside it, so nothing is lost when the article consists of several top-level elements.)
- **The byline is always extracted, and inline bylines are removed from the content by default** (see `keepInlineByline` above). A 3.x install that never set `articleByline` kept the byline in the content.
- **Relative URL fixing follows the WHATWG URL Standard** (what browsers and Readability.js do), via PHP 8.5's native `Uri\WhatWg\Url` or rowbot/url on PHP 8.4. Edge-case outputs may differ slightly from 3.x's RFC 3986 resolution (e.g. `https://example.com` serializes as `https://example.com/`).
- **Encoding:** input is parsed with the encoding declared in the document, defaulting to UTF-8 (as a browser would). The 3.x mb_* guessing hacks are gone — supply UTF-8 or make sure the document declares its charset.
diff --git a/src/Readerable.php b/src/Readerable.php
index 85b39c7..d580355 100644
--- a/src/Readerable.php
+++ b/src/Readerable.php
@@ -17,14 +17,14 @@ final class Readerable
* Mirrors isProbablyReaderable.
*
* @param \Dom\HTMLDocument|string $document the document, or an HTML string
- * @param int $minScore the minimum cumulated 'score' used to determine if the document is readerable
+ * @param float $minScore the minimum cumulated 'score' used to determine if the document is readerable
* @param int $minContentLength the minimum node content length used to decide if the document is readerable
* @param callable(\Dom\Element): bool|null $visibilityChecker the function used to determine if a node is visible
* @return bool Whether or not we suspect Readability::parse() will succeed at returning an article
*/
public static function isProbablyReaderable(
\Dom\HTMLDocument|string $document,
- int $minScore = 20,
+ float $minScore = 20,
int $minContentLength = 140,
?callable $visibilityChecker = null,
): bool {
diff --git a/test/ReaderableTest.php b/test/ReaderableTest.php
new file mode 100644
index 0000000..8c303ef
--- /dev/null
+++ b/test/ReaderableTest.php
@@ -0,0 +1,100 @@
+' . str_repeat('hello there ', $repeat) . '