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 •
edited
Loading

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.

@harshitha-cstk
docs: add AGENTS.md, Cursor rules, and project skills (DX-5378)
fe3bad5 
- 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

github-actions bot commented Mar 20, 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 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 •
edited
Loading

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
View reviewed changes
@aniket-shikhare-cstk
Copy link

aniket-shikhare-cstk commented Mar 20, 2026

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.

@harshitha-cstk
docs: expand sanity test env vars in testing rule and skill
5e4ebbd 
- 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

github-actions bot commented Mar 20, 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 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

Reviewers

@naman-contentstack naman-contentstack naman-contentstack approved these changes

@aman19K aman19K Awaiting requested review from aman19K

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@harshitha-cstk @aniket-shikhare-cstk @aman19K @naman-contentstack