Skip to content

chore: consolidate cursor rules to AGENTS.md - #1152

Merged
jonathannorris merged 2 commits into
mainfrom
chore-consolidate-rules-to-agents-md
Oct 17, 2025
Merged

chore: consolidate cursor rules to AGENTS.md#1152
jonathannorris merged 2 commits into
mainfrom
chore-consolidate-rules-to-agents-md

Conversation

@jonathannorris

Copy link
Copy Markdown
Member

Summary

Consolidates .cursor/rules/*.mdc files into a single AGENTS.md at repository root.

Why

  • AGENTS.md supported by both Cursor and GitHub Copilot
  • GitHub Copilot can use for PR reviews automatically
  • Single source of truth, simpler maintenance
  • Net reduction: 68 lines vs 148 lines

@jonathannorris
jonathannorris requested a review from a team as a code owner October 17, 2025 15:38
Copilot AI review requested due to automatic review settings October 17, 2025 15:38
@vercel

vercel Bot commented Oct 17, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
js-sdks-web-elements Ready Ready Preview Comment Oct 17, 2025 3:40pm
js-sdks-with-provider Ready Ready Preview Comment Oct 17, 2025 3:40pm
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
js-sdks-next-js-page-router Ignored Ignored Oct 17, 2025 3:40pm

Copilot AI 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.

Pull Request Overview

Consolidates scattered .cursor/rules/*.mdc files into a single AGENTS.md file at the repository root to improve maintainability and enable compatibility with both Cursor and GitHub Copilot.

  • Removes 8 separate .mdc rule files from .cursor/rules/ directory
  • Creates unified AGENTS.md with consolidated coding standards and conventions
  • Reduces overall documentation size from 148 to 68 lines

Reviewed Changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
AGENTS.md New consolidated file containing all coding standards, conventions, and project guidelines
.cursor/rules/testingMocks.mdc Removed - content moved to AGENTS.md testing section
.cursor/rules/projectStructure.mdc Removed - content moved to AGENTS.md project structure section
.cursor/rules/namingConventions.mdc Removed - content moved to AGENTS.md naming conventions section
.cursor/rules/internalLibraryImports.mdc Removed - content moved to AGENTS.md imports section
.cursor/rules/gitCommitConventions.mdc Removed - content moved to AGENTS.md commit conventions section
.cursor/rules/formatting.mdc Removed - content moved to AGENTS.md formatting section
.cursor/rules/configTooling.mdc Removed - content moved to AGENTS.md configuration section

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread AGENTS.md

## Testing and Mocks

- Source files in `lib/`, `sdk/`, should have corresponding test files.

Copilot AI Oct 17, 2025

Copy link

Choose a reason for hiding this comment

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

Missing word 'and' between lib/ and sdk/ for proper grammar: 'Source files in lib/ and sdk/ should have corresponding test files.'

Suggested change
- Source files in `lib/`, `sdk/`, should have corresponding test files.
- Source files in `lib/` and `sdk/` should have corresponding test files.

Copilot uses AI. Check for mistakes.
@jonathannorris
jonathannorris merged commit 25ea1da into main Oct 17, 2025
14 checks passed
@jonathannorris
jonathannorris deleted the chore-consolidate-rules-to-agents-md branch October 17, 2025 16:14
jonathannorris added a commit that referenced this pull request Jul 2, 2026
- js-yaml 3.14.1 -> 3.15.0 (medium, alerts #1175, #1176)
- js-yaml 4.1.0 -> 4.3.0 (medium, alerts #1171, #1172, #1173, #1174)
- webpack-dev-server -> 5.2.5 (medium, alerts #1167, #1168, #1169)
- http-proxy-middleware 2.0.9 -> 2.0.10 (medium, alerts #1164, #1165, #1166)
- @babel/core -> 7.29.7 (low, alerts #1139, #1140, #1163)
- undici -> 6.27.0 (low/medium/high, alerts #1154-#1161)
- tar -> 7.5.16 (medium, alerts #1149-#1152)
- ws 6.x -> 6.2.4, ws 7.x -> 7.5.11, ws 8.x -> 8.21.0 (high, alerts #1141, #1142, #1145)
- form-data 4.0.5 -> 4.0.6 (high, alerts #1143, #1144)
- protobufjs -> 7.6.4 (high/medium, alerts #1135, #1136)
- protobufjs-cli 1.2.2 -> 1.3.3 (high, alert #1132)
- launch-editor -> 2.14.1 (medium, alerts #1134, #1137, #1138)
- markdown-it 12.3.2 -> 14.3.0 (medium, alert #1133)
- joi 17.6.0 -> 17.13.4 (medium, alert #1125)
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