fix(chat): keep find stable across footnotes and pending chats, and match mention chips next to punctuation

[waleedlatif1]

Summary

• Chat find closed and cleared its query when a pending chat adopted its persisted id (undefined → id), although that's the same conversation. It now resets only when an existing scope changes
• Footnotes made the find index and the rendered DOM disagree on match counts and order: the index kept footnote definitions, while the DOM showed the marker, the moved definitions, a visually hidden heading, and back-links. Footnote definitions are now left out of the index, and [data-footnote-ref] / [data-footnotes] are excluded from the DOM walk. The chat's link renderer now forwards data-footnote-ref so the marker can be excluded
• Mention chips:
  • slash_command contexts render with the / prefix
  • the token match accepts trailing punctuation via a lookahead, so @Workflow, and back-to-back repeats (@A @A) now match, while @Workflow-2 still doesn't match @Workflow
  • overlapping matches resolve to the longest chip (@report vs @report.pdf, Workflow vs Workflow 2)
• The @/blocks/integration-matcher test mock moves to one central @sim/testing mock used by all four suites
• Responds to review threads on v0.9.4: db contention fixes, databricks genie, snowflake cortex, additional search connectors #8356

Type of Change

• Bug fix

Testing

• use-chat-find.test.tsx renders real ChatContent markup: find survives undefined → id but resets on a chat switch, and a footnoted message's match count equals its highlighted ranges. Each fails with its guard reverted
• utils.test.ts: punctuation, repeats, slash commands, the prefix boundary, and longest-overlap cases. The behavior cases fail on staging
• Cleanup passes (effects, state, memo, callback, React Query, EMCN, URL state, comments, test audit) found nothing
• Home, chat, and composer suites (238 tests), type-check, lint, and check:audits pass

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Sep 28, 2026 1:29am UTC

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Adjusts chat search and mention matching logic.

The PR appears safe to merge; no new actionable issue was identified in the changes since the previous review.

Summary

The PR keeps chat find open when a pending conversation receives its persisted ID, aligns footnote search results with rendered text, and improves mention-chip matching around punctuation and overlapping names. It also centralizes the integration matcher test mock and adds regression tests.

_{Reviews (2) · Last reviewed commit: "fix(chat): only treat a period as a ment..."}

[cubic-dev-ai]

All reported issues were addressed across 11 files

<sub>Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 11 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}