fix OAuth + drop isPullRequest

bench-hub.js

@@ -75,12 +75,22 @@ export async function beginHFSignIn() {
   if (!isHubConfigured()) {
     throw new Error('HF hub is not configured. Set HF_DATASET_REPO in bench-config.js.');
   }
-  // When served from an HF Space with `hf_oauth: true`, the Space injects
-  // OAuth client config via meta tags; @huggingface/hub picks it up when
-  // clientId is omitted. For local dev, running the sign-in flow requires a
-  // Space URL — so OAuth is effectively hosted-only.
+  // When served from an HF Space with `hf_oauth: true`, HF injects
+  // `window.huggingface.variables` with OAUTH_CLIENT_ID + OAUTH_SCOPES.
+  // @huggingface/hub needs the clientId passed explicitly.
+  const injected = (globalThis.window?.huggingface?.variables) || {};
+  const clientId = injected.OAUTH_CLIENT_ID;
+  if (!clientId) {
+    throw new Error(
+      'No OAUTH_CLIENT_ID injected — is this page served from an HF Space with `hf_oauth: true`?',
+    );
+  }
+  const scopes = injected.OAUTH_SCOPES
+    ? injected.OAUTH_SCOPES.split(/\s+/).filter(Boolean)
+    : HF_OAUTH_SCOPES;
   const url = await oauthLoginUrl({
-    scopes: HF_OAUTH_SCOPES,
+    clientId,
+    scopes,
     redirectUrl: location.origin + location.pathname,
   });
   location.assign(url);
@@ -129,14 +139,11 @@ export async function submitResultsToDataset(results, {
     credentials: { accessToken: token },
     file: { path, content: blob },
     commitTitle: `bench: ${machineSlug} / ${browser} / ${results.length} variants`,
-    isPullRequest: true,
   });
 
-  const prUrl = res?.pullRequestUrl
-    || (res?.hfHubUrl ? `https://huggingface.co${res.hfHubUrl}` : null);
   return {
     path,
     commit: res?.commit?.oid || null,
-    prUrl,
+    commitUrl: `https://huggingface.co/datasets/${datasetRepo}/blob/main/${path}`,
   };
 }