chore: Rename code-review.md to custom-codereview-guide.md

[xingyaoww]

Summary

Rename the repo-specific code review skill to use a unique name that won't conflict with the public code-review skill.

Changes

• Renamed .agents/skills/code-review.md → .agents/skills/custom-codereview-guide.md
• Updated name field from code-review to custom-codereview-guide

Why

With the previous naming (name: code-review), this skill completely overrode the default public code-review skill from OpenHands/extensions.

By using a unique name (custom-codereview-guide) with the same trigger (/codereview), both skills will now be activated:

1. The default public code review guidelines
2. The SDK-specific guidelines (Event Type Deprecation, etc.)

This gives reviewers the benefit of both general best practices AND repo-specific rules.

Related

• Docs PR: docs: Clarify skill naming and trigger behavior for custom skills docs#348

@xingyaoww can click here to continue refining the PR

---

Agent Server images for this PR

• GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server

Variants & Base Images

Variant	Architectures	Base Image	Docs / Tags
java	amd64, arm64	eclipse-temurin:17-jdk	Link
python	amd64, arm64	nikolaik/python-nodejs:python3.12-nodejs22	Link
golang	amd64, arm64	golang:1.21-bookworm	Link

Pull (multi-arch manifest)

# Each variant is a multi-arch manifest supporting both amd64 and arm64
docker pull ghcr.io/openhands/agent-server:fa3120d-python

Run

docker run -it --rm \
  -p 8000:8000 \
  --name agent-server-fa3120d-python \
  ghcr.io/openhands/agent-server:fa3120d-python

All tags pushed for this build

ghcr.io/openhands/agent-server:fa3120d-golang-amd64
ghcr.io/openhands/agent-server:fa3120d-golang_tag_1.21-bookworm-amd64
ghcr.io/openhands/agent-server:fa3120d-golang-arm64
ghcr.io/openhands/agent-server:fa3120d-golang_tag_1.21-bookworm-arm64
ghcr.io/openhands/agent-server:fa3120d-java-amd64
ghcr.io/openhands/agent-server:fa3120d-eclipse-temurin_tag_17-jdk-amd64
ghcr.io/openhands/agent-server:fa3120d-java-arm64
ghcr.io/openhands/agent-server:fa3120d-eclipse-temurin_tag_17-jdk-arm64
ghcr.io/openhands/agent-server:fa3120d-python-amd64
ghcr.io/openhands/agent-server:fa3120d-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-amd64
ghcr.io/openhands/agent-server:fa3120d-python-arm64
ghcr.io/openhands/agent-server:fa3120d-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-arm64
ghcr.io/openhands/agent-server:fa3120d-golang
ghcr.io/openhands/agent-server:fa3120d-java
ghcr.io/openhands/agent-server:fa3120d-python

About Multi-Architecture Support

• Each variant tag (e.g., fa3120d-python) is a multi-arch manifest supporting both amd64 and arm64
• Docker automatically pulls the correct architecture for your platform
• Individual architecture tags (e.g., fa3120d-python-amd64) are also available if needed

[all-hands-bot]

🟢 Good Taste

Simple, pragmatic solution to a real problem. Clean rename that allows both public and repo-specific code review skills to coexist.

Taste Rating: 🟢 Good taste - Elegant solution, zero complexity added

Verdict: ✅ LGTM - Solves the skill override issue with minimal changes