Repository navigation
feat(homepage): add an Artificial Intelligence group - #5962
Merged
Merged
Conversation
The maintainer asked for kagent to sit in its own "Artificial Intelligence" group on the Homepage dashboard, together with the web portals of Claude, ChatGPT and GitHub Copilot. Declare the group in the Homepage layout and in the Kyverno allow-list and its deny message, which test-restrict-homepage-service-groups keeps in agreement. Move the kagent route's card into it, and list the three portals statically, with their dashboard-icons logos. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
✅
|
Contributor
✅
|
The Kyverno group policy and the kagent route both apply in the infrastructure layer. In one deploy, the route's new group would reach admission before Kyverno has loaded the widened allow-list, so the update would be denied and the layer would fail. Ship the group and the three portals first; kagent's route moves into the group in a follow-up once the group is admitted. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
devantler
commented
Nov 9, 2026
devantler
left a comment
devantler
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 Generated by the Agentic Engineer (interactive Claude Code session)
Self-review (fallback — CodeRabbit, Codex and Cursor Bugbot unavailable)
Reviewed commit: a4a8396
- CodeRabbit: its included allowance on this repository is 1 review per hour. It used it on #5961 at 19:32Z and stated "0 remain after this review".
- Codex: usage limit since 2026-11-09T14:35:07Z; only the maintainer can lift it.
- Cursor Bugbot: usage limit since 2026-11-09T14:35:39Z; only the maintainer can lift it.
Scope. The Homepage layout gains an Artificial Intelligence group, three static services are listed under it, and the Kyverno group policy's allow-list and deny message both include the group.
Findings:
- Correctness: the layout, allow-list and deny message agree.
test-restrict-homepage-service-groupsreports 19 groups, and the policy still denies an undeclared group. The three hrefs are the official portals, and all three dashboard-icons resolve on the CDN. - Rollout: adding a group to the allow-list is safe on its own; static services do not pass through admission. Moving kagent's route into the group was taken out of this PR (a4a8396). The policy and the route both apply in the
infrastructurelayer, so in one deploy the route update could reach admission before Kyverno loads the widened list, and it would be denied. - nit: the layout and services comments already speak of kagent "beside" the tools. That becomes true with the follow-up PR that moves kagent's card.
Verdict: no P0/P1 findings
🤖 Generated with Claude Code
devantler
marked this pull request as ready for review
November 9, 2026 19:47
devantler
added a commit
that referenced
this pull request
Nov 9, 2026
GitHub Copilot now has its own card in the Artificial Intelligence service group (#5962), so the "Copilot Agents" bookmark in Internal Developer Platform duplicates it. The maintainer asked for it removed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The maintainer asked for kagent to sit in its own Artificial Intelligence group on the Homepage dashboard, together with the web portals of Claude, ChatGPT and GitHub Copilot.
What
homepage/config-map.yaml): a newArtificial Intelligencegroup with anmdi-robot-outlineicon, placed after Data.https://claude.ai/), ChatGPT (https://chatgpt.com/) and GitHub Copilot (https://github.com/copilot). Each uses its full-colour dashboard-icons logo, all three of which resolve on the CDN.restrict-homepage-service-groupslists the new group in both its allow-list and its deny message.Deferred: moving kagent's card into the group. That is a follow-up PR once this one has deployed. The group policy and the kagent route both apply in the
infrastructurelayer. In one deploy, the route's new group would reach admission before Kyverno has loaded the widened allow-list, the update would be denied, and the layer would fail. Earlier group additions only had routes in the laterappslayer, so they never hit this ordering.The existing "Copilot Agents" bookmark in Internal Developer Platform is unchanged.
Validation
test-restrict-homepage-service-groups: the layout, allow-list and deny message agree on 19 service groups, and the policy denies.test-validate-homepage-bookmarks,test-trivyignore-configmap-content-boundary, and the authorization tests, whose surface is unchanged.ksail --config ksail.prod.yaml workload validatepasses, and the changed files appear in its output. The Homepage chart render was skipped locally for lack of a registry credential helper; CI renders it.🤖 Generated with Claude Code