culture(schleswig_holstein): add ## Title section (khai-tests v0.1.19 contract)#558
Merged
Merged
Conversation
Follow-up to satisfy khai-tests v0.1.19's component contract: every
khai file must carry a ## Title chapter. The four SH content files
that don't already have one (position_language, piece_doppeleiche,
place_kiel, process_deichgraf) now do.
Each insertion is a two-line addition right after the existing
level-1 heading:
# Place: Kiel
+ ## Title
+ Kiel
+
## Owner
The existing `# <Kind>: <Name>` heading is preserved -- the engine
convention stands; the new chapter just satisfies the structural test.
Personas already have ## Title (with their role as subtitle), so
they pass without change. The SH position file (culture_german_sh_position.md)
also already passes -- only the language / piece / place / process
needed the new chapter.
Verified locally:
pytest --pyargs khai_tests.components.test_khai_{position,piece,place,process}
--khai-files="regions/europe/germany/schleswig_holstein/*.md"
-> passes (was: 4 failed across the four kinds)
https://claude.ai/code/session_01PaUEPspVUzy36LUZ4t1yEU
khai_tests.review on this PR
|
khai_tests.review on this PR
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Companion to
culture/enginefor the SH side. khai-tests v0.1.19 (recently pinned via #546) requires every khai component file to carry a## Titlechapter; the SH content files added in #549/#553 don't have one. The release PR #556 fails on:This PR adds the chapter to the four SH content files.
Pattern
Personas already carry
## Titlewith their role as subtitle, so they pass without change. The SH position file (culture_german_sh_position.md) also already passes — only the language / piece / place / process files needed it.Verified locally
Order
Land alongside
culture/engineandculture/united_kingdom. All three unblock #556.https://claude.ai/code/session_01PaUEPspVUzy36LUZ4t1yEU
Generated by Claude Code