Skip to content

Commit ba3ceca

Browse files
dmealingclaude
andcommitted
fix(agent-context): <csv> is not an allowlisted placeholder, and the grounding gate reads it as a generator name
The previous commit's opt-in table wrote `--generators <csv>`. `skill-catalog-grounding.test.ts` matches `--generators[= ]+(...)` and treats each comma-separated token as a generator name to resolve against the manifest, so `<csv>` read as a generator that does not exist — twice, one per table row. The gate was right and the prose was the outlier: every other place in these skills already writes `--generators <names>`, which is in the gate's placeholder set. So the table now writes it too, rather than `<csv>` being added to the allowlist — widening a grounding gate to admit a synonym is how it stops grounding. Corpus regenerated in the same commit, for the reason the last one was. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011cQxyVuTkdPmAduLiedNEg
1 parent 6c39053 commit ba3ceca

6 files changed

Lines changed: 12 additions & 12 deletions

File tree

  • agent-context/skills/metaobjects-codegen
  • fixtures/agent-context-conformance
    • java-kotlin-react-tanstack/expected/.claude/skills/metaobjects-codegen
    • java-react/expected/.claude/skills/metaobjects-codegen
    • python/expected/.claude/skills/metaobjects-codegen
    • ts-react-tanstack/expected/.claude/skills/metaobjects-codegen
    • ts-requirements/expected/.claude/skills/metaobjects-codegen

‎agent-context/skills/metaobjects-codegen/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,8 +552,8 @@ emitting this artifact. "Wired" is the only fact there is.
552552

553553
| Port | Where the selection is declared | An existing project upgrading gets it? |
554554
|---|---|---|
555-
| C# | `--generators <csv>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556-
| Python | `--generators <csv>` on `metaobjects gen` — required, no default | **No** — name `names` |
555+
| C# | `--generators <names>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556+
| Python | `--generators <names>` on `metaobjects gen` — required, no default | **No** — name `names` |
557557
| TypeScript | `metaobjects.config.ts` `generators: [...]` — the complete list | **No** — add `namesFile()` |
558558
| Java / Kotlin | the pom's `<generators>` — the complete list | **No** — add `SpringNamesGenerator` / `KotlinNamesGenerator` |
559559

‎fixtures/agent-context-conformance/java-kotlin-react-tanstack/expected/.claude/skills/metaobjects-codegen/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,8 +552,8 @@ emitting this artifact. "Wired" is the only fact there is.
552552

553553
| Port | Where the selection is declared | An existing project upgrading gets it? |
554554
|---|---|---|
555-
| C# | `--generators <csv>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556-
| Python | `--generators <csv>` on `metaobjects gen` — required, no default | **No** — name `names` |
555+
| C# | `--generators <names>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556+
| Python | `--generators <names>` on `metaobjects gen` — required, no default | **No** — name `names` |
557557
| TypeScript | `metaobjects.config.ts` `generators: [...]` — the complete list | **No** — add `namesFile()` |
558558
| Java / Kotlin | the pom's `<generators>` — the complete list | **No** — add `SpringNamesGenerator` / `KotlinNamesGenerator` |
559559

‎fixtures/agent-context-conformance/java-react/expected/.claude/skills/metaobjects-codegen/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,8 +552,8 @@ emitting this artifact. "Wired" is the only fact there is.
552552

553553
| Port | Where the selection is declared | An existing project upgrading gets it? |
554554
|---|---|---|
555-
| C# | `--generators <csv>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556-
| Python | `--generators <csv>` on `metaobjects gen` — required, no default | **No** — name `names` |
555+
| C# | `--generators <names>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556+
| Python | `--generators <names>` on `metaobjects gen` — required, no default | **No** — name `names` |
557557
| TypeScript | `metaobjects.config.ts` `generators: [...]` — the complete list | **No** — add `namesFile()` |
558558
| Java / Kotlin | the pom's `<generators>` — the complete list | **No** — add `SpringNamesGenerator` / `KotlinNamesGenerator` |
559559

‎fixtures/agent-context-conformance/python/expected/.claude/skills/metaobjects-codegen/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,8 +552,8 @@ emitting this artifact. "Wired" is the only fact there is.
552552

553553
| Port | Where the selection is declared | An existing project upgrading gets it? |
554554
|---|---|---|
555-
| C# | `--generators <csv>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556-
| Python | `--generators <csv>` on `metaobjects gen` — required, no default | **No** — name `names` |
555+
| C# | `--generators <names>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556+
| Python | `--generators <names>` on `metaobjects gen` — required, no default | **No** — name `names` |
557557
| TypeScript | `metaobjects.config.ts` `generators: [...]` — the complete list | **No** — add `namesFile()` |
558558
| Java / Kotlin | the pom's `<generators>` — the complete list | **No** — add `SpringNamesGenerator` / `KotlinNamesGenerator` |
559559

‎fixtures/agent-context-conformance/ts-react-tanstack/expected/.claude/skills/metaobjects-codegen/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,8 +552,8 @@ emitting this artifact. "Wired" is the only fact there is.
552552

553553
| Port | Where the selection is declared | An existing project upgrading gets it? |
554554
|---|---|---|
555-
| C# | `--generators <csv>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556-
| Python | `--generators <csv>` on `metaobjects gen` — required, no default | **No** — name `names` |
555+
| C# | `--generators <names>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556+
| Python | `--generators <names>` on `metaobjects gen` — required, no default | **No** — name `names` |
557557
| TypeScript | `metaobjects.config.ts` `generators: [...]` — the complete list | **No** — add `namesFile()` |
558558
| Java / Kotlin | the pom's `<generators>` — the complete list | **No** — add `SpringNamesGenerator` / `KotlinNamesGenerator` |
559559

‎fixtures/agent-context-conformance/ts-requirements/expected/.claude/skills/metaobjects-codegen/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -552,8 +552,8 @@ emitting this artifact. "Wired" is the only fact there is.
552552

553553
| Port | Where the selection is declared | An existing project upgrading gets it? |
554554
|---|---|---|
555-
| C# | `--generators <csv>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556-
| Python | `--generators <csv>` on `metaobjects gen` — required, no default | **No** — name `names` |
555+
| C# | `--generators <names>` on `dotnet meta gen` — required, no default | **No** — name `names` |
556+
| Python | `--generators <names>` on `metaobjects gen` — required, no default | **No** — name `names` |
557557
| TypeScript | `metaobjects.config.ts` `generators: [...]` — the complete list | **No** — add `namesFile()` |
558558
| Java / Kotlin | the pom's `<generators>` — the complete list | **No** — add `SpringNamesGenerator` / `KotlinNamesGenerator` |
559559

0 commit comments

Comments
 (0)