Notifications: FE prep for real message content

[romanivan-flamingo]

ClickUp: https://app.clickup.com/t/86ajn8hpg (parent PSA-loop task 86ajn8hmp)

FE side of "Notifications do not show the real message content". BE counterpart (merged): openframe-oss-lib#1741, openframe-saas-lib#707, openframe-saas-tenant#2552.

What this PR does

Content rendering (BE-independent):

• Defensive markdown/HTML strip - stripNotificationMarkup() in notifications-helpers.ts removes tags, links/images (keeping their text), bold/italic/strike, code, headings, blockquotes and list markers from title/description. Applied in mapNotificationNode (drawer tiles, popups), the NATS live handler (live tiles + browser OS toast) and the /notifications table rows. Newlines are preserved so the full-text hover tooltip keeps its paragraph structure.
• NATS createdAt - live notifications use the payload timestamp instead of stamping local Date.now(); local time remains the fallback.
• Table rendering priority - the real title leads the first column; the context-derived kind label moved to the details column.

BE schema adoption (per the merged BE PRs):

• schema.graphql - additive sync: ticketTitle on ticket-bearing contexts, nullable ticketId/ticketNumber/ticketTitle on ClientAiMessageContext.
• Shared fragment (notification-fields.ts) selects ClientAiMessageContext { dialogId, clientTicketId: ticketId } (aliased - relay refuses to merge ID! with ID); the mapper folds it into meta.ticketId.
• Navigation - CLIENT_AI_MESSAGE notifications with a ticket open the ticket's client chat tab; TICKET_ESCALATED_BY_USER (a first-class tile in the new Figma blocks) now selects ticketId and routes to Ticket Details.

Merge with main (ee035d5)

Main restructured the same area while this branch was open; the merge re-applies the changes onto the new structure (shared notification-fields.ts fragment, readNotificationNode read-once rows, generated-type mapper). The notification-settings surface deleted on main stays deleted.

Content suppression: dropped by BE before merge

Earlier commits of this branch carried a tenant-wide "Hide Message Content" toggle and its schema. The suppression parts of openframe-oss-lib#1741 were cut from the PR before it merged (verified against merge commit ce246608: no contentSuppressed, no updateNotificationContentSuppression, no redactor on main) - BE dropped notification settings "for now". All suppression traces are removed from this PR; enabled/typeSettings SDL stays as it is live on BE main.

Verification (QA, real data)

• Real content across the table, drawer and popups: ticket number + title as the headline, message excerpts with host attribution, approval summaries, word-boundary truncation with ellipsis; no markdown artifacts in fresh records.
• Escalated-ticket notification navigates to the ticket and auto-reads on open (unread 7 -> 6, record lands in History).
• npm run relay, npm run generate-enums, npm run type-check, Biome - clean. No console errors.
• Pending: live NATS popup / browser push spot-check with a fresh event; FCM/email verification on the BE side.

🤖 Generated with Claude Code

[coderabbitai]

📝 Walkthrough

Walkthrough

Notification text is sanitized across mapped and live notification paths. Notification timestamps use normalized or payload values. The table displays the actual title and shows the notification type before the description.

Changes

Notification normalization and display

Layer / File(s)	Summary
Notification markup normalization src/graphql/notifications/notifications-helpers.ts	Adds stripNotificationMarkup and applies it to mapped notification titles and descriptions.
Notification data path normalization src/app/(app)/notifications/components/notifications-section.tsx, src/app/components/notifications/notifications-data-provider.tsx	Notification rows use mapped text and normalized timestamps. Live notifications sanitize text and prefer the payload timestamp.
Notification column display src/app/(app)/notifications/components/notifications-columns.tsx	The title column renders the notification title. The details column renders the type before the description.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Suggested reviewers: aliaska-varieva

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the frontend changes that prepare notifications to display real message content.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch hotfix/notifications-real-content-fe

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/graphql/notifications/notifications-helpers.ts`:
- Line 241: Update the normalization regex in the notification helper to avoid
treating underscores adjacent to word characters as italic delimiters,
preserving identifiers such as job_status_failed while still stripping valid
underscore-delimited markup. Add coverage for both identifier text and
snake_case content.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 8db850b4-b5d8-4213-a528-b6ac9f56b540

📥 Commits

Reviewing files that changed from the base of the PR and between 93cbeda and 5edc59e.

📒 Files selected for processing (4)

• src/app/(app)/notifications/components/notifications-columns.tsx
• src/app/(app)/notifications/components/notifications-section.tsx
• src/app/components/notifications/notifications-data-provider.tsx
• src/graphql/notifications/notifications-helpers.ts