Skip to content

SDKS-5133: Add AI Agent Context Files to Repository#590

Open
SteinGabriel wants to merge 1 commit into
developfrom
SDKS-5133
Open

SDKS-5133: Add AI Agent Context Files to Repository#590
SteinGabriel wants to merge 1 commit into
developfrom
SDKS-5133

Conversation

@SteinGabriel

Copy link
Copy Markdown
Contributor

Summary

https://pingidentity.atlassian.net/browse/SDKS-5133

Adds a canonical AGENTS.md to the forgerock-javascript-sdk repository with full project overview, monorepo architecture,
build/test/lint commands, file-naming conventions, and SDK architectural patterns. Provider-specific files (CLAUDE.md, GEMINI.md,
.github/copilot-instructions.md) are thin redirects to AGENTS.md — single source of truth, no content drift.

Changes

Root

  • AGENTS.md (new) — canonical agent context: project overview, module structure (javascript-sdk, ping-protect, token-vault), e2e
    layout, key directory tree, internal file conventions (suffix-as-architecture table), and architectural principles
  • CLAUDE.md (new)@AGENTS.md import (Claude Code injects content at parse time)
  • GEMINI.md (new)@./AGENTS.md import directive (Gemini CLI injects content at parse time)
  • .github/copilot-instructions.md (new) — prose redirect + explicit read instruction + limitation note (Copilot has no file-include
    syntax)
  • .gitignore — adds CLAUDE.local.md to ignore list (personal instructions, local only)

Tests

No runtime behavior — these are markdown/config context files with no e2e flow to drive. Validation is structural:

  • AGENTS.md exists at root, non-empty, valid markdown headings
  • CLAUDE.md contains exactly the @AGENTS.md import line
  • GEMINI.md contains the @./AGENTS.md import directive
  • .github/copilot-instructions.md is a prose redirect to ../AGENTS.md
  • CLAUDE.local.md is git-ignored (git check-ignore CLAUDE.local.md exits 0)
  • GEMINI.md is NOT ignored (git check-ignore GEMINI.md exits nonzero)
  • pnpm lint and pnpm build pass with no regressions

How to test

1. Verify file structure

# AGENTS.md exists and is non-empty
test -s AGENTS.md && echo "ok"

# CLAUDE.md is the @AGENTS.md import
cat CLAUDE.md  # should show @AGENTS.md + note

# GEMINI.md uses @./AGENTS.md import syntax
cat GEMINI.md  # should show @./AGENTS.md + note

# copilot-instructions.md is a prose redirect
cat .github/copilot-instructions.md

# CLAUDE.local.md is git-ignored
git check-ignore CLAUDE.local.md  # exits 0

# GEMINI.md is NOT ignored
git check-ignore GEMINI.md  # exits nonzero (non-zero = not ignored)

2. Verify no build regression

pnpm lint
pnpm build

chore(context): convert CLAUDE.md to @AGENTS.md import

chore(context): add GEMINI.md redirect to AGENTS.md

chore(context): convert copilot-instructions.md to AGENTS.md redirect

chore(context): fix redirect files and gitignore for forgerock repo

chore(context): fix AGENTS.md note and token-vault dir tree
@changeset-bot

changeset-bot Bot commented Jun 26, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: fafe475

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@nx-cloud

nx-cloud Bot commented Jun 26, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit fafe475

Command Status Duration Result
nx affected -t build lint test docs e2e-ci ✅ Succeeded <1s View ↗
nx-cloud record -- nx format:check --verbose ✅ Succeeded 4s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-06-26 15:56:17 UTC

@pkg-pr-new

pkg-pr-new Bot commented Jun 26, 2026

Copy link
Copy Markdown

Open in StackBlitz

pnpm add https://pkg.pr.new/ForgeRock/forgerock-javascript-sdk/@forgerock/javascript-sdk@590
pnpm add https://pkg.pr.new/ForgeRock/forgerock-javascript-sdk/@forgerock/token-vault@590
pnpm add https://pkg.pr.new/ForgeRock/forgerock-javascript-sdk/@forgerock/ping-protect@590

commit: fafe475

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant