Skip to content

feat(skills): add compound-engineering implementation reference#1

Merged
aegntic merged 1 commit into
mainfrom
feat/compound-implementation-reference
Jun 5, 2026
Merged

feat(skills): add compound-engineering implementation reference#1
aegntic merged 1 commit into
mainfrom
feat/compound-implementation-reference

Conversation

@aegntic
Copy link
Copy Markdown
Owner

@aegntic aegntic commented Jun 5, 2026

Adds skills/compound-engineering with references for recurring stack-specific breakages so workflow commands can consume fix recipes directly.

What's new

  • skills/compound-engineering/SKILL.md — routing index mapping workflow phases to reference files
  • references/wagmi-v7-migration.md — v6→v7 migration patterns, ABI tuples, balance APIs, viem v2 rewrite
  • references/polymarket-auto-2026-05-09.md — 60+ TS fixes, runtime crash resolutions, production verification logs

Why
Local Hermes compound-engineering skill had actionable stack recipes that weren't surfaced into the workflow repo. This bridges commands/workflows/* execution with the implementation details those workflows actually hit.

Co-authored-by: Hermes hermes@nousresearch.com

Summary by Sourcery

Add a compound-engineering skill that documents stack-specific fix recipes and session notes for recurring frontend and wagmi/viem issues.

Documentation:

  • Document wagmi v6→v7 migration patterns, viem v2 contract read/write changes, and common TypeScript/type mismatch pitfalls.
  • Add session notes for the Polymarket Auto dashboard capturing resolved TS errors, runtime crashes, and verification steps.
  • Introduce a SKILL index for the compound-engineering implementation reference, listing available reference documents and usage guidance.

Summary by CodeRabbit

  • Documentation
    • Added compound engineering skills reference guide for implementation patterns and workflows
    • Added Polymarket Auto Dashboard session notes with completed fixes and verification results
    • Added migration guide with code examples and troubleshooting patterns for library updates

Adds skills/compound-engineering with references for recurring
stack-specific breakages so workflow commands can consume fix
recipes directly.

Co-authored-by: Hermes <hermes@nousresearch.com>
Copilot AI review requested due to automatic review settings June 5, 2026 17:02
@sourcery-ai
Copy link
Copy Markdown

sourcery-ai Bot commented Jun 5, 2026

Reviewer's Guide

Adds a new compound-engineering skill with an implementation reference index and detailed markdown references for wagmi v7 migration and a specific Polymarket dashboard repair session, so workflow commands can consume concrete stack-specific fix recipes.

File-Level Changes

Change Details Files
Introduce compound-engineering skill metadata and routing index for implementation references.
  • Define skill metadata (name, description, version, tags, triggers, model) in frontmatter.
  • Document how workflows should invoke this skill from existing commands.
  • List and categorize all reference markdown files that the skill exposes.
  • Add maintenance guidance for adding/removing reference documents.
skills/compound-engineering/SKILL.md
Add wagmi v6→v7 migration reference documenting breaking changes and preferred fix patterns.
  • Describe new patterns for balance fetching using useReadContract and optional removal of balance display.
  • Document migration from contract.write and viem getContract patterns to readContract/writeContract with separate clients.
  • Capture stricter ABI parsing rules and tuple handling in viem v2.
  • List common TypeScript error patterns and batch-fix workflow for large migrations.
  • Record session-specific fixes applied in the Polymarket Auto repo related to wagmi/viem changes and runtime issues.
skills/compound-engineering/references/wagmi-v7-migration.md
Add session-specific Polymarket Auto dashboard repair log as a reusable reference.
  • Document project context, stack, and risk profile for the Polymarket Auto dashboard.
  • Summarize how 60+ TypeScript errors were triaged and resolved via pattern-based batching.
  • Detail fixes for trading and risk/strategy tab runtime crashes, including ABI and response-shape issues.
  • Record partial wagmi v6→v7 migration decisions (e.g., removing useBalance) and verification steps for builds and API routes.
skills/compound-engineering/references/polymarket-auto-2026-05-09.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@ecc-tools
Copy link
Copy Markdown

ecc-tools Bot commented Jun 5, 2026

