diff --git a/packages/safe-chain/src/packagemanager/bun/createBunPackageManager.js b/packages/safe-chain/src/packagemanager/bun/createBunPackageManager.js index 88c84f5..9716261 100644 --- a/packages/safe-chain/src/packagemanager/bun/createBunPackageManager.js +++ b/packages/safe-chain/src/packagemanager/bun/createBunPackageManager.js @@ -12,7 +12,7 @@ export function createBunPackageManager() { // For bun, we use the proxy-only approach to block package downloads, // so we don't need to analyze commands. isSupportedCommand: () => false, - getDependencyUpdatesForCommand: async () => [], + getDependencyUpdatesForCommand: () => [], }; } @@ -26,7 +26,7 @@ export function createBunxPackageManager() { // For bunx, we use the proxy-only approach to block package downloads, // so we don't need to analyze commands. isSupportedCommand: () => false, - getDependencyUpdatesForCommand: async () => [], + getDependencyUpdatesForCommand: () => [], }; } diff --git a/packages/safe-chain/src/packagemanager/currentPackageManager.js b/packages/safe-chain/src/packagemanager/currentPackageManager.js index ef7e9aa..390d4d1 100644 --- a/packages/safe-chain/src/packagemanager/currentPackageManager.js +++ b/packages/safe-chain/src/packagemanager/currentPackageManager.js @@ -17,11 +17,18 @@ const state = { packageManagerName: null, }; +/** + * @typedef {Object} GetDependencyUpdatesResult + * @property {string} name + * @property {string} version + * @property {string} type + */ + /** * @typedef {Object} PackageManager * @property {(args: string[]) => Promise<{ status: number }>} runCommand * @property {(args: string[]) => boolean} isSupportedCommand - * @property {(args: string[]) => Promise<{name: string, version: string, type: string}[]>} getDependencyUpdatesForCommand + * @property {(args: string[]) => Promise | GetDependencyUpdatesResult[]} getDependencyUpdatesForCommand */ /** diff --git a/packages/safe-chain/src/packagemanager/npm/createPackageManager.js b/packages/safe-chain/src/packagemanager/npm/createPackageManager.js index 465bf60..fa72276 100644 --- a/packages/safe-chain/src/packagemanager/npm/createPackageManager.js +++ b/packages/safe-chain/src/packagemanager/npm/createPackageManager.js @@ -28,7 +28,7 @@ export function createNpmPackageManager() { /** * @param {string[]} args * - * @returns {Promise<{name: string, version: string, type: string}[]>} + * @returns {ReturnType} */ function getDependencyUpdatesForCommand(args) { const scanner = findDependencyScannerForCommand( diff --git a/packages/safe-chain/src/packagemanager/npm/dependencyScanner/commandArgumentScanner.js b/packages/safe-chain/src/packagemanager/npm/dependencyScanner/commandArgumentScanner.js index f19449b..c4f6bb6 100644 --- a/packages/safe-chain/src/packagemanager/npm/dependencyScanner/commandArgumentScanner.js +++ b/packages/safe-chain/src/packagemanager/npm/dependencyScanner/commandArgumentScanner.js @@ -16,7 +16,7 @@ import { hasDryRunArg } from "../utils/npmCommands.js"; /** * @typedef {Object} CommandArgumentScanner - * @property {(args: string[]) => Promise} scan + * @property {(args: string[]) => Promise | ScanResult[]} scan * @property {(args: string[]) => boolean} shouldScan */ diff --git a/packages/safe-chain/src/packagemanager/npm/dependencyScanner/nullScanner.js b/packages/safe-chain/src/packagemanager/npm/dependencyScanner/nullScanner.js index 449fed4..5c1d3bd 100644 --- a/packages/safe-chain/src/packagemanager/npm/dependencyScanner/nullScanner.js +++ b/packages/safe-chain/src/packagemanager/npm/dependencyScanner/nullScanner.js @@ -3,7 +3,7 @@ */ export function nullScanner() { return { - scan: async () => [], + scan: () => [], shouldScan: () => false, }; } diff --git a/packages/safe-chain/src/packagemanager/pnpm/createPackageManager.js b/packages/safe-chain/src/packagemanager/pnpm/createPackageManager.js index 193470b..c3046c8 100644 --- a/packages/safe-chain/src/packagemanager/pnpm/createPackageManager.js +++ b/packages/safe-chain/src/packagemanager/pnpm/createPackageManager.js @@ -41,7 +41,7 @@ export function createPnpxPackageManager() { /** * @param {string[]} args * @param {boolean} isPnpx - * @returns {Promise} + * @returns {ReturnType} */ function getDependencyUpdatesForCommand(args, isPnpx) { if (isPnpx) {