fix(cloud): must login with scopes

[Dav-14]

No description provided.

[coderabbitai]

Walkthrough

The NewAppDeployClient function in pkg/clients.go now specifies required OAuth scopes ("apps:Read" and "apps:Write") for the deployment client, replacing the previous empty scope list.

Changes

Cohort / File(s)	Summary
Deployment Client Scopes pkg/clients.go	Updated NewAppDeployClient to include "apps:Read" and "apps:Write" scopes instead of empty scope list, granting necessary app access permissions for deployment operations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 With scopes expanded, the deploys take flight,
Read and write permissions shining so bright,
Apps now accessible, permissions aligned,
A little config change, but peace of mind!

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to assess whether the description relates to the changeset.	Add a description explaining why the scopes are needed and what impact this change has on the deployment authentication flow.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix(cloud): must login with scopes' directly relates to the main change which updates the required scopes for the deployment app client.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/login_with_scopes

---

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