feat: Add CLAUDE.md files to improve AI processing by mrsabath · Pull Request #113 · rossoctl/cortex

mrsabath · 2026-02-12T17:00:09Z

Description

This PR creates CLAUDE.md files that make processing the repo by Agents much faster and easier.
Also it changes CI behavior (Go version bump) and includes a functional webhook behavior change (client-registration injection is now actually gated by the flag)

Fixes # (issue)

Type of Change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Documentation update
Refactoring (no functional changes)
Performance improvement
Test addition or update
CI/CD update

How Has This Been Tested?

Test A
Test B

Test Configuration:

Kubernetes version:
Toolhive webhook version:
OS:

Checklist

My code follows the style guidelines of this project
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules

Screenshots (if appropriate)

Additional Notes

Copilot

Pull request overview

Adds CLAUDE.md context files to help AI agents understand and navigate the monorepo and its main subprojects (webhook + AuthBridge).

Changes:

Added top-level CLAUDE.md summarizing repo structure, components, and cross-component contracts.
Added AuthBridge/CLAUDE.md detailing AuthBridge internals, flows, and operational gotchas.
Added kagenti-webhook/CLAUDE.md documenting webhook architecture, injection logic, and developer workflows.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File	Description
`CLAUDE.md`	Monorepo-wide AI context (structure, workflows, component relationships).
`AuthBridge/CLAUDE.md`	AuthBridge-specific AI context (AuthProxy + client-registration + demo flows).
`kagenti-webhook/CLAUDE.md`	Webhook-specific AI context (mutation/injection logic, repo conventions, test/dev commands).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

huang195 · 2026-02-18T14:39:05Z

@mrsabath I would recommend to add just the CLAUDE.md file(s) and not mix it up with changes in the *.go file. Let those changes in *.go files be another PR perhaps.

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Add CLAUDE.md files at root, AuthBridge/, and kagenti-webhook/ levels to provide structured context for AI coding agents working with this repository. Signed-off-by: Mariusz Sabath <mrsabath@gmail.com>

mrsabath · 2026-02-18T20:22:59Z

All the Copilot suggestions are addressed by issue #122

mrsabath · 2026-02-18T20:30:39Z

@mrsabath I would recommend to add just the CLAUDE.md file(s) and not mix it up with changes in the *.go file. Let those changes in *.go files be another PR perhaps.

@huang195 I split this PR into 2 PRs. This one addresses only the CLAUDE.md files and the other one is addressing the CI pipeline changes. Please review and approve.

huang195 · 2026-02-18T22:09:20Z

@mrsabath 2 issues:

Reference to github.com/huang195/auth-proxy is outdated. That was fixed by Morgan in one of her PRs.
Since we're trying to remove the dependency of having to set privileged: true, we should refrain from stating this in CLAUDE.md

Other than these, everything else looks fine.

Remove stale AuthProxy module name reference (github.com/huang195/auth-proxy was already renamed) and privileged mode statements (dependency being removed). Update AuthBridge/CLAUDE.md module name to match current go.mod.

huang195

Looks good. Thank you!

mrsabath changed the title ~~Feat: Add CLAUDE.md files to improve AI processing~~ feat: Add CLAUDE.md files to improve AI processing Feb 12, 2026

mrsabath marked this pull request as ready for review February 12, 2026 17:03

Copilot AI review requested due to automatic review settings February 12, 2026 17:03

Copilot started reviewing on behalf of mrsabath February 12, 2026 17:03 View session

Copilot AI reviewed Feb 12, 2026

View reviewed changes

Comment thread CLAUDE.md Outdated

Comment thread CLAUDE.md Outdated

Comment thread CLAUDE.md Outdated

Comment thread CLAUDE.md Outdated

Comment thread AuthBridge/CLAUDE.md Outdated

Comment thread AuthBridge/CLAUDE.md Outdated

Copilot AI review requested due to automatic review settings February 13, 2026 03:43

Copilot started reviewing on behalf of mrsabath February 13, 2026 03:43 View session

