andygrove opened a new pull request, #4442: URL: https://github.com/apache/datafusion-comet/pull/4442
## Which issue does this PR close? Closes #. ## Rationale for this change The existing `audit-comet-expression` skill walks a human through auditing one expression interactively. There is no way to march through the backlog of `[x]`-marked but un-audited entries in `spark_expressions_support.md` without a human in the loop on every step. These two new skills let an operator (or `/loop`) drive expression audits autonomously: each invocation picks the next un-audited expression, runs a full per-expression audit in a fresh worktree, and opens a draft PR plus any `requires-triage` issues for behavioural divergence. The skills live under `.claude/skills/` and are inert until invoked, so they have no effect on builds, tests, or anyone who is not running these slash commands. ## What changes are included in this PR? - `.claude/skills/audit-comet-expression-autonomous/SKILL.md` — non-interactive variant of `audit-comet-expression`. Audits a single expression end to end: pre-flight checks, worktree setup, Spark cross-version study, Comet/DataFusion implementation review, missing-coverage test additions, draft PR, and a separate `requires-triage` issue per behavioural divergence found. - `.claude/skills/audit-comet-expression-batch/SKILL.md` — dispatcher that scans `docs/source/contributor-guide/spark_expressions_support.md` for `[x]`-marked entries with no sub-bullets (the un-audited backlog) and dispatches `audit-comet-expression-autonomous` against them, one per invocation by default. Designed to be driven manually or by `/loop`. No code changes; documentation/automation only. ## How are these changes tested? Manual: dry-runs of the batch dispatcher against the current `spark_expressions_support.md` correctly identified candidates and dispatched the per-expression audit. The per-expression skill's pre-flight checks, worktree setup, and abort paths were exercised against both valid and invalid expression names. CI is not exercised by this change because nothing under `.claude/skills/` is loaded by the build or test runs. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
