docs: record or drop the fixed eleven-crate / no felis-test-* rule the skills cite #392

Closed

opened 2026-09-25 00:48:16 +09:00 by natsukium · 1 comment

natsukium commented

2026-09-25 00:48:16 +09:00

Owner

The implement-feature and test-strategy skills tell agents that crates/ is fixed at eleven crates and that no felis-test-* helper crates may be added. No doc records this rule. docs/reference/workspace.md only counts the crates today, and docs/explanation/architecture/overview.md "Workspace: the crate-boundary decision record" does not forbid adding one.

The docs consistency audit softened the skills to cite AGENTS.md "Project shape" instead of claiming a doc states the rule. Decide whether this is a real decision. If it is, record it (rationale plus a "Revisit if" trigger) in overview.md's crate-boundary record and point the skills there. If it is not, drop it from the skills.

The implement-feature and test-strategy skills tell agents that `crates/` is fixed at eleven crates and that no `felis-test-*` helper crates may be added. No doc records this rule. `docs/reference/workspace.md` only counts the crates today, and `docs/explanation/architecture/overview.md` "Workspace: the crate-boundary decision record" does not forbid adding one. The docs consistency audit softened the skills to cite AGENTS.md "Project shape" instead of claiming a doc states the rule. Decide whether this is a real decision. If it is, record it (rationale plus a "Revisit if" trigger) in overview.md's crate-boundary record and point the skills there. If it is not, drop it from the skills.

natsukium added the

type/design

label

2026-09-25 00:48:17 +09:00

natsukium referenced this issue

2026-09-25 00:48:41 +09:00

docs: fix inconsistencies found by a tree-wide docs and skills audit #395

natsukium referenced this issue from a commit

2026-09-25 00:54:55 +09:00

docs: fix inconsistencies found by a tree-wide docs and skills audit (#395)

natsukium commented

2026-09-25 11:45:19 +09:00

Author

Owner

Decision: record only the "no test-helper crates" part as a real decision in docs/explanation/architecture/overview.md's crate-boundary record (rationale: every crate is an extraction seam, and a test that reaches above its own crate root lives in the tests/ member; plus a "Revisit if" trigger). The crate count is a fact in docs/reference/workspace.md, not a rule: drop "fixed at eleven" from the skills and point them at the recorded decision.

Decision: record only the "no test-helper crates" part as a real decision in `docs/explanation/architecture/overview.md`'s crate-boundary record (rationale: every crate is an extraction seam, and a test that reaches above its own crate root lives in the `tests/` member; plus a "Revisit if" trigger). The crate count is a fact in `docs/reference/workspace.md`, not a rule: drop "fixed at eleven" from the skills and point them at the recorded decision.

natsukium referenced this issue from a pull request that will close it,

2026-09-25 11:55:50 +09:00

docs: record why the workspace has no test-helper crates #437

natsukium closed this issue

2026-09-25 12:30:05 +09:00

natsukium referenced this issue from a commit

2026-09-25 12:30:07 +09:00

docs: record why the workspace has no test-helper crates (#437)

Commenting is not possible because the repository is archived.