Skip to content

feat(subagent): show agent emoji header before output#227

Merged
birdmanmandbir merged 1 commit intomainfrom
worker/feat-show-agent-emoji
Mar 12, 2026
Merged

feat(subagent): show agent emoji header before output#227
birdmanmandbir merged 1 commit intomainfrom
worker/feat-show-agent-emoji

Conversation

@birdmanmandbir
Copy link
Contributor

Summary

  • Add emoji field to AgentFrontmatter in internal/sync/frontmatter.go so canonical subagent .md files can declare an emoji
  • Before streaming output, ttal subagent run <name> prints \n<emoji> <name>\n to stderr as a visual header
  • ttal explore prints 🔭 explore [--project/--repo/--url/--web <target>] header before each run
  • Added printAgentHeader helper to cmd/subagent.go shared by both commands
  • Added emojis to all 8 canonical subagent templates in templates/docs/agents/
Subagent Emoji
plan-reviewer 📋
pr-code-reviewer 🔍
pr-code-simplifier
pr-comment-analyzer 💬
pr-principles-reviewer 📐
pr-silent-failure-hunter 🕵️
pr-test-analyzer 🧪
pr-type-design-analyzer 🏗️

Test plan

  • make ci passes (fmt, vet, lint, test, build)
  • ttal subagent run pr-code-reviewer "..." shows 🔍 pr-code-reviewer header before output
  • ttal explore "..." --project ttal-cli shows 🔭 explore --project ttal-cli header
  • ttal sync deploys updated subagent files with emoji in frontmatter

Add emoji field to AgentFrontmatter so subagent .md files can declare
an emoji. Before streaming output, ttal subagent run prints the emoji
and agent name to stderr as a visual header. ttal explore prints a 🔭
header with the active mode label (e.g. "explore --project ttal-cli").

Add emoji values to all eight canonical subagent templates.
@birdmanmandbir
Copy link
Contributor Author

PR Review: feat(subagent): show agent emoji header before output

Reviewed by: pr-code-reviewer, pr-principles-reviewer, pr-silent-failure-hunter

Critical Issues

None

Important Issues

None

Observations

  • Output to stderrprintAgentHeader correctly writes to os.Stderr, keeping stdout clean for pipe-friendly usage. ✓
  • Placement timing — header is printed after config/limits resolution but before provider build, so provider errors won't produce a spurious header. ✓
  • Empty emoji guardprintAgentHeader handles empty emoji gracefully (falls back to name-only). ✓
  • Consistent coverage — all five explore variants (--project, --repo, --url, --web, general) correctly wired with 🔭 emoji and descriptive labels. ✓
  • exploreOpts label fallback — the if label == "" guard in runExploreAgent is defensive and correct, even though all current callers supply non-empty labels. ✓
  • Non-breaking field additionEmoji string on AgentFrontmatter deserializes to "" for agents without the field, handled gracefully downstream. ✓
  • Agent template emojis — all 8 agent files received semantically appropriate emojis.

VERDICT: LGTM

@birdmanmandbir birdmanmandbir merged commit dbf82ff into main Mar 12, 2026
1 check passed
@birdmanmandbir birdmanmandbir deleted the worker/feat-show-agent-emoji branch March 12, 2026 16:24
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