Fix duplicate quality criteria rendering in skill templates

[nhorton]

Summary

Fixed an issue where skill files were rendering duplicate "Quality Validation" sections when outputs had associated doc_spec quality criteria. The step-level quality criteria section now only appears when no doc_spec quality criteria exist, preventing redundant validation instructions.

Changes Made

• Template Logic: Updated both Claude and Gemini skill templates to detect when outputs have doc_spec quality criteria

  • Added namespace check to identify if any output has doc_spec.quality_criteria
  • Modified conditional rendering to suppress step-level quality validation when doc_spec criteria are present
  • Applies to skill-job-step.md.jinja (Claude) and skill-job-step.toml.jinja (Gemini)

• Skill Files: Fixed formatting issues in 24 skill definition files

  • Removed extra blank lines between constraint lists and section headers
  • Corrected malformed markdown where headers were concatenated to previous lines
  • Affected files: add_platform.*, commit.*, deepwork_jobs.*, deepwork_rules.*, manual_tests.*, and update.job skills

• Documentation: Updated CHANGELOG.md to document the fix with clear explanation of the behavior change

Implementation Details

The fix uses Jinja2 template logic to:

1. Iterate through all outputs to check for doc_spec quality criteria
2. Set a namespace flag if any output has doc_spec criteria
3. Conditionally render the "Quality Validation" section only when the flag is false

This ensures that when doc specs define quality criteria (shown under outputs), the step doesn't also render a separate quality validation section, eliminating confusion and redundancy.

https://claude.ai/code/session_01F5MXLigSjs6jiQU3rsFfHC

[nhorton]

Helpful to know the source, but I want to fix another way