Analyzing 200 commits...

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 5, 2026

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR introduces a new documentation skill system for compound engineering workflows. It adds a skill definition file that references two implementation guides: a comprehensive wagmi v6→v7 and viem v1→v2 migration reference with code patterns and troubleshooting, and session notes documenting fixes applied to the Polymarket Auto Dashboard.

Changes

Compound Engineering Skill Documentation

Layer / File(s) Summary
Skill definition and reference index
skills/compound-engineering/SKILL.md
Skill metadata (name, description, version, tags) and trigger types defined; lists available reference documents consumed by workflow steps, with maintenance guidance for evolving patterns.
Wagmi v6 to v7 migration reference
skills/compound-engineering/references/wagmi-v7-migration.md
Breaking changes documented: useBalanceuseReadContract for ERC20 tokens, contract.write.*writeContract for writes, viem parseAbi tuple handling, and viem v1 → v2 migration from getContract to direct publicClient.readContract / walletClient.writeContract calls. Includes TypeScript error checklist, batch repair workflow using grep and tsc, and session-specific fix patterns.
Polymarket Auto Dashboard session notes
skills/compound-engineering/references/polymarket-auto-2026-05-09.md
Session context and checklist of fixes: TypeScript compile error triage, trading tab ABI/runtime crash remediation via viem v2, risk/strategy tab performance normalization, partial wagmi balance migration. Verification confirms successful build and zero TypeScript errors.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 New skills bloom in careful prose,
Migration maps where chaos flows,
From wagmi six to seven's shores,
Reference guides unlock the doors—
A rabbit's gift of patterns pure. 🌟

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a new compound-engineering skill with implementation reference materials to the repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/compound-implementation-reference

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ecc-tools
Copy link
Copy Markdown

ecc-tools Bot commented Jun 5, 2026

Analysis Complete

Generated ECC bundle from 1 commits | Confidence: 50%

View Pull Request #2

Repository Profile
Attribute Value
Language TypeScript
Framework Not detected
Commit Convention conventional
Test Directory separate
Changed Files (3)
Metric Value
Files changed 3
Additions 217
Deletions 0

Top hotspots

Path Status +/-
skills/compound-engineering/references/wagmi-v7-migration.md added +145 / -0
skills/compound-engineering/SKILL.md added +40 / -0
skills/compound-engineering/references/polymarket-auto-2026-05-09.md added +32 / -0

Top directories

Directory Files Total changes
skills/compound-engineering/references 2 177
skills/compound-engineering 1 40
Analysis Depth Readiness (commit-history, 7%)

ECC Tools uses this to decide whether recommendations should stay at commit-history/setup guidance or expand into CI, security, harness, reference-set, AI-routing, and team backlog work.

Area Status Evidence / Next Step
Commit history Partial 1 commits sampled
CI/CD signals Missing Add workflow files or CI troubleshooting evidence so ECC Tools can reason about pipeline setup.
Security evidence Missing Add AgentShield, audit, SARIF, SBOM, or security review evidence so recommendations can cover security posture.
Harness configuration Missing Add Claude, Codex, OpenCode, Zed, dmux, MCP, plugin, or cross-harness config evidence for harness-agnostic recommendations.
Reference/eval evidence Missing Add fixtures, golden traces, reference sets, or evaluator benchmarks so deeper recommendations have regression evidence.
AI routing and cost controls Missing Add model-routing, budget, usage, or cost-control files before relying on AI-heavy automation recommendations.
Team handoff and project tracking Missing Add roadmap, runbook, project, Linear, or follow-up tracking docs so generated work can land in a team queue.
Reference Set Readiness (0/7, 0%)
Area Status Evidence / Next Step
Deep analyzer corpus Missing Add analyzer fixture, golden, benchmark, or reference-set files that can catch analyzer regressions.
RAG/evaluator comparison Missing Add retrieval or evaluator reference-set comparison fixtures with expected ranking behavior.
PR salvage/review corpus Missing Add stale-PR, review-thread, reopen-flow, or salvage reference cases for queue cleanup automation.
Discussion triage corpus Missing Add public discussion triage fixtures, golden cases, or reference sets for informational, answered, and no-response classifications.
Harness compatibility Missing Add cross-harness, adapter-compliance, or harness-audit evidence for Claude, Codex, OpenCode, Zed, dmux, and agent surfaces.
Security evidence Missing Attach security evidence such as SBOMs, SARIF, audit reports, or AgentShield evidence packs.
CI failure-mode evidence Missing Add captured CI failure logs, dry-run fixtures, or troubleshooting docs for common workflow failure modes.
Likely Future Issues (2)
Severity Signal Why it may show up
MEDIUM AI or harness analysis changes may ship without reference-set validation 3 reference-sensitive path(s) changed; 0 eval, golden trace, benchmark, or reference-set artifacts changed
MEDIUM Skill or agent guidance may ship without quality evidence 3 skill-quality path(s) changed; 0 examples, validator, eval, or reference artifacts changed
  • AI or harness analysis changes may ship without reference-set validation: The PR changes analyzer, skill, agent, command, or harness guidance without updating any obvious eval, golden trace, benchmark, or reference-set artifact.
  • Skill or agent guidance may ship without quality evidence: The PR changes skill, agent, command, or rule guidance without touching examples, validator, eval, or reference evidence.
