Skip to content

Add repository-scoped ehrQL authoring skill - #157

Draft
sebbacon wants to merge 3 commits into
mainfrom
agent/provide-ehrql-skill-https-github-com-seb-4cd7d8
Draft

sebbacon wants to merge 3 commits into
mainfrom
agent/provide-ehrql-skill-https-github-com-seb-4cd7d8

Conversation

@sebbacon

@sebbacon sebbacon commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • vendor the ehrQL dataset-authoring skill and bundled references under .agents/skills
  • adapt the skill to use the ehrQL executable supplied by the OpenSAFELY Codespace image
  • document automatic Codex discovery and the review-based update process

Testing

  • uv run --no-project --with pyyaml python /home/agent/.codex/skills/.system/skill-creator/scripts/quick_validate.py .agents/skills/ehrql-dataset-authoring
  • verified every reference linked from SKILL.md exists and is non-empty
  • git diff --check main...HEAD
  • scanned changed files for common secret patterns

sebbacon and others added 3 commits September 5, 2026 11:42
Copilot has no skill/plugin loader, so the Codex-style .agents/skills
vendoring was never consumed. Move the skill to a Copilot-visible setup:

- Add .github/instructions/ehrql.instructions.md (Copilot Chat, path-scoped
  to analysis/**, codelists/**, dummy-tables/**) and AGENTS.md (Copilot
  coding agent), both summarising the workflow and pointing at the library.
- Relocate .agents/skills/ehrql-dataset-authoring -> .github/skills/, drop
  the Codex-only .agents/README.md and agents/openai.yaml.
- Trim SKILL.md and references/source-index.md to what runs in this repo:
  drop the plugin-root / scaffold-script / SessionStart / uv / .venv
  machinery; ehrQL is invoked via `opensafely exec ehrql:v1` (matching
  project.yaml), with bare `ehrql` noted as the Codespace-image equivalent.
- Update README.md "AI-assisted ehrQL development" for the Copilot setup.

The 10 references/upstream/*.md ehrQL docs are unchanged.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VpaWHjkf2e8DiZkwVzHKPg
@sebbacon
sebbacon marked this pull request as draft September 14, 2026 10:33
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.

1 participant