Confirm DOProvider.Plan canonicalization already resolved (issue #63)

[Copilot]

Issue #63 requested collapsing DOProvider.Plan to the canonical platform.ComputePlan 2-statement bridge. PR #68 already shipped this change (merged 2026-05-05) and explicitly closed #63.

Findings

• internal/provider.go already contains the canonical form:

  func (p *DOProvider) Plan(ctx context.Context, desired []interfaces.ResourceSpec, current []interfaces.ResourceState) (*interfaces.IaCPlan, error) {
      plan, err := platform.ComputePlan(ctx, p, desired, current)
      return &plan, err
  }

• All tests pass clean (-race -count=1 ./...).

No code changes required. Issue #63 and this PR can both be closed.

[intel352]

iac-codemod refactor-apply report

Mode: dry-run
Sites: 1
Errors: 0

Skipped (// wfctl:skip-iac-codemod)

• internal/provider.go:238 DOProvider.Apply skipped

Full report (90-day retention) attached as workflow artifact.

[intel352]

@copilot please review this PR now that it is ready for review.

[Copilot]

Copilot wasn't able to review any files in this pull request.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

@copilot please review this PR now that it is ready for review.

Reviewed. The PR is in correct shape:

• DOProvider.Plan (lines 187–190): canonical 2-statement platform.ComputePlan bridge — no codemod findings, as confirmed by the report above.
• DOProvider.Apply (line 238): the // wfctl:skip-iac-codemod marker is intentional and correctly reflected in the codemod report as the one skipped site with 0 errors.

No issues found. Ready to merge / close issue #63.