Suggested Follow-up Work (2)
Type Suggested title Targets
PR analysis: add reference-set evidence for skills/compound-engineering/SKILL.md + skills/compound-engineering/references/polymarket-auto-2026-05-09.md skills/compound-engineering/SKILL.md, skills/compound-engineering/references/polymarket-auto-2026-05-09.md
PR skills: add quality evidence for skills/compound-engineering/SKILL.md + skills/compound-engineering/references/polymarket-auto-2026-05-09.md skills/compound-engineering/SKILL.md, skills/compound-engineering/references/polymarket-auto-2026-05-09.md
  • analysis: add reference-set evidence for skills/compound-engineering/SKILL.md + skills/compound-engineering/references/polymarket-auto-2026-05-09.md: Backfill eval, golden trace, benchmark, or reference-set evidence before another AI or harness-analysis change lands on the touched surface.
  • skills: add quality evidence for skills/compound-engineering/SKILL.md + skills/compound-engineering/references/polymarket-auto-2026-05-09.md: Backfill examples, validator, eval, or reference evidence before another skill or agent guidance change lands on the touched surface.

Copy-ready bodies

analysis: add reference-set evidence for skills/compound-engineering/SKILL.md + skills/compound-engineering/references/polymarket-auto-2026-05-09.md

## Summary
- Add reference-set or eval evidence for the recently changed AI, analyzer, skill, agent, command, or harness guidance surface.

## Why
- Backfill eval, golden trace, benchmark, or reference-set evidence before another AI or harness-analysis change lands on the touched surface.

## Touched paths
- `skills/compound-engineering/SKILL.md`
- `skills/compound-engineering/references/polymarket-auto-2026-05-09.md`

## Validation
- Add or update an eval, golden trace, benchmark, fixture, or reference-set artifact for the changed AI/harness behavior.
- Compare the changed behavior against the maintained reference set and record the pass/fail evidence.
- Confirm the follow-up evidence covers the same analyzer, skill, agent, command, or harness surface touched by this PR.

skills: add quality evidence for skills/compound-engineering/SKILL.md + skills/compound-engineering/references/polymarket-auto-2026-05-09.md

## Summary
- Add skill-quality evidence for the recently changed skill, agent, command, or rule guidance.

## Why
- Backfill examples, validator, eval, or reference evidence before another skill or agent guidance change lands on the touched surface.

## Touched paths
- `skills/compound-engineering/SKILL.md`
- `skills/compound-engineering/references/polymarket-auto-2026-05-09.md`

## Validation
- Add or update examples, trigger coverage, frontmatter validation, or skill validator fixtures for the changed guidance.
- Run the relevant skill, agent, command, or rule validation flow and record pass/fail evidence.
- Confirm the follow-up evidence covers the same guidance surface touched by this PR.
Generated Instincts (16)
Domain Count
git 4
code-style 9
testing 2
architecture 1

After merging, import with:

/instinct-import .claude/homunculus/instincts/inherited/compound-engineering-instincts.yaml

Files

  • .claude/ecc-tools.json
  • .claude/skills/compound-engineering/SKILL.md
  • .agents/skills/compound-engineering/SKILL.md
  • .agents/skills/compound-engineering/agents/openai.yaml
  • .claude/identity.json
  • .codex/config.toml
  • .codex/AGENTS.md
  • .codex/agents/explorer.toml
  • .codex/agents/reviewer.toml
  • .codex/agents/docs-researcher.toml
  • .claude/homunculus/instincts/inherited/compound-engineering-instincts.yaml

ECC Tools | Everything Claude Code

@aegntic aegntic merged commit 0215b3c into main Jun 5, 2026
2 of 3 checks passed
Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've found 2 issues

Prompt for AI Agents
Please address the comments from this code review:

## Individual Comments

### Comment 1
<location path="skills/compound-engineering/SKILL.md" line_range="3" />
<code_context>
+---
+name: compound-engineering
+description: Implementation reference library for compound engineering workflows. For workflow orchestration, use slash commands under `commands/workflows/` (constitution, plan, work, review, compound). This skill provides stack-specific fix recipes those workflows consume.
+version: 1.0.0
+tags: [reference, fixes, typescript, react, nextjs, wagmi, automation]
</code_context>
<issue_to_address>
**suggestion (typo):** Consider inserting 'that' for smoother grammar in the final sentence of the description.

The clause "stack-specific fix recipes those workflows consume" is grammatically awkward. Please add "that" after "recipes" to read "stack-specific fix recipes that those workflows consume" for clearer documentation.

```suggestion
description: Implementation reference library for compound engineering workflows. For workflow orchestration, use slash commands under `commands/workflows/` (constitution, plan, work, review, compound). This skill provides stack-specific fix recipes that those workflows consume.
```
</issue_to_address>

### Comment 2
<location path="skills/compound-engineering/SKILL.md" line_range="17" />
<code_context>
+
+# Compound Engineering — Implementation Reference
+
+Actionable fix recipes for recurring breakages. Use alongside `aegntic/compound-engineering` workflow commands.
+
+## Usage
</code_context>
<issue_to_address>
**issue (typo):** The identifier `aegntic/compound-engineering` likely contains a typo.

If this is meant to reference the repository, please change it to `agentic/compound-engineering` to avoid confusion.

```suggestion
Actionable fix recipes for recurring breakages. Use alongside `agentic/compound-engineering` workflow commands.
```
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@@ -0,0 +1,40 @@
---
name: compound-engineering
description: Implementation reference library for compound engineering workflows. For workflow orchestration, use slash commands under `commands/workflows/` (constitution, plan, work, review, compound). This skill provides stack-specific fix recipes those workflows consume.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

suggestion (typo): Consider inserting 'that' for smoother grammar in the final sentence of the description.

The clause "stack-specific fix recipes those workflows consume" is grammatically awkward. Please add "that" after "recipes" to read "stack-specific fix recipes that those workflows consume" for clearer documentation.

Suggested change
description: Implementation reference library for compound engineering workflows. For workflow orchestration, use slash commands under `commands/workflows/` (constitution, plan, work, review, compound). This skill provides stack-specific fix recipes those workflows consume.
description: Implementation reference library for compound engineering workflows. For workflow orchestration, use slash commands under `commands/workflows/` (constitution, plan, work, review, compound). This skill provides stack-specific fix recipes that those workflows consume.


# Compound Engineering — Implementation Reference

Actionable fix recipes for recurring breakages. Use alongside `aegntic/compound-engineering` workflow commands.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

issue (typo): The identifier aegntic/compound-engineering likely contains a typo.

If this is meant to reference the repository, please change it to agentic/compound-engineering to avoid confusion.

Suggested change
Actionable fix recipes for recurring breakages. Use alongside `aegntic/compound-engineering` workflow commands.
Actionable fix recipes for recurring breakages. Use alongside `agentic/compound-engineering` workflow commands.

@aegntic aegntic review requested due to automatic review settings June 5, 2026 17:26
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.

1 participant