Skip to content

Add Production Prompt Vault to Build Tools & Development - #374

Open
AymaneWebDEV wants to merge 3 commits into
PatrickJS:mainfrom
AymaneWebDEV:patch-1
Open

AymaneWebDEV wants to merge 3 commits into
PatrickJS:mainfrom
AymaneWebDEV:patch-1

Conversation

@AymaneWebDEV

@AymaneWebDEV AymaneWebDEV commented Sep 13, 2026

Copy link
Copy Markdown

Summary

Adds the open-source Developer & AI Prompt Vault to the Build Tools & Development section, featuring production-ready instruction sets and .cursorrules for clean architecture, SOLID refactoring, and hermetic unit testing.

Contribution Type

  • Documentation or README cleanup

Value To Cursor Users

Provides structured system prompts and deterministic .cursorrules templates for Cursor users who need strict architecture enforcement, early-return guard clauses, and automated AAA unit test harness generation.

Summary by CodeRabbit

  • Documentation
    • Updated the Production Prompt Vault entry in the Build Tools and Development section with a revised description and an in-repository link.
    • Added an optional production architecture and refactoring rule covering SOLID principles, clean modular design, separation of domain logic and side effects, type safety, immutability, and hermetic unit testing.

Adds the open-source Developer & AI Prompt Vault to the Build Tools & Development section. 

Includes production-ready instruction sets and .cursorrules enforcing clean architecture boundaries, strict SOLID refactoring rules, and hermetic unit test harness generation (AAA pattern) with boundary assertions.
@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: a2b65b06-f0b2-48f8-9629-d0a2969967a4

📥 Commits

Reviewing files that changed from the base of the PR and between 2824915 and b3e985a.

📒 Files selected for processing (1)
  • README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • README.md

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The pull request adds a Production Prompt Vault rule file with architecture, type-safety, and testing guidance. It also adds a README entry that links to the new in-repository rule.

Changes

Production Prompt Vault Resource

Layer / File(s) Summary
Add production guidelines and resource listing
rules/production-prompt-vault.mdc, README.md
The new rule defines SOLID design, modular architecture, separation of domain logic from side effects, type safety, immutability, and hermetic unit testing. The README links to the new rule and describes its guidance.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to b3e98

The new resource listing resolves to the included prompt-vault rule file; no actionable merge risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the change and its value, but it omits the required Added Or Changed Files, Quality Checklist, and Notes For Maintainers sections. It also selects Documentation or README clea… Add the missing template sections. List README.md and rules/production-prompt-vault.mdc with their purposes. Complete the Quality Checklist. Select New rules/*.mdc rule, or explain why another contribution type applies. Add source attributi…
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: adding the Production Prompt Vault to the Build Tools & Development section.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Description check

Explanation

The description explains the change and its value, but it omits the required Added Or Changed Files, Quality Checklist, and Notes For Maintainers sections. It also selects Documentation or README cleanup even though the pull request adds a new rules/*.mdc file.

Resolution

Add the missing template sections. List README.md and rules/production-prompt-vault.mdc with their purposes. Complete the Quality Checklist. Select New rules/*.mdc rule, or explain why another contribution type applies. Add source attribution or maintainer notes if needed.

  • Fix all pre-merge checks with AI
✨ 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.

Added guidelines for production-ready code architecture and testing.
Removed duplicate entry for 'Production Prompt Vault' in the README.
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