Update virustotal extension
Description
Add support for Windows and maintenance
Closes https://github.com/raycast/extensions/issues/23876
Checklist
- [x] I read the extension guidelines
- [x] I read the documentation about publishing
- [x] I ran
npm run buildand tested this distribution build in Raycast - [x] I checked that files in the
assetsfolder are used by the extension itself - [x] I checked that assets used by the
READMEare placed outside of themetadatafolder
Thank you for your contribution! :tada:
🔔 @sadv1r you might want to have a look.
You can use this guide to learn how to check out the Pull Request locally in order to test it.
📋 Quick checkout commands
BRANCH="ext/virustotal"
FORK_URL="https://github.com/0xdhrv/raycast-extensions.git"
EXTENSION_NAME="virustotal"
REPO_NAME="raycast-extensions"
git clone -n --depth=1 --filter=tree:0 -b $BRANCH $FORK_URL
cd $REPO_NAME
git sparse-checkout set --no-cone "extensions/$EXTENSION_NAME"
git checkout
cd "extensions/$EXTENSION_NAME"
npm install && npm run dev
Due to our current reduced availability, the initial review may take up to 10-15 business days.
@greptileai, could you please do a fresh review of this PR?
Greptile Summary
This PR adds Windows platform support to the VirusTotal extension by migrating from node-fetch to Node.js's native fetch API and converting the codebase to ES modules.
Key Changes:
- Added
"platforms": ["macOS", "Windows"]topackage.jsonto officially support Windows - Removed
node-fetchdependency in favor of native fetch (Node.js 18+), improving cross-platform compatibility - Converted to ES modules by adding
"type": "module"and updatingeslint.config.jsfrom CommonJS to ES module syntax - Updated dependencies (
@raycast/api,@raycast/utils,eslint, and dev dependencies) - Properly formatted CHANGELOG with
{PR_MERGE_DATE}placeholder in descending version order
Impact: The changes enable Windows users to use this extension. The migration to native fetch and ES modules modernizes the codebase without affecting functionality.
Confidence Score: 5/5
- This PR is safe to merge with no risk
- All changes are clean maintenance updates: removing an external dependency in favor of native APIs, proper ES module migration, and adding platform support. The CHANGELOG follows the required format. No logical errors or breaking changes introduced.
- No files require special attention
Important Files Changed
| Filename | Overview |
|---|---|
| extensions/virustotal/src/client.ts | Removed node-fetch dependency in favor of native fetch API, enabling better cross-platform compatibility |
| extensions/virustotal/package.json | Added Windows platform support, removed node-fetch dependency, updated to ES modules with "type": "module", and updated dependencies |
| extensions/virustotal/eslint.config.js | Converted from CommonJS to ES modules syntax to match package.json module type |
| extensions/virustotal/CHANGELOG.md | Added new version entry with proper {PR_MERGE_DATE} placeholder and maintained descending order |
Any update on this?
I'll take a look on this
Published to the Raycast Store: https://raycast.com/sadv1r/virustotal
:tada: :tada: :tada:
We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.