Loading...
Loading...
Get independent model opinions on an AI-proposed plan, fix, architecture, or idea.
npx skill4agent add adird/agent-shell-hamelech melech-consultmelech-consultmelech-consultmelech-consult| Consultation Format | For Implementation & Architecture Proposals | For Product & Idea Proposals |
|---|---|---|
| Double-Check / Second Opinion (Default) | Proofs a proposed code diff, refactor, DB schema, or bug fix for subtle flaws, race conditions, or missing edge cases. | Sanity-checks a proposed feature concept, workflow, or positioning against market reality. |
| Peer Review | Independent staff-engineer critique of a written RFC, plan, or API design before implementation. | Independent product-manager critique of a PRD, user journey, or pitch. |
| Expert Council | Triangulates the proposal across 2–3 competing technical lenses (e.g. Pragmatist / 80-20 vs. Scale / Concurrency vs. Security / Blast Radius). | Triangulates the proposal across 2–3 business lenses (e.g. User Value & WTP vs. Growth & Distribution vs. MVP Feasibility). |
| Devil's Advocate (Red Team) | Explicit mandate to break the proposed code: find why it will fail in production, deadlock, or degrade performance. | Explicit mandate to shoot down the idea: why users won't switch, why incumbents win, and flawed assumptions. |
invoke_subagentRoleModelinherit**Context**: AI proposed adding an in-memory cache with TTL for user permissions.
**User**: "Double-check this with another model before we write code."
**Consultant Prompt**:
You are a Staff Systems Engineer. Audit this proposed permission-caching design:
Proposal: In-memory cache with 60s TTL on the API gateway.
Constraints: Multi-tenant SaaS, instantaneous permission revocation required for offboarded admins.
Task: Find where this breaks, evaluate cache invalidation complexity, and recommend the soundest fix.
**Synthesized Verdict**:
- **Landmine Exposed**: 60s TTL violates the hard requirement of instantaneous revocation for security offboarding.
- **Consultant Recommendation**: Use Redis Pub/Sub invalidation events or check revocation status via Redis bitmap rather than a blind TTL.
- **Adjusted Plan**: Switch from pure TTL to a lightweight revocation check endpoint.**Context**: AI pitched building a custom automated visual diffing engine for PRs.
**User**: "Convene a council to proof this idea."
**Council Dispatched** (3 roles, 3 distinct models, 3 different providers; the main thread's model excluded):
1. *Pragmatist / 80-20 Lead* — provider A, deep-reasoning tier (Focus: build vs. buy, maintenance burden)
2. *Developer Experience Reviewer* — provider B (Focus: workflow friction, false-positive noise)
3. *Technical Architect* — provider C (Focus: headless browser rendering cost, CI latency)
**Synthesized Verdict**:
- **Consensus**: All 3 agree that building a custom diff engine from scratch has high maintenance overhead and false-positive flake.
- **Clash**: DX wants rich inline PR comments; Pragmatist recommends integrating existing GitHub Actions / Percy CLI instead of building a service.
- **Adjusted Plan**: Don't build a custom engine; write a 20-line GitHub Action wrapper around an existing open-source visual regression tool.melech-consultrole → modelinherit