Recovered: feat(agent): add Gemini CLI install target (#57 by @merlinsantiago982-cmd)#247
Conversation
WalkthroughThe PR introduces a registry for multiple agent skills, adds onboarding content, supports Gemini managed sections, updates installation and initialization reporting, refines managed-section detection, and expands CLI documentation and unit/e2e coverage. ChangesAgent skills and installation
Estimated code review effort: 4 (Complex) | ~60 minutes Possibly related issues
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed: private package registry requires authentication. Disable ESLint in CodeRabbit settings or use public packages. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Around line 64-70: Update the README quickstart text around testsprite setup
to describe installing both the verification and onboarding skills, replacing
the singular references in the setup description and agent guidance while
preserving the existing command examples and links.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: ffbc3408-8c31-4ef5-b60d-53ad397f9cb7
⛔ Files ignored due to path filters (1)
test/__snapshots__/help.snapshot.test.ts.snapis excluded by!**/*.snap,!**/*.snap
📒 Files selected for processing (15)
DOCUMENTATION.mdREADME.mddocs/cli-v1-agent-install/onboard-skill-template.mdskills/testsprite-onboard.skill.mdskills/testsprite-verify.codex.mdskills/testsprite-verify.skill.mdsrc/commands/agent.test.tssrc/commands/agent.tssrc/commands/init.tssrc/lib/agent-targets.test.tssrc/lib/agent-targets.tssrc/lib/skill-nudge.test.tssrc/lib/skill-nudge.tstest/e2e/agent-install.e2e.test.tstest/e2e/setup.e2e.test.ts
| `testsprite setup` prompts for your [API key](https://www.testsprite.com), verifies it, and installs the verification-loop skill for your coding agent (`claude`, `cursor`, `cline`, `antigravity`, `codex`, `gemini`, etc.) — one command, so your agent is wired to verify its own work. Non-interactive (CI / onboarding scripts): | ||
|
|
||
| ```bash | ||
| TESTSPRITE_API_KEY=sk-... testsprite setup --from-env --yes --agent claude | ||
| ``` | ||
|
|
||
| > **Pointing a coding agent (Claude Code, Cursor, Codex, Cline, …) at TestSprite?** Have it run `testsprite setup` first — that installs the verification skill, so the agent knows how to create, run, and triage tests on its own (instead of guessing from this README). New here? Start with the **[getting-started overview](https://docs.testsprite.com/cli/getting-started/overview)**. | ||
| > **Pointing a coding agent (Claude Code, Gemini CLI, Cursor, Codex, Cline, …) at TestSprite?** Have it run `testsprite setup` first — that installs the verification skill, so the agent knows how to create, run, and triage tests on its own (instead of guessing from this README). New here? Start with the **[getting-started overview](https://docs.testsprite.com/cli/getting-started/overview)**. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Quickstart understates what setup now installs.
testsprite setup now installs the full DEFAULT_SKILLS set (testsprite-verify and testsprite-onboard), consistent with init.ts's pluralized SETUP_DESCRIPTION. Lines 64 and 70 still describe only "the verification-loop skill" / "the verification skill" (singular), omitting the onboarding skill. Consider pluralizing to mention both the verification and onboarding skills so the docs match behavior.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.md` around lines 64 - 70, Update the README quickstart text around
testsprite setup to describe installing both the verification and onboarding
skills, replacing the singular references in the setup description and agent
guidance while preserving the existing command examples and links.
|
Hi @merlinsantiago982-cmd 👋 — this is the recovery of your PR #57 , which was auto-closed by an error in our 2026-07-09 release process. Your commits are intact — thank you for the contribution! It currently shows merge conflicts because Option A — merge (no force-push): git checkout BRANCH
git fetch upstream # your remote for TestSprite/testsprite-cli (may be "origin")
git merge upstream/main # resolve conflicts, commit
git pushOption B — rebase (cleaner history, needs force): git checkout BRANCH
git fetch upstream
git rebase upstream/main # resolve conflicts
git push --force-with-leaseOnce you push, the conflicts here resolve automatically and we'll take it into review. Sorry again for the extra step! |
2 participants
Summary
geminiagent target that installs TestSprite guidance intoGEMINI.mdSafety
GEMINI.mduses sentinel-delimited managed-section writes, so existing project instructions outside the TestSprite block are preservedTests
npm run format:checknpm run test -- src/lib/agent-targets.test.ts src/commands/agent.test.ts test/help.snapshot.test.tsnpm run typechecknpm run lintnpm run buildSummary by CodeRabbit
testsprite-onboardskill for creating an initial test suite.--skill.