Skip to content

chore(agents): place Polish before Bug Fixes in changelog sections - #1527

Merged
wbxl2000 merged 1 commit into
mainfrom
chore/sync-changelog-section-order
Jul 9, 2026
Merged

chore(agents): place Polish before Bug Fixes in changelog sections#1527
wbxl2000 merged 1 commit into
mainfrom
chore/sync-changelog-section-order

Conversation

@wbxl2000

@wbxl2000 wbxl2000 commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Related Issue

N/A — agent workflow tweak.

Problem

The sync-changelog skill ordered changelog sections as Features → Bug Fixes → Polish → Refactors → Other. We want Polish (优化) to appear before Bug Fixes (修复) so user-visible improvements are listed ahead of fixes in the docs changelog and the pre-release preview.

What changed

Reorder the section order in .agents/skills/sync-changelog/SKILL.md to Features → Polish → Bug Fixes → Refactors → Other, and update the section-definition table to match. pre-changelog inherits this order automatically since it delegates ordering to sync-changelog.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works. (N/A — skill doc change)
  • Ran gen-changesets skill, or this PR needs no changeset. (No changeset — skill doc, not bundled)
  • Ran gen-docs skill, or this PR needs no doc update. (N/A — skill doc change)

@changeset-bot

changeset-bot Bot commented Jul 9, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 732f94b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@wbxl2000
wbxl2000 merged commit 5a90d9d into main Jul 9, 2026
10 checks passed
@wbxl2000
wbxl2000 deleted the chore/sync-changelog-section-order branch July 9, 2026 16:08

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 732f94bbd9

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".


```text
Features → Bug Fixes → Polish → Refactors → Other
Features → Polish → Bug Fixes → Refactors → Other

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep pre-changelog ordering instructions in sync

Changing this canonical order leaves the preview workflow with contradictory hard-coded ordering: .agents/skills/pre-changelog/SKILL.md still says to bucket/render Features / Bug Fixes / Polish / Refactors / Other and lists the Chinese headings as 新功能 / 修复 / 优化 / 重构 / 其他 (lines 42-43, with the sample output showing 修复 before any polish section). For release PRs that contain both fixes and polish, running pre-changelog can therefore produce the old preview order while the post-release docs sync uses the new order, defeating the intended pre-release review of the final changelog layout.

Useful? React with 👍 / 👎.

7723qqq pushed a commit to 7723qqq/kimi-code that referenced this pull request Jul 11, 2026
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.

1 participant