Skip to content

docs(cli): auto-derive command group copy#11939

Open
JulianMaurin wants to merge 1 commit into
mainfrom
devs/JulianMaurin/cli-docs-auto-group-copy/auto-derive-cmd-group-copy--3e5bccdc
Open

docs(cli): auto-derive command group copy#11939
JulianMaurin wants to merge 1 commit into
mainfrom
devs/JulianMaurin/cli-docs-auto-group-copy/auto-derive-cmd-group-copy--3e5bccdc

Conversation

@JulianMaurin

Copy link
Copy Markdown
Contributor

groupDescription resolves a group's outward-facing copy as the editorial entry,
then the schema about, then a generated sentence; groupTopLevel exposes it as a
precomputed CliGroup.description. A command synced into cli-schema.json renders
non-blank without a hand-written entry, and sorts among the capabilities ahead of
the MAINTENANCE_GROUPS tail.

The grouping test now guards against editorial entries orphaned by a renamed or
dropped command rather than requiring an entry per command, and a build-time
warning lists groups still on fallback copy.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

Copilot AI review requested due to automatic review settings June 25, 2026 16:55
@JulianMaurin

JulianMaurin commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

This pull request is part of a Mergify stack:

# Pull Request Link
1 docs(cli): auto-derive command group copy #11939 👈
2 docs(api): generate a description for tags without editorial copy #11940

@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 25, 2026 16:55 Failure
@mergify

mergify Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 2 of 6 protections blocking · waiting on 👀 reviews

Protection Waiting on
🔴 👀 Review Requirements 👀 reviews
🔴 🔎 Reviews 👀 reviews
🟢 🤖 Continuous Integration
🟢 Enforce conventional commit
🟢 📕 PR description
🟢 🚦 Auto-queue

🔴 👀 Review Requirements

Waiting for

  • #approved-reviews-by >= 2
This rule is failing.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🔴 🔎 Reviews

Waiting for

  • #review-requested = 0
This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

Show 4 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@JulianMaurin JulianMaurin marked this pull request as ready for review June 25, 2026 16:56
@mergify mergify Bot requested a review from a team June 25, 2026 16:57

Copilot AI 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.

Pull request overview

This PR improves the CLI reference pages by precomputing an outward-facing command-group description (editorial → schema about → generated fallback), reworking group ordering so newly synced commands land with capability groups ahead of the maintenance tail, and adjusting tests to guard against stale editorial-map entries.

Changes:

  • Add CliGroup.description derived via a new groupDescription() helper (editorial → schema → generated fallback).
  • Update CLI index + group pages to render/use group.description, and add a build-time warning for groups still using fallback copy.
  • Update grouping tests to detect orphaned editorial entries and add dedicated tests for groupDescription() fallback tiers.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/pages/cli/index.astro Uses precomputed group descriptions, logs a warning for groups missing editorial copy, and mutes maintenance groups via MAINTENANCE_GROUPS.
src/pages/cli/[group].astro Uses group.description as the page meta description.
src/components/CliReference/cli-schema.ts Adds CliGroup.description, introduces groupDescription() and MAINTENANCE_GROUPS, and tweaks ordering to keep maintenance groups at the end.
src/components/CliReference/cli-schema.test.ts Replaces “must have editorial entry” assertions with stale-entry checks, and adds fallback behavior tests for groupDescription().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/components/CliReference/cli-schema.ts
Comment thread src/pages/cli/index.astro Outdated
groupDescription resolves a group's outward-facing copy as the editorial entry,
then the schema `about`, then a generated sentence; groupTopLevel exposes it as a
precomputed CliGroup.description. A command synced into cli-schema.json renders
non-blank without a hand-written entry, and sorts among the capabilities ahead of
the MAINTENANCE_GROUPS tail.

The grouping test now guards against editorial entries orphaned by a renamed or
dropped command rather than requiring an entry per command, and a build-time
warning lists groups still on fallback copy.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Change-Id: I3e5bccdcd1033d4bb51f383cddcd76c22c5b8947
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/cli-docs-auto-group-copy/auto-derive-cmd-group-copy--3e5bccdc branch from 5543d48 to f44358b Compare June 26, 2026 08:12
@JulianMaurin

Copy link
Copy Markdown
Contributor Author

Revision history

# Type Changes Reason Date
1 initial 5543d48 2026-06-26 08:12 UTC
2 content 5543d48 → f44358b 2026-06-26 08:12 UTC

@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 26, 2026 08:12 Failure
@mergify mergify Bot requested a review from a team June 26, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants