Skip to content

docs(DX-5378): AGENTS.md, Cursor rules, and project skills#531

Merged
harshitha-cstk merged 2 commits intodevelopmentfrom
feat/dx-5378-skills-rules
Mar 20, 2026
Merged

docs(DX-5378): AGENTS.md, Cursor rules, and project skills#531
harshitha-cstk merged 2 commits intodevelopmentfrom
feat/dx-5378-skills-rules

Conversation

@harshitha-cstk
Copy link
Contributor

@harshitha-cstk harshitha-cstk commented Mar 20, 2026

Summary

Adds developer-agent documentation and Cursor context for @contentstack/management:

  • AGENTS.md — repo overview, stack, commands, test/sanity env notes
  • .cursor/rules/ — dev-workflow, javascript, contentstack-javascript-cma, testing, code-review (always apply)
  • skills/ — code-review, testing, contentstack-javascript-cma, framework

No runtime code changes.

- AGENTS.md entry point for AI/automation
- .cursor/rules: dev-workflow, javascript, CMA SDK, testing, code-review
- skills: code-review, testing, contentstack-javascript-cma, framework

Made-with: Cursor
@harshitha-cstk harshitha-cstk requested a review from a team as a code owner March 20, 2026 07:51
@github-actions
Copy link

🔒 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 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ 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

@github-actions
Copy link

github-actions bot commented Mar 20, 2026

Coverage report for commit: 5e4ebbd
File: coverage/clover.xml

Cover ┌─────────────────────────┐ Freq.
   0% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  10% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  20% │ ██░░░░░░░░░░░░░░░░░░░░░ │  2.2%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  70% │ ██████░░░░░░░░░░░░░░░░░ │  8.7%
  80% │ ███████████████████████ │ 39.1%
  90% │ █████████████░░░░░░░░░░ │ 21.7%
 100% │ █████████████████░░░░░░ │ 28.3%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution 
Summary - Lines: 82.31% | Methods: 95.93% | Branches: 65.34%
FilesLinesMethodsBranches
lib
   contentstack.js100.00%100.00%100.00%
   contentstackClient.js83.02%92.86%69.49%
   contentstackCollection.js94.12%100.00%86.96%
   entity.js76.22%100.00%61.34%
lib/core
   Util.js77.68%94.44%61.65%
   concurrency-queue.js71.84%72.22%62.02%
   contentstackError.js100.00%100.00%100.00%
   contentstackHTTPClient.js85.71%92.31%80.00%
   errorMessages.js100.00%100.00%71.43%
   oauthHandler.js90.85%100.00%67.14%
   pkceStorage.js100.00%100.00%94.12%
lib/organization
   index.js72.73%100.00%48.68%
lib/organization/teams
   index.js89.47%100.00%61.11%
lib/organization/teams/stackRoleMappings
   index.js78.57%100.00%47.50%
lib/organization/teams/teamUsers
   index.js100.00%100.00%87.50%
lib/query
   index.js73.91%100.00%56.25%
lib/stack/asset/folders
   index.js100.00%100.00%94.44%
lib/stack/asset
   index.js86.42%100.00%72.34%
lib/stack/auditlog
   index.js91.67%100.00%69.70%
lib/stack/branch
   compare.js68.18%100.00%56.67%
   index.js91.11%100.00%68.18%
   mergeQueue.js83.33%100.00%57.89%
lib/stack/branchAlias
   index.js79.31%100.00%58.54%
lib/stack/contentType/entry
   index.js83.04%100.00%57.43%
lib/stack/contentType/entry/variants
   index.js79.31%100.00%58.97%
lib/stack/contentType
   index.js84.85%100.00%67.57%
lib/stack/deliveryToken
   index.js95.24%80.00%85.00%
lib/stack/deliveryToken/previewToken
   index.js21.43%25.00%7.14%
lib/stack/environment
   index.js100.00%100.00%94.74%
lib/stack/extension
   index.js93.88%100.00%80.95%
lib/stack/globalField
   index.js91.67%100.00%80.77%
lib/stack
   index.js80.66%92.86%67.74%
lib/stack/label
   index.js100.00%100.00%93.33%
lib/stack/locale
   index.js100.00%100.00%94.74%
lib/stack/managementToken
   index.js100.00%100.00%89.47%
lib/stack/release
   index.js80.00%100.00%58.33%
lib/stack/roles
   index.js100.00%100.00%94.12%
lib/stack/taxonomy
   index.js79.01%100.00%52.94%
lib/stack/taxonomy/terms
   index.js78.57%100.00%56.41%
lib/stack/variantGroup
   index.js81.58%100.00%57.14%
lib/stack/variantGroup/variants
   index.js77.50%100.00%52.00%
lib/stack/variants
   index.js76.32%100.00%50.00%
lib/stack/webhook
   index.js84.48%100.00%63.64%
lib/stack/workflow
   index.js83.64%100.00%66.10%
lib/stack/workflow/publishRules
   index.js100.00%100.00%94.74%
lib/user
   index.js91.43%100.00%73.08%

🤖 comment via lucassabreu/comment-coverage-clover

aman19K
aman19K previously approved these changes Mar 20, 2026
@aniket-shikhare-cstk
Copy link

Hi @harshitha-cstk

skills/testing/SKILL.md — Sanity test env variables

The Sanity tests section currently lists only EMAIL, PASSWORD, HOST, and ORGANIZATION. Several other env vars are needed for full coverage; without them, the related tests are skipped.

Suggestion: Expand the env vars section along these lines:

  • Env: Variables are documented in test/sanity-check/utility/testSetup.js.
    • Required (core): EMAIL, PASSWORD, HOST, ORGANIZATION — without these, setup fails.
    • Required for specific suites (tests skip when missing):
      • OAuth: CLIENT_ID, APP_ID, REDIRECT_URI
      • 2FA: TFA_EMAIL, TFA_PASSWORD
      • MFA: MFA_SECRET
      • Team / stack share: MEMBER_EMAIL
      • DAM 2.0: DAM_2_0_ENABLED=true
    • Config: PERSONALIZE_HOST, DELETE_DYNAMIC_RESOURCES (have defaults).
    • For custom regions, set HOST to the target API host.

- Document core vs suite-specific variables (OAuth, 2FA, MFA, MEMBER_EMAIL, DAM 2.0)
- Optional PERSONALIZE_HOST, DELETE_DYNAMIC_RESOURCES; runtime vars from setup

Made-with: Cursor
@github-actions
Copy link

🔒 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 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ 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

@harshitha-cstk harshitha-cstk requested a review from aman19K March 20, 2026 11:57
@harshitha-cstk harshitha-cstk merged commit d114865 into development Mar 20, 2026
13 checks passed
@harshitha-cstk harshitha-cstk deleted the feat/dx-5378-skills-rules branch March 20, 2026 11:58
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