Skip to content

Update AGENTS.md#92

Open
bjdodson-openai wants to merge 2 commits intoSnapchat:mainfrom
bjdodson-openai:patch-1
Open

Update AGENTS.md#92
bjdodson-openai wants to merge 2 commits intoSnapchat:mainfrom
bjdodson-openai:patch-1

Conversation

@bjdodson-openai
Copy link

@bjdodson-openai bjdodson-openai commented Mar 12, 2026

Description

Adding guidance to help agents more quickly understand how to deploy and debug Valdi code

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • Documentation improvement
  • Performance optimization
  • Test improvement
  • Other (please describe)

Testing

  • Tests pass locally (bazel test //...)
  • Added/updated tests for changes (if applicable)
  • Tested on multiple platforms (iOS/Android/Web/macOS as applicable)
  • Manual testing performed (describe below)

Testing Details

Checklist

  • Code follows project style guidelines
  • Documentation updated (if needed)
  • No breaking changes (or documented in description)
  • Commit messages follow conventional format
  • No secrets, API keys, or internal URLs included

Related Issues

Additional Context

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Thanks for your first contribution to Valdi!

A maintainer will review your PR soon. Here are a few things to check while you wait:

  • ✅ All tests pass (bazel test //...)
  • ✅ Your changes follow our coding standards
  • ✅ You've added tests for your changes (if applicable)
  • ✅ You've updated documentation (if needed)

Join our Discord community if you have questions!

Copy link
Collaborator

@clholgat clholgat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea, I'm not cool with node'ing bits of the cli directly

Copy link
Collaborator

@clholgat clholgat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're still executing bits of the CLI js directly, don't do that. Either reproduce the bazel commands directly or direct it to install the cli from the local repo.

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.

2 participants