Skip to content

Add agent instructions, Jacques the sous-chef persona and engineering skills - #46

Merged
stphnwlsh merged 7 commits into
mainfrom
feature/agents-and-skills
Nov 3, 2026
Merged

stphnwlsh merged 7 commits into
mainfrom
feature/agents-and-skills

Conversation

@stphnwlsh

@stphnwlsh stphnwlsh commented Nov 2, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Brings the agent setup over from Minimal API, so any coding agent works here the way we do.

  • AGENTS.md, a one-line CLAUDE.md and Jacques, our sous-chef persona (.agents/agents/jacques.md). These match Minimal API except for the GraphQL rows.
  • 38 skills in .agents/skills:
    • 25 shared skills, identical to Minimal API except for the name.
    • 6 GraphQL skills: expose-a-use-case (a query, mutation or type), api-conventions (naming, nullability, connections and error codes), dataloaders, schema-snapshot, presentation-integration-tests and nitro.
    • 7 repository and maintainer skills, which aren't included in the template package.
  • The template package ships AGENTS.md, CLAUDE.md, Jacques and the 31 engineering skills. I installed a packed template, and the generated solution builds and passes its tests.
  • CONTRIBUTING gets the "Working with an AI agent" section, and the PR template gets the skills checkbox.

A few presentation changes so the skills describe the code as it is:

  • A new Presentation.Tests.Unit project, so every template has the same four test projects. It tests our own code directly: ApplicationErrorFilter (the VALIDATION and NOT_FOUND codes, and errors it leaves alone), the cursor parsing and connection building in PageExtensions, and AuthenticationSettings, which moved over from the integration tests.
  • Each integration test class now creates its own application and database for every test and disposes of them afterwards. Before, all the tests in a class shared one static application.
  • Serilog is registered with AddSerilog. The version properties are now AssemblyVersion and AssemblyInformationalVersion.
  • The Nullable settings that Directory.Build.props already sets are gone from the presentation projects.

The docs site's file trees pick up the new test project in LeanMeanCleanApps/leanmeancleanapps.github.io#10.

Bring AGENTS.md, CLAUDE.md, Sous and the shared skills across from Minimal API, and add the GraphQL presentation skills: expose-a-use-case, api-conventions, dataloaders, schema-snapshot, presentation-integration-tests and nitro. Ship them in the template package, leaving out the repository and maintainer skills.

Line the presentation layer up with the skills: register Serilog with AddSerilog and name the version properties without spaces, give every integration test class its own application and database instead of a static one, and drop the Nullable settings that Directory.Build.props already sets.
@stphnwlsh stphnwlsh self-assigned this Nov 2, 2026
@github-actions github-actions Bot added enhancement New feature or request build labels Nov 2, 2026
@github-actions

github-actions Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Build Pipeline

✅ Ready to merge. Every stage passed.

Stage Result Details
Build 🟢 Passed Version 2026.11.03.161-pr-46 · Template package 194 KB
Test 🟢 Passed 197 passed · 0 failed · 0 skipped · Template installs, builds and passes its tests
Coverage 🟢 Passed 100% (555 of 555 lines) · Needs 95%
Static Analysis 🟢 Passed No open CodeQL alerts
Dependency Review 🟢 Passed 0 vulnerable packages · 0 licence problems
Container Scan 🟢 Passed Image starts and answers · No open Trivy alerts

Commit 3d2f37f · Full run

@stphnwlsh stphnwlsh changed the title Add agent instructions, the Sous persona and engineering skills Add agent instructions, Jacques the sous-chef persona and engineering skills Nov 3, 2026
@stphnwlsh
stphnwlsh merged commit f65486c into main Nov 3, 2026
13 checks passed
@stphnwlsh
stphnwlsh deleted the feature/agents-and-skills branch November 3, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

2 participants