Skip to content

update: added skill files#54

Open
cs-raj wants to merge 1 commit intodevelopmentfrom
update/DX-5369
Open

update: added skill files#54
cs-raj wants to merge 1 commit intodevelopmentfrom
update/DX-5369

Conversation

@cs-raj
Copy link
Copy Markdown
Contributor

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

Summary

Adds a repo-wide agent and contributor guide (AGENTS.md), a skills/ tree with YAML-frontmatter SKILL.md files, and a Cursor pointer (.cursor/rules/README.md) so editor-specific config does not duplicate canonical docs.

Documentation is repo-specific (dotnet CLI, xUnit, Coverlet, GitHub Actions, branch rules, NuGet) and includes explicit inventories (interfaces, enums, models, mocks, JSON fixtures) instead of vague shorthand.

The Contentstack Utils API skill includes code flow sections (RTE JsonToHtml, GQL edges, RenderContent, variant metadata) with Mermaid diagrams and pointers into Utils.cs, HtmlDocumentExtension, and Metadata.

Changes

Entry and index

  • AGENTS.md — Project purpose, stack, commands (including quick dotnet test Contentstack.Utils.sln), CI workflow table, skills index, Cursor note.
  • skills/README.md — When to use each skill.

Skills (skills/<name>/SKILL.md)

  • dev-workflow — Branch policy (stagingmaster), build/pack, test scripts, workflows, CODEOWNERS, Talisman, publish secrets wording.
  • testing — Test project packages, test-class map, Resources and mocks (explicit files), quick local test, coverage and CI alignment.
  • code-review — PR checklist and security notes.
  • contentstack-utils — Package and namespaces, models and interfaces, rendering hooks, variants JSON path, Code flows and diagrams, authoritative-source line.
  • csharp-style — Folder and file-level inventory for library layout.

Cursor

  • .cursor/rules/README.md — Points to AGENTS.md only.

Motivation

  • Single source of truth for humans and AI agents: commands, CI, and API boundaries.
  • Discoverability of how rendering works (flows and code pointers) without pasting full sources into markdown.

Notes

  • Documentation-only changes (plus .cursor pointer); no application code changes in this workstream.
  • Exclude this helper file from the PR if you prefer not to commit a PR template artifact; otherwise keep it for future releases.

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.

@cs-raj framework skill details are not added under skills folder.
Here is the reference: Link


## References

- [`skills/dev-workflow/SKILL.md`](../dev-workflow/SKILL.md) — CI and branch rules.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@cs-raj, Why are we referring to these skill files if they are already mentioned in agents.md?


## References

- [`skills/csharp-style/SKILL.md`](../csharp-style/SKILL.md) — layout and naming in this repo.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@cs-raj, Why are we referring to these skill files if they are already mentioned in agents.md?


## References

- [`skills/contentstack-utils/SKILL.md`](../contentstack-utils/SKILL.md) — what belongs in the public API.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@cs-raj, Why are we referring to these skill files if they are already mentioned in agents.md?


## References

- [`skills/testing/SKILL.md`](../testing/SKILL.md) — test project and coverage.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@cs-raj, Why are we referring to these skill files if they are already mentioned in agents.md?
Can't we move the common references in the Agents.md file to avoid redundant declarations?

Comment on lines +60 to +62
## Using Cursor (optional)

If you use **Cursor**, [`.cursor/rules/README.md`](.cursor/rules/README.md) only points to **[`AGENTS.md`](AGENTS.md)** at the repo root. All conventions live in **`skills/*/SKILL.md`**—same docs as for any other tool.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@cs-raj This should not be included in Agents.md. We can have this in cursor rules.

Comment on lines +5 to +13
## When to use which skill

| Skill folder | Use when |
|--------------|----------|
| [`dev-workflow`](dev-workflow/SKILL.md) | Branch rules, local build/pack, test scripts, CI workflows, versioning, security scans, CODEOWNERS, Talisman. |
| [`testing`](testing/SKILL.md) | Writing or running xUnit tests, fixtures, mocks, coverage, matching CI behavior. |
| [`code-review`](code-review/SKILL.md) | Preparing or reviewing PRs, release/docs expectations, dependency risk. |
| [`contentstack-utils`](contentstack-utils/SKILL.md) | Public API (`Utils`, GQL helpers, variants, `Options`), NuGet package boundaries, converters, dependencies. |
| [`csharp-style`](csharp-style/SKILL.md) | Where to put new types, namespaces, naming consistency, multi-target framework constraints. |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@cs-raj This is redundant with Agents.md block

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.

5 participants