Skip to content

Fix raw HTML in Google Calendar event descriptions - #2193

Open
martin-forge wants to merge 2 commits into
callumalpass:mainfrom
martin-forge:martin-forge/plain-text-google-calendar-descriptions
Open

Fix raw HTML in Google Calendar event descriptions#2193
martin-forge wants to merge 2 commits into
callumalpass:mainfrom
martin-forge:martin-forge/plain-text-google-calendar-descriptions

Conversation

@martin-forge

Copy link
Copy Markdown
Contributor

Problem

Google Calendar descriptions may contain HTML. TaskNotes treated the field as plain text, so users could see literal tags in event details, copied text, and generated notes.

Fix and benefit

Normalize descriptions once at the Google provider boundary using the Obsidian HTML sanitizer. Paragraphs, lists, and link destinations stay readable everywhere TaskNotes uses the description, while existing plain text—including angle-bracketed email addresses and placeholders—remains unchanged.

This complements #1882, which fixed TaskNotes-to-Google export; this PR fixes Google-to-TaskNotes input.

Validation

  • Focused service and utility tests: 50 passed
  • Typecheck, lint, and production test-vault build passed
  • Plugin reload reported no JavaScript errors
  • Full Jest suite has no new failures; all four failures reproduce on upstream main

Google Calendar descriptions may contain HTML, but TaskNotes uses them as plain text across event details, copied content, generated notes, and folder templates. Normalize them at the Google provider boundary while preserving paragraphs, lists, and link targets. Leave plain-text descriptions unchanged.
@martin-forge martin-forge changed the title Read Google Calendar descriptions as plain text Fix raw HTML in Google Calendar event descriptions Aug 3, 2026
The module only wires the Google provider boundary; ICS subscriptions read
descriptions through their own path and are not normalized here.
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