Skip to content

fix: improve notifications description for agent clarity#9

Merged
sweetmantech merged 1 commit intomainfrom
sweetmantech/myc-4338-cli-notifications-more-helpful-description-for-agents-to
Feb 25, 2026
Merged

fix: improve notifications description for agent clarity#9
sweetmantech merged 1 commit intomainfrom
sweetmantech/myc-4338-cli-notifications-more-helpful-description-for-agents-to

Conversation

@sweetmantech
Copy link
Copy Markdown
Contributor

@sweetmantech sweetmantech commented Feb 25, 2026

Summary

  • Updates the notifications command description so agents understand the recipient is auto-resolved from the API key
  • Before: "Send a notification email to the authenticated account" — agents looked for a --to flag, didn't find one, and never tried the command
  • After: "Send an email to the account owner. The recipient is automatically resolved from your API key — no --to flag needed. Only --subject is required."

Test plan

  • All 45 tests pass
  • Build succeeds
  • Verify agents use the command correctly in sandbox runs

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the notifications command help text to clarify that the recipient is resolved from the API key and only the --subject flag is required.

The previous description was vague enough that agents looked for a --to
flag, didn't find one, and never tried the command. The new description
explicitly states the recipient is auto-resolved from the API key.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 25, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ae1ee7d and efdd34e.

📒 Files selected for processing (1)
  • src/commands/notifications.ts

📝 Walkthrough

Walkthrough

Updated the description text of notificationsCommand to clarify that recipient resolution derives from the API key, eliminating the need for a --to flag, with only --subject being required. No functional changes to the command logic or behavior.

Changes

Cohort / File(s) Summary
Notification Command Description
src/commands/notifications.ts
Updated exported command description to clarify recipient resolution from API key and required flag requirements. No changes to function signatures or runtime behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A message once vague, now crystal and clear,
The recipient's path, no longer unclear,
With flags now defined in words bright and true,
The notifier hops forward, with nothing to rue! 🌟

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sweetmantech/myc-4338-cli-notifications-more-helpful-description-for-agents-to

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.

@sweetmantech sweetmantech merged commit cdc7976 into main Feb 25, 2026
1 of 2 checks passed
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