refactor: add missing IconClass type to icon props

[C1ANCYSz]

🔗 Linked issue

Partially resolves #1256

🧭 Context

The codebase already had an IconClass type defined in ~/types/icon
for Iconify icon class strings (e.g. i-lucide:home). However, some
component props that accept icon classes were still typed as plain string,
missing out on type safety.

📚 Description

Applied the existing IconClass type to props that accept Iconify icon
class strings in two components:

• app/components/CollapsibleSection.vue: icon?: string → icon?: IconClass
• Diff/SidebarPanel.vue: icon: string → icon: IconClass inside
  the sectionMeta computed record

Full resolution of #1256 would require a dedicated icon component that
enforces IconClass usage across the whole codebase, as noted by maintainers.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev (canary)	Ready	Preview, Comment	Apr 10, 2026 9:50pm

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Apr 10, 2026 9:50pm
npmx-lunaria	Ignored		Apr 10, 2026 9:50pm

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: efc4ef04-33cb-41c9-92bf-08b46434329d

📥 Commits

Reviewing files that changed from the base of the PR and between 64c2017 and 6b5ec24.

📒 Files selected for processing (2)

• app/components/CollapsibleSection.vue
• app/components/Diff/SidebarPanel.vue

✅ Files skipped from review due to trivial changes (2)

• app/components/CollapsibleSection.vue
• app/components/Diff/SidebarPanel.vue

---

📝 Walkthrough

Walkthrough

This pull request tightens TypeScript typings for icon properties in two Vue components: app/components/CollapsibleSection.vue now types its public icon prop as IconClass (imported from ~/types/icon) instead of string, and app/components/Diff/SidebarPanel.vue updates the sectionMeta computed type so each section’s icon is IconClass. No runtime logic, rendering, or control flow was changed.

Possibly related PRs

• chore: add type constraints for icon class names #1538: Adds and applies the IconClass type across components, the same code-level change used here.

Suggested reviewers

• danielroe

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	The PR partially addresses issue #1256 by applying existing IconClass type to props, but does not implement the suggested template-literal type pattern.	Clarify whether the existing IconClass type matches the template-literal approach suggested in #1256, or if further refinement is needed to fully satisfy the typing requirements.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description accurately relates to the changeset, explaining the context of adding IconClass type to icon props in two components.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to updating icon prop types in two components as described in the PR objectives and linked issue.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[ghostdevv]

Good catch! The import path just needs updating - also in future could you create a branch to PR from instead of using your main branch, as it makes it easier for us 🙏

[C1ANCYSz]

Thanks
Noted on the branch, will follow that going forward.