chore: use a pointer for oauth app global value so it can be changed

Without the pointer, it isn't possible to set the global value to false.

Signed-off-by: Donnie Adams <donnie@acorn.io>

apiclient/types/oauthapp.go

@@ -37,8 +37,8 @@ type OAuthAppManifest struct {
 	OptionalScope string `json:"optionalScope,omitempty"`
 	// This field is required, it correlates to the integration name in the gptscript oauth cred tool
 	Alias string `json:"alias,omitempty"`
-	// Global indicates if the OAuth app is globally applied to all agents.
-	Global bool `json:"global,omitempty"`
+	// Global indicates if the OAuth app is globally applied to all agents. Defaults to false.
+	Global *bool `json:"global,omitempty"`
 	// This field is only used by Salesforce
 	InstanceURL string `json:"instanceURL,omitempty"`
 }

pkg/gateway/types/oauth_apps.go

@@ -176,7 +176,7 @@ func MergeOAuthAppManifests(r, other types.OAuthAppManifest) types.OAuthAppManif
 	if other.OptionalScope != "" {
 		retVal.OptionalScope = other.OptionalScope
 	}
-	if other.Global {
+	if other.Global != nil {
 		retVal.Global = other.Global
 	}
 

pkg/render/render.go

@@ -162,7 +162,7 @@ func OAuthAppEnv(ctx context.Context, db kclient.Client, oauthAppNames []string,
 	activeIntegrations := map[string]v1.OAuthApp{}
 	for _, name := range slices.Sorted(maps.Keys(apps)) {
 		app := apps[name]
-		if !app.Spec.Manifest.Global || app.Spec.Manifest.ClientID == "" || app.Spec.Manifest.ClientSecret == "" || app.Spec.Manifest.Alias == "" {
+		if app.Spec.Manifest.Global == nil || !*app.Spec.Manifest.Global || app.Spec.Manifest.ClientID == "" || app.Spec.Manifest.ClientSecret == "" || app.Spec.Manifest.Alias == "" {
 			continue
 		}
 		activeIntegrations[app.Spec.Manifest.Alias] = app