Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 1 addition & 11 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ Tests live in `src/tests/BuildSdk.IntegrationTests/PurviewSkillValidationTests.c

## Purview.Skills.Testing

`src/src/SkillsTesting/` ships a second package: the Agent Skills conformance rule set, shaped exactly like the architecture rule engine (which now lives in its own repository, [`purview-dev/architests`](https://github.com/purview-dev/architests)) and for the same reason — the rules are defined once rather than reimplemented in every repository that authors skills.
`src/src/SkillsTesting/` ships a second package: the Agent Skills conformance rule set. It is shaped for the same reason as every rule engine: the rules are defined once rather than reimplemented in every repository that authors skills.

Three layers, deliberately separated:

Expand Down Expand Up @@ -230,16 +230,6 @@ The caller owns the ref in that case — append `#<ref>` to pin it. This path ne

Tests live in `src/tests/BuildSdk.IntegrationTests/PurviewSkillResolutionTests.cs`, including the guarantee that an ordinary restore-and-build writes no lockfile and creates no agent directory.

## Purview.Architecture.Testing has moved

The architecture rule engine (`Purview.Architecture.Testing`) used to live at `src/src/ArchitectureTesting/`, with its policy schema at the repository root and its skill in `skills/`. It now lives in its own repository, [`purview-dev/architests`](https://github.com/purview-dev/architests), which owns the package, the `architecture-policy.schema.json` schema, the tests and the `purview-architests` skill.

This repository no longer packs `Purview.Architecture.Testing`, and `.config/purview-build.json` no longer validates it. Install the skill from its new home:

```bash
npx skills add purview-dev/architests
```

## Repository map

- `src/src/SkillsTesting/` — packable agent-skill rule engine (`Purview.Skills.Testing`)
Expand Down
4 changes: 2 additions & 2 deletions skills/purview-buildsdk-skill-tests/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,8 +108,8 @@ Overrides: `RepositoryRoot` (default: the SDK's repository-root marker walk),
`SkillsFolderName` (default `skills`), and `SkipRule(string)`.

The evaluator takes a `SkillCatalogue`, so a rule can be exercised against a fixture folder rather
than a real repository — the same three-layer shape as the architecture rule engine in
[`purview-dev/architests`](https://github.com/purview-dev/architests).
than a real repository — the same three-layer shape this repository uses throughout: a reader, an
evaluator over an in-memory model, and a thin test base class.

### At pack time — `ValidatePurviewSkills`

Expand Down
4 changes: 1 addition & 3 deletions src/src/SkillsTesting/SkillsTesting.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,7 @@
<!--
The package name is Purview.Skills.Testing, not Purview.BuildSdk.SkillsTesting: it is
consumed by repositories that have no relationship to this SDK beyond building with it.
Setting RootNamespace keeps IDE0130's folder check aligned with that choice. Mirrors
Purview.Architecture.Testing, which is packaged the same way in its own repository
(purview-dev/architests) for the same reason.
Setting RootNamespace keeps IDE0130's folder check aligned with that choice.
-->
<RootNamespace>Purview.Skills.Testing</RootNamespace>
<AssemblyName>Purview.Skills.Testing</AssemblyName>
Expand Down
Loading