Skip to content

setup-target config reads ignore the legacy blender: wrapper #146

Description

@toufali

scripts/load-config.py supports a top-level blender: wrapper in a target's .blender/blender.yml (see the blender override branch), but every yq read in .github/actions/setup-target/action.yml uses flat paths only — node_version, python_version, install_command, repo_name, dismiss_unaffected, dismiss_min_confidence. So a wrapped repo config is silently ignored in the remediation job and every value falls back to its default.

Systemic (not specific to any one field) and it fails safe for dismissal (a wrapped repo gets the stricter high default), but it's a real footgun. Fix: make all setup-target reads check the wrapped path before the flat path and default, e.g. yq '.investigate.X // .blender.investigate.X // "default"'.

Surfaced by Copilot review of #145.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    Priority

    None yet

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions