Skip to content

docs(spec): correct stale eval-coverage Known Gap in meta-and-quality-tooling#561

Merged
potiuk merged 1 commit into
apache:mainfrom
justinmclean:meta-quality-spec-sync
Jun 26, 2026
Merged

docs(spec): correct stale eval-coverage Known Gap in meta-and-quality-tooling#561
potiuk merged 1 commit into
apache:mainfrom
justinmclean:meta-quality-spec-sync

Conversation

@justinmclean

Copy link
Copy Markdown
Member

Summary

The Known Gaps section claimed "~15 suites, coverage incomplete". All 44 shipped skills now have a matching suite in tools/skill-evals/evals/; the soft eval-coverage check (check #8) in skill-and-tool-validator enforces this going forward. Remove the stale back-fill claim and state the actual coverage.

Generated-by: Claude (Opus 4.7)

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:

…-tooling

The Known Gaps section claimed "~15 suites, coverage incomplete". All 44
shipped skills now have a matching suite in tools/skill-evals/evals/; the
soft eval-coverage check (check apache#8) in skill-and-tool-validator enforces
this going forward. Remove the stale back-fill claim and state the actual
coverage.

Generated-by: Claude (Opus 4.7)
@justinmclean justinmclean self-assigned this Jun 26, 2026

@potiuk potiuk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woohooo.

@potiuk potiuk merged commit 926480a into apache:main Jun 26, 2026
26 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.

2 participants