🌱 Add structured CI-observability summary to check-internal-links.yml - #6802
🌱 Add structured CI-observability summary to check-internal-links.yml#6802kubestellar-hive[bot] wants to merge 1 commit into
Conversation
scripts/check-internal-links.ts only printed decorative free-text lines. Emit one bounded, grep-friendly CI_OBSERVABILITY line (counts only, no file paths or link targets) and a short markdown table to $GITHUB_STEP_SUMMARY, matching the pattern already used for vitest.yml/fuzz-mdx.yml. Fixes #6801 Signed-off-by: kubestellar-hive[bot] <kubestellar-hive[bot]@users.noreply.github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Deploy Preview for kubestellar-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Hi @kubestellar-hive[bot]. Thanks for your PR. I'm waiting for a kubestellar member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Important Held for human sign-off on the direction, not on the code. This PR's only tracked rationale is #6801, which the hive filed itself — issue #6801 was filed by kubestellar-hive[bot] and no human has acknowledged it. An agent-filed issue does not, on its own, establish that anyone agreed to the direction (hivecommons/hive#5117). The change may well be right; nothing here is a review of it. To release the hold, acknowledge the direction on that issue — comment on it, assign yourself, or add the |
Fixes #6801
What
scripts/check-internal-links.ts(run bycheck-internal-links.ymlon every PR/push touchingdocs/content/**) only printed decorative free-text lines to stdout. There was no machine-readable summary and nothing written to$GITHUB_STEP_SUMMARY.Change
CI_OBSERVABILITY check=internal-links links_checked=<n> routes=<n> broken=<n> status=pass|fail. Only aggregate counts are included — no file paths or link targets (those stay in the existing free-text error detail, unbounded and only printed on failure as before).$GITHUB_STEP_SUMMARYwhen running in CI (no-op locally where the env var is unset).This mirrors the CI-observability convention already used for
vitest.yml/fuzz-mdx.ymlin this repo.Testing
npm run type-check— passesnpm run check-links(withGITHUB_STEP_SUMMARYset) — verified structured line + summary table output, 0 broken linksnpx vitest runon the two existing helper test suites (97 tests) — all passing, unaffected by this change (no test coverage change needed;main()is only invoked when run as a script, unchanged from prior behavior)No exporter, external data flow, or secrets introduced. CI-observability only.
— hive: agent=telemetry backend=copilot model=claude-sonnet-4-6