Skill-Namensfamilien: weekly-review -> gtd-weekly-review, dritte Person in allen Descriptions (#129)

Files changed:
- AGENTS.md
- CHANGES.md
- INSTALL.md
- README.md
- VERSION
- instructions/CONTRACT.md
- instructions/dev/stack-close/SKILL.md
- instructions/dev/stack-dev/SKILL.md
- instructions/gtd-weekly-review/SKILL.md
- instructions/setup-instance.md
- instructions/weekly-review/SKILL.md
- instructions/wiki-ingest/SKILL.md
- instructions/wiki-lint/SKILL.md
- instructions/wiki-manage/SKILL.md
- instructions/wiki-query/SKILL.md
- instructions/wiki-status/SKILL.md
- tools/chemenu/tests/test_instructions_cmd.py

tools/chemenu/tests/test_instructions_cmd.py

@@ -75,13 +75,13 @@ def test_the_real_repo_publishes_the_six_wiki_skills():
     } <= names
 
 
-def test_weekly_review_skill_names_no_provider():
+def test_gtd_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()
+    text = (config.INSTRUCTIONS_DIR / "gtd-weekly-review" / "SKILL.md").read_text(encoding="utf-8").lower()
     forbidden = [
         "super productivity",
         "azure devops",
@@ -91,7 +91,7 @@ def test_weekly_review_skill_names_no_provider():
         ".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}"
+    assert not hits, f"gtd-weekly-review/SKILL.md names a provider or its shape: {hits}"
 
 
 def test_instructions_dev_flat_file_is_discovered(layer):