* improve: enhance data-scientist based on automated review
- Replace pinned model claude-sonnet-4-5 with alias sonnet
- Add WebSearch to tools (needed for literature review/methodology selection tasks already in the prompt)
- Name concrete libraries: Fairlearn/AIF360 (bias/fairness), SHAP/LIME (interpretation), DoWhy/EconML/CausalML (causal inference), Optuna/Ray Tune/Hyperopt (hyperparameter tuning), DVC (versioning)
- Tighten data science checklist to measurable/testable criteria instead of vague adverbial items
- Upgrade pre-analysis clarifying questions into a structured, numbered gate that blocks proposing a model/methodology until key unknowns are resolved
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: repair double-escaped newlines in data-scientist description
Pre-existing bug where the description field used \\n instead of \n,
rendering as literal backslash-n text in the agent picker/dashboard.
Also tightens grammar in the pre-analysis gate added in the prior commit.
* fix: scope modeling requirements to modeling tasks in data-scientist gate
Addresses Greptile review feedback: the clarification gate and checklist
required a success-metric threshold and held-out-test/cross-validation
steps even for pure exploratory profiling, visualization, and clustering
work the agent explicitly supports.
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance droid agent based on automated review
- Fix frontmatter to use valid Claude Code subagent conventions (tools: Read, Grep, Glob, Bash, WebFetch, WebSearch instead of GitHub Copilot aliases; model: sonnet instead of pinned dated ID)
- Remove GitHub Copilot Integration section (contradicted actual .claude/agents/ install path) and replace with a Claude Code usage section
- Fix outdated --enabled-tools flag to current --restrict-tools/--additional-tools/--disabled-tools
- Remove stale hardcoded model IDs from droid exec --model examples, point to docs.factory.ai/models instead
- Add missing droid exec flags: --fork, -w/--worktree, --use-spec, -r/--reasoning-effort
- Add <example>/<commentary> blocks to the description frontmatter matching sibling agent conventions
- Clarify autonomy tier boundaries (medium = no push, high = push/deploy with safety checks)
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: remove dangling model-ID hint from GitHub Actions example
The parenthetical referenced a --model flag that was removed from
the command in an earlier edit, leaving a confusing dangling hint.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B3THSCtx7huxkRFmmUXne7
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance llm-architect based on automated review
- Add Glob and Grep to tools list (parity with sibling agents)
- Add SGLang to serving framework selection; remove stale vLLM version gates
- Add Anthropic Contextual Retrieval technique and Voyage AI embedding option
- Add GraphRAG/agentic retrieval escalation callout for multi-hop queries
- Add Unsloth as fine-tuning library option for LoRA/QLoRA
- Add ORPO and KTO preference-alignment methods to method-selection table
- Add "verify current model IDs" disclaimer to multi-model routing section
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: clarify ORPO vs DPO/KTO selection criterion in llm-architect
Both rows now key off dataset format (paired vs unpaired preferences)
instead of mixing in GPU count, so ORPO no longer overlaps with the
adjacent DPO/KTO rows.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Eckv6ssk3XuRXpbAzCxPgA
* fix: split DPO and GRPO into separate rows with correct prerequisites
GRPO is a reward/verifier-driven RL method, not a paired-preference
method — the previous merge wrongly implied it needs paired preference
data and a prior SFT checkpoint like DPO does.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Eckv6ssk3XuRXpbAzCxPgA
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance ai-engineer based on automated review
- Reposition scope as generalist for both classical ML and generative-AI/LLM engineering, with explicit handoffs to llm-architect, ml-engineer/machine-learning-engineer, and prompt-engineer
- Add Generative AI / LLM Engineering section (LLM API providers, RAG basics, agentic patterns, fine-tuning, evaluation frameworks)
- Add model: sonnet and WebSearch tool to frontmatter
- Replace fictional context-manager protocol with a concrete Requirements Gathering step
- Replace vague adjective checklists with measurable criteria and named tools/thresholds
- Sharpen frontmatter description examples with concrete frameworks and a boundary-clarifying example
- Disambiguate the "prompt-engineer" cross-reference (ai-specialists vs data-ai)
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: remove stray content marker, make requirements gathering conditional
Addresses Greptile review feedback on PR #775: an unmatched </content>
marker leaked into the installed prompt, and the requirements-gathering
step re-asked for details the user had already supplied.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QS1SncnafSWXMRXQa1yVBA
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance ai-ethics-advisor based on automated review
- Fix malformed YAML frontmatter: description examples were bare unindented
text outside the description field, breaking YAML parsing / delegation
matching. Merged into a single quoted description scalar per sibling
convention.
- Add missing model: sonnet field, consistent with other ai-specialists agents
- Add NIST AI 600-1 (Generative AI Profile) subsection with the 12 GenAI risk
categories
- Add EU AI Act compliance-timeline caveat with current/pending deadlines
- Cite OWASP Top 10 for LLM Applications (LLM01, LLM06) in Agentic AI Ethics
- Add LLM/Generative System Explainability subsection (chain-of-thought
faithfulness caveat, citation-grounded transparency)
- Add Aequitas toolkit, 80% disparate-impact rule, and Bash least-privilege
clarification
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* chore: sync generated content for ai-ethics-advisor
Regenerate only the per-component content artifact for
ai-ethics-advisor so the dashboard detail view reflects the updated
frontmatter/body. Full catalog regeneration (docs/components.json,
counts, search-index, download stats) is left to the scheduled
update-json-data workflow, since a local run here pulled stale/wrong
download counts that would have clobbered production data.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018wpuLQJaC6qskfDwUSioLF
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance product-manager based on automated review
- Remove fabricated JSON "protocol" blocks and invented metrics (fake agent-to-agent JSON requests, +$4.2M/84% satisfaction canned numbers) that don't map to real Claude Code mechanics and risk priming fabricated outputs
- Add model: sonnet to frontmatter (required field for agents)
- Replace ~20 flat keyword-list sections with concrete worked frameworks: RICE scoring formula, OKR worksheet, discovery-to-launch checklist with exit criteria, one-page PRD template, and stakeholder communication cadence
- Add explicit "How This Differs From Related Agents" section disambiguating from product-strategist (market/positioning strategy) and se-product-manager-advisor (GitHub issue creation), mirrored in description examples
- Fix cross-reference from "customer-success" to the actual component name "customer-success-manager"
- Add explicit Write/Edit output guidance (PRD/roadmap markdown documents) and anti-fabrication guidance for progress reporting
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: restore component download counts reset by catalog regeneration
The prior commit regenerated the catalog without Supabase credentials,
resetting download counts to 0 for ~1800 unrelated components. Restore
each file's original download counts from main while keeping the
product-manager description update.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012oa4wnU6RLfRH4gFWLFumC
* fix: soften product-manager/product-strategist disambiguation
product-strategist's own description explicitly covers feature
prioritization and roadmap planning, so the prior wording (claiming it
handles only market/positioning docs) contradicted the referenced
agent. Acknowledge the overlap instead, per cubic review feedback.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012oa4wnU6RLfRH4gFWLFumC
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: rename risk-manager to trading-risk-manager, add model field
- Resolve name collision with finance/risk-manager.md by renaming this
trading/portfolio-focused agent to trading-risk-manager (frontmatter
name + filename), sharpening the description to distinguish it from
the enterprise ERM-focused finance/risk-manager
- Add required model: sonnet field (was missing, failed component-reviewer
checklist)
- Add "When Invoked" context-gathering step and Human-in-the-Loop Pause
Criteria (account size, risk tolerance, leverage/derivatives, full-Kelly
requests)
- Add financial-advice disclaimer as a required Output element
- Add <example> blocks to description for reliable auto-invocation
- Recommend fractional (half/quarter) Kelly over full Kelly, and add
tail-risk hedging / leverage-liquidation risk / Sharpe-Sortino-Calmar
focus areas
- Add "Integration with Other Agents" section
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* chore: regenerate component catalog for trading-risk-manager rename
Propagates the risk-manager -> trading-risk-manager rename (and the
model/content updates) into the generated catalog artifacts so
browse/search/install no longer reference the deleted
business-marketing/risk-manager.md path. Also removes the now-orphaned
component-content JSON for the deleted path.
Addresses review feedback on PR #765.
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance backend-developer based on automated review
- Replace fictional context-manager JSON RPC messaging with concrete
Glob/Grep/Read-based discovery steps (matches api-designer's pattern)
- Update runtime baselines: Node.js 22+/24+, Python 3.12+ (3.13/3.14
fluency), Go 1.24+
- Reference OWASP API Security Top 10 (2023) explicitly, naming BOLA
and Broken Object Property Level Authorization, plus secret
management and short-lived token/MFA guidance
- Cross-reference backend-architect for upfront design decisions and
add it to Integration with other agents
- Add model: sonnet and color: green to frontmatter
- Add "Use PROACTIVELY" framing to the description
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: address PR review feedback on backend-developer agent
Make manifest discovery Glob patterns recursive (**/package.json etc.)
so nested services in monorepos are found, and reword the two
"service boundary" checklist bullets so they consume/confirm
backend-architect's design instead of contradicting the new
architecture handoff note.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019vHEpW64LEdeoV7jhZGRPR
* fix: remove remaining architecture-ownership claims from description examples
The <example> blocks in the frontmatter description still had
backend-developer "architect this service decomposition" and "design
service boundaries", contradicting the handoff to backend-architect
introduced earlier in this PR. Reworded both examples and the
"full stack from architecture decisions" commentary line so the
description is consistent with the body's ownership split.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019vHEpW64LEdeoV7jhZGRPR
* fix: make remaining discovery glob patterns recursive
openapi.yaml and docker-compose.yml were the only patterns in the
Discovery Protocol Glob step without the **/ prefix, inconsistent
with the monorepo-nested-service rationale applied to the other
patterns.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019vHEpW64LEdeoV7jhZGRPR
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance market-researcher based on automated review
- Add missing model: sonnet frontmatter field
- Forbid fabricated statistics in progress tracking and delivery notification (no more baked-in sample figures)
- Replace fictional "context manager" protocol with direct user-scoping questions
- Add Human-in-the-Loop Pause Criteria for ambiguous scope, conflicting data, unsourced estimates, and single-source claims
- Add Ethical & Legal Boundaries section (public sources only, no paywalled/login-gated access, cite sources and as-of dates, no implied primary research)
- Consolidate flat keyword lists into a "Core Practices" section and merge four redundant checklists into one Excellence checklist
- Add data-freshness guidance for market-size/growth/share figures
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: address greptile review feedback on market-researcher
- Only ask clarifying questions when the user hasn't already provided
target market/objective/scope, avoiding redundant interruptions on
fully-specified single-turn requests
- Don't pause for confirmation on every single-source claim (common
in market research); flag them instead per Ethical & Legal Boundaries
Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: make research-planning confirmation conditional too
The Research Planning step still unconditionally required confirming
scope/market/objectives with the user, reintroducing the redundant-
interaction issue already fixed in the When Invoked step. Now only
confirms when something is still missing or ambiguous.
Co-Authored-By: Claude Code <noreply@anthropic.com>
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance legal-advisor based on automated review
- Add required model field (sonnet)
- Add Human-in-the-Loop Pause Criteria for unconfirmed jurisdiction/facts
- Rewrite description with scenario-driven examples for auto-delegation
- Add When Invoked section to gather jurisdiction/data facts before drafting
- Add WebFetch tool and expand Key Regulations for 2026 (EU AI Act, US state laws, DPDPA)
- Move disclaimer into Output as a required document element
- Add Integration with Other Agents section
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: correct double-escaped newlines in legal-advisor description
The description field used \\n (literal backslash-n) instead of \n
(YAML newline escape), which rendered the <example>/<commentary>
blocks as a single wall of text.
Co-Authored-By: Claude Code <noreply@anthropic.com>
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance react-specialist based on automated review
- Update React version framing to React 19+; add React Compiler and
React 19 Actions/hooks guidance (use(), useActionState, useOptimistic,
useFormStatus)
- Remove Recoil (archived by Meta, unmaintained); add prioritized state
management guidance (TanStack Query, Zustand, Jotai, Redux Toolkit)
- Modernize testing stack: Vitest as default runner, MSW for mocking,
Playwright for E2E
- Add Vite/Next.js/Remix tooling guidance, note CRA deprecation
- Add concrete TypeScript compiler flags and WCAG 2.2 AA accessibility
criteria with tooling
- Trim redundant filler (duplicated Portals/error boundary/Suspense
bullets) and update ecosystem list (shadcn/ui, Radix UI)
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: correct WCAG 2.4.11 criterion citation in react-specialist
2.4.11 is "Focus Not Obscured (Minimum)" (Level AA), not "Focus
Appearance" (that's 2.4.13, Level AAA). The mislabeled guidance would
have led the agent to check focus indicator styling instead of
whether focused elements are hidden by overlapping content, while
still reporting AA compliance.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JbrpDkpsMuh5ckdeWiuqCU
---------
Co-authored-by: Claude <noreply@anthropic.com>
- Add missing model: sonnet field to frontmatter
- Remove fictional "context manager" / Communication Protocol JSON scaffolding, replaced with a direct ask-user-first invocation flow
- Replace fabricated example telemetry (invented revenue/NPS numbers) with instructions to report only actual confirmed session findings
- Add Human-in-the-Loop Pause Criteria section for health/churn classification, renewal/pricing decisions, and financial interventions
- Reframe the success checklist as illustrative reference points rather than assumed-achieved facts
- Note the industry shift toward ML-based/real-time health scoring and NRR as a headline metric
- Add a Source Boundaries section governing WebSearch/WebFetch use (public sources only, no pretexting, respect robots.txt, cite sources)
- Narrow tools list by dropping Edit (Read, Write, Glob, Grep, WebFetch, WebSearch)
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance marketing-attribution-analyst based on automated review
- Add required model field and rewrite description with Use-when framing
plus 3 <example> blocks that disambiguate from content-marketer,
business-analyst, and market-researcher
- Fix tools to match sibling analyst agents (drop Bash, add Glob/WebFetch/
WebSearch) since this is a research/analysis role, not a shell-execution one
- Add a "When Invoked" context-gathering step so the agent asks for the
user's real tracking stack, attribution window, and data sources before
producing any numbers
- Add a Measurement Strategy Framework (MMM + incrementality + MTA
triangulation) with spend-tier guidance
- Update GA4 attribution model guidance (Data-Driven Attribution is now the
default/only selectable model in most reports since 2023)
- Update tracking infrastructure guidance for Consent Mode v2, server-side
tagging, and Enhanced Conversions/server-side conversion APIs
- Correct cookie/privacy framing (Chrome reversed third-party cookie
deprecation; Privacy Sandbox trials shut down Oct 2025; Safari/Firefox
still block by default)
- Reference Google Meridian and Meta's Robyn as current standard MMM tools
- Expand the single-line Privacy Compliance bullet into concrete guidance
(Consent Mode v2 signals, data clean rooms, first-party data strategy)
- Add data-provenance caveats to report/dashboard templates so figures are
never fabricated
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: address Greptile review findings on attribution analyst
- Scope Consent Mode v2 to applicable Google Ads/Analytics features
rather than claiming it's mandatory for every EEA-facing site
- Add explicit consent-state/regional-rules boundary to Enhanced
Conversions / server-side conversion API guidance
- Soften spend-tier MMM guidance to depend on data history/variance
rather than spend level alone
- Define the previously-missing apply_saturation() helper used by
the MMM code example
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNgB3pCBw9SaHktzQWoFc
* fix: address second round of Greptile findings
- Ask for data-history length and channel spend variance during
intake, not just spend level, so MMM fitness can be assessed
before it's recommended
- Scope the Data Quality Monitoring consent checklist to applicable
Google Ads/Analytics features and include the ad_storage signal
- Guard apply_saturation() against zero-spend channels (max=0) to
avoid producing NaN features
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNgB3pCBw9SaHktzQWoFc
* fix: address cubic review findings on attribution analyst
- Correct GA4 attribution note to list the three current selectable
reports (Data-Driven Attribution, Paid and organic last click,
Google paid channels last click) instead of implying DDA is the
only selectable model
- Distinguish Google Meridian (genuinely Bayesian, posterior credible
intervals) from Meta's Robyn (ridge regression, bootstrapped
confidence intervals) in both the prose and the matching code
comment, which previously still lumped both under "Bayesian"
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNgB3pCBw9SaHktzQWoFc
* fix: close remaining checklist gaps flagged by Greptile
- Require confirmed data fitness (not spend tier alone) before the
Analysis Framework checklist marks MMM as the matched approach
- Require consent-state propagation and suppression of ineligible
events/identifiers as part of the conversion-API checklist item,
not just "configured where applicable"
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNgB3pCBw9SaHktzQWoFc
* fix: guard MMM example against missing spend observations
calculate_adstock() carries values forward recursively, so a single
NaN in a spend column poisons every subsequent period's adstock
value; since that's a plain numpy array (not a pandas Series), the
existing zero-guard in apply_saturation() doesn't catch it because
max() on an array containing NaN returns NaN, not 0. Fill missing
spend values before computing adstock so the illustrative pipeline
runs on real-world data with gaps.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNgB3pCBw9SaHktzQWoFc
* fix: stop silently imputing missing spend as zero in MMM example
Blanket fillna(0) conflated genuinely missing observations with
confirmed zero spend, which can materially bias adstock/attribution
for the affected channel. Fail loudly instead and require the caller
to make an explicit, documented choice per Greptile/cubic feedback.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TXNgB3pCBw9SaHktzQWoFc
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance content-marketer agent based on automated review
- Add model: sonnet frontmatter field for category consistency
- Replace duplicate "Content types" section with distinct emerging/high-ROI formats list
- Add "AI search & generative engine visibility" section (AI Overviews, LLM citation, llms.txt)
- Add "Owned audience strategy" section (zero-party/first-party data, privacy-first)
- Expand "Content quality" with explicit E-E-A-T signals and named-entity writing guidance
- Add "Limitations" section scoping Write/Edit usage and handoffs
- Add handoff lines to marketing-attribution-analyst, search-ai-optimization-expert, seo-specialist
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: correct seo-specialist handoff scope in content-marketer
seo-specialist has no Write/Edit/Bash access (audit/strategy only), so
routing technical SEO *implementation* to it left that work with no
actual owner. Split the handoff: audits/recommendations to
seo-specialist, implementation to frontend-developer or
wordpress-master.
Addresses Greptile review comment on PR #744.
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance accessibility-tester agent based on automated review
- Fix broken automated-scan command: npx axe-core-cli -> npx @axe-core/cli --exit (with WCAG tag flag guidance)
- Correct inaccurate legal-compliance framing; map each framework to its actual referenced WCAG version (Section 508 -> 2.0 AA, ADA Title II/III -> 2.1 AA, EAA/EN 301 549 -> approx 2.1 AA)
- Add Track 1b: scripted keyboard/focus/ARIA-state testing via @axe-core/playwright
- Qualify the automated-coverage statistic (30-40% industry-wide, axe-core ~57%) instead of an unqualified 40%
- Note WCAG 3.0 is still a W3C Working Draft, not expected before ~2029
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* address review feedback: fix missing AxeBuilder import, stale frontmatter examples
- Add missing `import { AxeBuilder } from '@axe-core/playwright'` to the
Track 1b code snippet so it's copy-pasteable
- Update the frontmatter description's second example to drop a now-past
"EAA June 2025 deadline" reference and match the per-framework WCAG
version mapping already used in the body
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LsDjHUg19zLAPpSYuQxH5A
* fix: add missing assertion to Track 1b Playwright example
The snippet collected axe results but never asserted on them, so a
copied test would exit 0 regardless of violations found.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LsDjHUg19zLAPpSYuQxH5A
* fix: expand Track 1b withTags to full WCAG 2.0/2.1/2.2 scope
withTags(['wcag22aa']) only activated the 9 new WCAG 2.2 criteria,
silently skipping the WCAG 2.0/2.1 rules that Track 1 already covers
via --tags wcag2a,wcag2aa,wcag21aa,wcag22aa. Align the two tracks.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LsDjHUg19zLAPpSYuQxH5A
* fix: add missing wcag21a tag to accessibility scan commands
The --tags/withTags lists omitted wcag21a (WCAG 2.1 Level A), causing
axe-core to skip rules like Character Key Shortcuts, Pointer Gestures,
Pointer Cancellation, Label in Name, and Motion Actuation. Applied to
both Track 1 (@axe-core/cli) and Track 1b (@axe-core/playwright).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LsDjHUg19zLAPpSYuQxH5A
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance competitive-analyst based on automated review
- Remove fabricated example statistics from progress/delivery templates
- Replace fictional context-manager JSON protocol with explicit
user-questions step and Human-in-the-Loop Pause Criteria
- Add Ethical & Legal Boundaries section for competitive intelligence
- Add model: sonnet to match sibling agents in business-marketing
- Consolidate repetitive bullet lists into denser Core Practices prose
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: resolve wording conflicts flagged by cubic review on PR #737
- Analysis step now allows sourced findings with explicit uncertainty
labels instead of an absolute corroboration requirement, matching the
later "flag single-source claims" guidance.
- Non-public/paywalled competitor access is now an outright refusal with
a redirect to public sources, not a pause-for-confirmation item (which
implied it was user-authorizable, conflicting with the absolute
prohibition in Ethical & Legal Boundaries).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PgWKsHrXsL4B1ZdaKgkCvk
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance legacy-modernizer based on automated review
- Add missing model: sonnet frontmatter field (was required but absent)
- Add explicit ownership/hand-off statement clarifying scope vs
architecture-modernizer, refactoring-specialist, cloud-migration-specialist,
and database-optimizer
- Add concrete "Common legacy stacks & upgrade paths" section (COBOL,
Java EE/WebLogic, .NET Framework, AngularJS, Python 2, PHP 5/7)
- Add "AI-assisted analysis" guidance (dependency mapping, codemods,
auto-generated characterization tests)
- Replace vague filler bullets ("Team trained effectively", "Celebrate wins",
"Business value delivered continuously") with concrete, actionable deliverables
- Diversify description examples: replaced a redundant strangler-fig example
with a distinct mainframe/COBOL + version-upgrade-only scenario
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: address greptile review feedback on legacy-modernizer
Align the third description example's context with what it actually
demonstrates (.NET Framework version-upgrade only), and de-duplicate
the implementation-approach bullets that repeated the checklist items
verbatim.
* fix: address cubic review feedback on legacy-modernizer accuracy
Scope the .NET Framework upgrade path (Web Forms/WCF have no direct
Upgrade Assistant port and need separate plans), note 2to3 was removed
in Python 3.13+, and clarify that Grep/Glob only inventories
dependencies rather than producing a validated call graph.
* fix: soften zero-risk claim in legacy-modernizer example commentary
Version-only upgrades can still change runtime/dependency/batch
behavior; describe risk minimization via validation and rollback
instead of guaranteeing zero production risk.
---------
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance bettoredge-value-finder based on automated review
- Fix name to kebab-case (bettoredge-value-finder) matching filename
- Add required tools/model frontmatter, scoped to Read + mcp__bettoredge__*
- Add system-prompt persona and "When invoked" workflow
- Add Guardrails section enforcing bankroll limits, no fabricated results,
and legal-age confirmation
- Caveat the bid/ask-midpoint value-detection methodology
- Add credential-handling guidance for live account login env vars
- Bump version to 1.0.1 to match published npm package
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: address PR review feedback on bettoredge-value-finder
- Remove unnecessary Read from tools frontmatter (agent only calls
MCP tools, none of which touch the filesystem)
- Fix bootstrap paradox: step 1 no longer unconditionally calls
bettoredge_setup, which only exists once the MCP server is running
- Move credential-handling warning ahead of the first instruction
that has users embed BETTOREDGE_EMAIL/PASSWORD
Co-Authored-By: Claude <noreply@anthropic.com>
* fix: clarify bettoredge_status is read-only in guardrail wording
bettoredge_status only surfaces current bankroll limits; it can't
change them. Point users to the BettorEdge platform for that instead
of implying the agent can modify limits through this tool.
Co-Authored-By: Claude <noreply@anthropic.com>
---------
Co-authored-by: Claude <noreply@anthropic.com>
- Add missing required `model: sonnet` frontmatter field
- Rewrite description with a "Use PROACTIVELY" trigger phrase and three <example> blocks for better delegation matching, dropping the unsupported "research-backed" claim (tools have no WebSearch/WebFetch)
- Fix broken "See Also" references: professional-effective-communication -> professional-communication skill, /compose-email command -> email-composer skill, remove non-existent tech-talks-craft skill and /feedback-composer command, add difficult-workplace-conversations skill for roleplay scenarios
- Add persona-drift resistance guidance to Roleplay Mode so simulated personas stay consistently resistant/motivated across an exchange instead of becoming generically agreeable
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance smart-contract-specialist based on automated review
- Narrow scope to architecture/design-pattern advisory role, resolving overlap with blockchain-developer
- Add model: sonnet and tools: Read, Grep, Glob, WebSearch, WebFetch frontmatter (least-privilege, no Write/Edit/Bash)
- Add "When to Stop and Ask" guardrails section
- Add 2026 EVM/Solidity coverage: EIP-1153 transient storage, EIP-7201 namespaced storage, EIP-7702 (Pectra) EOA delegation, via_ir pipeline
- Add security/verification toolchain advisory section (Slither, Aderyn, Echidna, Medusa, Certora, Halmos, forge snapshot)
- Add anti-hallucination delivery-summary instruction
- Add "Integration with Other Agents" section documenting handoff to blockchain-developer and smart-contract-auditor
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: remove unsourced gas-savings percentage for via_ir pipeline
Addresses cubic review feedback on PR #713 — the "10-30% gas reduction"
figure isn't backed by official Solidity docs, which only describe
via-IR as producing "better gas-optimized code" without a specific
percentage. Replaced with qualitative language consistent with the
file's own anti-hallucination guardrail.
Co-Authored-By: Claude Code <noreply@anthropic.com>
---------
Co-authored-by: Claude <noreply@anthropic.com>
- Add required tools and model frontmatter fields (audit-only allowlist: Read, Grep, Glob, Bash, WebSearch, WebFetch)
- Modernize security toolchain with Foundry, Aderyn, Echidna/Medusa, Certora Prover/Halmos
- Reference OWASP Smart Contract Top 10 and add explicit severity classification (Critical/High/Medium/Low/Informational)
- Cover cross-chain bridge exploits and business logic/tokenomics attack categories
- Add Integration with Other Agents section and When to Stop and Ask guardrail
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance shopify-expert based on automated review
- Fix invalid tools frontmatter (was VS Code Copilot Chat tool names, not Claude Code tools)
- Add model/color fields and upgrade description with delegation examples
- Correct REST vs GraphQL Admin API guidance for 2026 (REST legacy since Oct 2024, GraphQL mandatory for new public apps since Apr 2025)
- Replace Remix-specific app guidance with React Router v7 (@shopify/shopify-app-react-router)
- Add checkout.liquid deprecation deadline (Aug 26, 2026)
- Add Polaris Web Components and Hydrogen/headless (Oxygen, Storefront MCP) guidance
- Update Shopify CLI reference for CLI 4.0 flag changes
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: avoid false-positive javascript: XSS flag in shopify-expert
Automated security audit's HTML-injection check does a naive substring
match for "javascript:" and flagged the heading "...in JavaScript:" as
a javascript: protocol XSS pattern. Reword to sidestep the false
positive without changing meaning.
---------
Co-authored-by: Claude <noreply@anthropic.com>
- Add model field and rewrite description to state MCP dependencies and trigger phrases
- Add Prerequisites / Required MCP Setup section (Octopus Deploy + GitHub MCP servers, env vars, example config)
- Expand body into a structured workflow with ambiguity handling, missing-build-info fallback, and GitHub error handling
- Add example markdown release-notes output and a Troubleshooting & Limitations section
- Fix "release nots" typo
- Move from api-graphql/ to devops-infrastructure/ (correct category for a deployment/release-management agent)
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance graphql-security-specialist based on automated review
- Add model, color, and permissionMode to frontmatter; add Glob and Edit to tools
- Rewrite description with <example>/<commentary> blocks matching sibling agents
- Add alias/batching overload ("battering ram") and CSRF attack coverage
- Add Recommended Security Tooling section (graphql-armor, graphql-cop, InQL)
- Fix Input Sanitization: clarify DOMPurify is XSS-only, add SQL/NoSQL injection prevention guidance
- Extend Query Allowlisting with Automatic Persisted Queries / trusted documents pattern
- Add closing "Integration with other agents" section
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: correct CSRF comment and remove fabricated Apollo plugin import
Addresses cubic-dev-ai review on PR #679:
- CSRF middleware comment now accurately describes rejecting all GET
requests lacking the preflight header, not just mutations
- Replaced non-existent ApolloServerPluginPersistedDocuments import
with a manual requestDidStart/didResolveOperation hook, which is a
real documented Apollo Server 4 plugin API
---------
Co-authored-by: Claude <noreply@anthropic.com>
- Renamed Thinking-Beast-Mode.md -> thinking-beast-mode.md (kebab-case)
and moved from api-graphql/ to development-tools/ (no GraphQL/API
content; it's a generic autonomous engineering-task agent)
- Added required model field plus color/permissionMode, matching
sibling agent conventions
- Replaced non-existent tool references (fetch_webpage,
sequential_thinking, get_errors) with real Claude Code tools
(WebFetch, WebSearch, Bash for lint/test)
- Rewrote the marketing-fluff description into a concrete,
example-driven trigger description
- Stripped "quantum cognitive architecture" roleplay language and
consolidated the workflow into Investigate -> Plan -> Implement ->
Verify -> Report, keeping the legitimately good discipline (root
cause debugging, rigorous verification, don't stop until truly done)
- Added explicit guardrails (ask before destructive commands, stop on
genuine blockers)
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
* improve: enhance data-engineer based on automated review
- Add dbt transformation frameworks section (dbt Core, Fusion, tests, contracts, Semantic Layer)
- Add managed EL ingestion and CDC tooling (Fivetran, Airbyte, Meltano, Debezium)
- Name concrete data-quality/observability tools and data contracts (Great Expectations, Soda, Monte Carlo, Elementary, ODCS)
- Modernize orchestration section (Airflow 3.x, Dagster asset-centric model, mark Luigi as legacy)
- Add AI/LLM data pipeline expertise (vector DBs, embeddings, RAG data prep)
- Align frontmatter/structure with sibling data-scientist.md (model field, clarifying-questions preamble)
- Strengthen description for automatic delegation with PROACTIVELY phrasing and key tool names
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
* fix: use neutral Luigi label per review feedback
cubic flagged the unsourced "maintenance mode" claim about Luigi in
the orchestration section; use a neutral label instead.
---------
Co-authored-by: Claude <noreply@anthropic.com>
Add missing model field, sharpen proactive-use description, add AI
search/GEO visibility guidance, update Core Web Vitals to INP-based
thresholds, and cross-reference search-ai-optimization-expert.
Claude-Session: https://claude.ai/code/session_01GoWck91RfDye9q46JZDGUk
Co-authored-by: Claude <noreply@anthropic.com>
- Add missing model: sonnet frontmatter field to match sibling agents
- Rewrite system prompt to be Claude-specific: XML tag structuring,
role framing, multishot examples, extended thinking/effort parameter,
prompt caching, and grounded long-context answers
- Remove outdated JSON "Communication Protocol" handshake pattern in
favor of a plain-language Requirements Gathering step
- Add explicit boundaries vs. llm-architect, model-evaluator, and
task-decomposition-expert to reduce delegation ambiguity
- Reframe hardcoded example metrics (accuracy %, latency, cost) as
illustrative targets to confirm with the user rather than universal
thresholds
- Regenerate component catalog entries for prompt-engineer
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
- Add required model field (sonnet) and color field to frontmatter
- Add Write/Edit tools so the agent can produce the test plans/docs it describes
- Rewrite QA excellence and excellence checklists as concrete, verifiable criteria instead of vague adjectives
- Replace hardcoded fake metrics in progress tracking and delivery notification with placeholder templates and explicit anti-fabrication guidance
- Replace speculative JSON "Communication Protocol" context-manager pattern with plain-language context-gathering guidance
- Add AI-assisted and agentic testing practices section covering risk-based test planning and delegation to test-automator/playwright-tester
- Regenerate component catalog (docs/components.json, dashboard/public/components.json)
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
- Add error handling and run-status checks (try/catch, ApifyApiError, run.status) to JS/TS and Python examples
- Expand Apify MCP tools list with run management, storage access, and discovery tools (get-actor-log, get-dataset-items, get-key-value-store*, add-actor, rag-web-browser, etc.)
- Add webhook guidance (event types, `webhooks` call parameter, 200 response requirement)
- Add proxy configuration guidance (proxyConfiguration, RESIDENTIAL/DATACENTER groups, checkAccess opt-out)
- Add input schema validation step to the recommended workflow
- Add production deployment notes (scheduling, memory/timeout, idempotency) and testing/mocking examples for both languages
- Cover key-value store OUTPUT record retrieval alongside dataset pagination guidance
- Add model/color frontmatter fields and a "use proactively" trigger phrase to the description
- Fix duplicate H1 headings and refresh a stale example date
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
- Remove all hardcoded absolute paths (/Users/cam/VAULT01/...) and replace with relative paths rooted at ./
- Replace VAULT01-specific references with generic vault terminology
- Add model: sonnet field to frontmatter
- Add Grep and Edit to tools list; remove redundant LS
- Expand description with 3 concrete <example>/<commentary> blocks
- Add Native Fallback Workflow using Glob+Grep (no external script dependency)
- Make Script-Assisted Workflow optional with clear instructions to run from vault root
- Add MOC Hierarchy section explaining LYT three-tier system (Home/Topic/Sub-MOC)
- Add optional Dataview block to MOC template with note to remove if plugin absent
- Add vault-structure validation guidance before writing MOCs
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
- Add tools and model frontmatter fields (WebSearch, WebFetch; sonnet)
- Rewrite description with three concrete examples using example/commentary tags
- Add imperative "When Invoked" seven-step workflow checklist
- Add iterative retrieval loop section with explicit stopping conditions
- Add source credibility evaluation framework as a scored table
- Expand contradiction handling into an explicit protocol with example format
Automated review cycle | Co-Authored-By: Claude Code <noreply@anthropic.com>
Daniel Avila