Skip to content

feat: support Hover state import#19

Merged
intel352 merged 1 commit into
mainfrom
feat/import-metadata
May 24, 2026
Merged

feat: support Hover state import#19
intel352 merged 1 commit into
mainfrom
feat/import-metadata

Conversation

@intel352
Copy link
Copy Markdown
Contributor

@intel352 intel352 commented May 24, 2026

Summary

  • implement read-only provider import for Hover infra.dns and infra.dns_delegation resources
  • return typed gRPC Import state instead of an empty response
  • document wfctl import usage for existing Hover domains

Verification

  • GOWORK=off go test ./internal -run TestHoverIaCServer_Import_ReturnsResourceState -count=1
  • GOWORK=off go test ./... -race -count=1
  • GOWORK=off go build ./...
  • git diff --check

Adversarial review

  • Checked unsupported resource types, nil driver output, import-before-initialize behavior, CLI docs accuracy, and Hover unofficial API overpromise risk.
  • Fixed docs to use current wfctl import flags (--name, optional --id) instead of provider-direct flags.

@intel352 intel352 merged commit d179d11 into main May 24, 2026
4 checks passed
@intel352 intel352 deleted the feat/import-metadata branch May 24, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant