chore: remove unused QueryForDebug

[lionello]

Description

new agent uses existing QueryLogs function so QueryForDebug is no longer used.

Linked Issues

Checklist

• I have performed a self-review of my code
• I have added appropriate tests
• I have updated the Defang CLI docs and/or README to reflect my changes, if necessary

Summary by CodeRabbit

• Chores
  • Removed debug query functionality from CLI provider implementations.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR systematically removes the QueryForDebug method and related debugging functionality from the Provider interface and all its implementations (AWS BYOC, DigitalOcean BYOC, GCP BYOC, and Playground), eliminating CloudWatch and logging query capabilities for AI debugging across all provider backends.

Changes

Cohort / File(s)	Summary
Provider Interface & Implementations src/pkg/cli/client/provider.go, src/pkg/cli/client/byoc/aws/byoc.go, src/pkg/cli/client/byoc/do/byoc.go, src/pkg/cli/client/byoc/gcp/byoc.go, src/pkg/cli/client/playground.go	Removes QueryForDebug method from Provider interface and all implementations. AWS removes CloudWatch log tailing logic and slices import. GCP removes helper methods createDeploymentLogQuery and query along with io import. DO and Playground remove stub implementations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Always use etag for gcp cdCommands, remove useless code #1772: Both PRs modify GCP logging/tailing code in src/pkg/cli/client/byoc/gcp/byoc.go and remove related log query and tail functions.

Suggested reviewers

• jordanstephens

Poem

🐰 The debug methods hopped away,
No more logs to tail and play,
CloudWatch whispers fade to none,
Provider interfaces, simply done! ✨
Cleaner code hops in the sun. 🌞

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'chore: remove unused QueryForDebug' directly and clearly summarizes the main change: removal of the QueryForDebug method across multiple files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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