Skip to content

Conversation

@theFong
Copy link
Member

@theFong theFong commented Feb 2, 2026

Summary

  • Add brev claude-skill command to install/uninstall the Claude Code skill
  • Integrate skill installation prompt into brev login flow (detects if Claude Code is installed)
  • Add standalone bash installer script for non-brev users
  • Add comprehensive brev-cli skill with prompts, reference docs, and examples
  • Document Claude Code integration in README

Test plan

  • Run brev claude-skill to install the skill (requires post-merge testing - downloads from main)
  • Run brev claude-skill --uninstall to remove it
  • Run brev login and verify skill installation prompt appears when Claude Code is detected (requires integration testing)
  • Test standalone installer: curl -fsSL https://raw.githubusercontent.com/brevdev/brev-cli/main/scripts/install-claude-skill.sh | bash (requires post-merge testing)
  • Verify skill files are downloaded correctly to ~/.claude/skills/brev-cli/ (requires post-merge testing)

Note: Install tests require post-merge testing since the command downloads skill files from the main branch on GitHub.

🤖 Generated with Claude Code

Add `brev claude-skill` command to install the brev-cli skill for
Claude Code AI assistant. The skill enables natural language GPU
instance management.

Features:
- `brev claude-skill` command to install/uninstall
- Automatic prompt during `brev login` if Claude detected
- Standalone bash installer for non-brev users
- Full skill with prompts, examples, and reference docs

The skill enables commands like:
- "Create an A100 instance for ML training"
- "Search for GPUs with 40GB VRAM"
- "Stop all my running instances"

Install via:
  brev claude-skill
  # or
  curl -fsSL https://raw.githubusercontent.com/brevdev/brev-cli/main/scripts/install-claude-skill.sh | bash
@theFong theFong requested a review from a team as a code owner February 2, 2026 07:11
Use explicit octal notation (0o755, 0o644) for file permissions.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@theFong
Copy link
Member Author

theFong commented Feb 3, 2026

Manual QA Results ⚠️

Build & Static Analysis:

  • ✅ Build passes
  • go vet passes

Test Plan Verification:

  • ⚠️ brev claude-skill install - Cannot test pre-merge: Downloads from GitHub main branch where files don't exist yet
  • brev claude-skill --uninstall removes skill - Verified: Successfully removes skill directory
  • ⚠️ brev login skill prompt - Needs integration testing
  • ⚠️ Standalone installer script - Cannot test until files are on main

Skill Files Verification:

  • ✅ SKILL.md is well-structured with proper YAML frontmatter
  • ✅ All referenced prompts/ and reference/ files exist
  • ✅ Safety rules and common workflows documented

Known Issue:
The install command downloads from https://raw.githubusercontent.com/brevdev/brev-cli/main/.claude/skills/brev-cli - files don't exist on main until this PR is merged.

Suggested Improvements:

  1. Add --local flag to install from local repo files for testing
  2. Or embed files in binary during build
  3. Or use the PR branch URL during development

Post-Merge Testing Required:

  • Full install flow
  • Login integration prompt
  • Standalone bash installer

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