Skip to content

fix(html): isolate annotation snapshots and option updates - #1173

Merged
andiwand merged 1 commit into
mainfrom
review/155-annotation-state-ownership
Oct 6, 2026
Merged

andiwand merged 1 commit into
mainfrom
review/155-annotation-state-ownership

Conversation

@andiwand

@andiwand andiwand commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

🤖 Generated with Claude Code

Mutating annotation.list() or an options array could silently change pending annotations or pointer handling. Return independent snapshots and copy incoming option arrays. Validate all option names before applying an update, so an unknown key leaves settings unchanged.

Extended the existing highlight, ink and pointer-policy checks instead of adding duplicate fixtures. The parent failed five assertions; all 85 annotation checks now pass in Chrome. JavaScript syntax and the native asset-embedding build also pass.

@andiwand
andiwand force-pushed the review/154-python-build-minimum branch from dc60a00 to 90b6420 Compare October 6, 2026 19:34
Base automatically changed from review/154-python-build-minimum to main October 6, 2026 19:34
@andiwand
andiwand force-pushed the review/155-annotation-state-ownership branch from a68684c to 36f101f Compare October 6, 2026 19:35
@andiwand
andiwand merged commit fb32ac2 into main Oct 6, 2026
18 of 23 checks passed
@andiwand
andiwand deleted the review/155-annotation-state-ownership branch October 6, 2026 19:35
andiwand added a commit that referenced this pull request Oct 6, 2026
Both output repositories carry the pdf-annotation.js from #1171 and
#1173. A full run on 078390a matched every reference page byte for byte;
only that script changed. #1178, merged since, changes how a file copy
is written, not its bytes.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MxyTMutqSUJRGfxA8CyzMc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant