refactor(tutorial): use isProductAttachment type guard instead of inline predicate

[ndhuutai]

Summary

• In 5-custom-attachment-type/App.tsx, hasProductMessage used an inline predicate (attachment) => 'type' in attachment && attachment.type === 'product' that duplicated the existing isProductAttachment type guard defined earlier in the same file.
• Replaced the inline predicate with a direct reference to isProductAttachment.

Test plan

• No behaviour change — purely a refactor to reuse the existing type guard.

Summary by CodeRabbit

Documentation

• Examples: Updated tutorial to demonstrate improved patterns for detecting and handling custom attachment types in message processing workflows.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 36857dd7-ea75-4ce4-bb8b-65dd353cee2f

📥 Commits

Reviewing files that changed from the base of the PR and between 2418025 and de62181.

📒 Files selected for processing (1)

• examples/tutorial/src/5-custom-attachment-type/App.tsx

---

📝 Walkthrough

Walkthrough

Tutorial App.tsx refactors product attachment detection to use a reusable isProductAttachment type guard instead of inline predicate logic when computing whether the channel has product messages.

Changes

Product Attachment Detection Refactor

Layer / File(s)	Summary
Type guard refactor for product attachment detection examples/tutorial/src/5-custom-attachment-type/App.tsx	hasProductMessage computation now delegates to the module's reusable isProductAttachment type guard via message.attachments?.some(isProductAttachment) instead of inline predicate code.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit tidies up the code,
One line, a cleaner road,
Type guards in place, no inline mess—
Refactoring? Success! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is incomplete according to the template; it lacks the required Goal, Implementation details, and UI Changes sections with proper headings.	Restructure the description using the template sections (🎯 Goal, 🛠 Implementation details, 🎨 UI Changes) instead of Summary and Test plan.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main refactoring change: replacing an inline predicate with a reusable type guard.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ 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 refactor/use-isproductattachment-type-guard

---

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 and usage tips.}

[github-actions]

Size Change: 0 B

Total Size: 656 kB

ℹ️ View Unchanged

Filename	Size
dist/cjs/audioProcessing.js	1.74 kB
dist/cjs/emojis.js	2.54 kB
dist/cjs/index.js	255 kB
dist/cjs/mp3-encoder.js	814 B
dist/cjs/ReactPlayerWrapper.js	545 B
dist/cjs/useNotificationApi.js	49.8 kB
dist/css/emoji-picker.css	178 B
dist/css/emoji-replacement.css	456 B
dist/css/index.css	39.7 kB
dist/es/audioProcessing.mjs	1.65 kB
dist/es/emojis.mjs	2.47 kB
dist/es/index.mjs	252 kB
dist/es/mp3-encoder.mjs	768 B
dist/es/ReactPlayerWrapper.mjs	485 B
dist/es/useNotificationApi.mjs	48.6 kB

_{compressed-size-action}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.98%. Comparing base (640bf4b) to head (de62181).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3213      +/-   ##
==========================================
+ Coverage   83.95%   83.98%   +0.03%     
==========================================
  Files         439      439              
  Lines       13222    13222              
  Branches     4293     4293              
==========================================
+ Hits        11100    11104       +4     
+ Misses       2122     2118       -4     

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.