fix: use file icon for LICENSE files

[gioboa]

🔗 Linked issue

Close #2558

📚 Description

Use file icon for LICENSE files.

Before:

After:

Before creating the pull request, please make sure you do the following:

• Check that there isn't already a PR that solves the problem the same way. If you find a duplicate, please help us reviewing it.
• Ensure that PR title follows conventional commits (https://www.conventionalcommits.org)
• Update the corresponding documentation if needed.
• Include relevant tests that fail without this PR but pass with it.
• Add any additional context, tradeoffs, follow-ups, or things reviewers should be aware of.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Apr 18, 2026 1:19pm

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Apr 18, 2026 1:19pm
npmx-lunaria	Ignored		Apr 18, 2026 1:19pm

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

• Style

  • Updated the visual icon displayed for LICENSE files to use the standard default file icon.

• Tests

  • Added test coverage for LICENSE file icon mappings.

Walkthrough

The icon mapping for the LICENSE family of filenames (including LICENSE, LICENSE.md, LICENSE.txt, and their lowercase variants) is updated from vscode-icons-file-type-license to vscode-icons-default-file. Test coverage is expanded to validate these mappings.

Changes

Cohort / File(s)	Summary
Icon mapping updates app/utils/file-icons.ts	Updated FILENAME_ICONS entries for LICENSE files (LICENSE, LICENSE.md, LICENSE.txt, and lowercase variants) to use the default file icon instead of the specialised license icon.
Test coverage expansion test/unit/app/utils/file-icons.spec.ts	Added test assertions for exact-filename matching of LICENSE files across common casing and extension combinations, verifying all variants resolve to vscode-icons-default-file.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix: use file icon for LICENSE files' clearly and concisely describes the main change in the changeset.
Description check	✅ Passed	The description is directly related to the changeset, explaining the icon change for LICENSE files with before/after screenshots.
Linked Issues check	✅ Passed	The PR successfully addresses the requirement in #2558 by implementing file icon display for extensionless LICENSE files and variants with extensions.
Out of Scope Changes check	✅ Passed	All changes are scoped to the requirements in #2558: updating LICENSE file icon mappings and corresponding unit tests.

_{✏️ 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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@app/utils/file-icons.ts`:
- Around line 267-272: The FILENAME_ICONS mapping currently forces all LICENSE
variants to the generic icon; change it to only override extensionless LICENSE
names so extension-based fallbacks still apply: remove the keys 'LICENSE.md',
'LICENSE.txt', 'license.md', and 'license.txt' from the mapping in
app/utils/file-icons.ts (leave 'LICENSE' and 'license' if you want to target
only extensionless files), so getFileIcon's extension-based logic will return
markdown/text icons for .md/.txt files.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 53cb6251-b574-440a-bb23-eb800880cce9

📥 Commits

Reviewing files that changed from the base of the PR and between d9f8d56 and 64afe72.

📒 Files selected for processing (2)

• app/utils/file-icons.ts
• test/unit/app/utils/file-icons.spec.ts

[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!

[connorshea]

I don't think this is actually implementing the issue as-intended?

[gioboa]

Would it be possible to show a file icon for it?

Based on this description, I implemented it. Let me know if it's a different thing @nicolo-ribaudo

[ghostdevv]

hey, thank you for the PR! unfortunately, I think the issue was that the icon isn't showing up not that it should use the default file type. I had a look into why it's not showing up but haven't figured it out yet. Will close this for now and post my findings in the original issue 🙏

[gioboa]

Thanks @ghostdevv for your review 🙏