Skip to content

update: added skill files#165

Merged
cs-raj merged 2 commits intodevelopmentfrom
update/DX-5359
Apr 8, 2026
Merged

update: added skill files#165
cs-raj merged 2 commits intodevelopmentfrom
update/DX-5359

Conversation

@cs-raj
Copy link
Copy Markdown
Contributor

@cs-raj cs-raj commented Apr 8, 2026

DX-5359 — Contributor & agent documentation

Summary

Adds a single entry point and structured skills so humans and AI agents can navigate SDK conventions without duplicating Cursor-only rules. Short SKILL.md files link to exhaustive content under skills/*/references/.

What’s included (documentation)

  • AGENTS.md — Repo purpose, stack, commands, CI pointers, and an index of all skills.
  • skills/README.md — “When to use which skill” table; notes that deep content lives in references/.
  • skills/<area>/SKILL.md — Lean entries for:
    • dev-workflow
    • sdk-core-patterns (+ references/sdk-architecture.md, references/aspnetcore-integration.md)
    • query-building (+ references/query-patterns.md)
    • models-and-serialization (+ references/serialization-patterns.md)
    • error-handling (+ references/error-patterns.md)
    • testing (+ references/testing-patterns.md)
    • code-review (+ references/code-review-checklist.md, references/red-flags.md)
  • .cursor/rules/README.md — Points to AGENTS.md (no duplicate rule prose).
  • README.md — One line linking contributors to AGENTS.md.

How to review

  • Skim AGENTS.mdskills/README.md → any SKILL.md → follow references/ links for depth.
  • Confirm internal markdown links resolve from each SKILL.md to its references/*.md files.

@cs-raj cs-raj requested a review from a team as a code owner April 8, 2026 05:27
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 0 ✅ Passed
🔵 Low Severity 0 0 0 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

sunil-lakshman
sunil-lakshman previously approved these changes Apr 8, 2026
Copy link
Copy Markdown

@sunil-lakshman sunil-lakshman left a comment

Choose a reason for hiding this comment

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

LGTM

aman19K
aman19K previously approved these changes Apr 8, 2026
Copy link
Copy Markdown
Contributor

@reeshika-h reeshika-h left a comment

Choose a reason for hiding this comment

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

@cs-raj pls follow structure. The skill file should contain all details.

@cs-raj cs-raj dismissed stale reviews from aman19K and sunil-lakshman via 724d8fa April 8, 2026 06:44
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 0 ✅ Passed
🔵 Low Severity 0 0 0 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@cs-raj cs-raj merged commit 8287465 into development Apr 8, 2026
10 of 11 checks passed
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