Skip to content

fix(docs): consume shared release support status - #570

Merged
Eli Bosley (elibosley) merged 3 commits into
mainfrom
codex/docs-release-support-policy
Oct 22, 2026
Merged

Eli Bosley (elibosley) merged 3 commits into
mainfrom
codex/docs-release-support-policy

Conversation

@elibosley

@elibosley Eli Bosley (elibosley) commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Summary

The version archive now uses support status from releases-helper so Docs and Account follow one public-release policy.

Why this exists

Docs and Account calculated the support window independently. Account's list could hide a public RC, making it disagree with Docs. The releases service has the complete catalog and can calculate support before caller filtering.

Resolution

Use the supportStatus on /json?includePublic=1 entries for archive groups and badges. Derive series descriptions from the authoritative tagged releases and remove Docs' classifier. Docs keeps page data only; releases-helper owns classification.

Reviewer considerations

  • Deploy the linked releases service change before publishing this consumer.
  • Missing or unknown support status shows an explicit unavailable message instead of guessing a support category.
  • Metadata-only prereleases retain their existing download guidance. Release ordering, table layout, and download permissions are unchanged.

Behavior changes

The latest public 7.4 RC is supported, the latest 7.3 patch receives security updates, and 7.2.x is EOL. Superseded builds remain in the supported-series archive. Empty and one-series catalogs have truthful descriptions. This implements the requested shared policy.

Implementation summary

  • Consume endpoint support status for grouping, labels, and series descriptions.
  • Remove independent policy ranking.
  • Add visible loading and unavailable states for the shared contract.

Verification

  • Scoped TypeScript check for VersionsTable.tsx: passed.
  • Browser harness rendered the real component with output from the real service classifier/projection: all four statuses, RC download guidance, EOL tab, missing status, one/zero series, and request failure passed.
  • Six existing Vitest tests and three Node tests passed with their respective runners.
  • Formatting and diff checks passed. Full Docs build was not run, per repository instructions. Full-repo TypeScript has existing missing type/configuration errors.
  • Browser evidence covers the component with Docusaurus provider boundaries, not a full themed site or live production endpoint.

Risk

Publishing before the producer would show the unavailable message. Deploy releases-helper first; revert this consumer if rollout must be undone. No deployment is part of this PR.

Producer: https://github.com/unraid/cloudflare/pull/1633
Account companion: https://github.com/unraid/account/pull/1951

Repository checklist

  • Checked for an existing PR on this task branch.
  • No Markdown/MDX links, documentation filenames, or assets changed.
  • Full build: intentionally not run under repository validation guidance; scoped checks passed.

Related to CLD-518 — shared tracking for the releases service, Account, and Docs.

@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c727eaf6-1355-46b3-8ef5-16436871ab01
📥 Commits

Reviewing files that changed from the base of the PR and between f50879f and 301eab2.

📒 Files selected for processing: 2
  • .limetech/ai-review-markers/codex-docs-release-support-policy-5d2a73578aee.json
  • src/components/VersionsTable.tsx
+------------------------------------------------------------+
| If you don't finish then you're just busy, not productive. |
+------------------------------------------------------------+
  \
   \     /)/)
        ( '')
        /  |
      o(___)
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
unraid-docs 301eab2 Commit Preview URL

Branch Preview URL
Oct 22 2026, 06:14 PM

@elibosley
Eli Bosley (elibosley) marked this pull request as ready for review October 22, 2026 20:59
@elibosley
Eli Bosley (elibosley) merged commit c285422 into main Oct 22, 2026
4 of 5 checks passed
@elibosley
Eli Bosley (elibosley) deleted the codex/docs-release-support-policy branch October 22, 2026 21:00
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