extensions icon indicating copy to clipboard operation
extensions copied to clipboard

Update virustotal extension

Open 0xdhrv opened this issue 2 months ago • 3 comments

Description

Add support for Windows and maintenance

Closes https://github.com/raycast/extensions/issues/23876

Checklist

0xdhrv avatar Dec 22 '25 15:12 0xdhrv

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.

raycastbot avatar Dec 22 '25 15:12 raycastbot

@greptileai, could you please do a fresh review of this PR?

0xdhrv avatar Dec 22 '25 15:12 0xdhrv

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"] to package.json to officially support Windows
  • Removed node-fetch dependency in favor of native fetch (Node.js 18+), improving cross-platform compatibility
  • Converted to ES modules by adding "type": "module" and updating eslint.config.js from 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

greptile-apps[bot] avatar Dec 22 '25 15:12 greptile-apps[bot]

Any update on this?

SanketBaraiya avatar Dec 26 '25 04:12 SanketBaraiya

I'll take a look on this

0xdhrv avatar Dec 26 '25 08:12 0xdhrv

Published to the Raycast Store: https://raycast.com/sadv1r/virustotal

github-actions[bot] avatar Jan 02 '26 05:01 github-actions[bot]

:tada: :tada: :tada:

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

raycastbot avatar Jan 02 '26 05:01 raycastbot