Skip to content

Add crypto, macro, and US equities specialized agents#133

Closed
Mamurbakhromov wants to merge 1 commit intomsitarzewski:mainfrom
Mamurbakhromov:finance-agents
Closed

Add crypto, macro, and US equities specialized agents#133
Mamurbakhromov wants to merge 1 commit intomsitarzewski:mainfrom
Mamurbakhromov:finance-agents

Conversation

@Mamurbakhromov
Copy link
Copy Markdown

What does this PR do?

Agent Information (if adding/modifying an agent)

  • Agent Name:
  • Category:
  • Specialty:

Checklist

  • Follows the agent template structure from CONTRIBUTING.md
  • Includes YAML frontmatter with name, description, color
  • Has concrete code/template examples (for new agents)
  • Tested in real scenarios
  • Proofread and formatted correctly

@msitarzewski
Copy link
Copy Markdown
Owner

Hey @Mamurbakhromov — cool to see crypto/macro/equities coverage, this fills a genuine gap!

A couple things need fixing before we can merge:

  1. Emoji format — the emoji values in your frontmatter are hex codepoints (e.g., emoji: f4c9) instead of actual emoji characters. They need to be the rendered glyphs (e.g., emoji: 📊). This will break our conversion tooling as-is.

  2. Vibe diversity — "disciplined" appears in nearly every agent's vibe. Each agent should have a distinct personality. Take a pass and give each one a unique voice — what makes the Crypto Derivatives Risk Analyst feel different from the Macro Policy Regime Analyst?

Content quality is solid otherwise. Fix those two things and we're good to go!

@msitarzewski
Copy link
Copy Markdown
Owner

Hey @Mamurbakhromov — sorry this sat so long without follow-up. The crypto/macro/equities content is solid and fills a real gap.

I'm closing this to clear the backlog, but you're welcome to reopen once you've had a chance to address the two items from my earlier review:

  1. Emoji format — swap hex codepoints (e.g., emoji: f4c9) for actual emoji characters (e.g., emoji: 📊)
  2. Vibe diversity — give each agent a distinct personality instead of reusing "disciplined" across all nine

A rebase onto current main will be needed too since a lot has changed. Looking forward to seeing this come back!

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.

2 participants