cozystack

0 Fork 0

mirror of https://github.com/cozystack/cozystack.git synced 2026-10-30 00:27:04 +09:00

Files

T

Timofei Larkin 50f8548ba9 fix(ci): stop double-parsing every file in the API gate's content walk

branch-review on #3184 measured a real regression: switching CRD/
APIService discovery to content-based scanning made every file in
packages/+internal a candidate, and each was split into documents and
fully unmarshaled twice — once per parser — inflating a full snapshot
load from ~0.5s to ~3.4s.

Split each file's documents once and probe each document's kind once
before deciding which shape to try, so the common case (a document
that is neither a CRD nor an APIService — the overwhelming majority
of what the wider walk now sees) pays for at most one parse attempt
instead of two. Add a raw substring fast-reject, at both the
per-document and whole-file level, ahead of any YAML parsing: neither
kind this package cares about can appear without its own name
appearing as a literal substring first.

Also fixes two smaller gaps flagged in the same review: an untrusted
PR head checkout could point discovery at an arbitrarily large file
with no size guard (capped at 8 MiB), and a group's merged APIService
resource only recorded the first contributing file's path, dropping
the others from the report.

Assisted-By: Claude <noreply@anthropic.com>
Signed-off-by: Timofei Larkin <lllamnyp@gmail.com>

2026-07-06 15:48:07 +03:00

classify_test.go

fix(ci): address review of the API gate

2026-07-01 14:51:33 +03:00

classify.go

fix(ci): address review of the API gate

2026-07-01 14:51:33 +03:00

model.go

fix(ci): catch templated CRDs and APIService-only groups in the API gate

2026-07-06 15:48:07 +03:00

parse_test.go

fix(ci): stop double-parsing every file in the API gate's content walk