Skip to content

fix: annotate failing TDD tests in GitHub diff#4126

Open
Copilot wants to merge 2 commits into
mainfrom
copilot/annotate-test-failure-in-diff
Open

fix: annotate failing TDD tests in GitHub diff#4126
Copilot wants to merge 2 commits into
mainfrom
copilot/annotate-test-failure-in-diff

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 17, 2026

  • Understand the problem: TDD workflow emits no GitHub diff annotations when tests pass on base (TDD violation)
  • Add annotation logic in unit job: emit ::error for added it()/test() lines when TDD fails
  • Add annotation logic in puppeteer job
  • Extract shared awk script to .github/scripts/annotate-tdd-violations.sh
  • Unit job restores .github/scripts/ from FETCH_HEAD; puppeteer job already restores from FETCH_HEAD (now includes scripts/)
  • All CI checks passing on PR fix: annotate failing TDD tests in GitHub diff #4126

…eteer jobs

When tests pass on the base branch (TDD violation), parse the saved patch
file and emit ::error annotations for each added it()/test() call so the
violation is highlighted directly in the PR file changes view.

Agent-Logs-Url: https://github.com/cybersemics/em/sessions/ac2ea786-fff7-4dcc-afd2-fb716755c7b5

Co-authored-by: raineorshine <750276+raineorshine@users.noreply.github.com>
Move the awk annotation block to .github/scripts/annotate-tdd-violations.sh
to avoid duplication between unit and puppeteer jobs. Both jobs now restore
.github/scripts/ from FETCH_HEAD and delegate to the shared script.

Agent-Logs-Url: https://github.com/cybersemics/em/sessions/ac2ea786-fff7-4dcc-afd2-fb716755c7b5

Co-authored-by: raineorshine <750276+raineorshine@users.noreply.github.com>
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