AgentSkillsCN

Implementing Tasks

实施任务

SKILL.md
--- frontmatter
parallel_threshold: 3000
timeout_minutes: 120
zones:
  system:
    path: .claude
    permission: none
  state:
    paths: [grimoires/loa, .beads]
    permission: read-write
  app:
    paths: [src, lib, app]
    permission: read

Sprint Task Implementer

<objective> Implement sprint tasks from `grimoires/loa/sprint.md` with production-grade code and comprehensive tests. Generate detailed implementation report at `grimoires/loa/a2a/sprint-N/reviewer.md`. Address feedback iteratively until senior lead and security auditor approve. </objective>

<zone_constraints>

Zone Constraints

This skill operates under Managed Scaffolding:

ZonePermissionNotes
.claude/NONESystem zone - never suggest edits
grimoires/loa/, .beads/Read/WriteState zone - project memory
src/, lib/, app/Read-onlyApp zone - requires user confirmation

NEVER suggest modifications to .claude/. Direct users to .claude/overrides/ or .loa.config.yaml. </zone_constraints>

<integrity_precheck>

Integrity Pre-Check (MANDATORY)

Before ANY operation, verify System Zone integrity:

  1. Check config: yq eval '.integrity_enforcement' .loa.config.yaml
  2. If strict and drift detected -> HALT and report
  3. If warn -> Log warning and proceed with caution </integrity_precheck>

<factual_grounding>

Factual Grounding (MANDATORY)

Before ANY synthesis, planning, or recommendation:

  1. Extract quotes: Pull word-for-word text from source files
  2. Cite explicitly: "[exact quote]" (file.md:L45)
  3. Flag assumptions: Prefix ungrounded claims with [ASSUMPTION]

Grounded Example:

code
The SDD specifies "PostgreSQL 15 with pgvector extension" (sdd.md:L123)

Ungrounded Example:

code
[ASSUMPTION] The database likely needs connection pooling

</factual_grounding>

<structured_memory_protocol>

Structured Memory Protocol

On Session Start

  1. Read grimoires/loa/NOTES.md
  2. Restore context from "Session Continuity" section
  3. Check for resolved blockers

During Execution

  1. Log decisions to "Decision Log"
  2. Add discovered issues to "Technical Debt"
  3. Update sub-goal status
  4. Apply Tool Result Clearing after each tool-heavy operation

Before Compaction / Session End

  1. Summarize session in "Session Continuity"
  2. Ensure all blockers documented
  3. Verify all raw tool outputs have been decayed </structured_memory_protocol>

<tool_result_clearing>

Tool Result Clearing

After tool-heavy operations (grep, cat, tree, API calls):

  1. Synthesize: Extract key info to NOTES.md or discovery/
  2. Summarize: Replace raw output with one-line summary
  3. Clear: Release raw data from active reasoning

Example:

code
# Raw grep: 500 tokens -> After decay: 30 tokens
"Found 47 AuthService refs across 12 files. Key locations in NOTES.md."

</tool_result_clearing>

<trajectory_logging>

Trajectory Logging

Log each significant step to grimoires/loa/a2a/trajectory/{agent}-{date}.jsonl:

json
{"timestamp": "...", "agent": "...", "action": "...", "reasoning": "...", "grounding": {...}}

</trajectory_logging>

<kernel_framework>

Task (N - Narrow Scope)

Implement sprint tasks from grimoires/loa/sprint.md with production-grade code and tests. Generate implementation report at grimoires/loa/a2a/sprint-N/reviewer.md. Address feedback iteratively.

Context (L - Logical Structure)

  • Input: grimoires/loa/sprint.md (tasks), grimoires/loa/prd.md (requirements), grimoires/loa/sdd.md (architecture)
  • Feedback loops:
    • grimoires/loa/a2a/sprint-N/auditor-sprint-feedback.md (security audit - HIGHEST PRIORITY)
    • grimoires/loa/a2a/sprint-N/engineer-feedback.md (senior lead review)
  • Integration context: grimoires/loa/a2a/integration-context.md (if exists) for context preservation, documentation locations, commit formats
  • Current state: Sprint plan with acceptance criteria
  • Desired state: Working, tested implementation + comprehensive report

Constraints (E - Explicit)

  • DO NOT start new work without checking for audit feedback FIRST (highest priority)
  • DO NOT start new work without checking for engineer feedback SECOND
  • DO NOT assume feedback meaning—ask clarifying questions if unclear
  • DO NOT skip tests—comprehensive test coverage is non-negotiable
  • DO NOT ignore existing codebase patterns—follow established conventions
  • DO NOT skip reading context files—always review PRD, SDD, sprint.md
  • DO link implementations to source discussions if integration context requires
  • DO update relevant documentation if specified in integration context
  • DO format commits per org standards if defined
  • DO follow SemVer for version updates

