Initialize managed settings before Go events

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: joshspicer

go/client.go

@@ -906,8 +906,12 @@ func (c *Client) CreateSession(ctx context.Context, config *SessionConfig) (*Ses
 	// message is dispatched) so notifications for the new session id are
 	// routed to a registered session.
 	initializeSession := func(sessionID string) (*Session, error) {
-		s := newSession(sessionID, c.client, "")
-		s.managedSettings = config.EnableManagedSettings != nil && *config.EnableManagedSettings
+		s := newSession(
+			sessionID,
+			c.client,
+			"",
+			config.EnableManagedSettings != nil && *config.EnableManagedSettings,
+		)
 
 		s.registerTools(config.Tools)
 		s.registerPermissionHandler(config.OnPermissionRequest)
@@ -1228,8 +1232,12 @@ func (c *Client) ResumeSessionWithOptions(ctx context.Context, sessionID string,
 
 	// Create and register the session before issuing the RPC so that
 	// events emitted by the CLI (e.g. session.start) are not dropped.
-	session := newSession(sessionID, c.client, "")
-	session.managedSettings = config.EnableManagedSettings != nil && *config.EnableManagedSettings
+	session := newSession(
+		sessionID,
+		c.client,
+		"",
+		config.EnableManagedSettings != nil && *config.EnableManagedSettings,
+	)
 
 	session.registerTools(config.Tools)
 	session.registerPermissionHandler(config.OnPermissionRequest)

go/session.go

@@ -366,10 +366,16 @@ func canvasResultError(err error) error {
 }
 
 // newSession creates a new session wrapper with the given session ID and client.
-func newSession(sessionID string, client *jsonrpc2.Client, workspacePath string) *Session {
+func newSession(
+	sessionID string,
+	client *jsonrpc2.Client,
+	workspacePath string,
+	managedSettings bool,
+) *Session {
 	s := &Session{
 		SessionID:         sessionID,
 		workspacePath:     workspacePath,
+		managedSettings:   managedSettings,
 		client:            client,
 		clientSessionAPIs: &rpc.ClientSessionAPIHandlers{},
 		handlers:          make([]sessionHandler, 0),