nightcityblade opened a new pull request, #354:
URL: https://github.com/apache/airflow-steward/pull/354

   ## Summary
   
   - update spec-loop docs to use the current `tools/skill-and-tool-validator` 
path
   - replace stale `skill-validate` references with the renamed 
`skill-and-tool-validate` CLI
   - keep the documentation aligned with the repo's current validator layout 
for issue #351
   
   ## 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)
   - [x] 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)
   - [x] Other: `git grep -nE 'skill-validator|skill-validate\\b' -- 
':!tools/skill-and-tool-validator' | grep -v skill-and-tool || true`
   - [x] Other: attempted `uv run --project tools/skill-and-tool-validator 
--group dev skill-and-tool-validate`, but local `uv` is pinned to 0.10.7 and 
the repo requires `>=0.11.8`
   
   ## 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
   - [x] **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 #351
   
   ## Notes for reviewers (optional)
   
   The change is docs-only. I left the test-plan note about the `uv` version 
mismatch so the failed local validator invocation is explicit rather than 
silently omitted.
   


-- 
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]

Reply via email to