Skip to content

chore(agents): clarify commit conventions - #1921

Merged
Benoît Cortier (CBenoit) merged 4 commits into
masterfrom
cbenoit-clarify-ci-commit-type
Aug 9, 2026
Merged

chore(agents): clarify commit conventions#1921
Benoît Cortier (CBenoit) merged 4 commits into
masterfrom
cbenoit-clarify-ci-commit-type

Conversation

@CBenoit

@CBenoit Benoît Cortier (CBenoit) commented Aug 9, 2026

Copy link
Copy Markdown
Member

Classify Conventional Commit types by affected surface so automation fixes remain ci and non-feature public API boundary changes use fix.

Standardize agents as the scope for agent instructions and reusable skills.

Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

Select Conventional Commit types by affected surface so automation fixes remain classified as CI changes.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings August 9, 2026 16:35
@CBenoit

Copy link
Copy Markdown
Member Author

Implementation notes:

  • Mirrors the Conventional Commit type guidance adopted in IronRDP.
  • Keeps CI, workflow, and automation fixes classified as ci while reserving fix for product or runtime bugs.
  • The repository-local Tessl helper reports the skill as valid.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Clarifies Conventional Commit type selection for automation fixes.

Changes:

  • Prioritizes affected surface when selecting commit types.
  • Defines CI fixes as ci.
  • Adds breaking-change guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .agents/skills/commit-and-pr-conventions/SKILL.md Outdated
Keep the semantic type accurate when a change is breaking and represent breakingness with its dedicated marker.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Distinguish externally consumed API-boundary changes from internal restructuring while preserving CI classification by affected surface.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Use the agents scope for agent instructions and reusable skills, matching the convention established in IronRDP.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@CBenoit Benoît Cortier (CBenoit) changed the title chore(skills): clarify CI commit type chore(agents): clarify commit conventions Aug 9, 2026
@CBenoit
Benoît Cortier (CBenoit) requested a balanced review from Copilot August 9, 2026 16:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 2 out of 2 changed files in this pull request and generated no new comments.

@CBenoit
Benoît Cortier (CBenoit) merged commit 5050935 into master Aug 9, 2026
43 checks passed
@CBenoit
Benoît Cortier (CBenoit) deleted the cbenoit-clarify-ci-commit-type branch August 9, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants