Skip to content

feat(youtube-analyst): add Agent Starter Pack#1195

Closed
maeve2024 wants to merge 3 commits intogoogle:mainfrom
tristanhussain:feat/youtube-analyst-add-starter-pack-clean
Closed

feat(youtube-analyst): add Agent Starter Pack#1195
maeve2024 wants to merge 3 commits intogoogle:mainfrom
tristanhussain:feat/youtube-analyst-add-starter-pack-clean

Conversation

@maeve2024
Copy link
Copy Markdown

  • Add [tool.agent-starter-pack] and agent-starter-pack dev dep so the agent can be used as an ASP template.
  • Add flexible auth in init.py (google.auth.default + env setdefault) so it works with ADC, service accounts, and Cloud Run/GKE without hardcoding project/location.
  • Make ASP the main path in README and recommend uv one-liner first so users get a production-ready setup by default; clone-and-run is the fallback.
  • Remove load_artifacts from the root agent so ASP-generated projects pass integration tests when the Runner has no artifact service (avoids "Artifact service is not initialized").
  • Add pytest-asyncio and asyncio_mode = "auto" so generated projects’ async tests run without extra config.
  • Regenerate uv.lock for new dev deps.

TESTS:
image
image

…ed-project tests

- Add [tool.agent-starter-pack] and agent-starter-pack dev dep so the agent can be used as an ASP template.
- Add flexible auth in __init__.py (google.auth.default + env setdefault) so it works with ADC, service accounts, and Cloud Run/GKE without hardcoding project/location.
- Make ASP the main path in README and recommend uv one-liner first so users get a production-ready setup by default; clone-and-run is the fallback.
- Remove load_artifacts from the root agent so ASP-generated projects pass integration tests when the Runner has no artifact service (avoids "Artifact service is not initialized").
- Add pytest-asyncio and asyncio_mode = "auto" so generated projects’ async tests run without extra config.
- Regenerate uv.lock for new dev deps.
@eliasecchig
Copy link
Copy Markdown
Collaborator

LGTM thanks for doing it!

zeroasterisk
zeroasterisk previously approved these changes Mar 18, 2026
@zeroasterisk zeroasterisk dismissed their stale review March 18, 2026 12:42

Reverting: I was only supposed to approve the workflow runs, not the PR itself.

@zeroasterisk
Copy link
Copy Markdown
Collaborator

Closing in favor of #1337 (newer version of this YouTube Analyst ASP integration). Thanks for the contribution! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants