fix: apply suggestion by @coyotte508

Author: axel7083

packages/hub/src/lib/cache-management.ts

@@ -16,7 +16,7 @@ function getHuggingFaceHubCache(): string {
 	return process.env["HUGGINGFACE_HUB_CACHE"] ?? getDefaultCachePath();
 }
 
-export function getHFHubCache(): string {
+export function getHFHubCachePath(): string {
 	return process.env["HF_HUB_CACHE"] ?? getHuggingFaceHubCache();
 }
 
@@ -70,7 +70,7 @@ export interface HFCacheInfo {
 }
 
 export async function scanCacheDir(cacheDir: string | undefined = undefined): Promise<HFCacheInfo> {
-	if (!cacheDir) cacheDir = getHFHubCache();
+	if (!cacheDir) cacheDir = getHFHubCachePath();
 
 	const s = await stat(cacheDir);
 	if (!s.isDirectory()) {

packages/hub/src/lib/download-file-cache.spec.ts renamed to packages/hub/src/lib/download-file-to-cache-dir.spec.ts

@@ -4,9 +4,9 @@ import { dirname, join } from "node:path";
 import { lstat, mkdir, stat, symlink, writeFile, rename } from "node:fs/promises";
 import { pathsInfo } from "./paths-info";
 import type { Stats } from "node:fs";
-import { getHFHubCache, getRepoFolderName } from "./cache-management";
+import { getHFHubCachePath, getRepoFolderName } from "./cache-management";
 import { toRepoId } from "../utils/toRepoId";
-import { downloadFileToCacheDir } from "./download-file-cache";
+import { downloadFileToCacheDir } from "./download-file-to-cache-dir";
 
 vi.mock('node:fs/promises', () => ({
 	writeFile: vi.fn(),
@@ -34,7 +34,7 @@ function _getBlobFile(params:  {
 	etag: string;
 	cacheDir?: string, // default to {@link getHFHubCache}
 }) {
-	return join(params.cacheDir ?? getHFHubCache(), getRepoFolderName(toRepoId(params.repo)), "blobs", params.etag);
+	return join(params.cacheDir ?? getHFHubCachePath(), getRepoFolderName(toRepoId(params.repo)), "blobs", params.etag);
 }
 
 // utility test method to get snapshot file path
@@ -44,7 +44,7 @@ function _getSnapshotFile(params:  {
 	revision : string;
 	cacheDir?: string, // default to {@link getHFHubCache}
 }) {
-	return join(params.cacheDir ?? getHFHubCache(), getRepoFolderName(toRepoId(params.repo)), "snapshots", params.revision, params.path);
+	return join(params.cacheDir ?? getHFHubCachePath(), getRepoFolderName(toRepoId(params.repo)), "snapshots", params.revision, params.path);
 }
 
 describe('downloadFileToCacheDir', () => {

packages/hub/src/lib/download-file-cache.ts renamed to packages/hub/src/lib/download-file-to-cache-dir.ts

@@ -1,4 +1,4 @@
-import { getHFHubCache, getRepoFolderName } from "./cache-management";
+import { getHFHubCachePath, getRepoFolderName } from "./cache-management";
 import { dirname, join } from "node:path";
 import { writeFile, rename, symlink, lstat, mkdir, stat } from "node:fs/promises";
 import type { CommitInfo, PathInfo } from "./paths-info";
@@ -63,7 +63,7 @@ export async function downloadFileToCacheDir(
 ): Promise<string> {
 	// get revision provided or default to main
 	const revision = params.revision ?? "main";
-	const cacheDir = params.cacheDir ?? getHFHubCache();
+	const cacheDir = params.cacheDir ?? getHFHubCachePath();
 	// get repo id
 	const repoId = toRepoId(params.repo);
 	// get storage folder

packages/hub/src/lib/index.ts

@@ -8,7 +8,7 @@ export * from "./delete-file";
 export * from "./delete-files";
 export * from "./delete-repo";
 export * from "./download-file";
-export * from "./download-file-cache";
+export * from "./download-file-to-cache-dir";
 export * from "./file-download-info";
 export * from "./file-exists";
 export * from "./list-commits";