torben/chemenu
1
1
Fork 0
Code Issues 25 Pull Requests Actions Packages Releases 32 Activity

types/type-spec.schema.yaml ist veraltet und wird von nichts angewandt #105

New Issue
Closed
opened 2026-09-15 15:17:04 +00:00 by torben · 1 comment
torben commented 2026-09-15 15:17:04 +00:00
Owner
Copy Link
Copy Source

Befund

types/type-spec.schema.yaml deklarierte additionalProperties: false, kannte aber die Felder
root: und capture_fields: nicht, obwohl types/instruction.md (root: repo) und
types/source.md (capture_fields: [fidelity, authority]) beide tragen und
type_resolver.get_root()/get_capture_fields() sie lesen. Gegen das Schema validiert waeren
beide Type-Specs ungueltig gewesen - und niemandem fiel es auf, weil Type-Spec-Frontmatter
nirgends gegen das Schema validiert wurde.

Entscheidung

Wie im Befund empfohlen: eingeschaltet, nicht als reine Dokumentation gekennzeichnet.

Umsetzung (erledigt)

  • root: (Enum kb/repo) und capture_fields: (Liste wie page_ref_fields:) im Schema
    ergaenzt. Dazu guidance:, das seit #104 real benutzt wird, aber ebenfalls nie im Schema stand.
  • docs verify bekam eine neue Pruefung (check_type_spec_frontmatter()): jede Datei unter
    types/ mit type: types/type-spec.md validiert jetzt gegen types/type-spec.schema.yaml,
    wiederverwendet resolver.list_type_specs() statt eines zweiten Parse-Durchlaufs.
  • types/type-spec.md § Validation Contract und beide docs verify-Zeilen in
    tools/CONTRACT.md nennen die neue Pruefung.
  • Nebenbei behoben: instructions/dev/doc-pull-through.md verwies noch auf "AGENTS.md § File
    naming lists all four" docs/-Seiten - der Zaehler in AGENTS.md selbst stand schon auf fuenf
    (plus eine sechste, nur von CLAUDE.md verlinkte).

Akzeptanzkriterien

  • root: und capture_fields: stehen im Schema, mit den Mustern, die get_root() und
    get_capture_fields() tatsaechlich akzeptieren (root: nur kb oder repo).
  • Jede Datei unter types/ mit type: types/type-spec.md validiert gegen
    types/type-spec.schema.yaml - nachgewiesen durch
    test_this_repos_type_specs_validate_against_their_own_schema, das ueber alle Type-Specs
    des Repos laeuft (nicht durch einen Einzelfall), und test_an_unknown_type_spec_field_is_reported
    fuer die Gegenrichtung.
  • Ein Type-Spec mit einem im Schema unbekannten Frontmatterfeld wird gemeldet (eingeschaltet,
    also diese Haelfte des Kriteriums).
  • docs verify, instructions verify und pytest gruen (1263 Tests, 2 neu).

Verifiziert

  • tools/wikitool docs verify / instructions verify: gruen
  • pytest in tools/: 1263 gruen (2 neue Tests fuer diese Aenderung)
  • check_type_spec_frontmatter() gegen den realen Baum: 0 Befunde (alle acht bestehenden
    Type-Specs validieren)

Grenzuebertritt

Additiv in beide Richtungen - eine bestehende Instanz validiert bereits (0 Befunde gegen den
realen Baum), ein Type-Spec ohne die drei neuen Felder bleibt unveraendert gueltig. --patch,
kein --breaking, keine neue Migration noetig.

Veroeffentlicht als 6.0.0-beta.7, Commit 55f65c1.

