Skip to content

Fix registry:migrate parity gaps and bugs vs hc (#207) - #233

Merged
sawka-harness merged 11 commits into
mainfrom
fix/har-migration
Sep 22, 2026
Merged

sawka-harness merged 11 commits into
mainfrom
fix/har-migration

Conversation

@BlueLabelscotch

Copy link
Copy Markdown
Collaborator

Summary

Closes #207. Brings harness execute registry:migrate to parity with hc's registry migrate, fixing all reported bugs and adding all missing features.

Bugs

  • 1. Failed artifacts don't fail the runRun() now tallies StatusFail records and joins a %d of %d artifact(s) failed to migrate error into the returned error, so any failure exits non-zero instead of printing "Migration completed successfully."
  • 2. packageFilters silently dropped — added packageFilters to Config (types/selector.go) with per-artifact-type granularity (util/selector_util.go), applied during registry/package traversal.
  • 3. Unknown artifactType not rejected at config loadvalidateConfig now calls IsKnownArtifactType, rejecting unknown/mistyped types before any network calls.

Missing features

  • 5. --result-file — writes one JSON-lines record per artifact (status, error, skip reason) to a configurable path, written before the exit-code decision so automation sees the full picture even on failure.
  • 6. --summary flag — prints condensed Success/Skipped/Failed/Total counts instead of the full per-file table.
  • 7. Missing artifact types (CRAN, RUBY, TERRAFORM) — full adapter/migratable/test support added for all three
  • 8. No skip reasonFileStat now carries a Reason field (already_exists, no_content) populated at every skip site, so it's clear why an artifact was skipped.
  • 9. Config file not documented in help — added a long help block to execute registry:migrate covering an example config, supported artifact types, packageFilters granularity, and include/exclude pattern behavior.

@sawka-harness
sawka-harness merged commit 9cc390a into main Sep 22, 2026
1 check passed
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.

[HAR] harness execute registry:migrate — parity gaps and bugs vs hc

2 participants