fix(server): return errors instead of panicking in InitializeConfigs (#3397)

## Description

`InitializeOfflineConfigs` previously called `panic(err)` when the
instrumentation or logger could not be retrieved from the context. This
changes those to graceful error returns (`return nil, nil,
fmt.Errorf(...)`) so callers (e.g. `skills-generate`) get a clean error
instead of a crash.

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>

Author: 3 people

internal/server/server.go

@@ -89,12 +89,12 @@ func InitializeConfigs(ctx context.Context, cfg ServerConfig) (
 	ctx = util.WithUserAgent(ctx, metadataStr)
 	instrumentation, err := util.InstrumentationFromContext(ctx)
 	if err != nil {
-		panic(err)
+		return nil, nil, nil, nil, nil, nil, nil, fmt.Errorf("failed to get instrumentation from context: %w", err)
 	}
 
 	l, err := util.LoggerFromContext(ctx)
 	if err != nil {
-		panic(err)
+		return nil, nil, nil, nil, nil, nil, nil, fmt.Errorf("failed to get logger from context: %w", err)
 	}
 
 	// initialize and validate the sources from configs
@@ -274,12 +274,12 @@ func InitializeOfflineConfigs(ctx context.Context, cfg ServerConfig) (
 ) {
 	instrumentation, err := util.InstrumentationFromContext(ctx)
 	if err != nil {
-		panic(err)
+		return nil, nil, fmt.Errorf("failed to get instrumentation from context: %w", err)
 	}
 
 	l, err := util.LoggerFromContext(ctx)
 	if err != nil {
-		panic(err)
+		return nil, nil, fmt.Errorf("failed to get logger from context: %w", err)
 	}
 
 	toolsMap, err := initializeTools(ctx, cfg, instrumentation, l)