fix: apply suggestion by @coyotte508

axel7083 · axel7083 · commit f78e189777a3 · 2024-10-30T13:56:22.000+01:00
diff --git a/packages/hub/src/lib/dataset-info.ts b/packages/hub/src/lib/dataset-info.ts
@@ -31,7 +31,7 @@ export async function datasetInfo<
 	]).toString();
 
 	const response = await (params.fetch || fetch)(
-		`${params?.hubUrl || HUB_URL}/api/datasets/${params.name}${params.revision ? `/revision/${params.revision}` : ''}?${search.toString()}`,
+		`${params?.hubUrl || HUB_URL}/api/datasets/${params.name}/revision/${encodeURIComponent(params.revision ?? "HEAD")}?${search.toString()}`,
 		{
 			headers: {
 				...(accessToken ? { Authorization: `Bearer ${accessToken}` } : {}),
diff --git a/packages/hub/src/lib/model-info.ts b/packages/hub/src/lib/model-info.ts
@@ -31,7 +31,7 @@ export async function modelInfo<
 	]).toString();
 
 	const response = await (params.fetch || fetch)(
-		`${params?.hubUrl || HUB_URL}/api/models/${params.name}${params.revision ? `/revision/${params.revision}` : ''}?${search.toString()}`,
+		`${params?.hubUrl || HUB_URL}/api/models/${params.name}/revision/${encodeURIComponent(params.revision ?? "HEAD")}?${search.toString()}`,
 		{
 			headers: {
 				...(accessToken ? { Authorization: `Bearer ${accessToken}` } : {}),
diff --git a/packages/hub/src/lib/space-info.ts b/packages/hub/src/lib/space-info.ts
@@ -32,7 +32,7 @@ export async function spaceInfo<
 	]).toString();
 
 	const response = await (params.fetch || fetch)(
-		`${params?.hubUrl || HUB_URL}/api/spaces/${params.name}${params.revision ? `/revision/${params.revision}` : ''}?${search.toString()}`,
+		`${params?.hubUrl || HUB_URL}/api/spaces/${params.name}/revision/${encodeURIComponent(params.revision ?? "HEAD")}?${search.toString()}`,
 		{
 			headers: {
 				...(accessToken ? { Authorization: `Bearer ${accessToken}` } : {}),

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ export async function datasetInfo<`
`31`	`31`	`]).toString();`
`32`	`32`
`33`	`33`	`const response = await (params.fetch \|\| fetch)(`
`34`		- `${params?.hubUrl \|\| HUB_URL}/api/datasets/${params.name}${params.revision ? `/revision/${params.revision}` : ''}?${search.toString()}`,
	`34`	+ `${params?.hubUrl \|\| HUB_URL}/api/datasets/${params.name}/revision/${encodeURIComponent(params.revision ?? "HEAD")}?${search.toString()}`,
`35`	`35`	`{`
`36`	`36`	`headers: {`
`37`	`37`	...(accessToken ? { Authorization: `Bearer ${accessToken}` } : {}),
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ export async function spaceInfo<`
`32`	`32`	`]).toString();`
`33`	`33`
`34`	`34`	`const response = await (params.fetch \|\| fetch)(`
`35`		- `${params?.hubUrl \|\| HUB_URL}/api/spaces/${params.name}${params.revision ? `/revision/${params.revision}` : ''}?${search.toString()}`,
	`35`	+ `${params?.hubUrl \|\| HUB_URL}/api/spaces/${params.name}/revision/${encodeURIComponent(params.revision ?? "HEAD")}?${search.toString()}`,
`36`	`36`	`{`
`37`	`37`	`headers: {`
`38`	`38`	...(accessToken ? { Authorization: `Bearer ${accessToken}` } : {}),