docs: update CLI and helm reference for v2.30.1#308
docs: update CLI and helm reference for v2.30.1#308github-actions[bot] wants to merge 2 commits into
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
|
Claude finished @mbevc1's task in 1m 39s —— View job Reviewing PR #308
Review SummaryThis PR updates CLI reference docs for v2.30.1 and adds a new consolidated Helm chart page. The CLI changes are auto-generated "live example" output refreshes — updated snapshot indices, timestamps, commit SHAs, and a couple of new response fields. Those all look correct. The main concern is around the new Findings
|
| Change type | Files |
|---|---|
| Updated GitHub commit SHA links | kosli_assert_artifact, kosli_attest_artifact, kosli_attest_custom, kosli_attest_generic, kosli_attest_junit, kosli_attest_pullrequest_github, kosli_attest_pullrequest_gitlab, kosli_attest_sonar, kosli_begin_trail |
| Refreshed live example output (timestamps, indices, data) | kosli_diff_snapshots, kosli_get_environment, kosli_get_snapshot, kosli_list_environments, kosli_list_flows, kosli_list_snapshots, kosli_log_environment |
New response fields in output (flow_name, trail_name) |
kosli_get_attestation |
| Updated CLI version string | overview |
These are all mechanical, auto-generated updates with no content issues. Internal links use root-relative paths. No relative links found.
Bottom line
The CLI reference updates are clean and ready. The helm/k8s_reporter.mdx situation needs a decision on whether it replaces or coexists with the existing sub-pages before merge — navigation must be updated either way.
| @@ -0,0 +1,397 @@ | |||
| --- | |||
There was a problem hiding this comment.
Critical: This new page is not listed in config/navigation.json, so it won't appear in the site navigation. Per CLAUDE.md: "Never create a page file without also adding it to navigation."
Additionally, the existing split pages (helm/k8s_reporter/overview, prerequisites, installing, etc.) are still present in the filesystem and navigation. This consolidated single-page file appears to duplicate their content. The intended relationship needs clarification:
- If this replaces the sub-pages: remove the old
helm/k8s_reporter/directory files, replace the navigation entries with"helm/k8s_reporter", and add redirects. - If this coexists: add
"helm/k8s_reporter"to navigation (e.g. as the group landing page) and differentiate the content.
Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Automated update of CLI and Helm reference documentation for release
v2.30.1.