accordproject / template-archive / 39490482219

90%

push

github

feat(cicero-core): read data.json as sample data when there is no sample.json (#964)

* feat(cicero-core): read data.json as sample data when there is no sample.json

cicero-core reads and archives a template's sample data as sample.json,
while `cicero draft` defaults to data.json, the name the template library
and many templates use. So `cicero archive` dropped a data.json template's
sample data.

TemplateLoader now falls back to data.json (fromDirectory and fromArchive)
when there is no sample.json, warning when both exist; toArchive still writes
only sample.json. `cicero draft` defaults to sample.json when the template
has one, else data.json as before.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Matt Roberts <code@rbrts.uk>

* fix(cicero-core): parse only the sample data file that is used

pickSampleData parsed data.json even when sample.json won, so a malformed
data.json beside a valid sample.json stopped fromDirectory and fromArchive.
It now reads both as text and parses only the one it uses.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Matt Roberts <code@rbrts.uk>

---------

Signed-off-by: Matt Roberts <code@rbrts.uk>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

253 of 288 branches covered (87.85%)

Branch coverage included in aggregate %.

14 of 14 new or added lines in 2 files covered. (100.0%)

807 of 894 relevant lines covered (90.27%)

557.9 hits per line