Skip to content

[log] config/config_stdin: add debug logging to helper functions#7239

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
log/enhance-config-stdin-e7be582a2619c653
Draft

[log] config/config_stdin: add debug logging to helper functions#7239
github-actions[bot] wants to merge 1 commit into
mainfrom
log/enhance-config-stdin-e7be582a2619c653

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Add logStdin.Printf calls to three previously unlogged functions in internal/config/config_stdin.go.

Changes

assignLegacyIntAlias

Log when a deprecated snake_case alias (connect_timeout / tool_timeout) is applied. This surfaces configs that should migrate to the camelCase equivalents (connectTimeout / toolTimeout) — visible via DEBUG=config:config_stdin.

stripExtensionFieldsForValidation

Log the server count when stripping gateway-specific extension fields (guard, auth, tool_response_filters) before schema validation. Helpful for tracing the validation pipeline.

convertStdinServerConfig (HTTP branch)

Log when custom connectTimeout or toolTimeout are configured on an HTTP server, making non-default timeout configuration visible during debugging.

Validation

  • go build — compiles cleanly
  • go test ./... — all 26 Go packages pass
  • gofmt — no formatting changes needed
  • ⚠️ Rust guard tests skipped — crates.io blocked by network firewall (unrelated to Go changes)

File modified

  • internal/config/config_stdin.go (7 lines added, 0 deleted)

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • index.crates.io

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "index.crates.io"

See Network Configuration for more information.

Generated by Go Logger Enhancement · sonnet46 15.7M ·

Add logStdin.Printf calls to three previously unlogged functions:

- assignLegacyIntAlias: log when a deprecated snake_case alias
  (connect_timeout/tool_timeout) is applied, helping identify configs
  that should migrate to camelCase equivalents
- stripExtensionFieldsForValidation: log server count processed when
  stripping gateway-specific extension fields before schema validation
- convertStdinServerConfig (HTTP branch): log when custom
  connectTimeout or toolTimeout are configured on an HTTP server

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants