kcl-lang / kcl-openapi / 33054399940

64%

Build Type

push

github

Committed by web-flow

Commit Message

feat: --existing-models flag reuses pre-generated KCL models (fixes #18) (#160)

* feat: --existing-models flag reuses pre-generated KCL models (fixes #18)

Add a repeatable `--existing-models <alias>=<dir>` CLI option (and the
underlying GenOpts.ExistingModels field). For each directory the scanner
walks *.k files, extracts `schema <Name>` declarations, and records
them in a map keyed by the simple name. When the generator later
encounters a spec definition whose simple name matches, it skips the
file generation and instead emits `import <alias>` plus references of
the form `<alias>.<SchemaName>` for any $refs to it.

The behavior is wired through:
  - new LoadExistingModels helper in existing.go
  - newTypeResolverWith in types.go to seed the resolver with the map
  - resolveSchemaRef / resolveObject short-circuits to the alias
  - makeGenDefinitionHierarchy returns an External stub for matches

Tests cover: schema-name extraction, alias uniqueness, duplicate-name
detection, end-to-end reuse with a $ref'd model, and the no-match
baseline.

Signed-off-by: Claude <timi@noreply.anthropic.com>

* fix(generator): include both directories in LoadExistingModels conflict error

When a schema name is declared in two existing-model directories the
error used to print a placeholder `"<other>"` for the previous
directory. Track the source path of each schema and include both
absolute paths plus aliases in the message so operators can locate
the duplicates immediately.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Claude <claude@anthropic.com>
Signed-off-by: Claude <timi@noreply.anthropic.com>

* test(generator): assert error message names both dirs/aliases

The "schema in multiple aliases fails" subtest only asserted that some
error was returned. Strengthen it to require the new error format
(both directories plus both aliases are present), so the "<other>"
placeholder regression cannot reappear unnoticed.

Co-Authored-By: Claude Opus ... (continued)

Coverage Stats

97 of 117 new or added lines in 5 files covered. (82.91%)

33 existing lines in 2 files now uncovered.

2462 of 3683 relevant lines covered (66.85%)

1.47 hits per line