Skip to content

Surface DAG depth warnings via config.Resolve (currently only feature.Order emits) #12

@bilby91

Description

@bilby91

feature.Order emits WarnDeepFeatureChain (depth ≥ 16) and returns *FeatureDAGTooDeepError (depth ≥ 64). Today these only fire from inside Engine.Up's feature-fetch path — config.Resolve runs feature ordering against partial metadata (no installsAfter/dependsOn known yet), so depth warnings are never surfaced to callers using Resolve standalone for inspection.

Plan

Re-run feature.Order after fetch in Engine.Up (already happens) AND emit any warnings from there as config.Warning entries on Workspace.Config.Warnings. That preserves Resolve-without-fetch as fast/cheap and surfaces the depth signal where it actually matters (after metadata is known).

Smaller scope: just thread the warnings through. No behavior change; just visibility.

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/configConfig / devcontainer.json parsing + resolveenhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions