[xsahil03x]

StreamMessageText now links web addresses written without a scheme, such as youtube.com, as the iOS, Android and React Native chat SDKs already do (FLU-1094).

What changed

stream_core

• Adds findLinks, findLinkAt and LinkMatch. They find web links with or without a scheme. An address without one counts only when it ends in an IANA top-level domain, so config.json stays plain text.
• The domain list is generated from IANA by melos run generate:tlds, a part of link_finder.dart that only the finder uses. The update_top_level_domains workflow regenerates it monthly and opens a PR when it changes.
• This brings back a bundled top-level domain list on purpose, after chore(ui): drop the bundled IANA TLD list stream-chat-flutter#2654 (FLU-490) removed one. That list only double-checked addresses that already had a scheme. Here the list is the only thing that tells youtube.com apart from config.json, and the monthly refresh deals with it going stale.
• It lives in stream_core rather than stream_core_flutter so that v11's stream_chat and stream_chat_flutter_core, which depend only on stream_core, can use the same matcher.

stream_core_flutter (now depends on stream_core)

• StreamMessageText links every web address through findLinkAt, using one LinkSyntax subclass.
• Addresses without a scheme, www. ones included, open over https://, the scheme the backend's preview scraper and stream-chat-js's composer already use. Before, www. opened over http://.
• Addresses right after a quote or Chinese/Japanese text are now linked.
• An address inside a link label stays part of that link. This hooks into markdown's own createNode, so inline, reference, nested and escaped labels all behave.
• Custom inlineSyntaxes still take priority. Email and ftp:// links still come from markdown's autolinks.

How it was verified

• Tests: 66 finder tests in stream_core and 18 widget tests that tap the rendered link. Every guard in the finder (21) and in the syntax (6) was broken one at a time and caught by a test.

• Analyze, format and check:barrels are clean. The full stream_core_flutter suite fails only its 57 goldens locally, the same count as on main.

• packages/stream_core/benchmark/link_finder_benchmark.dart (benchmark_harness, AOT):

  Input	Time per call
  Typical message (~200 chars)	29 µs
  3.6k prose with links	141 µs
  5k Cyrillic	1.6 ms
  10k crafted CJK + dots	4.8 ms
  20k ) or .	0.09 ms

  Every case scales linearly with length, including inputs built to slow a link scanner down.

• Markdown parse with the syntax compared to without (AOT): messages with links parse faster, because it takes over from the GFM autolinker. Link-free text costs +5 µs on a typical message and +53 µs per 1k characters. A widget pump comparison shows no measurable difference for messages with links.

🤖 Generated with Claude Code

https://claude.ai/code/session_012vTgFwqhUc2wKCWrw635yC

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 39 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 52f97a6e-de08-4601-a7ad-1dc2f0af05c0

Commits

Reviewing files that changed from the base of the PR and between 3fdd421 and 66884fb.

Files selected for processing: 15

• .github/workflows/update_top_level_domains.yml
• STYLE_GUIDE.md
• melos.yaml
• packages/stream_core/CHANGELOG.md
• packages/stream_core/benchmark/link_finder_benchmark.dart
• packages/stream_core/lib/src/utils.dart
• packages/stream_core/lib/src/utils/link_finder.dart
• packages/stream_core/lib/src/utils/link_finder.tlds.g.dart
• packages/stream_core/pubspec.yaml
• packages/stream_core/test/utils/link_finder_test.dart
• packages/stream_core_flutter/CHANGELOG.md
• packages/stream_core_flutter/lib/src/components/message/stream_message_text.dart
• packages/stream_core_flutter/pubspec.yaml
• packages/stream_core_flutter/test/components/message/stream_message_text_links_test.dart
• scripts/generate_top_level_domains.dart

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.}

[codecov]

Codecov Report

❌ Patch coverage is 97.80220% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.98%. Comparing base (3fdd421) to head (66884fb).

Files with missing lines	Patch %	Lines
...ackages/stream_core/lib/src/utils/link_finder.dart	97.45%	4 Missing ⚠️

Additional details and impacted files

@@                    Coverage Diff                    @@
##           ci/pana-sibling-paths     #205      +/-   ##
=========================================================
+ Coverage                  71.46%   71.98%   +0.51%     
=========================================================
  Files                        215      216       +1     
  Lines                       8884     9066     +182     
=========================================================
+ Hits                        6349     6526     +177     
- Misses                      2535     2540       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[VelikovPetar]

The generator looks solid (it validates the download, writes nothing on failure, and skips no-op writes). My concern is that nothing ever runs it. It isn't part of generate:all and isn't in CI, so "periodically" in practice means "when someone reports a link that doesn't work". That's the same staleness problem that got the old list removed in GetStream/stream-chat-flutter#2654.

Since the script already makes no change when the set is unchanged, a scheduled GitHub Action (monthly, say) could run melos run generate:tlds and open a PR whenever the file changes. Updates would cost nothing and stay reviewable. If that's out of scope for this PR, could we at least add a ticket or a line in the release checklist so it doesn't depend on someone remembering?

[xsahil03x]

Added in 66884fb. The update_top_level_domains workflow runs the generator on the 1st of every month and opens a chore(repo) PR with the bot's token, so its checks run. Since the script leaves the file alone when the set hasn't changed, a quiet month opens nothing. It can also be triggered by hand.