Skip to content

SDKS-5034 Add AGENTS.md as canonical AI agent context#165

Merged
rodrigoareis merged 4 commits into
developfrom
SDKS-5034
Jun 26, 2026
Merged

SDKS-5034 Add AGENTS.md as canonical AI agent context#165
rodrigoareis merged 4 commits into
developfrom
SDKS-5034

Conversation

@rodrigoareis

@rodrigoareis rodrigoareis commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

JIRA Ticket

SDKS-5034 "[iOS] Add context to the SDK repository"

Description

Add the open-source, tool-agnostic AGENTS.md standard at the repo root and ensure CLAUDE.md with a one-line redirect file** (See AGENTS.md.) rather than a symlink. This avoids symlink portability issues (Windows core.symlinks=false) while keeping maintenance burden at zero — the redirect only needs updating if AGENTS.md is renamed. Claude Code reads both AGENTS.md and CLAUDE.md natively, so it picks up the full content from AGENTS.md directly.

Checklist:

  • I ran all unit tests and they pass
  • I added test case coverage for my changes

Adds AGENTS.md as the single source of truth for AI agent guidance in this repo. Absorbs the still-valid content from GEMINI.md (platform, toolchain, external deps, CI/CD) and removes the existing CLAUDE.md content, replacing it with a one-line redirect. Removes AGENTS.md and CLAUDE.md from .gitignore so they are shared across all contributors.

Phases:
- Phase 1: create AGENTS.md from CLAUDE.md + GEMINI.md; update .gitignore
- Phase 2: replace CLAUDE.md with `See AGENTS.md.`

Refs: SDKS-5034
@rodrigoareis rodrigoareis changed the title [docs] SDKS-5034 Add AGENTS.md as canonical AI agent context SDKS-5034 Add AGENTS.md as canonical AI agent context Jun 11, 2026

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.

LGTM

@vahancouver vahancouver left a comment

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.

left some comments

Comment thread CLAUDE.md Outdated
@@ -0,0 +1 @@
See AGENTS.md.

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.

you can double check, but apparently this wouldn't work. you need to use the file reference like this
@AGENTS.md

Comment thread AGENTS.md
PingOath ← PingTamperDetector
PingPush ← PingNetwork, PingTamperDetector
PingAuthMigration ← PingOath, PingPush
```

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.

PingDeviceClient ←PingCommons , PingNetwork is missing

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.

PingNetwork is on row #115

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.

Sorry I think I misspoke. I meant PingDeviceClient also depends on PingNetwork. that part is missing

Comment thread AGENTS.md
@@ -0,0 +1,153 @@
# AGENTS.md

This file provides guidance to AI coding agents (Claude Code, Cursor, GitHub Copilot, Gemini, etc.) when working with code in this repository. It follows the open AGENTS.md convention.

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.

Should we mention Gemini here if we are still ignoring GEMINI.md file?

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.

I think it's ok to mention it in general, even if we are not including a GEMINI.md file

@vahancouver vahancouver left a comment

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.

LGTM

Signed-off-by: Rodrigo Reis <rodrigo.reis@forgerock.com>
@rodrigoareis
rodrigoareis merged commit 41c5503 into develop Jun 26, 2026
8 of 9 checks passed
@rodrigoareis
rodrigoareis deleted the SDKS-5034 branch June 26, 2026 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants