Fix: relocate RESEARCH_FINDINGS.md and TOOL_SCAFFOLD.md to spec/

[4eckd]

Summary

• Moves skills/RESEARCH_FINDINGS.md → spec/RESEARCH_FINDINGS.md
• Moves skills/TOOL_SCAFFOLD.md → spec/TOOL_SCAFFOLD.md

Root Cause

The Check Skill Contribution Quality CI job (in skill-contribution-check.yml) parses git diff --name-only origin/main...HEAD and extracts every entry under skills/ as a skill directory name. It then requires each "skill" to contain SKILL.md and LICENSE.txt.

RESEARCH_FINDINGS.md and TOOL_SCAFFOLD.md are project documentation files — not skills — but they were committed directly into skills/, causing the check to emit:

❌ Validation failed:
  - RESEARCH_FINDINGS.md: Missing SKILL.md
  - RESEARCH_FINDINGS.md: Missing LICENSE.txt
  - TOOL_SCAFFOLD.md: Missing SKILL.md
  - TOOL_SCAFFOLD.md: Missing LICENSE.txt

Deliverables

Deliverable	Status
Root cause identified	✅
skills/RESEARCH_FINDINGS.md relocated to spec/	✅
skills/TOOL_SCAFFOLD.md relocated to spec/	✅
No skill content modified	✅
No workflow files modified	✅

Success Metrics

• Check Skill Contribution Quality check passes on the tools-roadmap PR (PR Tools roadmap #35)
• All other checks remain green (Validate Skills, Verify Licenses, Check YAML Consistency, etc.)
• Zero skill directories affected — only file locations changed
• spec/ directory now contains all project-level reference documents

Test Plan

• Confirm Check Skill Contribution Quality passes on this PR
• Confirm PR Tools roadmap #35 (tools-roadmap → main) CI goes green after merge
• Verify spec/RESEARCH_FINDINGS.md and spec/TOOL_SCAFFOLD.md are accessible

https://claude.ai/code/session_01DwJPGbN7xTeGXDaoNMbN7A