fix: Pass installation ID from env for usage report (#2140)

disq · web-flow · commit 4d36bfba0a57 · 2025-05-02T11:04:27.000Z
Follow-up to #2106 Plugins use regular team API keys for usage reports, not platform-specific local API keys.
diff --git a/premium/usage.go b/premium/usage.go
@@ -285,7 +285,7 @@ func NewUsageClient(meta plugin.Meta, ops ...UsageClientOptions) (UsageClient, e
 		}
 		u.teamName = teamName
 	}
-	u.installationID = determineInstallationID(u.tokenClient.GetTokenType())
+	u.installationID = determineInstallationID()
 
 	u.backgroundUpdater()
 
@@ -728,13 +728,8 @@ func (u *BatchUpdater) getTeamNameByTokenType(tokenType auth.TokenType) (string,
 	}
 }
 
-func determineInstallationID(tokenType auth.TokenType) string {
-	switch tokenType {
-	case auth.SyncRunAPIKey, auth.SyncTestConnectionAPIKey:
-		return os.Getenv("_CQ_INSTALLATION_ID")
-	default:
-		return ""
-	}
+func determineInstallationID() string {
+	return os.Getenv("_CQ_INSTALLATION_ID")
 }
 
 type NoOpUsageClient struct {

Original file line number	Diff line number	Diff line change
`@@ -285,7 +285,7 @@ func NewUsageClient(meta plugin.Meta, ops ...UsageClientOptions) (UsageClient, e`
`285`	`285`	`}`
`286`	`286`	`u.teamName = teamName`
`287`	`287`	`}`
`288`		`- u.installationID = determineInstallationID(u.tokenClient.GetTokenType())`
	`288`	`+ u.installationID = determineInstallationID()`
`289`	`289`
`290`	`290`	`u.backgroundUpdater()`
`291`	`291`
`@@ -728,13 +728,8 @@ func (u *BatchUpdater) getTeamNameByTokenType(tokenType auth.TokenType) (string,`
`728`	`728`	`}`
`729`	`729`	`}`
`730`	`730`
`731`		`-func determineInstallationID(tokenType auth.TokenType) string {`
`732`		`- switch tokenType {`
`733`		`- case auth.SyncRunAPIKey, auth.SyncTestConnectionAPIKey:`
`734`		`- return os.Getenv("_CQ_INSTALLATION_ID")`
`735`		`- default:`
`736`		`- return ""`
`737`		`- }`
	`731`	`+func determineInstallationID() string {`
	`732`	`+ return os.Getenv("_CQ_INSTALLATION_ID")`
`738`	`733`	`}`
`739`	`734`
`740`	`735`	`type NoOpUsageClient struct {`