fix: resolve pre-existing Biome lint warnings in spicetifyWrapper.js

[tmchow]

Two pre-existing Biome lint warnings in jsHelper/spicetifyWrapper.js, discovered during #3773 review:

1. Line 426: Number.parseInt(i) missing explicit radix parameter. Added , 10 for base-10 parsing.
2. Line 2366: refreshCount destructured from useReducer but never read. Changed to [, refresh] to discard the unused value.

Both are mechanical fixes with no behavior change. The refresh function from the same destructuring is still used on line 2367.

This contribution was developed with AI assistance (Claude Code).

Summary by CodeRabbit

• Bug Fixes

  • Improved version number parsing logic for enhanced compatibility handling with Spicetify versions.

• Refactor

  • Removed unused variable binding to streamline code.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6eeb986c-262f-4701-a424-9b929322240e

📥 Commits

Reviewing files that changed from the base of the PR and between 1c77973 and 8a2b34f.

📒 Files selected for processing (1)

• jsHelper/spicetifyWrapper.js

---

📝 Walkthrough

Walkthrough

The changes refine version number parsing in a Cosmos proxy handler by adding an explicit base-10 radix to Number.parseInt(), and simplify a useReducer tuple assignment in _renderNavLinks by removing an unused binding for refreshCount.

Changes

Cohort / File(s)	Summary
Version Parsing & Variable Cleanup jsHelper/spicetifyWrapper.js	Modified Number.parseInt() call to include explicit radix parameter (base 10) for version component parsing; removed unused refreshCount variable from useReducer tuple assignment in _renderNavLinks.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A radix makes parsing so clear,
Ten's the base, no ambiguity here!
Unused bindings we swiftly discard,
Clean code hopping through the yard. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main changes: fixing pre-existing Biome lint warnings in spicetifyWrapper.js. It is clear, concise, and directly related to the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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