Skip to content

unit-1: bundle messaging-performance-analyzer + MDR analyzer#9

Closed
sent-dm wants to merge 1 commit into
devfrom
batch/unit-1-mpa
Closed

unit-1: bundle messaging-performance-analyzer + MDR analyzer#9
sent-dm wants to merge 1 commit into
devfrom
batch/unit-1-mpa

Conversation

@sent-dm
Copy link
Copy Markdown
Contributor

@sent-dm sent-dm commented May 19, 2026

Scope

Part of the parallel /batch restructure (unit 1 of 13). Bundles messaging-performance-analyzer with its own reference + script payload so the skill is self-contained.

Changes

  • Move references/mdr-status-codes.md into skills/messaging-performance-analyzer/references/ (the only skill that cites it).
  • Add references/performance-diagnosis-playbook.md (~140 lines): decision tree for which signal to investigate first (sender-profile state vs webhook drop vs MDR funnel vs error codes), per-channel diagnostic patterns (SMS carrier filtering, WhatsApp template issues, RCS fallback chain breaks), cross-skill handoff matrix, and Sent-support escalation criteria.
  • Add scripts/analyze_mdr_funnel.py (Python 3.11+, stdlib only): parses CSV or JSON MDR exports, prints counts and drop-off percentages across QUEUED -> ROUTED -> SENT -> DELIVERED -> READ, exits 0 on healthy funnel, 3 on anomaly (with stage + percentage on stderr), 2 on bad input. Supports --threshold N (default 20).
  • Add scripts/fixtures/{good,bad}.json: 100-message synthetic exports. good stays at <=10% drop per stage; bad has a 66.7% drop at SENT -> DELIVERED.
  • Update SKILL.md: add a Bundled-references-and-scripts table with the invocation line, namespace Related Skills with sent-skills:, and link the top-level glossary.

Verification

  • bash scripts/validate-skills.sh -> exits 0
  • python3 skills/messaging-performance-analyzer/scripts/analyze_mdr_funnel.py .../good.json -> exit 0
  • python3 skills/messaging-performance-analyzer/scripts/analyze_mdr_funnel.py .../bad.json -> exit 3 (SENT -> DELIVERED at 66.7%)
  • --help text present; CSV input also exercised manually
  • Every references/ / scripts/ path in SKILL.md resolves on disk

Out of scope

Other skills' SKILL.md files, scripts/validate-skills.sh, CI workflows, root markdown, .claude-plugin/, and plan.md — per the unit's hard constraints.

- Move references/mdr-status-codes.md into the skill's own references/
  so the file lives with the only skill that consumes it.
- Add references/performance-diagnosis-playbook.md: decision tree for
  which signal to investigate first, per-channel diagnostic patterns,
  cross-skill handoff matrix, and Sent-support escalation criteria.
- Add scripts/analyze_mdr_funnel.py (Python 3.11+, stdlib only): parses
  CSV or JSON MDR exports, computes per-stage funnel counts and drop-off
  percentages across QUEUED -> ROUTED -> SENT -> DELIVERED -> READ, exits
  non-zero when any stage exceeds --threshold (default 20%).
- Add scripts/fixtures/{good,bad}.json: 100-message synthetic exports for
  smoke-testing the analyzer (good is healthy, bad has a >50% drop at
  SENT -> DELIVERED).
- Update SKILL.md with a Bundled-references-and-scripts table and a
  pointer to the top-level Sent glossary; namespace Related-Skills
  refs with sent-skills:.
sent-dm added a commit that referenced this pull request May 19, 2026
…analyzer

Manually rebased onto dev (worker branched from pre-baseline main).
Per-skill content is identical to PR #9.

- Move references/mdr-status-codes.md → skills/messaging-performance-analyzer/references/
- Add performance-diagnosis-playbook.md
- Add analyze_mdr_funnel.py (stdlib only) + good/bad fixtures
- Add glossary cross-link to Related skills
- Replace aspirational script names in bundled-resources table with the shipped one

Closes #9

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@sent-dm
Copy link
Copy Markdown
Contributor Author

sent-dm commented May 19, 2026

Merged manually into dev via commit referenced in Closes #9. The PR's branch was based on pre-baseline main (before optimized SKILL.md commit), so gh pr merge couldn't auto-merge against the optimized dev. The per-skill content (new references, scripts, fixtures) is identical to what this PR shipped; SKILL.md additions were re-applied on top of dev's optimized version.

@sent-dm sent-dm closed this May 19, 2026
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