Skip to content

test: multi-commit CHANGELOG coverage E2E#633

Open
sternryan wants to merge 1 commit intogarrytan:mainfrom
sternryan:test/changelog-multi-commit
Open

test: multi-commit CHANGELOG coverage E2E#633
sternryan wants to merge 1 commit intogarrytan:mainfrom
sternryan:test/changelog-multi-commit

Conversation

@sternryan
Copy link
Copy Markdown

Summary

  • Adds an E2E test that creates a temp repo with 6 commits across 4 themes (features, bug fix, infra, docs), runs the CHANGELOG generation section from ship/SKILL.md, and verifies all themes appear in the generated CHANGELOG
  • Extracts only the CHANGELOG section from SKILL.md into the test fixture (per CLAUDE.md rule against copying full SKILL.md files)
  • Classified as periodic tier in E2E_TIERS (non-deterministic LLM behavior)

Test plan

  • bun test test/skill-validation.test.ts passes (365/365)
  • bun test test/gen-skill-docs.test.ts passes (236/236)
  • bun test test/touchfiles.test.ts passes (23/23)
  • Run with EVALS=1 bun test test/skill-e2e-changelog.test.ts to verify CHANGELOG generation covers all themes

Add E2E test that creates a temporary git repo with 6 commits across
4 themes (features, bug fix, infrastructure, docs), runs the CHANGELOG
generation step from ship/SKILL.md, and verifies all themes appear in
the output.

- Extracts only the CHANGELOG section from SKILL.md (not the full file)
- Classified as periodic tier (non-deterministic LLM behavior)
- Verifies version header, theme coverage (3+ of 4 categories)
- Registered in E2E_TOUCHFILES and E2E_TIERS
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