Merge pull request #3739 from OpenNeuroOrg/fix-cli-auth-default

nellh · web-flow · commit 9feb26b5353a · 2026-01-26T15:03:28.000-08:00
fix(cli): Fix default getRepoAccess argument order
diff --git a/cli/src/commands/git-credential.ts b/cli/src/commands/git-credential.ts
@@ -21,7 +21,7 @@ interface GraphQLError {
   }
 }
 
-export async function getRepoAccess(instance: string, datasetId?: string) {
+export async function getRepoAccess(datasetId?: string, instance?: string) {
   const config = getConfig(instance)
   const req = await fetch(`${config.url}/crn/graphql`, {
     method: "POST",
@@ -68,7 +68,7 @@ export async function gitCredentialAction(
   }
   if ("path" in credential && credential.path) {
     const datasetId = credential.path.split("/").pop()
-    const { token } = await tokenGetter(instance, datasetId)
+    const { token } = await tokenGetter(datasetId, instance)
     const output: Record<string, string> = {
       username: "@openneuro/cli",
       password: token,

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ interface GraphQLError {`
`21`	`21`	`}`
`22`	`22`	`}`
`23`	`23`
`24`		`-export async function getRepoAccess(instance: string, datasetId?: string) {`
	`24`	`+export async function getRepoAccess(datasetId?: string, instance?: string) {`
`25`	`25`	`const config = getConfig(instance)`
`26`	`26`	const req = await fetch(`${config.url}/crn/graphql`, {
`27`	`27`	`method: "POST",`
`@@ -68,7 +68,7 @@ export async function gitCredentialAction(`
`68`	`68`	`}`
`69`	`69`	`if ("path" in credential && credential.path) {`
`70`	`70`	`const datasetId = credential.path.split("/").pop()`
`71`		`- const { token } = await tokenGetter(instance, datasetId)`
	`71`	`+ const { token } = await tokenGetter(datasetId, instance)`
`72`	`72`	`const output: Record<string, string> = {`
`73`	`73`	`username: "@openneuro/cli",`
`74`	`74`	`password: token,`