Skip to content

validate-arch-includes reports 153 warnings on current repo — needs cleanup plan #769

@gltanaka

Description

@gltanaka

Problem

Running pdd validate-arch-includes on the current repo produces 153 warnings:

  • Architecture entries referencing prompt files that don't exist on disk (e.g. candidate_models_Python.prompt, config_loader_Python.prompt)
  • Prompts that <include> modules not listed in architecture.json dependencies (e.g. llm_invoke, load_prompt_template referenced by many modules)
  • Architecture.json listing dependencies that prompts don't actually <include>

The CI workflow step (added in PR #757) emits a GitHub warning annotation on every PR. With 153 existing mismatches, this warning is permanently present and will be ignored.

Proposed fix

  1. Triage the 153 warnings — which are stale architecture entries to remove vs missing deps to add
  2. Fix the most common pattern (many modules include llm_invoke_example.py / load_prompt_template_example.py without corresponding architecture deps)
  3. Once warnings are under control, consider making the CI step blocking

Context

Discovered during E2E validation of PR #757. The mismatches are pre-existing drift, not introduced by the PR.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Status

    Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions