fix: remove obsolete contradiction test for pkm-curation

The skill no longer contains the contradictions this test was checking for
(prefer-reorganization phrasing, hardcoded cat requirement). Test file
served its purpose during the fix and is no longer needed.
This commit is contained in:
2026-06-27 23:12:25 -04:00
parent c7ed24ea51
commit 74572c1b78
@@ -1,48 +0,0 @@
# PKM Curation Skill Tests
These tests verify the operating rules in `pkm-curation/SKILL.md` are consistent and harness-agnostic.
## Test: Operating rules prioritize curation over reorganization
**Given:** `common/pkm/pkm-curation/SKILL.md`
**When:** The skill's Operating Rules section is read
**Then:** The first rule states "Prefer curation over reorganization."
**And:** The skill does NOT contain the phrase "Prefer reorganization over curation."
**Notes:**
- This ensures the skill purpose ("curate an Obsidian-style vault") aligns with its operating rules.
---
## Test: No harness-specific tool mandates in operating rules
**Given:** `common/pkm/pkm-curation/SKILL.md`
**When:** The skill's Operating Rules section is read
**Then:** The skill does NOT mandate `cat` as a required tool.
**And:** Any file-read requirement uses tool-neutral language like:
- "Read every file completely before moving, renaming, or modifying it."
- Or harness-specific guidance like "Use the `read` tool."
**Notes:**
- Avoids conflicts with different harness tool suites (pi, opencode, codex, etc.)
---
## Test: File-read requirement is clear and checkable
**Given:** `common/pkm/pkm-curation/SKILL.md`
**When:** The skill's Operating Rules section is read
**Then:** There exists a rule that explicitly requires reading files completely before modification.
**And:** The rule is expressed in a way that can be verified programmatically or by inspection.
**Notes:**
- The `read` tool or equivalent full-file read must be the expected behavior.