Skip to content

[WIP] Remove network.firewall.log-level field from configuration - #18012

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/remove-firewall-log-level-field
Closed

[WIP] Remove network.firewall.log-level field from configuration#18012
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/remove-firewall-log-level-field

Conversation

Copilot AI commented Feb 24, 2026

Copy link
Copy Markdown
Contributor
  • Create pkg/cli/codemod_firewall_log_level.go - new codemod to remove log-level from network.firewall
  • Create pkg/cli/codemod_firewall_log_level_test.go - tests for the new codemod
  • Register new codemod in pkg/cli/fix_codemods.go
  • Remove LogLevel field from FirewallConfig in pkg/workflow/firewall.go
  • Simplify pkg/workflow/firewall_validation.go (remove ValidateLogLevel and log-level check)
  • Remove log-level override in pkg/workflow/awf_helpers.go (always use default constant)
  • Remove log-level extraction in pkg/workflow/frontmatter_extraction_security.go
  • Remove log-level extraction in pkg/workflow/engine.go
  • Remove log-level from pkg/parser/schemas/main_workflow_schema.json
  • Delete pkg/workflow/firewall_log_level_test.go and update pkg/workflow/firewall_test.go
  • Update pkg/cli/fix_codemods_test.go (count and order)
  • Update docs: docs/src/content/docs/reference/network.md
  • Update autocomplete: docs/public/editor/autocomplete-data.json
  • Build and tests pass, code review clean

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI requested a review from pelikhan February 24, 2026 00:53
@pelikhan pelikhan closed this Feb 24, 2026
Copilot stopped work on behalf of pelikhan due to an error February 24, 2026 01:09
Copilot stopped work on behalf of pelikhan due to an error February 24, 2026 01:24
@github-actions
github-actions Bot deleted the copilot/remove-firewall-log-level-field branch April 2, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants