Skip to content

docs: Issue status review — DoR/DoD completion audit - #2265

Closed
ashleyshaw wants to merge 3 commits into
developfrom
docs/audit-dor-dod
Closed

docs: Issue status review — DoR/DoD completion audit#2265
ashleyshaw wants to merge 3 commits into
developfrom
docs/audit-dor-dod

Conversation

@ashleyshaw

@ashleyshawashleyshaw commented Aug 21, 2026

Copy link
Copy Markdown
Member

Linked issues

Closes#1930, #2051
Relates to #2186, #2147, #2049

What changed

Audience & placement

  • Audience: Contributors, issue maintainers, project managers
  • Location:.github/ (control-plane documentation)

Notes

Changelog

Added

Checklist (Global DoD / PR)

  • All AC met and demonstrated
  • Tests added/updated (not applicable — documentation/issue audit only)
  • Accessibility checklist completed (not applicable — documentation):
    • N/A — documentation only
  • Docs/readme/changelog updated (CHANGELOG.md entry added; audit document created)
  • Security checklist completed (not applicable — documentation only):
    • N/A — documentation only
  • Code/design reviews — ready for review
  • CI green; linked issues closed; release notes prepared (pending CI completion)

Generated by Claude Code

@claude
docs: Issue status review — DoR/DoD completion audit (2026-08-21)
431fd37
- Closed#1930: PR template validator false negative (merged via #2007, #2010)
- Closed#2051: Frontmatter freshness CI issues (merged via #2180)
- Updated #2186: Template validation script false positives with DoR/DoD
- Updated #2147: Phase 1 validation tests progress tracking with DoR/DoD framework
- Verified #2049: OpenSpec Labels Phase 2-3 epic (complete, merged)
All high-priority issues now have complete Definition of Ready and Definition of Done sections reflecting merged work into develop branch.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018MoY39EXD6tnbbjs3JVUqT
@claude
docs: Issue status review — DoR/DoD completion audit
eb5b8f1
- Add comprehensive audit document reviewing 5 critical issues
- Close#1930 (PR template validator false negative)
- Close#2051 (Frontmatter freshness CI issues)
- Enhance #2186 with complete DoR/DoD sections
- Update #2147 with progress tracking and DoR/DoD framework
- Verify #2049 OpenSpec Labels Phase 2-3 Epic (complete)
- Apply standardized DoR/DoD framework across all issues
- Update CHANGELOG.md with complete entry
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018MoY39EXD6tnbbjs3JVUqT
@claude
fix: Apply CodeRabbit review recommendations to PR#2260
7e51411
- Standardise UK English spelling in CHANGELOG (standardized → standardised)
- Fix branch naming violation: remove forbidden claude/ prefix from ISSUE_STATUS_REVIEW
- Correct issue count summary (2 closed, 0 opened, 2 enhanced)
Resolves CodeRabbit review findings on PR #2260.
@coderabbitai

coderabbitaiBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: d2d22c1b-c400-4981-a5b9-f6689541e988

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

❌ Branch Name Validation Failed

The branch name docs/audit-dor-dod does not follow the LightSpeed branching strategy.

Required Format

{type}/{scope}-{short-title}
  • type: one of the allowed prefixes (lowercase)
  • scope: lowercase, hyphens only (no underscores or uppercase)
  • title: lowercase, hyphens only (no underscores or uppercase)

Allowed Branch Types

feat, fix, hotfix, release, refactor, chore, docs, test, perf, ci, build, deps, security, revert, research, design, a11y, ux, i18n, ops, proto, ds, api, schema, telemetry, content, seo, config, migrate, qa, uat, audit, codex

Valid Examples

  • ✓ feat/branch-naming-enforcement
  • ✓ fix/validation-script-bug
  • ✓ chore/update-dependencies
  • ✓ docs/branching-strategy-guide
  • ✓ hotfix/critical-security-patch

Invalid Examples

  • ✗ claude/my-branch (type "claude" not allowed)
  • ✗ Feature/MyBranch (uppercase not allowed)
  • ✗ fix-bug (missing type prefix)
  • ✗ feat/my_feature (underscores not allowed)
  • ✗ feat/MyFeature (uppercase not allowed)

Solution

Rename your branch to follow the pattern and update the PR.

For more information, see docs/BRANCHING_STRATEGY.md.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🚫 This PR description is missing required template content.

Missing required section(s): Global DoD checklist

Please update the PR body using one of the repository PR templates:

Empty placeholders, unchecked checklist boxes, and stub issue references do not count.

@mergify

mergifyBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

⚠️ The sha of the head commit of this PR conflicts with #2260. Mergify cannot evaluate rules on this PR. Once #2260 is merged or closed, Mergify will resume processing this PR. ⚠️

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

investigate: PR template validator false negative on 'Changelog' and 'Global DoD Checklist' sections

2 participants

@ashleyshaw@claude