Skip to content

chore: improve GenAI tests in PR descriptions#543

Merged
ovitrif merged 3 commits intomasterfrom
chore/ai-pr-tests-update
May 6, 2026
Merged

chore: improve GenAI tests in PR descriptions#543
ovitrif merged 3 commits intomasterfrom
chore/ai-pr-tests-update

Conversation

@ovitrif
Copy link
Copy Markdown
Collaborator

@ovitrif ovitrif commented May 6, 2026

This PR ports the shared agent PR-description guidance from synonymdev/bitkit-android#928 to the iOS repo.

Description

This PR:

  1. Moves durable /pr and /release command specs into .agents/commands.
  2. Keeps .claude/commands as a compatibility symlink to the same files.
  3. Updates /pr guidance so generated QA notes split manual validation from automated check coverage.
  4. Adds iOS-specific examples for manual QA and automated coverage summaries.

Linked Issues/Tasks

Ports synonymdev/bitkit-android#928

Screenshot / Video

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
  • Symlink validation: .claude/commands resolves to ../.agents/commands, and both command specs are reachable through it.
  • Release command move validated: .agents/commands/release.md matches the original tracked command file exactly.

@ovitrif ovitrif requested review from jvsena42 and piotr-iohk May 6, 2026 13:31
@ovitrif ovitrif added this to the 2.3.0 milestone May 6, 2026
@ovitrif ovitrif enabled auto-merge May 6, 2026 13:58
@ovitrif ovitrif self-assigned this May 6, 2026
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 disabled auto-merge May 6, 2026 14:15
@ovitrif
Copy link
Copy Markdown
Collaborator Author

ovitrif commented May 6, 2026

Shouldn't have to wait or integration tests and unit tests on PRs that don't change code contributing to output binaries during compilation.

@ovitrif ovitrif merged commit 422de71 into master May 6, 2026
9 of 12 checks passed
@ovitrif ovitrif deleted the chore/ai-pr-tests-update branch May 6, 2026 14:16
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