Skip to content

♻️ (agent-files) [NO-ISSUE]: Unified agent files#1493

Merged
fAnselmi-Ledger merged 1 commit into
developfrom
feat/no-issue-unify-agent-files
May 18, 2026
Merged

♻️ (agent-files) [NO-ISSUE]: Unified agent files#1493
fAnselmi-Ledger merged 1 commit into
developfrom
feat/no-issue-unify-agent-files

Conversation

@fAnselmi-Ledger
Copy link
Copy Markdown
Contributor

📝 Description

This PR unifies all agent-related configuration (Claude Code and Cursor) under a single agent-files/ directory, eliminating the previous duplication between .claude/ and .cursor/.

Layout

  agent-files/
  ├── commands/             # Shared slash commands (create-pr, trigger-snapshot-release)
  ├── skills/               # Shared skills (backmerge, changeset, commit, release)
  ├── scripts/release/      # Shared release scripts (.cjs)
  ├── claude/               # Claude-specific entry points + settings
  │   ├── commands/         → symlinks to ../../commands/*
  │   ├── skills/           → symlinks to ../../skills/*
  │   └── settings.local.json
  └── cursor/               # Cursor-specific entry points + hooks/rules
      ├── commands/         → symlinks to ../../commands/*
      ├── rules/            → symlinks to ../../commands/*
      ├── skills/           → symlinks to ../../skills/*
      ├── scripts/
      │   ├── hooks/        # Cursor-only hook scripts
      │   └── release       → symlink to ../../scripts/release
      └── hooks.json

.claude and .cursor at the repo root are now symlinks to agent-files/claude and agent-files/cursor, so both tools keep working with zero config changes on the consumer side.

Why

  • Single source of truth for commands, skills, and release scripts — no more drift between Claude and Cursor copies
  • Adding a new command/skill = adding it once under agent-files/{commands,skills}/ and symlinking from each agent folder
  • Keeps tool-specific bits (Cursor rules/, hooks.json; Claude settings.local.json) co-located with the tool they belong to

❓ Context

  • JIRA or GitHub link: [NO-ISSUE]

✅ Checklist

  • [] Covered by automatic tests — N/A, tooling-only change with no runtime code
  • [] Changeset is provided — N/A, no published package affected
  • [] Documentation is up-to-date
  • Impact of the changes:
    • .claude/ and .cursor/ are now symlinks → repo must be cloned on a filesystem that supports symlinks (any non-Windows-default checkout, or Windows with core.symlinks=true)
    • All commands/skills/scripts previously under .cursor/ moved to agent-files/ (renames only, no content changes for .cjs and shared files)
    • Cursor and Claude Code users should see no behavioral change

Copilot AI review requested due to automatic review settings May 18, 2026 10:09
@fAnselmi-Ledger fAnselmi-Ledger requested a review from a team as a code owner May 18, 2026 10:09
@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
device-sdk-ts-sample Ready Ready Preview, Comment May 18, 2026 10:16am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
doc-device-management-kit Ignored Ignored May 18, 2026 10:16am

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Consolidates Claude Code and Cursor agent configuration under a single agent-files/ directory, eliminating duplication by sharing commands, skills, and release scripts via symlinks. .claude/ and .cursor/ at the repo root become symlinks to agent-files/{claude,cursor}/, keeping both tools working without consumer-side changes.

Changes:

  • Move shared commands, skills, and release .cjs scripts into agent-files/{commands,skills,scripts/release}
  • Add tool-specific entry points: agent-files/claude/ (settings) and agent-files/cursor/ (hooks, rules, hook scripts)
  • Update path references in skill docs and hooks from .cursor/... to agent-files/...

Reviewed changes

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

Show a summary per file
File Description
agent-files/skills/{release,backmerge,commit,changeset}/SKILL.md Add YAML frontmatter (name/description) and rewrite script paths to agent-files/scripts/release/
agent-files/scripts/release/*.cjs (preflight, discover, set-private, pin-deps, bump, changelog, cleanup, create-pr, revert-private, unpin-deps, config) New canonical location for release automation scripts (moved from .cursor/scripts/release/)
agent-files/cursor/scripts/hooks/{format,post-task-checks}.cjs New canonical location for Cursor-only hook scripts
agent-files/cursor/hooks.json Update hook command paths to agent-files/cursor/scripts/hooks/...
agent-files/commands/{create-pr,trigger-snapshot-release}.md Move shared slash-command docs into the unified directory
agent-files/claude/settings.local.json Add Claude Code permission allow-list
.cursor/commands/{commit,changeset}.md Delete stubs (replaced by skills under agent-files/skills/)

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

Comment thread agent-files/claude/settings.local.json Outdated
Comment thread agent-files/claude/settings.local.json Outdated
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
8.9% Duplication on New Code (required ≤ 3%)
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@github-actions
Copy link
Copy Markdown
Contributor

Danger Check Results

Messages

⚠️

No changeset file found. Please make sure this is intended or add a changeset file.

Danger: All checks passed successfully! 🎉

Generated by 🚫 dangerJS against 1feae6a

@fAnselmi-Ledger fAnselmi-Ledger added this pull request to the merge queue May 18, 2026
Merged via the queue into develop with commit 485ea7d May 18, 2026
24 of 25 checks passed
@fAnselmi-Ledger fAnselmi-Ledger deleted the feat/no-issue-unify-agent-files branch May 18, 2026 10:51
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.

3 participants