Skip to content

feat(config): validate .gittensory.yml against schema (fail clearly pre-review) #2056

Description

@JSONbored

Add an explicit manifest validation pass that surfaces every parse warning as a single grouped, actionable message before a review runs, so invalid config values fail clearly (an acceptance criterion of #1670: 'invalid config values fail clearly before a review runs'). Deterministic; a fully-valid manifest ⇒ byte-identical (no message).

Deliverables

  • Add a helper that collects the existing parseFocusManifest warnings[] and formats them into one grouped, deduped advisory surface (reuse the warnings already produced by every parser)
  • Emit the grouped validation notice on the review path when warnings exist; suppress entirely when there are none (byte-identical)
  • Unit tests: multiple warnings grouped, no warnings = no message, ordering stable; branch coverage on the has-warnings/no-warnings arms
  • Document manifest validation behavior in the config docs

References

  • src/signals/focus-manifest.ts:305 (warnings accumulation)
  • src/signals/focus-manifest.ts:1013 (parseFocusManifest)
  • part of #1670 (epic #1680)

Part of #1670.


size: M · gittensor:feature

Metadata

Metadata

Assignees

No one assigned

    Labels

    gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.help wantedExtra attention is needed

    Projects

    Status
    Done

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions