Agent skill
ds-implement
Phase 3 of /ds workflow. Execute analysis tasks with output-first verification.
Install this agent skill to your Project
npx add-skill https://github.com/edwinhu/workflows/tree/main/skills/ds-implement
SKILL.md
Overview
Apply output-first verification at every step of analysis implementation. This is Phase 3 of the /ds workflow.
Contents
- The Iron Law of DS Implementation - EVERY step MUST produce visible output
- Delegation - Main chat orchestrates, subagents analyze
- What Output-First Means
- Red Flags
- SAS Language Routing - Load SAS enforcement when PLAN.md specifies SAS
- Implementation Process
- Verification Patterns - See
references/verification-patterns.md - Common Failures
- Gate: Exit Implementation
Implementation (Output-First Verification)
Implement analysis with mandatory visible output at every step. NO TDD - instead, every code step MUST produce and verify output.
EVERY CODE STEP MUST PRODUCE VISIBLE OUTPUT. This is not negotiable.
Before moving to the next step, you MUST:
- Run the code
- See the output (print, display, plot)
- Verify output is correct/reasonable
- Document in
.planning/LEARNINGS.md - Only THEN proceed to next step
This applies even when YOU think:
- "I know this works"
- "It's just a simple transformation"
- "I'll check results at the end"
- "The code is straightforward"
If you're about to write code without outputting results, STOP. </EXTREMELY-IMPORTANT>
Delegation
You orchestrate. Subagents analyze. For every task in PLAN.md, use the delegation skill:
Read ${CLAUDE_SKILL_DIR}/../../skills/ds-delegate/SKILL.md and follow its instructions.
This is MANDATORY. ds-delegate contains the Task agent templates, output-first protocol details, methodology review patterns, and rationalization prevention. Do not attempt to summarize or shortcut it.
If you're about to write analysis code directly, STOP and read ds-delegate.
If you wrote analysis code in main chat, DELETE it immediately and dispatch a Task agent instead. Code written in main chat is contaminated by orchestrator context and must not be kept.
Delete & Restart Protocol
| Scenario | Action |
|---|---|
| You wrote > 3 lines of analysis code in main chat | DELETE immediately. Restart via Task agent. |
| You ran a cell, realized it should have been in Task agent | DELETE the cell output and cell. Re-do via Task agent. |
| You started a transformation in main chat | STOP. DELETE what you've done. Spawn Task agent instead. |
| "Just finish this quick analysis here" | STOP — if it's quick enough to finish, it's quick enough for a Task agent. Delete and restart. |
Helpfulness Check: If you kept main-chat code "because it worked," you bypassed the orchestration protocol. Working code written in the wrong place skips verification and review — it is anti-helpful to the user. Delete it. </EXTREMELY-IMPORTANT>
What Output-First Means
| DO | DON'T |
|---|---|
| Print shape after each transform | Chain operations silently |
| Display sample rows | Trust transformations work |
| Show summary stats | Wait until end to check |
| Verify row counts | Assume merges worked |
| Check for unexpected nulls | Skip intermediate checks |
| Plot distributions | Move on without looking |
The Mantra: If not visible, it cannot be trusted.
Rationalization Table
| Excuse | Reality | Do Instead |
|---|---|---|
| "I'll combine these steps to save time" | Combined steps hide which one failed. Your efficiency creates undiagnosable bugs. | One operation per verification cycle |
| "Output looks roughly right" | "Roughly" means you didn't actually check. Roughly right is precisely wrong. | Compare against PLAN.md expected output with specific numbers |
| "LEARNINGS.md is overhead" | Undocumented steps are invisible to review. Skipping logs means review can't verify your work. | Log every task completion — it takes 30 seconds |
| "This task is trivial, no need to delegate" | Trivial tasks in main chat accumulate into unverified analysis. | Delegate everything, verify everything |
| "I'll read the agent's output later" | Running code without reading the output is running blind. The agent may have reported errors you missed. | Read agent output immediately, verify claims against actual data |
Drive-Aligned Framing
| Drive | Shortcut | Consequence |
|---|---|---|
| Helpfulness | Skipping output verification | You assumed the output was correct because the code ran. The results are wrong — your assumption is the error the user publishes. You were anti-helpful. |
| Helpfulness | Claiming task complete without checking output | You logged "COMPLETE" in LEARNINGS.md without verifying the output. The task may have silently failed — the user acts on results that don't exist. |
| Competence | Running all tasks sequentially yourself | You ran everything yourself to feel productive. Quality suffered without delegation and review — your productivity was incompetence disguised as efficiency. |
| Efficiency | Not delegating to Task agents | You wrote analysis in main chat because it felt faster. Now it must be deleted and redone. Your shortcut doubled the work — anti-efficient. |
| Approval | Skipping LEARNINGS.md logging | You skipped documentation to move faster. The reviewer cannot verify your work. The user loses trust when review fails. You lost their approval. |
Red Flags - STOP Immediately
| Thought | Why It's Wrong | Do Instead |
|---|---|---|
| "I'll check at the end" | STOP - you're letting errors compound silently | Check after every step |
| "This transform is simple" | STOP - simple code can still be wrong | Output and verify |
| "I know merge worked" | STOP - you've assumed this before and been wrong | Check row counts |
| "Data looks fine" | STOP - you're confusing "looks" with verification | Print stats, show samples |
| "I'll batch the outputs" | STOP - you're about to lose your ability to isolate issues | Output per operation |
| "Just a quick plot in main chat" | STOP - you're about to violate delegation | Spawn a Task agent |
| "I'll combine these steps to save time" | STOP - combined steps hide which one failed | One operation per verification cycle |
| "Output looks roughly right" | STOP - "roughly" means you didn't actually check | Compare against .planning/PLAN.md expected output |
| "LEARNINGS.md is overhead" | STOP - undocumented steps are invisible to review | Log every task completion to .planning/LEARNINGS.md, it takes 30 seconds |
| "This task is trivial, no need to delegate" | STOP - trivial tasks in main chat accumulate into unverified analysis | Delegate everything, verify everything |
Implementation Strategy Choice
After prerequisites pass and PLAN.md verified, check for parallelization potential:
Skip this choice when:
- PLAN.md has fewer than 4 tasks
- All tasks are dependent (every task is
after Nwith no independent groups) - Tasks form a pipeline (clean → merge → aggregate → model)
CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMSis not available
Otherwise, ask the user:
AskUserQuestion(questions=[{
"question": "How should we implement the analysis tasks in PLAN.md?",
"header": "Strategy",
"options": [
{"label": "Sequential (Default)", "description": "One task at a time with output-first verification. Safest, most DS work is sequential."},
{"label": "Agent team (parallel)", "description": "Spawn analyst per independent task group. Only for truly independent analysis branches (descriptive stats by subgroup, model comparisons). Requires reconciliation."}
],
"multiSelect": false
}])
If Sequential: Proceed to Implementation Process below (current behavior).
If Agent team: Skip to Agent Team Implementation (Parallel).
SAS Language Routing
If PLAN.md specifies Implementation Language: SAS or Mixed, load SAS enforcement BEFORE dispatching any SAS tasks. Paste the enforcement block into every SAS subagent prompt.
Full SAS enforcement rules: See references/sas-enforcement.md
Implementation Process Flowchart
┌─────────────────────────┐
│ Read PLAN.md + Load │
│ ds-delegate + ETL refs │
└───────────┬─────────────┘
▼
┌─────────────────────────┐
│ For each task in PLAN │◄──────────────────────┐
│ (in dependency order) │ │
└───────────┬─────────────┘ │
▼ │
┌─────────────────────────┐ │
│ Dispatch Task agent │ │
│ (per ds-delegate) │ │
└───────────┬─────────────┘ │
▼ │
┌─────────────────────────┐ ┌──────────────┐ │
│ Read agent output │────→│ Output wrong │ │
│ Verify output present │ │ or missing? │ │
│ + reasonable │ └──────┬───────┘ │
└───────────┬─────────────┘ │ │
│ OK ▼ │
│ ┌──────────────────┐ │
│ │ STOP. Investigate │ │
│ │ Log issue. Fix. │ │
│ │ Re-verify. │ │
│ └──────────────────┘ │
▼ │
┌─────────────────────────┐ │
│ Log to LEARNINGS.md │ │
│ (Task N: COMPLETE) │ │
└───────────┬─────────────┘ │
▼ │
More tasks? ──── YES ─────────────────────┘
│
NO
▼
┌─────────────────────────┐
│ Exit Gate: Compare │
│ PLAN.md vs LEARNINGS │
│ (all tasks accounted?) │
└───────────┬─────────────┘
▼
┌─────────────────────────┐
│ Invoke ds-validate │
└─────────────────────────┘
This flowchart IS the specification. If the narrative below and this flowchart disagree, the flowchart wins.
Topic Change Protocol
If user sends an off-topic message during implementation, follow C6 from ds-common-constraints.md:
- Announce: "Pausing ds-implement to address your request."
- Handle: Process the request (normal tools allowed outside the loop).
- Announce: "Resuming ds-implement. Reading state files for current progress."
- Reload: Read LEARNINGS.md and PLAN.md to restore context.
- Resume: Continue from where you left off.
Do NOT silently switch context. Silent switches kill the implementation loop.
Implementation Process
Step 1: Read Plan, Load Shared Enforcement, and Delegation Skill
Read ${CLAUDE_SKILL_DIR}/../../references/constraints/ds-common-constraints.md for the full constraint index.
For implementation phase, load these specific constraints:
Read ${CLAUDE_SKILL_DIR}/../../references/constraints/ds-data-quality-checks.md
Read ${CLAUDE_SKILL_DIR}/../../references/constraints/ds-post-subagent-boundary.md
Read ${CLAUDE_SKILL_DIR}/../../references/constraints/ds-deviation-rules.md
Load conventions for implementation phase:
Read ${CLAUDE_SKILL_DIR}/../../references/constraints/ds-common-conventions.md for the full convention index.
Read ${CLAUDE_SKILL_DIR}/../../references/conventions/ds-assumption-over-evidence.md
Read ${CLAUDE_SKILL_DIR}/../../references/conventions/ds-deferred-verification.md
Read ${CLAUDE_SKILL_DIR}/../../references/conventions/ds-topic-change-protocol.md
Read ${CLAUDE_SKILL_DIR}/../../references/conventions/ds-escape-patterns.md
Read(".planning/PLAN.md")
Read ${CLAUDE_SKILL_DIR}/../../skills/ds-delegate/SKILL.md and follow its instructions.
Follow the task order defined in the plan. Use ds-delegate's templates for every task.
ETL Strategy Enforcement — load domain-specific references based on PLAN.md:
If PLAN.md contains an ## ETL Strategy section, the user made decisions during planning that MUST be enforced during implementation. Check each subsection and load the corresponding enforcement:
| PLAN.md Section | Enforcement Reference | Inject Into |
|---|---|---|
Implementation Language: SAS or Mixed |
SAS ETL enforcement (skills/wrds/references/sas-etl.md) |
Every SAS subagent prompt |
Filter Strategy table present |
ETL enforcement (skills/ds-implement/references/etl-enforcement.md) § Filter Push-Down |
Subagent prompts for data loading tasks |
Parallelism Plan table present |
ETL enforcement (skills/ds-implement/references/etl-enforcement.md) § Parallelism |
Implementation strategy choice |
Data Flow with intermediates |
ETL enforcement (skills/ds-implement/references/etl-enforcement.md) § Caching |
Subagent prompts for tasks producing/consuming intermediates |
Scale-Up Testing Plan table present |
ETL enforcement (skills/ds-implement/references/etl-enforcement.md) § Scale-Up + domain reference (e.g., gemini-batch/references/scale-up-testing.md) |
Before any batch submission task |
To load these references, discover the plugin cache path first:
- Read
${CLAUDE_SKILL_DIR}/../../skills/wrds/references/sas-etl.mdand follow its instructions. - Read
${CLAUDE_SKILL_DIR}/../../skills/ds-implement/references/etl-enforcement.mdand follow its instructions.
If PLAN.md has NO ETL Strategy section: Skip this — proceed directly to Step 2.
Context Monitoring
Before starting each task, check context availability:
| Level | Remaining Context | Action |
|---|---|---|
| Normal | >35% | Proceed with task |
| Warning | 25-35% | Complete current task, then invoke ds-handoff |
| Critical | ≤25% | Invoke ds-handoff immediately — no new tasks |
At Warning level: After current task completes, invoke:
Read ${CLAUDE_SKILL_DIR}/../../skills/ds-handoff/SKILL.md and follow its instructions.
Why: A multi-task analysis pipeline with 20% context remaining produces degraded output. Better to handoff cleanly and resume fresh.
Step 2: Execute Each Task via Delegation
For each task in PLAN.md:
- Dispatch analyst subagent (per ds-delegate pattern)
- Verify outputs are present and reasonable
- Dispatch methodology reviewer (for statistical tasks)
- Log findings to LEARNINGS.md
Step 3: Log to LEARNINGS.md
Document every significant step:
## Task N: [Description] - COMPLETE
**Input:** [Describe input state]
**Operation:** [What was done]
**Output:**
- Shape: [final shape]
- Key findings: [observations]
**Verification:** [How you confirmed it worked]
**Next:** [What comes next]
Task Summary (MANDATORY after each task)
After a task passes review, append a structured summary to LEARNINGS.md:
## Task N: [task description]
---
task: N
status: completed
implements: [DATA-01, STAT-03]
affects: [notebooks/analysis.py, data/processed/]
key-files:
created: [list of new files]
modified: [list of changed files]
deviations: {r1: 0, r2: 1, r3: 0, r4: 0}
---
One-liner: [SUBSTANTIVE summary — not "Task complete" but "Merged CRSP-Compustat panel with winsorized returns at 1%/99%"]
Changes: [what was added/modified and why]
Output: [output files produced and their contents]
One-liner rule: Must be SUBSTANTIVE. Good: "Panel regression with firm and year FE, clustered SEs, 3 robustness checks". Bad: "Completed task 3".
Verification Patterns
See references/verification-patterns.md for detailed code patterns for:
- Data loading, filtering, merging
- Aggregation and model training
- Batch pipeline scale-up testing (submission, validation, cost extrapolation)
- Quick reference table by operation type
See references/etl-enforcement.md for ETL strategy enforcement:
- Filter push-down (database vs application vs hybrid)
- Parallelism (Task agents vs SGE vs sequential)
- Intermediate caching (parquet vs CSV vs SQLite)
- Scale-up testing domain routing
Scale-Up Testing Protocol (Batch/ETL Operations)
Triggers when PLAN.md includes a Scale-Up Testing Plan table. NO FULL BATCH WITHOUT A SUCCESSFUL TEST BATCH. This is not negotiable.
Three stages: Test (~10 items, always required) -> Intermediate (~100, if total >500) -> Large (~1,000, if total >5,000). Each stage has quality gates that must pass before scaling up.
Full protocol, rationalization table, and red flags: See references/scale-up-testing.md
Common Failures to Avoid
| Failure | Why It Happens | Prevention |
|---|---|---|
| Silent data loss | Merge drops rows | Print row counts before/after |
| Hidden nulls | Join introduces nulls | Check null counts after joins |
| Wrong aggregation | Groupby logic error | Display sample groups |
| Type coercion | Pandas silent conversion | Verify dtypes after load |
| Off-by-one | Date filtering edge cases | Print min/max dates |
If Output Looks Wrong
- STOP - do not proceed
- Investigate - print more details
- Document - log the issue in LEARNINGS.md
- Ask - if unclear, ask user for guidance
- Fix - only proceed after output verified
Never hide failures. Bad output documented is better than silent failure.
No Pause Between Tasks
| Thought | Reality |
|---|---|
| "Task done, should check in with user" | You're wasting context. User wants ALL tasks done. Keep going. |
| "User might want to see intermediate results" | You're assuming wrong. User will see results at the END. Continue. |
| "Natural pause point" | You're making excuses. Only pause when ALL tasks complete or you're blocked. |
| "Should summarize this step" | You're procrastinating. Summarize AFTER all tasks. Keep moving. |
Your pausing between tasks is procrastination disguised as courtesy. </EXTREMELY-IMPORTANT>
Deviation Rules
When subagents encounter unplanned issues during implementation, follow this 4-rule system:
| Rule | Trigger | Action | Permission |
|---|---|---|---|
| R1: Bug | Data integrity bugs, wrong joins, type errors, off-by-one in date ranges, NaN propagation, index alignment errors | Fix → verify output with output-first protocol → track [Rule 1 - Bug] |
Auto |
| R2: Missing Critical | Missing null handling, no dedup check after merge, missing row count verification, no dtype validation, missing outlier handling | Add → verify → track [Rule 2 - Missing Critical] |
Auto |
| R3: Blocking | Missing dependency/package, wrong file path, data file unavailable, API rate limit, memory error on large data | Fix blocker → verify proceeds → track [Rule 3 - Blocking] |
Auto |
| R4a: Data Assumption | Data doesn't match expected shape/schema/distribution — expected panel but got cross-section, unexpected nulls in key column, different date range than specified, unexpected categories | STOP → present finding with evidence → track [Rule 4a - Data Assumption] |
Ask user |
| R4b: Methodology Change | Analysis approach needs changing — different model needed, different sample definition, different variable construction, need to add/remove control variables | STOP → present decision with alternatives → track [Rule 4b - Methodology] |
Ask user |
Priority: R4a/R4b (STOP) > R1-R3 (auto) > unsure → escalate as R4.
Edge cases:
- Unexpected nulls in non-key column → R2 (add handling)
- Unexpected nulls in key/ID column → R4a (data assumption violated)
- Package version mismatch → R3 (blocking)
- Need different statistical test → R4b (methodology change)
- Wrong merge type (left vs inner) → R1 (bug)
- Data has different granularity than expected → R4a (assumption)
Tracking format per task:
Each task summary in .planning/LEARNINGS.md should end with:
Deviations: N auto-fixed (R1: X, R2: Y, R3: Z). R4 escalations: [list or "none"].
Agent Team Implementation (Parallel)
Full protocol: See references/agent-team-protocol.md for prerequisites, spawn prompt template, lead monitoring, reconciliation (3 passes), and usage guidelines.
Key points:
- Run foundation tasks sequentially FIRST, then spawn parallel teammates
- Each teammate gets exclusive data scope and output files
- After all teammates complete, lead performs 3 reconciliation passes (Collect, Verify, Methodology)
- Default is sequential. Only use agent teams for 4+ tasks with true independence (different datasets/subsets, no shared output files)
Gate: Exit Implementation
Checkpoint type: human-verify (all tasks pass — machine-verifiable)
Before proceeding to validation, execute this gate:
- IDENTIFY: Read
.planning/PLAN.md— list every task by number and name - RUN: Read
.planning/LEARNINGS.md— find entries for each task - READ: For each task, confirm LEARNINGS.md contains:
- A "Task N: [Name] - COMPLETE" entry
- Verified output (shape, stats, or sample)
- No unresolved issues flagged
- VERIFY: Count tasks in PLAN.md vs completed entries in LEARNINGS.md. They MUST match.
Staleness Check: LEARNINGS.md must be updated in THIS session, not reused from prior work.
- Does each task entry reference current outputs (file paths, cell numbers)?
- If LEARNINGS.md is stale from a prior session, UPDATE it with fresh entries before claiming completion.
Stale LEARNINGS.md = false gate pass = unverified work = the user gets results no one actually checked.
- CLAIM: Only if all tasks accounted for, proceed to review
If ANY task is missing from LEARNINGS.md, implement it before proceeding.
Claiming all tasks are done without checking LEARNINGS.md against PLAN.md is NOT HELPFUL — missing tasks mean incomplete analysis the user relies on. </EXTREMELY-IMPORTANT>
Phase Complete
After passing the exit gate, IMMEDIATELY discover and read the validation phase:
Read ${CLAUDE_SKILL_DIR}/../../skills/ds-validate/SKILL.md and follow its instructions. Follow its instructions to validate outputs before review.
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
audit-fix-loop
This skill should be used when the user asks to 'iteratively improve', 'audit and fix', 'hill-climb quality', 'grade and improve', 'score and fix', 'audit loop', 'quality loop', or needs structured iterative improvement of an artifact using scored independent audits. Also use when the user invokes a ralph loop for quality improvement rather than task completion.
ds-spec-reviewer
Internal skill used by ds-brainstorm at Phase 1 exit gate. Dispatches a reviewer subagent to verify SPEC.md completeness before planning. NOT user-facing.
pptx-render
Use when the user asks to "render pptx", "show pptx slide", "compare with pptx", "pptx to image", "export pptx slide", "original slide", "show me the original", "what does the pptx look like", or needs to extract a specific PPTX slide's content for visual comparison.
obsidian-organize
Organize Obsidian notes according to clawd's preferences. Use when user asks to "organize notes", "move notes to right folder", "clean up vault", "tidy vault", "file this note", or when creating new notes in the Obsidian vault. Also use when moving, renaming, or categorizing notes, or when the vault root has stray files.
dev-verify
This skill should be used when the user asks to 'verify completion', 'check that tests pass', 'confirm feature works', or REQUIRED Phase 7 of /dev workflow (final). Enforces fresh runtime evidence before claiming completion.
dev
This skill should be used when the user asks to 'start a feature', 'build a feature', 'implement a feature', 'develop', 'new feature', or needs the full 7-phase development workflow with TDD enforcement.
Didn't find tool you were looking for?