refactor: pull verification manager out to workspace level

[adityachoudhari26]

Summary by CodeRabbit

• Refactor
  • Verification manager initialization is now injected into the release manager rather than created internally.
  • Verification manager is now directly accessible as a top-level component within the workspace.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request refactors verification manager provisioning from internal creation within the release manager to external dependency injection at the Workspace level. The verification manager is now a first-class component of the Workspace struct and explicitly passed to the release manager during construction.

Changes

Cohort / File(s)	Summary
Release Manager Constructor Update apps/workspace-engine/pkg/workspace/releasemanager/manager.go, apps/workspace-engine/pkg/workspace/releasemanager/manager_test.go	Updated New() constructor to accept an additional verificationManager parameter. Removed internal verification manager creation and now uses the injected instance. Test updated to create and wire the verification manager before passing it to the constructor.
Workspace Integration apps/workspace-engine/pkg/workspace/workspace.go	Added new verificationManager field to Workspace struct. Initialization now creates a verification manager instance and passes it to the release manager during construction. Added public VerificationManager() accessor method and updated action registration to use the standalone verification manager.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Once nested deep where none could see,
The verifier now stands proud and free!
Workspace wires it with care,
Dependencies dance through the air! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: moving the verification manager from the release manager level to the workspace level, which is the core refactoring across all three modified files.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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