Copilot AI reviewed Feb 13, 2026

View reviewed changes

Comment thread CLAUDE.md Outdated

Comment thread CLAUDE.md Outdated

Copilot AI review requested due to automatic review settings February 13, 2026 13:56

Copilot started reviewing on behalf of mrsabath February 13, 2026 13:56 View session

Copilot AI reviewed Feb 13, 2026

View reviewed changes

Comment thread kagenti-webhook/CLAUDE.md

Comment thread kagenti-webhook/CLAUDE.md

Comment thread CLAUDE.md Outdated

Comment thread CLAUDE.md Outdated

Comment thread CLAUDE.md

mrsabath requested a review from Copilot February 13, 2026 15:56

Copilot started reviewing on behalf of mrsabath February 13, 2026 15:56 View session

Copilot AI reviewed Feb 13, 2026

View reviewed changes

Comment thread .github/workflows/ci.yaml

Comment thread .github/workflows/ci.yaml Outdated

Comment thread kagenti-webhook/internal/webhook/injector/pod_mutator.go Outdated

Copilot AI review requested due to automatic review settings February 13, 2026 19:13

Copilot started reviewing on behalf of mrsabath February 13, 2026 19:13 View session

mrsabath changed the title ~~feat: Add CLAUDE.md files to improve AI processing~~ feat: Add CLAUDE.md files to improve AI processing and update CI flow Feb 13, 2026

Copilot AI reviewed Feb 13, 2026

View reviewed changes

Comment thread .github/workflows/ci.yaml

Comment thread kagenti-webhook/internal/webhook/injector/pod_mutator.go Outdated

Copilot AI review requested due to automatic review settings February 18, 2026 19:48

mrsabath force-pushed the claude-code branch from e404a56 to e176e13 Compare February 18, 2026 19:48

mrsabath mentioned this pull request Feb 18, 2026

fix: CI go-version from go.mod, gate client-registration injection, improve isAlreadyInjected #122

Merged

3 tasks

Copilot started reviewing on behalf of mrsabath February 18, 2026 19:49 View session

Copilot AI reviewed Feb 18, 2026

View reviewed changes

Comment thread kagenti-webhook/CLAUDE.md

Comment thread CLAUDE.md

Comment thread CLAUDE.md Outdated

Comment thread kagenti-webhook/CLAUDE.md

mrsabath force-pushed the claude-code branch from e176e13 to 62373bd Compare February 18, 2026 19:54

feat: Add CLAUDE.md files to improve AI processing

f955649

Add CLAUDE.md files at root, AuthBridge/, and kagenti-webhook/ levels to provide structured context for AI coding agents working with this repository. Signed-off-by: Mariusz Sabath <mrsabath@gmail.com>

mrsabath force-pushed the claude-code branch from 62373bd to f955649 Compare February 18, 2026 20:18

mrsabath changed the title ~~feat: Add CLAUDE.md files to improve AI processing and update CI flow~~ feat: Add CLAUDE.md files to improve AI processing Feb 18, 2026

fix: Remove outdated gotchas from CLAUDE.md files

be33f00

Remove stale AuthProxy module name reference (github.com/huang195/auth-proxy was already renamed) and privileged mode statements (dependency being removed). Update AuthBridge/CLAUDE.md module name to match current go.mod.

huang195 reviewed Feb 18, 2026

View reviewed changes

huang195 approved these changes Feb 18, 2026

View reviewed changes

huang195 merged commit f2d4e9a into rossoctl:main Feb 18, 2026
4 of 5 checks passed

Uh oh!

Conversation

mrsabath commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of Change

How Has This Been Tested?

Checklist

Screenshots (if appropriate)

Additional Notes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

huang195 commented Feb 18, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mrsabath commented Feb 18, 2026

Uh oh!

mrsabath commented Feb 18, 2026

Uh oh!

huang195 commented Feb 18, 2026

Uh oh!

huang195 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

mrsabath commented Feb 12, 2026 •

edited

Loading