docs(skills): stop forbidding the target register and correct the card rules - #137
Merged
Merged
Conversation
…d rules The authoring rules are the de-facto schema an LLM generates from, and two of them worked against the project. typography-readability required a line-height between 1.4 and 1.6 and declared that below 1.2 lines collide and text becomes unreadable. Sub-1.0 leading is the single most identifiable trait of the register this project is building toward, and both shipped templates violated the rule — dark-premium at 1.04 throughout, 1600-style between 0.80 and 1.02. It also contradicted the brutalist rule with no precedence between them. There is now a statement register documented with measured scale and tracking, and an explicit precedence. Measurement also showed the two templates use opposite tracking signs — 1600-style positive, dark-premium negative — so both are described rather than one being treated as the error. The card rules were not stale but reversed. stat-cards advised using stat over counter for centring; measurement shows counter centres exactly on the content box while stat has no intrinsic, overflows its own box, and renders nothing at all when three sit in a flex row. counter-standalone's claimed missing baseline correction is not reproducible; the real defect is horizontal and validator-silent. grid-card-height's BAD case does not reproduce on this build. card-flex-layout never mentioned that 23 component types are invisible without an explicit height — its own good example works only because its icon carries one. The component count in SKILL.md said 51 against 57 in the enum, and qr_code was missing from the catalogue entirely. CLAUDE.md already carried the correct count; that part of the brief was stale. All 23 fenced JSON blocks in the touched files were extracted and validated, including two pre-existing broken ones that the sweep caught. Closes #130
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
1 participant
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.
Closes #130. Part of the chantier #123.
Independent of the other six workstreams: disjoint file set, compiles and tests green on its own against
main. Mergeable in any order.Full rationale, measurements and evidence are in the commit message and in #130.