Skip to content

feat: add Full-Stack AI Engineering Playbook rules#301

Open
hashem-cruncher wants to merge 2 commits into
PatrickJS:mainfrom
hashem-cruncher:main
Open

feat: add Full-Stack AI Engineering Playbook rules#301
hashem-cruncher wants to merge 2 commits into
PatrickJS:mainfrom
hashem-cruncher:main

Conversation

@hashem-cruncher

@hashem-cruncher hashem-cruncher commented Jun 2, 2026

Copy link
Copy Markdown

Summary

Added a comprehensive fullstack-ai-engineering-playbook.mdc rule. This rule provides enterprise-grade AI behavior constraints, focusing on token optimization, strict clean architecture boundaries (Frontend/Backend), and anti-hallucination directives. It also updates the README to include this new rule under the "Build Tools and Development" section.

Contribution Type

  • New Cursor rule file or rules folder
  • Update/fix to an existing rule
  • New rules/*.mdc rule
  • Documentation or README cleanup

Value To Cursor Users

This rule helps developers prevent common LLM mistakes such as over-engineering, modifying out-of-scope files, and breaking architectural boundaries. By enforcing strict constraints and a "Do Not Guess" policy, it significantly reduces token waste and maintains a clean, scalable codebase for full-stack applications.

Added Or Changed Files

  • rules/fullstack-ai-engineering-playbook.mdc
  • README.md

Quality Checklist

  • The contribution includes original rule content, or clearly credits the source.
  • New rule files use a descriptive kebab-case filename, such as react-typescript.mdc.
  • New rules/*.mdc files include frontmatter with a non-empty description, relevant globs, and alwaysApply: false unless the rule is universal.
  • README links use canonical GitHub URLs for repo files and point to the correct category.
  • The text is neutral and useful, not sales copy.
  • This is not a standalone external tool, product, directory, marketplace, or service listing.
  • No secrets, tokens, affiliate links, tracking links, or unrelated product claims are included.
  • I checked for duplicate or near-duplicate existing entries.

Notes For Maintainers

This rule is a condensed, highly practical extraction from a larger open-source AI engineering playbook I maintain. It has been formatted strictly according to your .mdc guidelines and frontmatter requirements. Thank you for maintaining this awesome list!

Summary by CodeRabbit

  • Documentation
    • Added a new AI engineering playbook providing comprehensive guidelines for enterprise-grade full-stack development, covering frontend, backend, and security best practices.

Added enterprise-grade AI behavior rules and full-stack guidelines to the playbook, focusing on architectural standards, token optimization, and security practices.
- Added a new 'Project Initialization' section detailing distinct paths for brand new projects (Scenario A) and legacy/existing codebases (Scenario B).
- Restructured the 'Daily Operating Workflow' to clearly separate feature development, debugging, and code review phases.
- Highlighted token optimization strategies to minimize API costs and prevent context bloat.
- Organized the 'System Index' with direct links to all playbook files for faster navigation inside the IDE.
- Improved overall readability, visual hierarchy, and formatting.
@coderabbitai

coderabbitai Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a new Cursor rules file that documents enterprise-grade AI behavior guidelines, including core directives, frontend/backend architecture boundaries, and security practices. The playbook is registered in README.md with a link and description.

Changes

Full-Stack AI Engineering Playbook

Layer / File(s) Summary
Add Full-Stack AI Engineering Playbook rule
rules/fullstack-ai-engineering-playbook.mdc, README.md
New rule file establishes Cursor rule metadata (globs: *, alwaysApply: false) and documents AI directives for core behavior (no guessing, scope containment, token economy), React/Next.js frontend boundaries, FastAPI/Node.js backend architecture, and security practices (no hardcoded secrets, malicious client data handling, injection prevention). README entry catalogs the new playbook under "Build Tools and Development".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • PatrickJS

Poem

🐰 A playbook hops into the fold,
With rules of gold for AI bold,
Frontend bounds and backend care,
Security woven everywhere!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding a full-stack AI engineering playbook rule file.
Description check ✅ Passed The description follows the template well, with summary, contribution type, value, files, checklist, and maintainer notes filled in.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@rules/fullstack-ai-engineering-playbook.mdc`:
- Line 3: The rule currently uses a non-recursive pattern "globs: *" which won't
match nested files; update the globs entry in
rules/fullstack-ai-engineering-playbook.mdc to use a recursive pattern like
"globs: **/*" so it matches files in subdirectories—locate the "globs" key in
that file and replace the "*" value with "**/*" to align with the other rules.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ce106d39-514e-4b96-8876-a23741a9a71a

📥 Commits

Reviewing files that changed from the base of the PR and between b044f95 and 1bb4a3f.

📒 Files selected for processing (2)
  • README.md
  • rules/fullstack-ai-engineering-playbook.mdc

@@ -0,0 +1,33 @@
---
description: Enterprise-grade AI behavior rules, token optimization, and full-stack guidelines. Extracts from the Cursor AI Engineering Playbook.
globs: *

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.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Read-only verification: inspect glob conventions used by existing rules
fd -e mdc . rules | xargs rg -n '^globs:\s*' 
echo "---- potential non-recursive catch-all patterns ----"
fd -e mdc . rules | xargs rg -n '^globs:\s*\*$'

Repository: PatrickJS/awesome-cursorrules

Length of output: 19331


Fix globs to ensure the rule applies to nested filesrules/fullstack-ai-engineering-playbook.mdc uniquely uses globs: * while the repo’s other rules use recursive patterns like globs: **/*; update it to match nested paths.

Proposed fix
-globs: *
+globs: **/*
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
globs: *
globs: **/*
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@rules/fullstack-ai-engineering-playbook.mdc` at line 3, The rule currently
uses a non-recursive pattern "globs: *" which won't match nested files; update
the globs entry in rules/fullstack-ai-engineering-playbook.mdc to use a
recursive pattern like "globs: **/*" so it matches files in
subdirectories—locate the "globs" key in that file and replace the "*" value
with "**/*" to align with the other rules.

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.

1 participant