Skip to content

chore(ai-tutors): regenerate knowledge base for lesson-04-your-first-skill - #928

Closed
justinmclean wants to merge 1 commit into
apache:mainfrom
justinmclean:ai-tutors-regenerate2
Closed

chore(ai-tutors): regenerate knowledge base for lesson-04-your-first-skill#928
justinmclean wants to merge 1 commit into
apache:mainfrom
justinmclean:ai-tutors-regenerate2

Conversation

@justinmclean

Copy link
Copy Markdown
Member

Summary

Step-6 (debugging-skills) and step-7 (portable-skills) were added to the learning progression after this tutor was authored; the injector now adds them to the "Where to go next" section and renumbers the eval-driven- development reference from step-5 to steps-5–7.

Generated-by: Claude (claude-sonnet-4-6)

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

  • [X ] 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:

…skill

Step-6 (debugging-skills) and step-7 (portable-skills) were added to the
learning progression after this tutor was authored; the injector now adds
them to the "Where to go next" section and renumbers the eval-driven-
development reference from step-5 to steps-5–7.

Generated-by: Claude (claude-sonnet-4-6)
@potiuk

potiuk commented Aug 1, 2026

Copy link
Copy Markdown
Member

Closing — superseded by #1010, which regenerated lesson-04 as part of fixing the nested-code-fence bug in the injector itself.

Worth recording why this is a close rather than a merge: the regeneration here now predates two source-page changes, so merging it would roll lesson-04 backwards:

main (via #1010) this PR
skill invocation /magpie-write-skill /write-skill
capability labels capability:review, :fix, :intake :security, :release, :contributor-growth

Neither difference is fence-related — they are ordinary content drift that #1010's regeneration picked up because it ran later. inject-knowledge-base.py --check on current main no longer lists lesson-04, so this PR's goal is already met.

lesson-08-eval-driven-development.md is still stale on main — it was stale before any of the fence work, from unrelated source-page edits. That one still wants a regeneration commit, and would be a direct follow-up to this.

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