refactor: remove connect strings from packager property

[phillebaba]

Description

This change moves the connect strings map to the installed charts struct. This removes potential race conditions that could be caused by multiple threads writing to the same struct property.

Related Issue

Depends on #2940 Relates to #2576

Checklist before merging

• [ ] Test, docs, adr added or updated as needed
• [ ] Contributor Guide Steps followed

[netlify[bot]]

✅ Deploy Preview for zarf-docs canceled.

Name	Link
🔨 Latest commit	cebae127a3999a8348553c8e5fbc75644644949b
🔍 Latest deploy log	https://app.netlify.com/sites/zarf-docs/deploys/66d15d2aee63d80008f7a02e

[codecov[bot]]

Codecov Report

Attention: Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/pkg/packager/deploy.go	0.00%	16 Missing :warning:
src/pkg/cluster/zarf.go	0.00%	8 Missing :warning:

Files with missing lines	Coverage Δ
src/pkg/packager/common.go	53.06% <ø> (ø)
src/pkg/packager/dev.go	0.00% <ø> (ø)
src/types/k8s.go	66.66% <ø> (ø)
src/pkg/cluster/zarf.go	19.26% <0.00%> (-0.46%)	:arrow_down:
src/pkg/packager/deploy.go	5.86% <0.00%> (-0.02%)	:arrow_down: