CLI-447 Use the declarative framework for migrations

[damien-urruty-sonarsource]

---

Summary by Gitar

• New declarative framework:
  • Introduced IntegrationRegistry core logic to centrally manage and validate integration declarations.
  • Added ALL_INTEGRATIONS entry point for automated registration of existing integrations.
• Migration logic:
  • Added migrateDeclarativeIntegrations to post-update.ts to automatically reapply features and prune stale states after CLI upgrades.
  • Integrated migrateClaudeCodeHooks with the new framework to skip legacy migration when declarative tracking is present.
• Testing enhancements:
  • Added comprehensive unit tests for registry validation and integration seeding.
  • Added integration tests to verify that declarative Claude hooks are refreshed correctly during CLI upgrades.

_{This will update automatically on new commits.}

[sonarqubecloud]

Agentic Analysis: Early Results

Agentic Analysis and Context Augmentation are available on your project. Here are some issues that could have been prevented. Follow the links to learn how to put them into action.

1 issue(s) found across 1 file(s):

Rule	File	Line	Message
typescript:S3776	src/lib/post-update.ts	370	Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.

_{Analyzed by SonarQube Agentic Analysis in 7.6 s}

[hashicorp-vault-sonar-prod]

CLI-447

[sophio-japharidze-sonarsource]

LGTM! Added a few questions/comments

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
1 Accepted issue
0 New dependency risks

Measures
0 Security Hotspots
97.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[gitar-bot]

Code Review ✅ Approved 1 resolved / 1 findings

Refactors the CLI to use a declarative migration framework managed by IntegrationRegistry, addressing redundant null checks and simplifying state pruning during upgrades.

✅ 1 resolved

✅ Quality: Redundant null check on feature after filtering by same map

📄 src/lib/post-update.ts:120-124
At line 121-123 of post-update.ts, featuresById.get(installedFeature.featureId) is checked for undefined, but knownFeatures (line 111-113) was already filtered to only include features whose featureId exists in featuresById. The guard is dead code. While harmless, it adds unnecessary noise.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}