Remove duplicated roster policy from tests and docs table #43

New issue

Closed

opened 2026-07-23 06:12:09 +00:00 by coilyco-ops · 0 comments

coilyco-ops commented

2026-07-23 06:12:09 +00:00

Member

Problem\n\nThe compatibility-matrix slice landed correctly in #42, but its verification duplicated the complete KDL matrix in `internal/person/person_test.go`, contrary to the repository rule that tests never encode tunable configuration. The person-contract walkthrough also used a prose Markdown table, contrary to the flat-bullets voice rule.\n\n## What to change\n\n* Keep `internal/person/person.kdl` as the only executable source of the role-personality matrix.\n* Remove embedded roster value duplication from tests while preserving behavioral coverage for parsing and roster rendering.\n* Render the documented matrix as flat anchored bullets rather than a prose table.\n* Preserve the approved matrix and all shipped behavior.\n\n## Acceptance criteria\n\n* No test enumerates the canonical role-personality matrix or asserts copied embedded-person values.\n* Parser and roster behavior remain covered with purpose-built test fixtures where needed.\n* `docs/person-contract.md` contains no prose table.\n* `ward exec test` passes.\n* The commit closes this issue and lands on canonical main.

## Problem\n\nThe compatibility-matrix slice landed correctly in #42, but its verification duplicated the complete KDL matrix in `internal/person/person_test.go`, contrary to the repository rule that tests never encode tunable configuration. The person-contract walkthrough also used a prose Markdown table, contrary to the flat-bullets voice rule.\n\n## What to change\n\n* Keep `internal/person/person.kdl` as the only executable source of the role-personality matrix.\n* Remove embedded roster value duplication from tests while preserving behavioral coverage for parsing and roster rendering.\n* Render the documented matrix as flat anchored bullets rather than a prose table.\n* Preserve the approved matrix and all shipped behavior.\n\n## Acceptance criteria\n\n* No test enumerates the canonical role-personality matrix or asserts copied embedded-person values.\n* Parser and roster behavior remain covered with purpose-built test fixtures where needed.\n* `docs/person-contract.md` contains no prose table.\n* `ward exec test` passes.\n* The commit closes this issue and lands on canonical main.

coilyco-ops referenced this issue from a commit

2026-07-23 06:15:35 +00:00

fix: keep roster policy single-sourced

coilyco-ops closed this issue

2026-07-23 06:15:35 +00:00

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

coilyco-flight-deck/agent-compose#43

No description provided.

Rows
Columns