docs: Add missing CHANGELOG entries for Phase 1-3 completion (PRs #1609–#1624) - #1629
Conversation
#1624) Added comprehensive changelog entries for recently merged PRs: - PR #1613: Phase 3 label validation enforcement (script + workflow) - PR #1624: Phase 1–3 completion documentation report - PR #1611: Label creation governance in CLAUDE.md and AGENTS.md - PR #1610: Fixed Copilot review feedback on audit reports - PR #1609: Phase 1 critical fixes (badges + release workflow) Also added Phase 4 documentation entries for LABELING.md updates, LABELING_FAQ.md, and LABELING_EXAMPLES.md. Updated frontmatter last_updated to 2026-08-07. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 59 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughUpdated ChangesChangelog updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
⏱️ Aging and SLA annotation
Maintained by project-meta-sync workflow. |
github-actions
Bot
added
status:needs-review
|
✅ Template check passed after update. Thanks for fixing the PR description. |
github-actions
Bot
added
priority:normal
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CHANGELOG.md`:
- Line 45: Update the new changelog entries to use UK English: change
“authorization” to “authorisation” and “organization-wide” to
“organisation-wide,” preserving all other content.
- Around line 61-66: Update the three new [Unreleased] entries around the Phase
1–3 completion, label creation governance, and Phase 4 label validation
documentation descriptions to include the corresponding issue link alongside
each existing PR link. Preserve the current PR and documentation links, and use
the issue each PR closes or tracks rather than treating documentation links as a
substitute.
- Line 43: Update the Phase 3 changelog entry to remove “labeling” from the
listed workflow validation triggers, unless the corresponding workflows are also
changed to handle labeled and unlabeled events. Keep the remaining documented
triggers and enforcement claims accurate.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 480fe15e-ea76-41a3-81bb-73038cd18ff0
📒 Files selected for processing (1)
CHANGELOG.md
📜 Review details
⏰ Context from checks skipped due to timeout. (8)
- GitHub Check: Testing
- GitHub Check: coderabbit-gate
- GitHub Check: Validate Mermaid Diagrams
- GitHub Check: Unified Labeling, Status, and Type Assignment
- GitHub Check: Mergify Merge Protections
- GitHub Check: Analyze (javascript-typescript)
- GitHub Check: Summary
- GitHub Check: Analyze (python)
⚠️ CI failures not shown inline (2)
GitHub Actions: Validate PR Template / validate-pr-template: docs: Add missing CHANGELOG entries for Phase 1-3 completion (PRs #1609–#1624)
Conclusion: failure
##[group]Run actions/github-script@v7
with:
script: const { validatePullRequestBody } = require('./scripts/validation/template-helpers.cjs');
const marker = '<!-- template-enforcement -->';
const pr = context.payload.pull_request;
const author = pr.user?.login || '';
const isDependabot = author === 'dependabot[bot]' || author === 'app/dependabot';
const isImgbot = author === 'imgbot[bot]' || author === 'app/imgbot';
if (isDependabot || isImgbot) {
core.info(`Skipping PR template validation for bot author ${author}.`);
return;
}
const validation = validatePullRequestBody(pr.body || '', pr.labels || [], pr.head?.ref || '');
const comments = await github.paginate(github.rest.issues.listComments, {
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: pr.number,
per_page: 100
});
const previous = comments.find((comment) =>
comment.user?.type === 'Bot' && comment.body?.includes(marker)
);
if (validation.missing.length === 0) {
if (previous) {
await github.rest.issues.updateComment({
owner: context.repo.owner,
repo: context.repo.repo,
comment_id: previous.id,
body: `${marker}\n✅ Template check passed after update. Thanks for fixing the PR description.`
});
}
return;
}
const message = [
marker,
'🚫 This PR description is missing required template content.',
'',
`Missing required section(s): ${validation.missing.join(', ')}`,
'',
'Please update the PR body using one of the repository PR templates:',
'- https://github.com/lightspeedwp/.github/blob/develop/.github/pull_request_template.md',
'- https://github.com/lightspeedwp/.github/tree/develop/.github/PULL_REQUEST_TEMPLATE',
'',
'Empty placeholders, unchecked checklist boxes, and stub issue references do not count.'
].join('\n');
if (previous) {
await github.rest.issues.updateComment({
owner: context.repo.owner,
repo: context.repo.repo,
comment_id: previous.id,
body: message
});
} else {
await github.rest.issues....
GitHub Actions: Validate PR Template / 0_validate-pr-template.txt: docs: Add missing CHANGELOG entries for Phase 1-3 completion (PRs #1609–#1624)
Conclusion: failure
##[group]Run actions/github-script@v7
with:
script: const { validatePullRequestBody } = require('./scripts/validation/template-helpers.cjs');
const marker = '<!-- template-enforcement -->';
const pr = context.payload.pull_request;
const author = pr.user?.login || '';
const isDependabot = author === 'dependabot[bot]' || author === 'app/dependabot';
const isImgbot = author === 'imgbot[bot]' || author === 'app/imgbot';
if (isDependabot || isImgbot) {
core.info(`Skipping PR template validation for bot author ${author}.`);
return;
}
const validation = validatePullRequestBody(pr.body || '', pr.labels || [], pr.head?.ref || '');
const comments = await github.paginate(github.rest.issues.listComments, {
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: pr.number,
per_page: 100
});
const previous = comments.find((comment) =>
comment.user?.type === 'Bot' && comment.body?.includes(marker)
);
if (validation.missing.length === 0) {
if (previous) {
await github.rest.issues.updateComment({
owner: context.repo.owner,
repo: context.repo.repo,
comment_id: previous.id,
body: `${marker}\n✅ Template check passed after update. Thanks for fixing the PR description.`
});
}
return;
}
const message = [
marker,
'🚫 This PR description is missing required template content.',
'',
`Missing required section(s): ${validation.missing.join(', ')}`,
'',
'Please update the PR body using one of the repository PR templates:',
'- https://github.com/lightspeedwp/.github/blob/develop/.github/pull_request_template.md',
'- https://github.com/lightspeedwp/.github/tree/develop/.github/PULL_REQUEST_TEMPLATE',
'',
'Empty placeholders, unchecked checklist boxes, and stub issue references do not count.'
].join('\n');
if (previous) {
await github.rest.issues.updateComment({
owner: context.repo.owner,
repo: context.repo.repo,
comment_id: previous.id,
body: message
});
} else {
await github.rest.issues....
🧰 Additional context used
📓 Path-based instructions (4)
**/*
📄 CodeRabbit inference engine (AGENTS.md)
**/*: Use UK English and optimise solutions for clarity, scalability, maintainability, and profitable outcomes.
Prefer minimal, modular solutions; justify heavier tools based on return on investment and maintenance cost.
**/*: Do not create instruction files with areferencesfrontmatter field; use inline links or footer sections instead.
Do not move existing agents, instructions, or schemas without a migration issue recording the source path, target path, and validation plan.
Do not add WordPress plugin- or theme-specific project code to the.githubcontrol-plane repository.
Do not place reports or task trackers indocs/or the repository root.
Do not commitnode_modules/,build/, or other generated artefacts.
Files:
CHANGELOG.md
**/*.md
📄 CodeRabbit inference engine (AGENTS.md)
**/*.md: Documentation and AI infrastructure files must follow their applicable documented standards, including required structure, frontmatter, quality gates, and validation.
Run Markdown linting withnpm run lint:mdand validate frontmatter withnpm run validate:frontmatterwhere applicable.
Files:
CHANGELOG.md
**/*.{md,mdx}
📄 CodeRabbit inference engine (CLAUDE.md)
Use UK English throughout documentation and content, including spellings such as “optimise”, “organisation”, “colour”, and “behaviour”.
Files:
CHANGELOG.md
CHANGELOG.md
⚙️ CodeRabbit configuration file
CHANGELOG.md: Review CHANGELOG.md:
- Confirm entries follow Keep a Changelog 1.1.0 format.
- Each entry under [Unreleased] must include a PR link and issue link.
- Verify entries use the correct section headings (Added, Changed, Fixed, Deprecated, Removed, Security, Documentation, Performance).
- Check UK English spelling throughout.
Files:
CHANGELOG.md
🔇 Additional comments (2)
CHANGELOG.md (2)
47-47: LGTM!
6-6: 📐 Maintainability & Code QualityNo action needed.
🔍 Reviewer Summary for PR #1629CI Status: ✅ Recommendations
|
- Changed 'authorization' to 'authorisation' (UK English) - Changed 'organization-wide' to 'organisation-wide' (UK English) - Changed 'organized' to 'organised' (UK English) - Added issue links to Phase entries (#1290, #1592, #1612) - Removed 'labeling' from workflow trigger list (not configured) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
|
Tick the box to add this pull request to the merge queue (same as
|
Fixed extractFrontmatterSafely to properly handle frontmatter longer than 10 lines by searching the entire file for closing --- rather than limiting to first 10 lines. Tests: All 19 footer injection tests now pass Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Signed-off-by: Ash Shaw <ashley@lightspeedwp.agency>
PR #1629 was merged after PR #1630, creating duplicate entries in the CHANGELOG for Phase 1-3 completion work. This fix removes the duplicate entries while preserving the more complete versions from PR #1630. Removed duplicates: - Phase 3 label validation enforcement (PR #1613) - Phase 1 critical fixes (PR #1609) - Address Copilot review feedback (PR #1610) - Phase 1–3 completion documentation (PR #1624) - Label creation governance (PR #1611) - Phase 4 label validation documentation (PR #1613) Closes #1629 (merge conflict resolution) Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
1 participant
Summary
Added missing CHANGELOG entries for recently merged PRs in the label governance enforcement initiative. Fixed UK English spelling, added issue links, and corrected workflow trigger documentation.
Also added Phase 4 documentation entries for LABELING.md updates, LABELING_FAQ.md, and LABELING_EXAMPLES.md.
Linked issues
Closes #1612 (label validation enforcement)
Closes #1290 (restructuring epic)
Relates to Phase 1-3 label governance audit
Changes
✅ Added comprehensive CHANGELOG entries in Fixed and Added sections
✅ Merged with existing origin/develop entries (preserves detailed descriptions)
✅ Updated frontmatter last_updated timestamp to 2026-08-07
✅ UK English spelling (authorisation, organisation-wide)
✅ Added issue links to Phase entries (#1290, #1592, #1612)
✅ Corrected workflow trigger documentation
✅ Markdownlint compliance verified
Changelog
Fixed
Added
No breaking changes. Documentation-only updates.
Checklist (Global DoD / PR)
Generated with Claude Code