[log] Add debug logging to 5 pkg/ files

[github-actions]

Adds meaningful DEBUG=* debug logging to five files that previously had no logging or minimal logging, following the pkg:filename naming convention from AGENTS.md.

Files changed

File	Logger	New log calls
pkg/parser/schema_validation.go	schemaValidationLog (new)	4 — logs entry to both public validate functions and reports forbidden-field findings
pkg/parser/schedule_time_utils.go	existing scheduleTimeUtilsLog	2 — logs time-string input in parseTime() and warns on unrecognised weekday names in mapWeekday()
pkg/parser/yaml_error.go	existing yamlErrorLog	1 — logs the line-number adjustment offset in adjustLineNumbersInFormattedError()
pkg/workflow/mcp_config_playwright_renderer.go	existing mcpPlaywrightLog	2 — logs rendering options (copilot fields, inline args) and custom-arg counts
pkg/workflow/copilot_mcp.go	existing copilotMCPLog	1 — logs whether localhost URL rewriting is enabled per custom MCP tool

All log arguments are constants or already-computed local variables — no side-effecting expressions.

Validation

• make build passes (binary compiles without errors)
• go test is unavailable in the sandbox due to network restrictions preventing module downloads, but the successful build confirms correct syntax and type-checking

Test plan

• Enable DEBUG=parser:schema_validation and compile a workflow — expect log lines on entry and for any forbidden-field hits
• Enable DEBUG=parser:schedule_time_utils and compile a schedule workflow with a UTC offset — expect Parsing time string: lines
• Enable DEBUG=workflow:mcp_config_playwright_renderer and compile a Playwright workflow — expect option and custom-arg logs
• Enable DEBUG=workflow:copilot_mcp and compile a Copilot workflow with custom MCP — expect rewrite-enabled log per tool

🤖 Generated with [Claude Code]((claude.com/redacted)

Generated by Go Logger Enhancement

Warning

⚠️ Firewall blocked 1 domain

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

• proxy.golang.org

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

network:
  allowed:
    - defaults
    - "proxy.golang.org"

See Network Configuration for more information.

• expires on Mar 2, 2026, 8:28 PM UTC

[Copilot]

Pull request overview

This PR adds DEBUG=*-style debug logging to five previously unlogged (or minimally logged) Go source files in pkg/, following the pkg:filename namespace convention established in AGENTS.md.

Changes:

• Introduces a new schemaValidationLog logger in pkg/parser/schema_validation.go and adds 4 debug log calls at function entry points and forbidden-field detection
• Adds 2 debug log calls to existing loggers in pkg/parser/schedule_time_utils.go (time string input) and pkg/parser/yaml_error.go (line-adjustment offset)
• Adds 2 debug log calls to the existing mcpPlaywrightLog in pkg/workflow/mcp_config_playwright_renderer.go (rendering options and custom arg counts)
• Adds 1 debug log call to the existing copilotMCPLog in pkg/workflow/copilot_mcp.go (localhost URL rewriting decision per tool)

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
pkg/parser/schema_validation.go	Adds new logger import and variable, plus 4 debug log calls for validation entry points and forbidden field detection
pkg/parser/schedule_time_utils.go	Adds debug logging for time string input in parseTime() and unrecognized weekday names in mapWeekday()
pkg/parser/yaml_error.go	Adds a single debug log for the line-adjustment offset in adjustLineNumbersInFormattedError()
pkg/workflow/mcp_config_playwright_renderer.go	Adds debug logging for rendering options and custom arg application
pkg/workflow/copilot_mcp.go	Adds debug logging for the localhost URL rewriting decision per MCP tool

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The new log line at line 91 redundantly includes toolName in the format string ("Localhost URL rewriting for tool %s: enabled=%t"), even though toolName was already logged on the immediately preceding line 86 ("Rendering custom MCP config for tool: %s"). Since both log lines are always executed together, repeating toolName adds noise to the log output. The new line should convey only the new information — i.e., whether localhost URL rewriting is enabled — without repeating the tool name that is already in context from the previous log message.

Suggested change

	copilotMCPLog.Printf("Localhost URL rewriting for tool %s: enabled=%t", toolName, rewriteLocalhost)
	copilotMCPLog.Printf("Localhost URL rewriting: enabled=%t", rewriteLocalhost)