[Copilot]

The Code Simplifier was opening PRs for micro-changes (e.g. replacing a two-line map lookup + nil check with a single direct access) that provide negligible value and cost more reviewer time than they save.

Changes

• New "Bar for merit" section — a simplification must clear at least one of:

  • Style conformance: code is a clear outlier vs. the rest of the codebase
  • Helper reuse: existing utility already does what the code reimplements inline
  • Significant clarity/DRY: meaningfully reduces complexity (e.g. 10+ tangled lines → 2–3, or eliminates a real copy-paste pattern)

• Explicit "Do not submit" rule for idiom swaps — renaming variables, reordering equivalent expressions, or collapsing a two-line pattern into one should result in noop

• Expanded candidate search criteria in Step 1 to steer the agent toward style outliers and inline logic that reimplements an existing helper, rather than micro-optimisations

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• https://api.github.com/repos/ruby/setup-ruby/git/ref/tags/v1
  • Triggering command: /usr/bin/gh gh api /repos/ruby/setup-ruby/git/ref/tags/v1 --jq .object.sha -c=4 -nolocalimports -importcfg /tmp/go-build4059744779/b329/importcfg -pack /home/REDACTED/go/pkg/mod/github.com/sourcegraph/conc@v0.3.0/pool/context_pool.go /home/REDACTED/go/pkg/mod/github.com/sourcegraph/conc@v0.3.0/pool/error_pool.go (http block)
  • Triggering command: /usr/bin/gh gh api /repos/ruby/setup-ruby/git/ref/tags/v1 --jq .object.sha -c=4 -nolocalimports -importcfg /tmp/go-build4059744779/b334/importcfg -pack /home/REDACTED/go/pkg/mod/github.com/spf13/pflag@v1.0.10/bool.go /home/REDACTED/go/pkg/mod/github.com/spf13/pflag@v1.0.10/bool_func.go (http block)
  • Triggering command: /usr/bin/gh gh api /repos/ruby/setup-ruby/git/ref/tags/v1 --jq .object.sha /tmp/go-build4059744779/b338/symabis -c=4 -nolocalimports -importcfg /tmp/go-build4059744779/b338/importcfg -pack -asmhdr (http block)
• https://api.github.com/user
  • Triggering command: /usr/bin/gh gh api user --jq .login -c=4 -nolocalimports -importcfg /tmp/go-build4059744779/b315/importcfg -pack /home/REDACTED/go/pkg/mod/golang.org/x/text@v0.34.0/message/catalog/catalog.go /home/REDACTED/go/pkg/mod/golang.org/x/text@v0.34.0/message/catalog/dict.go (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Code Simplifier improvements</issue_title>
<issue_description>See elastic/beats#48971

Small changes like this are not particularly useful. Lets focus on changes that either bring a block of code into the style of the codebase (remove an outlier), that utilize a helper function/etc instead of rewriting logic, or that greatly improve clarity or improve DRY.

Let's avoid tiny changes that take longer to review than they provide benefit</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Code Simplifier improvements #172

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[github-actions]

Requesting changes: this introduces an unintended behavior regression where issue-triggered /ai runs will create draft PRs by default because the draft-prs: "false" override was removed.

---

What is this? | From workflow: PR Review Agent

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not. for issue #177

[github-actions]

🟡 MEDIUM: draft-prs override was dropped, changing PR creation behavior

Removing the with.draft-prs: "false" input means this caller now inherits the reusable workflow default (draft-prs: "true" in .github/workflows/gh-aw-mention-in-issue.lock.yml), so /ai issue flows will start creating draft PRs instead of ready-for-review PRs.

This is a behavior change from the previous trigger configuration and can slow triage/automation that expects non-draft PRs.

[github-actions]

The failed check in run 22202395891 is from Verify compiled workflows → Ensure lock files are up to date and the root cause is that make compile still produced uncommitted generated changes.

From the job log, the step fails on a dirty worktree after compile and prints modifications across generated workflow outputs (including agentics-maintenance.yml and many gh-aw-*.lock.yml files), then exits with code 1.

Recommended remediation for any follow-up branch: run make compile with the repo’s current toolchain and commit all generated workflow/lock updates in one commit before re-running CI.

Checks/commands reviewed:

• GitHub Actions job 64218340559 logs (Verify compiled workflows)
• Failing step: Ensure lock files are up to date
• Failure condition observed: Working tree dirty after make compile.

---

What is this? | From workflow: PR Checks Fixer

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.