Skip to content

[log] feat: add debug logging to artifact sets, manifests, virtual FS, completion, and schema triggers - #25168

Merged
pelikhan merged 1 commit into
mainfrom
add-debug-logging-batch-2-4050761e75d65813
Apr 7, 2026
Merged

[log] feat: add debug logging to artifact sets, manifests, virtual FS, completion, and schema triggers#25168
pelikhan merged 1 commit into
mainfrom
add-debug-logging-batch-2-4050761e75d65813

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Adds meaningful Printf/Print debug log calls to five under-logged files so that operators and developers can trace execution through key code paths without needing a debugger.

Files changed

FileLoggerWhat was added
pkg/cli/logs_artifact_set.gocli:logs_artifact_setLog artifact-set validation results, filter resolution (sets → artifact names), and missing-entry detection per output directory
pkg/workflow/safe_update_manifest.goworkflow:safe_update_manifestLog manifest construction (raw secret/action counts → normalised counts) and successful extraction from lock-file headers
pkg/parser/virtual_fs.goparser:virtual_fsLog builtin-file existence checks, and frontmatter cache hits, misses, and stores (including concurrent-race no-ops)
pkg/cli/completion_command.gocli:completionLog entry to install and uninstall subcommand handlers
pkg/parser/schema_triggers.goparser:schema_triggersLog early-return branches and final result of IsLabelOnlyEvent and IsNonConflictingCommandEvent

Validation

  • make build passes (zero compilation errors)
  • go test ./pkg/parser/... ./pkg/cli/... ./pkg/workflow/... passes (all packages green)
  • No test files modified
  • No side effects in logger arguments (all args are primitive values or pre-computed variables)
  • Logger names follow pkg:filename convention

Generated by Go Logger Enhancement · ● 1.3M ·

  • expires on Apr 9, 2026, 9:41 PM UTC

…letion, and schema triggers
Adds meaningful debug log calls to five under-logged files so that
operators and developers can trace execution through key code paths
without adding breakpoints:
- pkg/cli/logs_artifact_set.go: log artifact set validation, filter
resolution, and missing-entry detection results
- pkg/workflow/safe_update_manifest.go: log manifest construction
(secret/action counts) and extraction from lock-file headers
- pkg/parser/virtual_fs.go: log builtin-file existence checks and
frontmatter cache hits, misses, and stores
- pkg/cli/completion_command.go: log install/uninstall shell-completion
subcommand invocations
- pkg/parser/schema_triggers.go: log early-return branches and final
result in IsLabelOnlyEvent and IsNonConflictingCommandEvent
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automationenhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@pelikhan