fix: add missing param to four eyes schema

[mbevc1]

Extend the schema and allow passing params to rego policy

[claude]

Claude finished @mbevc1's task in 1m 46s —— View job

---

PR Review: fix: add missing param to four eyes schema

• Gather context and read changed files
• Run static analysis checks
• Review code quality and correctness
• Post review feedback

---

Static Analysis

All checks pass:

• go build ./... — clean
• go vet ./... — clean
• go mod tidy — no drift

Review

The change is correct and necessary. The rego policy already references data.params.attestation_name (line 25 of four-eyes-policy.rego), and the Go code in evaluateHelpers.go:232-234 conditionally includes "params" in the output map. Without this schema entry, any output containing params would fail validation against additionalProperties: false on the schema's root object.

One minor observation: the Go code gates params behind showInput && params != nil (line 232), meaning params only appears in the output when --show-input is also passed. The schema description says "surfaced in the output" without mentioning this prerequisite — that's fine for a schema (it just declares what's allowed), but worth knowing.

Placement is good — params is inserted between current_commit and input, which is a logical grouping since both params and input are optional diagnostic fields.

No tests needed — this is a standalone JSON schema file; correctness is enforced by the schema structure itself and there are no existing schema-validation tests to update.

Summary

Clean, minimal fix. LGTM — the schema was missing a property that the code already emits, and additionalProperties: false would reject any output containing it. Good catch.