Skip to content

Update AI instructions#5627

Merged
DimoHG merged 4 commits intomainfrom
feature/update-ai-instructions
Mar 16, 2026
Merged

Update AI instructions#5627
DimoHG merged 4 commits intomainfrom
feature/update-ai-instructions

Conversation

@valkirilov
Copy link
Copy Markdown
Member

@valkirilov valkirilov commented Mar 12, 2026

What

Reorganize AI agent instructions to reduce token consumption.

Previously all 7 rules were always-applied (~1,470 lines injected into every conversation):

  • now only essential rules (code-quality, git-safety) are always-on
  • domain rules activate via file globs (back-end, front-end, e2e-tsting)
  • workflow references (commits, branches, PRs) are converted to on-demand skills, instead of being rules

Note

Low Risk
Low risk: documentation/config-only changes for AI agent guidance; no runtime application code paths are affected.

Overview
Reduces always-applied AI instructions by adding YAML frontmatter (description, globs, alwaysApply: false) to domain rules (frontend, backend, testing, e2e-testing) so they activate only for matching file paths.

Adds an always-on git-safety rule, converts branches/commits/PR guidance into .ai/skills/*/SKILL.md, updates Cursor mappings under .cursor/ to point at the new rule/skill locations, and adjusts AGENTS.md references accordingly.

Written by Cursor Bugbot for commit 559ab59. This will update automatically on new commits. Configure here.

…-scoped

Change alwaysApply from true to false and add file glob patterns so these
rules only activate when working with relevant files, reducing token
consumption in unrelated conversations.

Made-with: Cursor
Add frontmatter and symlinks for two previously inactive rule files:
- git-safety (alwaysApply: true) for protected branch guardrails
- e2e-testing (glob-scoped to tests/e2e-playwright/) for Playwright standards

Made-with: Cursor
Move workflow-oriented rules (commits, branches, pull-requests) from
.ai/rules/ to .ai/skills/ so they load on demand instead of being
always injected. Symlink from .cursor/skills/ for Cursor discovery.
Update AGENTS.md references to point to new locations.

Made-with: Cursor
@valkirilov valkirilov self-assigned this Mar 12, 2026
@jit-ci
Copy link
Copy Markdown

jit-ci Bot commented Mar 12, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@valkirilov valkirilov marked this pull request as ready for review March 12, 2026 07:59
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 12, 2026

Code Coverage - Backend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 92.65% 14928/16113
🟡 Branches 75.02% 4640/6185
🟢 Functions 86.44% 2333/2699
🟢 Lines 92.45% 14261/15425

Test suite run success

3286 tests passing in 303 suites.

Report generated by 🧪jest coverage report action from 559ab59

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 12, 2026

Code Coverage - Integration Tests

Status Category Percentage Covered / Total
🟢 Statements 80.68% 17156/21264
🟡 Branches 63.51% 7813/12301
🟡 Functions 69.3% 2407/3473
🟢 Lines 80.26% 16128/20093

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Comment thread .ai/rules/testing.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.15% 22894/27868
🟡 Branches 67.38% 9626/14286
🟡 Functions 77.18% 6211/8047
🟢 Lines 82.59% 22414/27140

Test suite run success

6352 tests passing in 759 suites.

Report generated by 🧪jest coverage report action from 559ab59

Comment thread .ai/rules/backend.md
alwaysApply: true
description: NestJS backend development patterns, module structure, services, controllers, DTOs, and error handling
globs: redisinsight/api/**/*.ts
alwaysApply: false
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.

Copy link
Copy Markdown
Contributor

@KrumTy KrumTy left a comment

Choose a reason for hiding this comment

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

hope it works for augment 🤞

@DimoHG DimoHG merged commit c6111c4 into main Mar 16, 2026
34 checks passed
@DimoHG DimoHG deleted the feature/update-ai-instructions branch March 16, 2026 12:18
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.

4 participants