Skip to content

chore(agent): add AGENTS.md and CLAUDE.md (symlink)#6

Merged
feloy merged 2 commits intoopenkaiden:mainfrom
feloy:agents-md
Feb 27, 2026
Merged

chore(agent): add AGENTS.md and CLAUDE.md (symlink)#6
feloy merged 2 commits intoopenkaiden:mainfrom
feloy:agents-md

Conversation

@feloy
Copy link
Copy Markdown
Contributor

@feloy feloy commented Feb 27, 2026

Add an AGENTS.md file (generated with /init skill and a few changes), and a CLAUDE.md symlinking to it

Co-Authored-By: Claude Code (Claude Sonnet 4.5) <noreply@anthropic.com>

Signed-off-by: Philippe Martin <phmartin@redhat.com>
@feloy feloy mentioned this pull request Feb 27, 2026
4 tasks
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 27, 2026

📝 Walkthrough

Walkthrough

Adds new top-level documentation files describing AI agent usage and guidance for the repository, including project overview, CLI/skills architecture, command/skill creation, testing, and conventions.

Changes

Cohort / File(s) Summary
Agent documentation
AGENTS.md, CLAUDE.md
New documentation files. AGENTS.md provides usage guidance for AI agents: project overview, build/test commands, CLI (Cobra) structure, skills system, how to add commands/skills/tests, copyright/header and dependency notes. CLAUDE.md added as a short auxiliary doc entry.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main changes: adding AGENTS.md and CLAUDE.md (symlink), matching the actual file additions in the changeset.
Description check ✅ Passed The description is directly related to the changeset, explaining that AGENTS.md was added and CLAUDE.md is a symlink to it.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
AGENTS.md (1)

72-72: Avoid hardcoding the calendar year in evergreen guidance.

Line 72 (“The current year is 2026.”) will age quickly; prefer neutral wording or referencing a maintained source.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@AGENTS.md` at line 72, Replace the hardcoded sentence "The current year is
2026." in AGENTS.md with neutral wording or a reference to a maintained source
(e.g., "the current year" or "see the /copyright-headers skill for the current
year") so the guidance stays evergreen; locate the exact line containing "The
current year is 2026." and update it to avoid embedding a calendar year.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@AGENTS.md`:
- Line 72: Replace the hardcoded sentence "The current year is 2026." in
AGENTS.md with neutral wording or a reference to a maintained source (e.g., "the
current year" or "see the /copyright-headers skill for the current year") so the
guidance stays evergreen; locate the exact line containing "The current year is
2026." and update it to avoid embedding a calendar year.

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 46e332e and 198efbd.

📒 Files selected for processing (2)
  • AGENTS.md
  • CLAUDE.md

@feloy feloy requested review from benoitf and jeffmaury February 27, 2026 13:50
AGENTS.md Outdated
## Dependencies

- Cobra (github.com/spf13/cobra): CLI framework
- Go 1.25.7
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think in other files, we try to keep at the end a + to says 'this version and the upcoming one' so we don't stick to a given one

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

side note: why for example not use go 1.26.0 (the latest one)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

side note: why for example not use go 1.26.0 (the latest one)

I'm happy not using a too much recent version, I'll update ASAP

# Check code coverage
go test -cover ./...
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something about formatter / go fmt to format ?

Co-Authored-By: Claude Code (Claude Sonnet 4.5) <noreply@anthropic.com>

Signed-off-by: Philippe Martin <phmartin@redhat.com>
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

♻️ Duplicate comments (1)
AGENTS.md (1)

92-94: ⚠️ Potential issue | 🟡 Minor

Keep Go version guidance synchronized with go.mod/toolchain.

Go 1.25+ in docs can drift from the enforced project version. Consider documenting the exact baseline from go.mod (and optionally the tested max range) to avoid contributor mismatch.

#!/bin/bash
# Verify documented Go version vs repository source of truth.
# Expected: AGENTS.md version guidance should match go.mod/toolchain directive.

set -euo pipefail

echo "== go.mod locations =="
fd '^go\.mod$' -a

echo
echo "== go.mod headers (first 40 lines) =="
fd '^go\.mod$' -x sh -c 'echo "--- $1 ---"; sed -n "1,40p" "$1"' sh {}

echo
echo "== Version mentions in docs =="
rg -n 'Go\s+[0-9]+\.[0-9]+(\+)?|toolchain\s+go[0-9]+\.[0-9]+' AGENTS.md README.md
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@AGENTS.md` around lines 92 - 94, AGENTS.md currently lists "Go 1.25+" which
can drift from the repository source of truth; update the Go version guidance in
AGENTS.md to match the exact module/toolchain version declared in go.mod (and/or
the toolchain directive in go.work if present) — replace the "Go 1.25+" text
with the exact version string found in go.mod (e.g., the module's go directive)
and optionally add a tested range note (e.g., "tested with Go X.Y and above") to
keep docs and the project toolchain in sync.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In `@AGENTS.md`:
- Around line 92-94: AGENTS.md currently lists "Go 1.25+" which can drift from
the repository source of truth; update the Go version guidance in AGENTS.md to
match the exact module/toolchain version declared in go.mod (and/or the
toolchain directive in go.work if present) — replace the "Go 1.25+" text with
the exact version string found in go.mod (e.g., the module's go directive) and
optionally add a tested range note (e.g., "tested with Go X.Y and above") to
keep docs and the project toolchain in sync.

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 198efbd and 19f384b.

📒 Files selected for processing (1)
  • AGENTS.md

@feloy feloy requested a review from benoitf February 27, 2026 14:15
@feloy feloy merged commit 3a58134 into openkaiden:main Feb 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants