Skip to content

chore: improve GenAI tests in PR descriptions#928

Merged
ovitrif merged 4 commits intomasterfrom
chore/fix-ai-pr-tests-gen
May 6, 2026
Merged

chore: improve GenAI tests in PR descriptions#928
ovitrif merged 4 commits intomasterfrom
chore/fix-ai-pr-tests-gen

Conversation

@ovitrif
Copy link
Copy Markdown
Collaborator

@ovitrif ovitrif commented May 4, 2026

This PR improves the shared /pr instructions so AI-generated PR descriptions split QA notes into focused manual validation and visible automated check coverage.

Description

This PR:

  1. Requires generated QA notes to use ### QA Notes with #### Manual Tests and #### Automated Checks.
  2. Adds a concrete manual-test template with unchecked boxes, bold numbering, navigation arrows, regression labels, and variation sub-lists.
  3. Routes automated commands, test suites, and CI coverage notes into the automated-checks section, including the workflow (after merge) caveat.
  4. Moves durable /pr and /release command specs into .agents/commands while keeping .claude/commands as a compatibility symlink.

Preview

N/A

QA Notes

Manual Tests

  • 1. Repo checkout → inspect .claude/commands: pr.md and release.md resolve through the symlink.
  • 2. Agent docs → open AGENTS.md: durable PR command behavior points to .agents/commands/pr.md.
  • 3. /pr --dry in Claude Code → generate a PR body: QA Notes contains separate Manual Tests and Automated Checks sections.

Automated Checks

  • git diff --check

@ovitrif ovitrif force-pushed the chore/fix-ai-pr-tests-gen branch from 7322d27 to 788add8 Compare May 4, 2026 16:48
@ovitrif ovitrif changed the title docs: share agent command specs docs: improve pr qa instructions May 4, 2026
@ovitrif ovitrif changed the title docs: improve pr qa instructions chore: improve ai generated tests in pr descriptions May 4, 2026
@ovitrif ovitrif changed the title chore: improve ai generated tests in pr descriptions chore: improve GenAI tests in PR descriptions May 4, 2026
@ovitrif ovitrif added this to the 2.3.0 milestone May 4, 2026
@ovitrif ovitrif requested review from jvsena42 and piotr-iohk May 4, 2026 17:08
@ovitrif ovitrif self-assigned this May 4, 2026
jvsena42
jvsena42 previously approved these changes May 4, 2026
Copy link
Copy Markdown
Member

@jvsena42 jvsena42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread .agents/commands/pr.md Outdated
@claude

This comment has been minimized.

@ovitrif ovitrif enabled auto-merge May 5, 2026 11:30
@ovitrif ovitrif requested a review from jvsena42 May 5, 2026 19:37
piotr-iohk
piotr-iohk previously approved these changes May 6, 2026
Comment thread .agents/commands/pr.md Outdated
@ovitrif
Copy link
Copy Markdown
Collaborator Author

ovitrif commented May 6, 2026

@piotr-iohk & @jvsena42 this is ready for another pass, added new commit to keep automated checks grouped under their own heading.

Once approved, I will port it to iOS.

@ovitrif ovitrif disabled auto-merge May 6, 2026 09:21
Comment thread .agents/commands/pr.md Outdated
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 6, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

Copy link
Copy Markdown
Collaborator

@piotr-iohk piotr-iohk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ovitrif ovitrif merged commit d63dfff into master May 6, 2026
17 checks passed
@ovitrif ovitrif deleted the chore/fix-ai-pr-tests-gen branch May 6, 2026 12:04
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.

3 participants