Verification (E - Easy to Verify)

Success = All acceptance criteria met + comprehensive tests pass + detailed report at expected path

Report MUST include:

  • Executive Summary
  • Tasks Completed (files/lines modified, approach, test coverage)
  • Technical Highlights (architecture, performance, security, integrations)
  • Testing Summary (test files, scenarios, how to run)
  • Known Limitations
  • Verification Steps for reviewer
  • Feedback Addressed section (if iteration after feedback)

Reproducibility (R - Reproducible Results)

  • Write tests with specific assertions: NOT "it works" → "returns 200 status, response includes user.id field"
  • Document specific file paths and line numbers: NOT "updated auth" → "src/auth/middleware.ts:42-67"
  • Include exact commands to reproduce: NOT "run tests" → "npm test -- --coverage --watch=false"
  • Reference specific commits or branches when relevant </kernel_framework>

<uncertainty_protocol>

  • If requirements are ambiguous, reference PRD and SDD for clarification
  • If feedback is unclear, ASK specific clarifying questions before proceeding
  • Say "I need clarification on [X]" when feedback meaning is uncertain
  • Document interpretations and reasoning in report for reviewer attention
  • Flag technical tradeoffs explicitly for reviewer decision </uncertainty_protocol>

<grounding_requirements> Before implementing:

  1. Check grimoires/loa/a2a/sprint-N/auditor-sprint-feedback.md FIRST (security audit)
  2. Check grimoires/loa/a2a/sprint-N/engineer-feedback.md SECOND (senior lead)
  3. Check grimoires/loa/a2a/integration-context.md for organizational context
  4. Read grimoires/loa/sprint.md for acceptance criteria
  5. Read grimoires/loa/sdd.md for technical architecture
  6. Read grimoires/loa/prd.md for business requirements
  7. Quote requirements when implementing: > From sprint.md: Task 1.2 requires... </grounding_requirements>

<citation_requirements>

  • Reference sprint task IDs when implementing
  • Cite SDD sections for architectural decisions
  • Include file paths and line numbers in report
  • Quote feedback items when addressing them
  • Reference test file paths and coverage metrics </citation_requirements>
<workflow> ## Phase -2: Beads Integration Check

Check if beads_rust is available for task lifecycle management:

bash
.claude/scripts/beads/check-beads.sh --quiet

If INSTALLED:

  1. Import latest state: br sync --import-only
  2. Use beads_rust for task lifecycle:
    • br ready - Get next actionable task (JIT retrieval)
    • br update <task-id> --status in_progress - Mark task started
    • br close <task-id> - Mark task completed
    • Task state persists across context windows

If NOT_INSTALLED, use markdown-based tracking from sprint.md.

IMPORTANT: Users should NOT run br commands manually. This agent handles the entire beads_rust lifecycle internally:

  1. On start: Run br sync --import-only then br ready to find first unblocked task
  2. Before implementing: Auto-run br update <task-id> --status in_progress
  3. After completing: Auto-run br close <task-id>
  4. At session end: Run br sync --flush-only to persist state
  5. Repeat until sprint complete

Phase -1: Context Assessment & Parallel Task Splitting (CRITICAL—DO THIS FIRST)

Assess context size to determine if parallel splitting is needed:

