infra: regen-js-client skill#48
Conversation
First team-shared skill for AI coding assistants in this repo. Documents the NSwag-generated V2 client regen against site-api-repository's swagger, plus the four traps that have silently broken the JS test suite in the past: swagger-override-filepath, baseUrl trailing-slash, jsdom multipart-Blob skip (#658052), and OpenApiTag client-split. README.md explains discovery (auto-scan of .claude/skills/*/SKILL.md frontmatter; vendor-neutral bodies so other assistants can read them) and authoring conventions. No change to the repo-level README intentionally; the skills system is scoped to its own directory in client libraries.
Replaces the reference to the (now-deleted) preview-NuGet user-story memory with a direct link to the regen-dotnet-client skill, which is the canonical home for the dotnet client publish workflow and the in-flight #659276 Phase 2 details.
Other AI assistants (Copilot Chat, Codex CLI, etc.) don't auto-discover `.claude/skills/` by frontmatter the way Claude Code does. Add the two common conventions — `AGENTS.md` at repo root and `.github/copilot-instructions.md` — each containing a short pointer to `.claude/skills/README.md`. Skill bodies stay the single source of truth; these files just bridge the auto-load gap for non-Claude tools.
|
Cross-assistant portability follow-up (also asked on the server PR — https://v-dev-tfs.laserfiche.com/DefaultCollection/Cloud/_git/site-api-repository/pullrequest/170733). By content: the By auto-load: only Claude Code scans
Just added (
Each is ~6 lines and points at Same addition on the two companion PRs:
|
…riptions The previous version named the primary workflow and reproduced its description inline. That sentence duplicated the same description already present in the SKILL.md frontmatter and the .claude/skills/README.md table — five places total once you include the workspace-level index. Make AGENTS.md and .github/copilot-instructions.md pure pointers to .claude/skills/README.md (which is the canonical per-repo index). The skills directory stays the single source of truth for skill content and applicability.
|
Follow-up: trimmed Commit: |
2 participants
Summary
First team-shared playbook for AI coding assistants in this repo. No code changes.
.claude/skills/regen-js-client/SKILL.mdcovers the NSwag regen flow for the V2 client (packages/lf-repository-api-client-v2) and the four traps that have silently broken the test suite in the past:--swagger-override-filepathis mandatory — without it,Entry.fromJSregenerates as throw-only and everyimportEntry/getEntry/listEntriescall fails at deserialization..envbaseUrl must NOT have a trailing slash — NSwag prepends/to every path; trailing slash yields//v2/…and 404s.describe.skipIf(SKIP_UNDER_JSDOM)until #658052 closes. The current 12 wrapped files are listed in the skill.[OpenApiTag]client-split — NSwag'sMultipleClientsFromFirstTagAndOperationIdmode generates one client class per unique tag; an outlier tag on the server side silently disappears the method fromentriesClient..claude/skills/README.mdexplains discovery (Claude Code auto-scans SKILL.md frontmatter; bodies are vendor-neutral) and authoring conventions.Architecture rationale:
site-api-repository/docs/design-ai-assistant-skills.md(in companion PR).Companion PRs
site-api-repository: add-v2-endpoint skill + design doclf-repository-api-client-dotnet: regen-dotnet-client skill