You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Deferred minor findings from the independent review of #261 (PR #290), consciously accepted at merge as follow-ups:
Multi-registry version reporting — Business Rule "version reported per registry where applicable" is unimplemented. The single .pair/.kb-version.json marker + resolveCurrentVersion operate at dataset granularity; RegistryConfig has no version field. Multi-registry projects get one aggregate version.
Git-source version check — version-resolver.ts returns current-unavailable for git sources ("not supported yet"). Git-backed KBs can never report a current version.
Downgrade migration URL — buildMigrationUrl(installed, current) produces a reversed (non-existent) docs URL when installed is newer than current (downgrade). Needs a semver-aware guard (no comparator exists yet) or fall back to the migrations index. Out of CLI KB version check: installed vs current + migration pointer #261 AC1 scope (upgrade only).
Round-trip e2e — DoD "install old fixture → drift → update → clean" is verified in isolated tests, not a single chained end-to-end test.
Deferred minor findings from the independent review of #261 (PR #290), consciously accepted at merge as follow-ups:
.pair/.kb-version.jsonmarker +resolveCurrentVersionoperate at dataset granularity;RegistryConfighas no version field. Multi-registry projects get one aggregate version.version-resolver.tsreturnscurrent-unavailablefor git sources ("not supported yet"). Git-backed KBs can never report a current version.buildMigrationUrl(installed, current)produces a reversed (non-existent) docs URL when installed is newer than current (downgrade). Needs a semver-aware guard (no comparator exists yet) or fall back to the migrations index. Out of CLI KB version check: installed vs current + migration pointer #261 AC1 scope (upgrade only).Source: PR #290 review (orchestrator, 2026-07-11).