Skip to content

Update agent-integration skill docs#555

Open
alishakawaguchi wants to merge 3 commits intomainfrom
alisha/improve-agent-integration-skill
Open

Update agent-integration skill docs#555
alishakawaguchi wants to merge 3 commits intomainfrom
alisha/improve-agent-integration-skill

Conversation

@alishakawaguchi
Copy link
Contributor

@alishakawaguchi alishakawaguchi commented Feb 28, 2026

Summary

  • Improved agent-integration skill documentation with clearer parameter descriptions and derivation guidance
  • Rewrote implementer docs with E2E-first TDD approach (unit tests written last) and detailed implementation patterns
  • Simplified researcher and test-writer docs for clarity and consistency with actual codebase patterns

Test plan

  • Verify /agent-integration skill triggers correctly with updated SKILL.md
  • Run through researcher phase to confirm updated instructions are clear
  • Run through test-writer phase to confirm updated instructions work
  • Run through implementer phase to confirm E2E-first TDD flow works

🤖 Generated with Claude Code

Entire-Checkpoint: ea560cc631ef
@alishakawaguchi alishakawaguchi requested a review from a team as a code owner February 28, 2026 00:22
Copilot AI review requested due to automatic review settings February 28, 2026 00:22
@cursor
Copy link

cursor bot commented Feb 28, 2026

PR Summary

Low Risk
Documentation-only changes, but they materially alter the recommended agent integration workflow (E2E-first tiers, unit tests last) and required artifacts (AGENT.md), so incorrect guidance could slow or misdirect future integrations.

Overview
Updates the agent-integration skill docs to standardize on E2E-first TDD: build and iterate by running existing ForEachAgent E2E tests in explicit tiers, defer all unit tests until the end, and use /debug-e2e + frequent /commit checkpoints.

Shifts Phase 1 output from a generic compatibility report to a required agent-specific cmd/entire/cli/agent/$AGENT_PACKAGE/AGENT.md one-pager that becomes the single source of truth for later phases, updates parameter conventions (AGENT_PACKAGE, AGENT_KEY), and narrows scope to hook-capable agents.

Refocuses Phase 2 (write-tests) to only create/verify the E2E agent runner (no new scenarios), and rewrites researcher guidance to rely on agent-guide.md plus external probing while explicitly avoiding other internal source reads.

Written by Cursor Bugbot for commit ee048ec. Configure here.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the agent-integration skill documentation to standardize around an E2E-first TDD workflow, with a single-source-of-truth agent one-pager (AGENT.md) feeding the runner and implementation phases.

Changes:

  • Refocused the test-writer phase on creating only an E2E agent runner (no new scenarios / no unit tests).
  • Reworked the researcher phase to output a structured AGENT.md one-pager for downstream phases.
  • Rewrote implementer guidance to enforce E2E-first development with unit tests written last.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 10 comments.

File Description
.claude/skills/agent-integration/test-writer.md Shifts phase to “runner only” and adds compile/registration verification steps.
.claude/skills/agent-integration/researcher.md Replaces feasibility report output with a persisted AGENT.md one-pager template.
.claude/skills/agent-integration/implementer.md Introduces strict E2E-tier workflow and consolidates unit tests into the final step.
.claude/skills/agent-integration/SKILL.md Updates skill description, parameter derivation guidance, and E2E-first TDD rule set.

alishakawaguchi and others added 2 commits February 27, 2026 16:25
…meter

- Fix single-dash `-agent` to double-dash `--agent` in implementer.md
- Fix `test:e2e:$SLUG` colon syntax to `test:e2e --agent $SLUG` in
  implementer.md and test-writer.md
- Clarify Step 3's `mise run test` is a compile-only sanity check
- Fix "Step 13" → "Step 14" cross-reference in output section
- Add missing `AGENT_SLUG` parameter to SKILL.md parameter table

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Entire-Checkpoint: 17a7922e66f6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants