Skip to content

Prefilter existing metadata reconciliation rows - #1555

Merged
ricardo-devis-agullo merged 4 commits into
masterfrom
advisor/009-prefilter-metadata-reconciliation
Aug 20, 2026
Merged

Prefilter existing metadata reconciliation rows#1555
ricardo-devis-agullo merged 4 commits into
masterfrom
advisor/009-prefilter-metadata-reconciliation

Conversation

@ricardo-devis-agullo

@ricardo-devis-agullo ricardo-devis-agullo commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • fetch existing committed metadata rows once before reconciliation inserts
  • skip matching component name/version pairs while preserving race-safe conflict handling and migration counts
  • cover prefiltering, empty inputs, prefetch failures, concurrency, and CLI reporting
  • remove completed implementation plans and their README references

Test plan

  • npm --workspace packages/oc run build
  • npx mocha --timeout 20000 packages/oc/test/unit/registry-domain-metadata-migration.js packages/oc/test/unit/cli-facade-registry-migrate-metadata.js
  • npm --workspace packages/oc run test-silent
  • npm run test-silent

Generated with Devin

ricardo-devis-agullo and others added 4 commits August 20, 2026 09:40
Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@ricardo-devis-agullo
ricardo-devis-agullo merged commit 34323f2 into master Aug 20, 2026
3 checks passed
@ricardo-devis-agullo
ricardo-devis-agullo deleted the advisor/009-prefilter-metadata-reconciliation branch August 20, 2026 07:58
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