fix: quote YAML colon in humanaios-findings-scan/SKILL.md; harden validate_skills against parse crashes

[Copilot]

tools/skills/humanaios-findings-scan/SKILL.md had a bare : sequence inside an unquoted YAML description value, causing yaml.safe_load to throw a ScannerError and abort the entire validate_skills scan. Additionally, validate_skills propagated any yaml.YAMLError uncaught, so a single bad file killed all subsequent checks.

Changes

• tools/skills/humanaios-findings-scan/SKILL.md — wrap description in a >- folded block scalar; the offending ): findings colon-space sequence is now unambiguous to the YAML parser.

• tools/validate_skills.py

  • Add FrontmatterParseError exception class.
  • extract_frontmatter now catches yaml.YAMLError and re-raises as FrontmatterParseError with a human-readable message instead of propagating the raw exception.
  • All five scanning functions (validate_all, check_deprecated_successors, check_governance_zone, check_meta_exempt, require_deprecated) wrap extract_frontmatter in try/except FrontmatterParseError. In validate_all the error is appended to the error list (preserving nonzero exit); in the --check functions the bad file is skipped and the scan continues.

# Before: yaml.YAMLError escapes and aborts the loop
def extract_frontmatter(md_path):
    ...
    return yaml.safe_load(match.group(1))  # raises on bad YAML

# After: raises a typed, labelled exception; callers decide how to handle
class FrontmatterParseError(Exception): ...

def extract_frontmatter(md_path):
    ...
    try:
        return yaml.safe_load(match.group(1))
    except yaml.YAMLError as exc:
        raise FrontmatterParseError(
            f"YAML PARSE ERROR [{md_path.parent.name}]: {exc}"
        ) from exc

Post-fix, validate_skills scans all files, reports every bad file (including the pre-existing humanaios-realtime-drift parse error now surfaced), and exits nonzero — no traceback.

[Copilot]

Pull request overview

This PR prevents validate_skills from aborting on a single malformed SKILL.md YAML frontmatter by (1) fixing an invalid YAML description in humanaios-findings-scan/SKILL.md, and (2) hardening the validator to convert PyYAML parse exceptions into a typed error that callers can handle while continuing the scan.

Changes:

• Quote/structure the humanaios-findings-scan YAML description using a folded block scalar to avoid : parsing ambiguity.
• Introduce FrontmatterParseError and wrap yaml.safe_load so parse failures become labeled, human-readable errors.
• Update all scan/check functions to catch FrontmatterParseError so validation continues (and validate_all still fails the run with aggregated errors).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
tools/validate_skills.py	Converts YAML parse crashes into typed, reportable errors and keeps scans running across bad files.
tools/skills/humanaios-findings-scan/SKILL.md	Fixes YAML frontmatter parsing by moving description to a folded block scalar.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[humanaios-ui]

@copilot resolve the merge conflicts in this pull request

[Copilot]

@copilot resolve the merge conflicts in this pull request

Resolved by merging origin/main into this branch and fixing the conflict in tools/seed_org_defaults.py. Included in commit 6359537.