Skip to content

fix: Correct OPENSPEC frontmatter and add document footers - #1711

Merged
ashleyshaw merged 6 commits into
developfrom
fix/project-documentation-openspec-frontmatter
Aug 10, 2026
Merged

fix: Correct OPENSPEC frontmatter and add document footers#1711
ashleyshaw merged 6 commits into
developfrom
fix/project-documentation-openspec-frontmatter

Conversation

@ashleyshaw

@ashleyshawashleyshaw commented Aug 10, 2026

Copy link
Copy Markdown
Member

Summary

Fixes frontmatter validation and footer formatting issues in OPENSPEC specification documents for active projects. These were validation issues discovered after PR #1703 merged, plus version bumps to satisfy freshness validation.

Changes

  1. Frontmatter Validation: Corrected invalid status values and missing required fields

    • Changed status from 'in-progress'/'implemented' to valid enum value 'active'
    • Fixed version field formatting (number → string)
    • Added missing description fields
    • Added owners field for all specifications
  2. Document Footers: Added proper footer sections to all OPENSPEC files

    • Added metadata footers with version, update date, and status
    • Resolves footer validation errors in CI/CD checks
  3. Version Bumps: Updated document versions to reflect body changes

    • badges-workflow-integration: v1.2.0 → v1.2.1
    • github-actions-v7-upgrade: 1.1 → 1.1.1
    • issue-metadata-triage-expansion: 1.1.0 → 1.1.1

Files Modified

  • .github/projects/active/github-actions-v7-upgrade-2026-08-09/OPENSPEC_ANALYSIS.md
  • .github/projects/active/badges-workflow-integration-2026-08-08/OPENSPEC_ANALYSIS.md
  • .github/projects/active/issue-metadata-triage-expansion/OPENSPEC.md

Linked issues

Closes#1709
Relates to #1672
Relates to #1673
Relates to #1679

Changelog

Fixed

  • Correct OPENSPEC frontmatter validation errors (status enum, version formatting)
  • Add proper document footers to specification files with metadata tracking
  • Bump document versions to reflect body changes (footer additions)

Checklist (Global DoD / PR)

  • Code changes complete and tested locally
  • All frontmatter validation passing
  • Document footers properly formatted
  • Version numbers updated to reflect body changes
  • No breaking changes
  • Documentation updated

ashleyshawand others added 2 commits August 10, 2026 18:27
Fixed invalid status values and missing required fields in OPENSPEC analyses:
- Changed status from 'in-progress'/'implemented' to 'active' (valid enum)
- Changed version from number to string format ("1.1")
- Added missing description fields
- Added owners field to all specification documents
- Standardized frontmatter field formatting
All frontmatter validation now passes (4/4 files validated, 0 errors).
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Added footer sections to all OPENSPEC analysis and specification documents
with metadata and status information for maintainability and tracking.
Resolves footer validation errors in CI/CD checks.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@github-actions

github-actionsBot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

✅ Template check passed after update. Thanks for fixing the PR description.

@github-actionsgithub-actionsBot added status:needs-review Awaiting code review type:bug Bug or defect priority:normal Default priority type:chore Chore / small hygiene change meta:needs-changelog Requires a changelog entry before merge labels Aug 10, 2026
@github-actions

github-actionsBot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

⏱️ Aging and SLA annotation

  • Age: 0 day(s)
  • SLA state: Within SLA
  • Thresholds: warn at 7 days, breach at 14 days
  • Last updated: 2026-08-10T16:46:31.983Z

Maintained by project-meta-sync workflow.

@coderabbitai

coderabbitaiBot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@ashleyshaw, you've reached your PR review limit, so we couldn't start this review.

Next review available in:16 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 @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: c1bc0694-5be9-4758-aa6c-8d7080baf607

📥 Commits

Reviewing files that changed from the base of the PR and between f707028 and 2777cb3.

📒 Files selected for processing (4)
  • .github/projects/active/badges-workflow-integration-2026-08-08/OPENSPEC_ANALYSIS.md
  • .github/projects/active/github-actions-v7-upgrade-2026-08-09/OPENSPEC_ANALYSIS.md
  • .github/projects/active/issue-metadata-triage-expansion/OPENSPEC.md
  • CHANGELOG.md

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.

