fix: Initialize IaC providers for dump subcommand

[ajkerrigan]

what

Call c7n-left's initialize_iac() to load the Terraform provider as part of running sinistral dump.

why

For a case where c7n-left dump -d . succeeds, sinistral dump -d . fails with:

AttributeError: 'NoneType' object has no attribute 'initialize'

Because sinistral bypasses some top-level CLI logic that c7n-left runs by default, so it handles provider registration for sinistral run while loading policies. It never loads the provider for the dump case.

testing

Tested locally, added a regression test.

docs

N/A

[tvansteenburgh]

LGTM, thanks @ajkerrigan

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

Architecture diagram

sequenceDiagram
    participant User
    participant Sinistral as Sinistral CLI (dump cmd)
    participant LeftWrapper as LeftWrapper (click)
    participant C7NLeft as c7n-left CLI (left_dump)
    participant Provider as IaC Provider Registry
    participant FS as Filesystem

    Note over Sinistral,Provider: sinistral dump invokes c7n-left dump directly,<br/>bypassing c7n-left top-level CLI setup

    User->>Sinistral: invoke dump subcommand
    Sinistral->>LeftWrapper: make_parser(ctx)
    LeftWrapper->>LeftWrapper: merge left_dump params<br/>(skip duplicates)
    LeftWrapper-->>Sinistral: parser ready

    Sinistral->>Provider: initialize_iac()
    Provider->>Provider: register Terraform provider<br/>and any IaC providers

    alt directory param missing
        Sinistral->>Sinistral: default directory to "."
    end

    Sinistral->>FS: get_provider(directory)
    FS-->>Sinistral: provider for IaC type (or None)

    alt no supported IaC files
        Sinistral-->>User: UsageError: No supported IaC files found
    else provider available
        Sinistral->>C7NLeft: invoke left_dump(ctx)
        C7NLeft->>FS: parse .tf/.json files
        FS-->>C7NLeft: resource graph
        C7NLeft-->>Sinistral: dump output (JSON)
        Sinistral-->>User: graph + variables
    end

    Note over Sinistral,FS: Regression tests invoke CLI in a subprocess<br/>to avoid import side-effect registration</br>from prior in-process c7n-left usage

Loading

_{Re-trigger cubic}