Skip to content

docs: sync the Cursor rules file with the other AI context files - #71787

Open
MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:docs/sync-cursor-ai-rules
Open

MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:docs/sync-cursor-ai-rules

Conversation

@MeAkib

@MeAkib MeAkib commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

The downloadable AI context files were updated with the Angular v22 rules (OnPush by default, model(), linkedSignal(), Signal Forms, no CommonModule, @Service), but the Cursor rules file was missed and still recommended Reactive forms and providedIn: 'root'. Add the same rules to it, along with the accessibility, NgOptimizedImage, relative path and globals rules it was also missing.

Also update the persona and example in the other files from Angular v20 to v22, and drop the providedIn: 'root' rule that contradicted the @Service rule below it, including in best-practices.md, which the other files are synced from.

The downloadable AI context files were updated with the Angular v22
rules (OnPush by default, `model()`, `linkedSignal()`, Signal Forms, no
`CommonModule`, `@Service`), but the Cursor rules file was missed and
still recommended Reactive forms and `providedIn: 'root'`. Add the same
rules to it, along with the accessibility, `NgOptimizedImage`, relative
path and globals rules it was also missing.

Also update the persona and example in the other files from Angular v20
to v22, and drop the `providedIn: 'root'` rule that contradicted the
`@Service` rule below it, including in `best-practices.md`, which the
other files are synced from.
@pullapprove
pullapprove Bot requested a review from atscott November 3, 2026 09:09
@angular-robot angular-robot Bot added the area: docs Related to the documentation label Nov 3, 2026
@ngbot ngbot Bot added this to the Backlog milestone Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs Related to the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant