fix(PL-6888): fix struct tags for applicationSource

[davidmdm]

Summary by CodeRabbit

• Bug Fixes
  • Improved application configuration serialization by omitting empty source paths, charts, and Helm settings.
  • Prevents unnecessary empty fields from appearing in generated configuration data.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2303abc9-1b0d-4b45-a1ed-c9f724cffb8a

📥 Commits

Reviewing files that changed from the base of the PR and between 75f6a30 and b08e6b8.

📒 Files selected for processing (1)

• cmd/operator/argocd/application.go

---

📝 Walkthrough

Walkthrough

ApplicationSource now omits empty path and chart values and zero-valued helm data during JSON serialization.

Changes

Application source serialization

Layer / File(s)	Summary
Omit empty source fields cmd/operator/argocd/application.go	Path and Chart use omitempty; Helm uses omitzero in their JSON tags.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• nestoca/joy-operator#15: Updates the related ApplicationSource path handling in the same file.

Suggested reviewers: silphid

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/PL-6888/fix-syncPolicy-defaults

---

_{Comment @coderabbitai help to get the list of available commands.}