Skill weekly-review: wikitool review's findings become decisions (#119)
Files changed: - AGENTS.md - CHANGES.md - README.md - VERSION - instructions/weekly-review/SKILL.md - tools/chemenu/tests/test_instructions_cmd.py
This commit is contained in:
1 parent
6324024d7a
commit
44909c9e47
6 files changed
+144
-2
No files matched your search
@@ -75,6 +75,25 @@ def test_the_real_repo_publishes_the_six_wiki_skills():
|
||||
} <= names
|
||||
|
||||
|
||||
def test_weekly_review_skill_names_no_provider():
|
||||
"""#119 D25/#127 AC1: the skill is the one document meant to read
|
||||
identically in every instance, whichever task-tracker provider it runs
|
||||
against - so its body must name none of them, and none of a provider's
|
||||
file or API shape either. Reads the real repo's file, not a fixture,
|
||||
because the claim is about what ships, not about the discovery logic."""
|
||||
text = (config.INSTRUCTIONS_DIR / "weekly-review" / "SKILL.md").read_text(encoding="utf-8").lower()
|
||||
forbidden = [
|
||||
"super productivity",
|
||||
"azure devops",
|
||||
"superproductivity",
|
||||
"db.json",
|
||||
"rest api",
|
||||
".wikitool-tasks.json",
|
||||
]
|
||||
hits = [term for term in forbidden if term in text]
|
||||
assert not hits, f"weekly-review/SKILL.md names a provider or its shape: {hits}"
|
||||
|
||||
|
||||
def test_instructions_dev_flat_file_is_discovered(layer):
|
||||
dev_dir = layer / "instructions" / "dev"
|
||||
dev_dir.mkdir()
|
||||
|
||||
Reference in new issue
Block a user