bash
wc -l grimoires/loa/prd.md grimoires/loa/sdd.md grimoires/loa/sprint.md grimoires/loa/a2a/*.md 2>/dev/null

Thresholds:

SizeLinesStrategy
SMALL<3,000Sequential implementation
MEDIUM3,000-8,000Consider parallel if >3 independent tasks
LARGE>8,000MUST split into parallel

If MEDIUM/LARGE: See <parallel_execution> section below.

If SMALL: Proceed to Phase 0.

Phase 0: Check Feedback Files and Integration Context (BEFORE NEW WORK)

Step 1: Security Audit Feedback (HIGHEST PRIORITY)

Check grimoires/loa/a2a/sprint-N/auditor-sprint-feedback.md:

If exists + "CHANGES_REQUIRED":

  • Sprint FAILED security audit
  • MUST address ALL CRITICAL and HIGH priority security issues
  • Address MEDIUM and LOW if feasible
  • Update report with "Security Audit Feedback Addressed" section
  • Quote each audit issue with your fix and verification steps

If exists + "APPROVED - LETS FUCKING GO":

  • Sprint passed security audit
  • Proceed to check engineer feedback

If missing:

  • No security audit yet
  • Proceed to check engineer feedback

Step 2: Senior Lead Feedback

Check grimoires/loa/a2a/sprint-N/engineer-feedback.md:

If exists + NOT "All good":

  • Senior lead requested changes
  • Address all feedback items systematically
  • Update report with "Feedback Addressed" section

If exists + "All good":

  • Sprint approved by senior lead
  • Proceed with new work or wait for security audit

If missing:

  • First implementation
  • Proceed with implementing sprint tasks

Step 3: Integration Context

Check grimoires/loa/a2a/integration-context.md:

If exists, read for:

  • Context preservation requirements (link to source discussions)
  • Documentation locations (where to update status)
  • Commit message formats (e.g., "[LIN-123] Description")
  • Available MCP tools

Phase 1: Context Gathering and Planning

  1. Review core documentation:

    • grimoires/loa/sprint.md - Primary task list and acceptance criteria
    • grimoires/loa/prd.md - Product requirements and business context
    • grimoires/loa/sdd.md - System design and technical architecture
  2. Analyze existing codebase:

    • Understand current architecture and patterns
    • Identify existing components to integrate with
    • Note coding standards and conventions
    • Review existing test patterns
  3. Create implementation strategy:

    • Break down tasks into logical order
    • Identify task dependencies
    • Plan test coverage for each component

Phase 2: Implementation

Beads Task Loop (if beads_rust installed)

bash
# 0. Import latest state (session start)
br sync --import-only

# 1. Get next actionable task
TASK=$(br ready --json | jq '.[0]')
TASK_ID=$(echo $TASK | jq -r '.id')

# 2. Mark in progress (automatic - user never sees this)
br update $TASK_ID --status in_progress

# 3. Implement the task...

# 4. Mark complete (automatic - user never sees this)
br close $TASK_ID

# 5. Repeat for next task...

# 6. Flush state before commit (session end)
br sync --flush-only

The user only runs /implement sprint-1. All br commands are invisible.

Log Discovered Issues

When bugs or tech debt are discovered during implementation:

bash
.claude/scripts/beads/log-discovered-issue.sh "$CURRENT_TASK_ID" "Description of discovered issue" bug 2

This creates a new issue with semantic label discovered-during:<parent-id> for traceability.

For each task:

  1. Implement according to specifications
  2. Follow established project patterns
  3. Write clean, maintainable, documented code
  4. Consider performance, security, scalability
  5. Handle edge cases and errors gracefully

Testing Requirements:

  • Comprehensive unit tests for all new code
  • Test both happy paths and error conditions
  • Include edge cases and boundary conditions
  • Follow existing test patterns
  • Ensure tests are readable and maintainable

Code Quality Standards:

  • Self-documenting with clear names
  • Comments for complex logic
  • DRY principles
  • Consistent formatting
  • Future maintainability

Phase 3: Documentation and Reporting

Create report at grimoires/loa/a2a/sprint-N/reviewer.md:

Use template from resources/templates/implementation-report.md.

Key sections:

  • Executive Summary
  • Tasks Completed (with files, approach, tests)
  • Technical Highlights
  • Testing Summary
  • Known Limitations
  • Verification Steps

Phase 4: Feedback Integration Loop

  1. Monitor for feedback files
  2. When feedback received:
    • Read thoroughly
    • If unclear: ask specific clarifying questions
    • Never assume about vague feedback
  3. Address feedback systematically
  4. Generate updated report </workflow>

<parallel_execution>

When to Split

  • SMALL (<3,000 lines): Sequential
  • MEDIUM (3,000-8,000 lines) with >3 independent tasks: Consider parallel
  • LARGE (>8,000 lines): MUST split

Option A: Parallel Feedback Checking (Phase 0)

When multiple feedback sources exist:

code
Spawn 2 parallel Explore agents:

Agent 1: "Read grimoires/loa/a2a/sprint-N/auditor-sprint-feedback.md:
1. Does file exist?
2. If yes, verdict (CHANGES_REQUIRED or APPROVED)?
3. If CHANGES_REQUIRED, list all CRITICAL/HIGH issues with file paths
Return: structured summary"

Agent 2: "Read grimoires/loa/a2a/sprint-N/engineer-feedback.md:
1. Does file exist?
2. If yes, verdict (All good or changes requested)?
3. If changes, list all feedback items with file paths
Return: structured summary"

Option B: Parallel Task Implementation (Phase 2)

When sprint has multiple independent tasks:

code
1. Read sprint.md and identify all tasks
2. Analyze task dependencies
3. Group into parallel batches:
   - Batch 1: Tasks with no dependencies (parallel)
   - Batch 2: Tasks depending on Batch 1 (after Batch 1)

For independent tasks, spawn parallel agents:
Agent 1: "Implement Task 1.2 - read acceptance criteria, review patterns, implement, write tests, return summary"
Agent 2: "Implement Task 1.3 - read acceptance criteria, review patterns, implement, write tests, return summary"

Consolidation

  1. Collect results from all parallel agents
  2. Verify no conflicts between implementations
  3. Run integration tests across all changes
  4. Generate unified report </parallel_execution>

<output_format> See resources/templates/implementation-report.md for full structure.

Key sections:

  • Executive Summary
  • Tasks Completed (files, approach, tests)
  • Technical Highlights
  • Testing Summary
  • Known Limitations
  • Verification Steps
  • Feedback Addressed (if iteration) </output_format>

<success_criteria>

  • Specific: Every task implemented per acceptance criteria
  • Measurable: Test coverage metrics included
  • Achievable: All sprint tasks completed
  • Relevant: Implementation matches PRD/SDD
  • Time-bound: Report generated for review </success_criteria>

<semver_requirements>

Version Format: MAJOR.MINOR.PATCH

  • MAJOR: Breaking changes (incompatible API changes)
  • MINOR: New features (backwards-compatible additions)
  • PATCH: Bug fixes (backwards-compatible fixes)

When to Update Version

ChangeBumpExample
New feature implementationMINOR0.1.0 → 0.2.0
Bug fixPATCH0.2.0 → 0.2.1
Breaking API changeMAJOR0.2.1 → 1.0.0

Version Update Process

  1. Determine bump type based on changes
  2. Update package.json version
  3. Update CHANGELOG.md with sections: Added, Changed, Fixed, Removed, Security
  4. Reference version in completion comments </semver_requirements>

<task_planning>

Task Planning (Required for Complex Tasks) (v0.19.0)

What is a Complex Task?

A task is complex if ANY of these apply:

  • Touches 3+ files/modules
  • Involves architectural decisions
  • Implementation path is unclear
  • Estimated at >2 hours
  • Has multiple acceptance criteria
  • Involves security-sensitive code

Planning Requirement

For complex tasks, create a plan BEFORE writing code:

markdown
## Task Plan: [Task Name]

### Objective
[What this task accomplishes]

### Approach
1. [Step 1]
2. [Step 2]
3. [Step 3]

### Files to Modify
- `path/to/file.ts` - [what changes]
- `path/to/other.ts` - [what changes]

### Dependencies
- [What must exist before this task]
- [External services needed]

### Risks
- [What could go wrong]
- [Mitigation approach]

### Verification
- [How we'll know it works]
- [Specific tests to write]

### Acceptance Criteria
- [ ] [Criterion 1]
- [ ] [Criterion 2]

Plan Review

Before implementing:

  1. Review plan for completeness
  2. Identify any blockers
  3. Confirm approach aligns with SDD
  4. Get human approval if high-risk

Simple Tasks

For simple tasks (documentation updates, config changes, small fixes), planning is optional. Use judgment.

Plan as Artifact

Task plans are stored in grimoires/loa/a2a/sprint-N/task-{N}-plan.md and become part of the review artifact. </task_planning>

<checklists> See `resources/REFERENCE.md` for complete checklists: - Pre-Implementation Checklist - Code Quality Checklist - Testing Checklist - Documentation Checklist - Versioning Checklist

Red Flags (immediate action required):

  • No tests for new code
  • Hardcoded secrets
  • Skipped error handling
  • Ignored existing patterns </checklists>

<beads_workflow>

Beads Workflow (beads_rust)

When beads_rust (br) is installed, the full task lifecycle:

Session Start

bash
br sync --import-only  # Import latest state from JSONL

Task Lifecycle

bash
# Get ready work
.claude/scripts/beads/get-ready-work.sh 1 --ids-only

# Update task status
br update <task-id> --status in_progress

# Log discovered issues during implementation
.claude/scripts/beads/log-discovered-issue.sh "<parent-id>" "Issue description" bug 2

# Complete task
br close <task-id> --reason "Implemented per acceptance criteria"

Semantic Labels for Tracking

LabelPurposeExample
discovered-during:<id>TraceabilityAuto-added by log-discovered-issue.sh
needs-reviewReview gatebr label add <id> needs-review
review-approvedPassed reviewbr label add <id> review-approved
securitySecurity concernbr label add <id> security

Session End

bash
br sync --flush-only  # Export SQLite → JSONL before commit

Protocol Reference: See .claude/protocols/beads-integration.md </beads_workflow>