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

[ndhuutai]

Summary

• In 5-custom-attachment-type/App.tsx, hasProductMessage was using 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

• Refactor
  • Simplified attachment-type detection logic in the tutorial example by consolidating how product attachments are validated and filtered from channel messages.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f5380e5f-a1b6-4a9c-ab15-e78f22639c2b

📥 Commits

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

📒 Files selected for processing (1)

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

Disabled knowledge base sources:

• Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

---

📝 Walkthrough

Walkthrough

Tutorial example refactored to use the existing isProductAttachment type guard instead of an inline type check when detecting product attachments in channel messages.

Changes

Attachment type guard consolidation

Layer / File(s)	Summary
Use isProductAttachment type guard in hasProductMessage examples/tutorial/src/5-custom-attachment-type/App.tsx	The hasProductMessage computation now uses the existing isProductAttachment type guard to check whether message attachments contain a product, replacing the previous inline attachment.type === 'product' check.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A type guard once scattered and lone,
Now bundled with others, no longer on its own,
The tutorial hops cleaner, code shares its refrain,
One simple extraction removes the domain's strain,
Simplicity blooms in the garden of change!

✨ 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 claude/inspiring-nobel-0675d3

---

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.90%. Comparing base (640bf4b) to head (b5a2d9a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3212      +/-   ##
==========================================
- Coverage   83.95%   83.90%   -0.05%     
==========================================
  Files         439      439              
  Lines       13222    13222              
  Branches     4293     4293              
==========================================
- Hits        11100    11094       -6     
- Misses       2122     2128       +6     

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