Skip to content

[deckhouse-cli] Update vendored package v0.0.31 - #439

Merged
ldmonster merged 2 commits into
mainfrom
chore/update-vendored-package-v0.0.31
Aug 5, 2026
Merged

[deckhouse-cli] Update vendored package v0.0.31#439
ldmonster merged 2 commits into
mainfrom
chore/update-vendored-package-v0.0.31

Conversation

@Glitchy-Sheep

@Glitchy-Sheep Glitchy-Sheep commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

d8 package is compiled into d8, so it never updates on its own. This syncs the vendored code with upstream v0.0.31: two new verify rules, a Russian translation file in the bootstrap templates, and a render fix for array examples.

What changed

New rules in the openapi linter of d8 package verify:

  • rule enum: enum values in OpenAPI schemas must be CamelCase and start with a capital letter (warning)
  • rule bilingual: every schema under openapi/ must have a doc-ru-*.yaml translation next to it (error)
  • values.yaml and *-tests.yaml fixtures are skipped by bilingual
  • a doc-ru-*.yaml without a matching schema is also an error - that catches a misspelled name
  • d8 package doc lists both rules

Bootstrap templates:

  • bootstrap module and bootstrap app now generate openapi/doc-ru-settings.yaml
  • so new packages pass the bilingual rule out of the box

Render:

  • a bare x-example on an array property now becomes a one-element list
  • before, toYaml and range in templates got a scalar instead of a list
  • the stubbed publicDomainTemplate now matches the real platform format (%s.%s.domain.io)

Verification

  • 163 files of the vendored tree match upstream v0.0.31 (8899c68), byte for byte
  • two files differ on purpose: packagecmd.go (d8 only) and pkg/cmd/cmd.go (no version subcommand)
  • go.mod needs nothing new; task build and task test are green, the vendored tree lints clean
  • smoke on a bootstrapped app package: bad enum values warn, a deleted doc-ru-settings.yaml fails verify, a scalar array example renders as a list

Notes

  • a second commit fixes 4 wsl_v5 issues in internal/snapshot/ that made task lint:check red on main (missing blank lines, no logic changes)

- verify: new "openapi/enum" rule. Enum values in OpenAPI schemas must
  be CamelCase and start with a capital letter.
- verify: new "openapi/bilingual" rule. Every schema under openapi/
  must have a doc-ru-*.yaml translation next to it. values.yaml and
  test fixtures are exempt.
- bootstrap: module and application templates now include
  openapi/doc-ru-settings.yaml, so new packages pass the bilingual
  rule out of the box.
- render: a bare x-example on an array property now becomes a
  one-element list, so toYaml and range in templates get the right
  shape.
- render: the stubbed publicDomainTemplate now matches the real
  platform format ("%s.%s.domain.io").

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
Four statements in archive/atomic.go and transport/http.go lacked a
blank line above them. They made "task lint:check" red on main since
the snapshot command landed (e4cd67a).

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
@Glitchy-Sheep Glitchy-Sheep self-assigned this Aug 5, 2026
@Glitchy-Sheep
Glitchy-Sheep marked this pull request as ready for review August 5, 2026 11:27
@ldmonster
ldmonster merged commit 348612f into main Aug 5, 2026
9 checks passed
@ldmonster
ldmonster deleted the chore/update-vendored-package-v0.0.31 branch August 5, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants