test(bash-lint): cover GateContext::PullRequest via pr-filter-tier1 fixture

[github-actions]

What the lint found

The baseline lint was green (exit 0), so this is a proactive coverage improvement (Step 4c).

Audit revealed that src/compile/filter_ir.rs has two GateContext variants that each emit a - bash: step:

• GateContext::PipelineCompletion → "Evaluate pipeline filters" — already tracked in REQUIRED_STEP_DISPLAY_NAMES and exercised by pipeline-filter-agent.md
• GateContext::PullRequest → "Evaluate PR filters" — not tracked, and pr-filter-tier1-agent.md / pr-filter-tier2-agent.md were absent from the lint's FIXTURES list

Both variants share the same codegen path in filter_ir.rs, but without the PR fixture in FIXTURES there was no guarantee the PR-gate bash body would ever reach shellcheck.

How it was fixed

• tests/bash_lint_tests.rs: added "pr-filter-tier1-agent.md" to FIXTURES so the compiler's PR-filter output is compiled and linted on every CI run.
• tests/bash_lint_tests.rs: added "Evaluate PR filters" to REQUIRED_STEP_DISPLAY_NAMES (with source comment pointing to filter_ir.rs) so the test fails loudly if the fixture is removed or the display name changes.

No production code was changed.

Verification

ENFORCE_BASH_LINT=1 cargo test --test bash_lint_tests -- --nocapture
  test every_listed_fixture_exists ... ok
  test compiled_bash_bodies_pass_shellcheck ... ok
  test result: ok. 2 passed; 0 failed

cargo test
  test result: ok. (all tests pass)

cargo clippy --all-targets
  (no new errors)

Warning

Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

• dev.azure.com
• releaseassets.githubusercontent.com

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

network:
  allowed:
    - defaults
    - "dev.azure.com"
    - "releaseassets.githubusercontent.com"

See Network Configuration for more information.

Generated by Bash Step Hygiene Auditor · ● 2.9M · ◷

[jamesadevine]

@copilot resolve the merge conflicts in this pull request

[Copilot]

@copilot resolve the merge conflicts in this pull request

Resolved and pushed in commit fc959f0. I merged origin/main, fixed the conflict in tests/bash_lint_tests.rs by keeping both required step assertions, and re-ran the bash lint integration test successfully.