The perform an observation algorithm's second step asserts:
index.bs:666
Assert: |traversable|'s active document is not fully active.
This appears inverted. A top-level traversable's active document is always fully active per HTML, and observation is meant to run against live pages currently in view (the surrounding prose says implementations reserve it "for when the user is interacting with a browser agent while web content is in view"). As written, the assertion would fail for exactly the documents you intend to observe.
For contrast, registerTool() rejects when the document is not fully active (index.bs:343), which confirms the intended polarity for an active, observable document.
Suggested fix:
Assert: |traversable|'s active document is fully active.
Introduced in #164.
The perform an observation algorithm's second step asserts:
index.bs:666
Assert: |traversable|'s active document is not fully active.
This appears inverted. A top-level traversable's active document is always fully active per HTML, and observation is meant to run against live pages currently in view (the surrounding prose says implementations reserve it "for when the user is interacting with a browser agent while web content is in view"). As written, the assertion would fail for exactly the documents you intend to observe.
For contrast, registerTool() rejects when the document is not fully active (index.bs:343), which confirms the intended polarity for an active, observable document.
Suggested fix:
Assert: |traversable|'s active document is fully active.
Introduced in #164.