Skip to content

docs: add missing pairing family row to adoption table#892

Merged
rbowen merged 1 commit into
apache:mainfrom
harshjainnn:docs/add-pairing-family-to-adoption-table
Jul 15, 2026
Merged

docs: add missing pairing family row to adoption table#892
rbowen merged 1 commit into
apache:mainfrom
harshjainnn:docs/add-pairing-family-to-adoption-table

Conversation

@harshjainnn

Copy link
Copy Markdown
Contributor

Summary

  • Added a row for the pairing family to the adoption table in docs/index.md
  • The pairing family (pairing-self-review, pairing-multi-agent-review) ships under skills/ and has its own docs/pairing/README.md, but was never added to the adoption table, so adopters browsing the table wouldn't discover it

Type of change

  • Skill change (.claude/skills/<name>/) — eval fixtures updated below
  • Tool / bridge contract (tools/<system>/*.md)
  • Python package (tools/*/ with pyproject.toml)
  • Groovy reference impl
  • Cross-cutting (RFC, AGENTS.md, sandbox, privacy-LLM)
  • Documentation (docs/, README.md, CONTRIBUTING.md)
  • Project template (projects/_template/)
  • CI / dev loop (prek, workflows, validators)
  • Other:

Test plan

  • prek run --all-files passes
  • For Python packages touched: uv run pytest / ruff check / mypy passes
  • For Groovy bridges touched: command-line invocation tested end-to-end
  • For skill changes: eval suite passes for the affected skill
    (PYTHONPATH=tools/skill-evals/src python3 -m skill_evals.runner tools/skill-evals/evals/<skill>/)
  • For skill behaviour changes: a new or updated eval fixture is included in this PR
    (a regression test for the bug fixed / the behaviour added — see CONTRIBUTING.md)
  • Other: confirmed the table now lists 10 families, matching the 10 present under skills/; link target pairing/README.md resolves and its scope note ("no ASF-specific assumptions") matches the "Any project" scope used in the new row

RFC-AI-0004 compliance

  • HITL — any new mutation is gated on explicit user confirmation
  • Sandbox — no new unrestricted host access; network reach declared in the adapter
  • Vendor neutrality — placeholders (<PROJECT>, <tracker>, <upstream>, <security-list>) used in all skill / tool prose (the check-placeholders prek hook is the mechanical gate)
  • Conversational + correctable — agentic-override path documented if behaviour is adopter-tunable
  • Write-access discipline — no autonomous outbound messages; drafts only, sent on confirmation
  • Privacy LLM — private content does not reach a non-approved LLM; redactor invoked where needed

Linked issues

Closes #872

Notes for reviewers (optional)

Placed the new row directly after pr-management rather than at the end, since pairing is a pre-flight step in the same PR-authoring workflow — open to moving it if a different position reads better. No other doc references needed updating (the doctoc TOC block only tracks headings, not table rows).

The pairing family (pairing-self-review, pairing-multi-agent-review)
ships under skills/ and has its own docs/pairing/README.md, but was
never added to the adoption table in docs/index.md. Adopters browsing
the table wouldn't discover it.

Fixes apache#872
@rbowen

rbowen commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Thanks!

@rbowen rbowen merged commit 7b808e6 into apache:main Jul 15, 2026
9 checks passed
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.

docs/index.md: missing pairing family from adoption table

2 participants