Skip to content

feat(workflows): make test-gap-finder open PRs for coverage gaps#680

Merged
jamesadevine merged 8 commits into
mainfrom
copilot/adapt-test-gap-analysis-workflow
May 21, 2026
Merged

feat(workflows): make test-gap-finder open PRs for coverage gaps#680
jamesadevine merged 8 commits into
mainfrom
copilot/adapt-test-gap-analysis-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 21, 2026

Summary

The test-gap-finder workflow now contributes missing tests via pull request instead of opening an issue report. This keeps outcomes actionable by moving from “identify gaps” to “land targeted test coverage changes.”

  • Behavior change

    • Replaced issue-oriented output with PR-oriented output in workflow front matter and instructions.
    • Updated run guidance to implement up to three focused tests directly in tests/**.
  • Safety and scope controls

    • Configured safe-outputs to create-pull-request with max: 1 and allowed-files: ["tests/**"].
    • Preserved issues: read permission required by the configured GitHub toolset while shifting write behavior to PR creation.
  • Authoring guidance updates

    • Rewrote Step 5 from “create issue if warranted” to “implement tests + open PR.”
    • Added explicit high-value test criteria (security-sensitive paths, error branches, untested public behavior).
    • Clarified GH_AW_CI_TRIGGER_TOKEN prerequisite for PR CI triggers and made noop tool invocation explicit when no safe additions are found.
  • Generated artifact sync

    • Regenerated test-gap-finder.lock.yml from the updated .md workflow definition so runtime behavior matches the new PR-first policy.
safe-outputs:
  create-pull-request:
    max: 1
    allowed-files:
      - "tests/**"

Test plan

  • Recompiled the workflow (gh aw compile test-gap-finder) and updated the lock file with no compile warnings.

Copilot AI and others added 8 commits May 21, 2026 06:15
…format

Agent-Logs-Url: https://github.com/githubnext/ado-aw/sessions/07b25765-7ae6-4745-8cbb-b26dedd90431

Co-authored-by: jamesadevine <4742697+jamesadevine@users.noreply.github.com>
@jamesadevine jamesadevine marked this pull request as ready for review May 21, 2026 06:39
@jamesadevine jamesadevine merged commit 4087579 into main May 21, 2026
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