@ashleyshawashleyshaw self-assigned this Aug 10, 2026
@ashleyshaw
ashleyshaw enabled auto-merge (squash) August 10, 2026 16:34
@ashleyshaw
ashleyshaw disabled auto-merge August 10, 2026 16:36
@github-actionsgithub-actionsBot removed the type:chore Chore / small hygiene change label Aug 10, 2026
Updated version numbers to satisfy frontmatter freshness validation:
- badges-workflow-integration: v1.2.0 → v1.2.1
- github-actions-v7-upgrade: 1.1 → 1.1.1
- issue-metadata-triage-expansion: 1.1.0 → 1.1.1
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@github-actions

github-actionsBot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

🔍 Reviewer Summary for PR #1711

CI Status:success
Files changed: 4
Risk Distribution: 0 critical, 3 high, 0 medium, 1 low

Recommendations

  • Ready to proceed pending human review

@ashleyshawashleyshaw removed the meta:needs-changelog Requires a changelog entry before merge label Aug 10, 2026
- OPENSPEC frontmatter/footer validation fixes (PR #1711)
- Confidence-status assertion determinism in handle-needs-triage tests (PR #1712)
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@github-actionsgithub-actionsBot added area:documentation Docs & guides lang:md Markdown content/docs meta:needs-changelog Requires a changelog entry before merge labels Aug 10, 2026
ashleyshaw added a commit that referenced this pull request Aug 10, 2026
- OPENSPEC frontmatter/footer validation fixes (PR #1711)
- Confidence-status assertion determinism in handle-needs-triage tests (PR #1712)
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
ashleyshawand others added 2 commits August 10, 2026 18:48
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Resolved multiple-footer validation errors by removing old footer text
and keeping only the latest structured footer with document metadata.
- badges-workflow-integration: v1.2.0 → v1.2.1
- github-actions-v7-upgrade: 1.1.1 (updated from 1.1)
- issue-metadata-triage-expansion: 1.1.1 (updated from 1.1.0)
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@ashleyshaw
ashleyshaw merged commit 802ea55 into developAug 10, 2026
27 of 29 checks passed
@ashleyshaw
ashleyshaw deleted the fix/project-documentation-openspec-frontmatter branch August 10, 2026 17:14
ashleyshaw added a commit that referenced this pull request Aug 10, 2026
…1712)
* fix: Make confidence-status assertions deterministic in handle-needs-triage tests
Address CodeRabbit findings on PR #1703:
1. Assert result.status is specifically 'warning' for low-confidence,
unlabeled fixture at 0.95 threshold (line 247)
2. Derive expected status from returned inference confidences instead of
accepting all three statuses (lines 294-296)
- Returns 'preview' if any inference meets 0.85 threshold
- Returns 'warning' if no inference meets threshold
All 27 handle-needs-triage tests passing.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
* docs: Add changelog entries for PR #1711 and PR #1712
- OPENSPEC frontmatter/footer validation fixes (PR #1711)
- Confidence-status assertion determinism in handle-needs-triage tests (PR #1712)
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
* docs: Update CHANGELOG.md last_updated to 2026-08-10
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
* fix: Remove duplicate footer sections from OPENSPEC files
Resolved multiple-footer validation errors by removing old footer text
and keeping only the latest structured footer with document metadata.
- badges-workflow-integration: v1.2.0 → v1.2.1
- github-actions-v7-upgrade: 1.1.1 (updated from 1.1)
- issue-metadata-triage-expansion: 1.1.1 (updated from 1.1.0)
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
---------
Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:documentationDocs & guideslang:mdMarkdown content/docsmeta:needs-changelogRequires a changelog entry before mergepriority:normalDefault prioritystatus:needs-reviewAwaiting code reviewtype:bugBug or defect

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR #1703: Phase 5 handoff notes - Resolve merge conflicts and CI failures

1 participant

@ashleyshaw