## Befund `types/type-spec.schema.yaml` deklarierte `additionalProperties: false`, kannte aber die Felder `root:` und `capture_fields:` nicht, obwohl `types/instruction.md` (`root: repo`) und `types/source.md` (`capture_fields: [fidelity, authority]`) beide tragen und `type_resolver.get_root()`/`get_capture_fields()` sie lesen. Gegen das Schema validiert waeren beide Type-Specs ungueltig gewesen - und niemandem fiel es auf, weil Type-Spec-Frontmatter nirgends gegen das Schema validiert wurde. ## Entscheidung Wie im Befund empfohlen: **eingeschaltet**, nicht als reine Dokumentation gekennzeichnet. ## Umsetzung (erledigt) - `root:` (Enum `kb`/`repo`) und `capture_fields:` (Liste wie `page_ref_fields:`) im Schema ergaenzt. Dazu `guidance:`, das seit #104 real benutzt wird, aber ebenfalls nie im Schema stand. - `docs verify` bekam eine neue Pruefung (`check_type_spec_frontmatter()`): jede Datei unter `types/` mit `type: types/type-spec.md` validiert jetzt gegen `types/type-spec.schema.yaml`, wiederverwendet `resolver.list_type_specs()` statt eines zweiten Parse-Durchlaufs. - `types/type-spec.md` § Validation Contract und beide `docs verify`-Zeilen in `tools/CONTRACT.md` nennen die neue Pruefung. - Nebenbei behoben: `instructions/dev/doc-pull-through.md` verwies noch auf "AGENTS.md § File naming lists all four" `docs/`-Seiten - der Zaehler in AGENTS.md selbst stand schon auf fuenf (plus eine sechste, nur von CLAUDE.md verlinkte). ## Akzeptanzkriterien - [x] `root:` und `capture_fields:` stehen im Schema, mit den Mustern, die `get_root()` und `get_capture_fields()` tatsaechlich akzeptieren (`root`: nur `kb` oder `repo`). - [x] Jede Datei unter `types/` mit `type: types/type-spec.md` validiert gegen `types/type-spec.schema.yaml` - nachgewiesen durch `test_this_repos_type_specs_validate_against_their_own_schema`, das ueber alle Type-Specs des Repos laeuft (nicht durch einen Einzelfall), und `test_an_unknown_type_spec_field_is_reported` fuer die Gegenrichtung. - [x] Ein Type-Spec mit einem im Schema unbekannten Frontmatterfeld wird gemeldet (eingeschaltet, also diese Haelfte des Kriteriums). - [x] `docs verify`, `instructions verify` und `pytest` gruen (1263 Tests, 2 neu). ## Verifiziert - `tools/wikitool docs verify` / `instructions verify`: gruen - `pytest` in `tools/`: 1263 gruen (2 neue Tests fuer diese Aenderung) - `check_type_spec_frontmatter()` gegen den realen Baum: 0 Befunde (alle acht bestehenden Type-Specs validieren) ## Grenzuebertritt Additiv in beide Richtungen - eine bestehende Instanz validiert bereits (0 Befunde gegen den realen Baum), ein Type-Spec ohne die drei neuen Felder bleibt unveraendert gueltig. `--patch`, kein `--breaking`, keine neue Migration noetig. Veroeffentlicht als `6.0.0-beta.7`, Commit `55f65c1`.
torben added the prio/plannedsize/Sarea/kbkind/defect labels 2026-09-15 15:17:04 +00:00
torben commented 2026-09-15 17:47:35 +00:00
Author
Owner
Copy Link
Copy Source

root:, capture_fields:, guidance: ins Schema nachgetragen, docs verify validiert jetzt jeden Type-Spec dagegen, doc-pull-through.mds veralteter "vier"-Verweis auf fuenf korrigiert. 6.0.0-beta.7, Commit 55f65c1.

`root:`, `capture_fields:`, `guidance:` ins Schema nachgetragen, `docs verify` validiert jetzt jeden Type-Spec dagegen, `doc-pull-through.md`s veralteter "vier"-Verweis auf fuenf korrigiert. `6.0.0-beta.7`, Commit `55f65c1`.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area/corpus
Betrifft Demo-/Testbett-Frage, Inhalt und Umfang von kb/
 area/distribution
Betrifft Auslieferung, Upgrade, Versionierung einer Instanz
 area/kb
Betrifft kb/-Schema, Contract, Confidence, Lint, Wissensbasis
 area/process
Betrifft den Entwicklungsprozess selbst, nicht den Stack als Artefakt
 area/workflow
Betrifft Git, Merge, Branching, Publish, PRs
 kind/build
Spezifiziert, wartet nur noch auf Umsetzungszeit
 kind/decision
Wartet auf eine Betreiberentscheidung
 kind/defect
Befund: Doku und Realitaet, oder zwei Dokus, widersprechen sich
 prio/blocking
Blockiert oder beschaedigt laufende Arbeit - als naechstes
 prio/planned
Traegt bald Zinsen - eingeplant
 prio/waiting
Sinnvoll, wartet auf einen Ausloeser
 size/L
Mehrere Sitzungen oder offene Designfragen vor dem ersten Commit
 size/M
Mehrere Dateien, Contract- oder Instruction-Aenderung, eigener Testaufwand
 size/S
Eine Sitzung, ein Publish, klar umrissener Schnitt
 status/blocked
Wartet auf ein anderes, noch offenes Issue - nicht eigenstaendig bearbeitbar
 status/incoming
Vom Menschen angelegte Anforderung, unvollständig, muss durch den Stack komplettiert werden.
 status/unconfirmed
Gemeldeter Verdacht, noch nicht gegen tatsaechliches Verhalten geprueft - size/prio vorlaeufig
No Label area/kb kind/defect prio/planned size/S
Milestone
No items
No Milestone
Assignees
Clear assignees
torben
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: torben/chemenu#105
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?