Add Email Delivery Reporter agent in tdx CLI format#437
Merged
toru-takahashi merged 3 commits intotreasure-data:masterfrom Apr 17, 2026
Merged
Add Email Delivery Reporter agent in tdx CLI format#437toru-takahashi merged 3 commits intotreasure-data:masterfrom
toru-takahashi merged 3 commits intotreasure-data:masterfrom
Conversation
Add the Email Delivery Reporter agent for Treasure Data Engage with a flat directory structure compatible with `tdx agent push`. Key design decisions: - knowledge_bases/ at project root (not nested in agent/) so tdx push recognizes all resources - DOMAIN placeholder in KB filename and references for clear setup - KB tables explicitly defined (events, error_events, subscription_events) - Remove unused Timezone field from Overall Summary form (not referenced in system prompt or specs) - Remove unsupported Subject filter from Campaign Details form (not implemented in CampaignSummary spec) - README written as self-contained setup guide with explicit 5-point DOMAIN replacement checklist Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new “Email Delivery Reporter” agent package under engage-box/ intended to be deployable via tdx agent push, including database/text knowledge bases, form interfaces, and English/Japanese prompt/docs.
Changes:
- Introduces a new
Email Delivery Dashboardagent config + system prompt for generating React/Plotly dashboards. - Adds DB + text knowledge bases and two form interfaces for “Overall Summary” and “Campaign Details” flows.
- Adds root README and Japanese reference docs for setup/usage guidance.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| engage-box/email-delivery-reporter/tdx.json | Adds the tdx project manifest. |
| engage-box/email-delivery-reporter/knowledge_bases/delivery_email_DOMAIN.yml | Defines the database KB and exposed tables/queries. |
| engage-box/email-delivery-reporter/knowledge_bases/OverallSummary_Spec.md | Adds the Overall Summary report specification (text KB). |
| engage-box/email-delivery-reporter/knowledge_bases/CampaignSummary_Spec.md | Adds the Campaign Details report specification (text KB). |
| engage-box/email-delivery-reporter/form_interfaces/Overall Summary.yml | Adds the “Overall Summary” form interface and prompt template. |
| engage-box/email-delivery-reporter/form_interfaces/Campaign Details.yml | Adds the “Campaign Details” form interface and prompt template. |
| engage-box/email-delivery-reporter/Email Delivery Dashboard/prompt.md | Adds the English system prompt describing execution/tooling rules. |
| engage-box/email-delivery-reporter/Email Delivery Dashboard/agent.yml | Adds the agent definition, outputs, and KB tool wiring. |
| engage-box/email-delivery-reporter/docs/japanese/prompt_ja.md | Adds Japanese reference translation of the system prompt. |
| engage-box/email-delivery-reporter/docs/japanese/README_JA.md | Adds Japanese setup/usage documentation. |
| engage-box/email-delivery-reporter/docs/japanese/OverallSummary_Spec_ja.md | Adds Japanese reference translation of the Overall Summary spec. |
| engage-box/email-delivery-reporter/docs/japanese/CampaignSummary_Spec_ja.md | Adds Japanese reference translation of the Campaign Details spec. |
| engage-box/email-delivery-reporter/README.md | Adds a root, self-contained Quick Start/setup guide for CLI deployment. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Add missing tables to KB config (daily_summary, event_master, email_events) referenced by report specs - Fix contradictory filter config: campaign_id and journey_id were both required:true with exclusive_with, making valid requests impossible. Changed to required:false with "exactly one" rule in filter_notes - Align prompt tool names: render_plotly_chart → new_plot to match the actual output function defined in agent.yml - Update docs/japanese/ to reflect new file structure (remove references to old filenames like system_prompt.md, knowledge_base_*.md) - Remove Subject filter references from README_JA.md - Fix cross-reference paths in Japanese spec files - Add GNU sed variant to README for Linux/Windows compatibility Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Resolved conflict in engage-box/email-delivery-reporter/README.md by: - Keeping the new tdx CLI format structure from improve-email-delivery-reporter - Integrating valuable content from master (Overview, Features, Report Components) - Maintaining English as primary language (Japanese docs in separate files) - Combining troubleshooting sections from both versions Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
toru-takahashi
approved these changes
Apr 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
tdx agent pushknowledge_bases/placed at project root (not nested inagent/) so all resources are recognized during pushDOMAINplaceholder in KB filename and@refreferences for clear, explicit setup instructionsevents,error_events,subscription_events) instead of empty arrayTimezonefield from Overall Summary form (not referenced in system prompt or report specs)Subjectfilter from Campaign Details form (not implemented in CampaignSummary spec)File Structure
Test plan
tdx agent push . -f🤖 Generated with Claude Code