Skip to content

Align commit-message generation prompt with OpenStack commit message best practices#16

Merged
electrocucaracha merged 12 commits into
mainfrom
copilot/improve-commit-message-output
May 17, 2026
Merged

Align commit-message generation prompt with OpenStack commit message best practices#16
electrocucaracha merged 12 commits into
mainfrom
copilot/improve-commit-message-output

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 17, 2026

This PR aligns the default commit-message prompt with OpenStack-style guidance and resolves follow-up review feedback for consistency.

Changes Made

  • Updated src/ai_prepare_commit_msg/prompts/default.yml to refine commit-message guidance toward clear prose and consistent formatting rules.
  • Removed conflicting guidance about bullet usage in the body section so bullets are consistently disallowed.
  • Added OpenStack to .github/.wordlist.txt to fix the spellcheck CI failure triggered by README.md.

Validation

  • Verified the failing CI job root cause from workflow logs (Run misspell).
  • Confirmed prompt YAML remains valid after edits.
  • Ran final automated validation (parallel_validation): Code Review passed and CodeQL reported trivial-change skip.

Copilot AI and others added 9 commits May 17, 2026 22:07
Agent-Logs-Url: https://github.com/electrocucaracha/ai-prepare-commit-msg/sessions/8b6e28a3-fef2-41f0-bb1a-3773ad2970ef

Co-authored-by: electrocucaracha <2782150+electrocucaracha@users.noreply.github.com>
Agent-Logs-Url: https://github.com/electrocucaracha/ai-prepare-commit-msg/sessions/8b6e28a3-fef2-41f0-bb1a-3773ad2970ef

Co-authored-by: electrocucaracha <2782150+electrocucaracha@users.noreply.github.com>
Agent-Logs-Url: https://github.com/electrocucaracha/ai-prepare-commit-msg/sessions/8b6e28a3-fef2-41f0-bb1a-3773ad2970ef

Co-authored-by: electrocucaracha <2782150+electrocucaracha@users.noreply.github.com>
Agent-Logs-Url: https://github.com/electrocucaracha/ai-prepare-commit-msg/sessions/8b6e28a3-fef2-41f0-bb1a-3773ad2970ef

Co-authored-by: electrocucaracha <2782150+electrocucaracha@users.noreply.github.com>
Comment thread src/ai_prepare_commit_msg/prompts/default.yml Outdated
Agent-Logs-Url: https://github.com/electrocucaracha/ai-prepare-commit-msg/sessions/42f30900-f245-4e9b-81af-ff3aebaf62b1

Co-authored-by: electrocucaracha <2782150+electrocucaracha@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the default commit-message prompt and README wording to align generated messages with OpenStack-style commit-message guidance, while adding the new proper noun to the spellcheck wordlist.

Changes:

  • Revises body guidance in the default prompt toward concise problem/change/impact prose.
  • Updates README positioning from Google engineering practices to OpenStack commit-message practices.
  • Adds OpenStack to the custom spellcheck wordlist.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/ai_prepare_commit_msg/prompts/default.yml Updates commit-message generation rules for headers, body structure, and validation.
README.md Documents the project’s commit-message style as OpenStack-oriented.
.github/.wordlist.txt Allows OpenStack in spellcheck validation.

Comment thread src/ai_prepare_commit_msg/prompts/default.yml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@electrocucaracha electrocucaracha merged commit ff92e52 into main May 17, 2026
15 checks passed
@electrocucaracha electrocucaracha deleted the copilot/improve-commit-message-output branch May 17, 2026 22:38
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