Skip to content

toolkit: align skills with the repo (.NET 10, npm, singular tables, drop toString/toJSON, move lang-template) #756

Description

@genwave-radio

📚 Problem

Skills state facts the repo contradicts, so builders follow them and reviewers FAIL on them:

Claim Where Reality
.NET 9 csharp-best-practices (+references/idioms.md), aspnetcore-patterns, security-api (+references/aspnetcore.md), docker-linux-ops (+templates/Dockerfile.dotnet) .NET 10
bun test / bunx build-loop, builder, reviewer, hooks npm (admin-ui/package-lock.json)
Plural table names, plpgsql business logic postgres-dba singular tables; logic in C#
toString()/toJSON() on every class typescript-best-practices, builder, reviewer not a repo convention
lang-template in active skill inventory .claude/skills/lang-template a template, never invoked

✅ Fix

One PR: .NET 9 → 10 everywhere; bun → npm; postgres-dba singular tables and drop the plpgsql section; delete the toString/toJSON rule from the skill and both agents; move lang-template to .claude/templates/. Language skills should defer to CONTRIBUTING.md L1–L10 rather than restate a generic house style.

Agrees with Codex A05 and part of A09.

📎 Source

docs/AI_TOOLKIT_AUDIT_CLAUDE.md F7.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2Medium prioritydocumentationImprovements or additions to documentation

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions