Skip to content

unit-5: bundle template-builder-ui references + negative trigger#5

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

unit-5: bundle template-builder-ui references + negative trigger#5
sent-dm wants to merge 1 commit into
devfrom
batch/unit-5-tbui

Conversation

@sent-dm
Copy link
Copy Markdown
Contributor

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

Scope (unit 5 of 13 in batch restructure)

Touches skills/template-builder-ui/ only.

Changes

  • SKILL.md frontmatter: added negative-trigger sentence to description ruling out generic templating engines (Jinja, Handlebars, Mustache, email/HTML, non-Sent template systems). Description stays well under 1024 chars.
  • SKILL.md body: linked three new bundled references from the Validation Strategy section; added top-level references/sent-glossary.md pointer to Related Skills.
  • NEW references/template-validation-matrix.md (~120 lines) — per-channel validation rule table for SMS / WhatsApp / RCS with how each failure should surface (inline error / blocked save / soft warning).
  • NEW references/template-ui-wireflows.md (~115 lines) — UX flow spec for create-from-scratch, clone, Meta import, JSON paste, edit (status-aware locks), submit, status transitions, and reject-recover.
  • NEW references/template-status-handling.md (~80 lines) — lifecycle states (Draft / Pending / Approved / Rejected), editable fields per state, Meta PAUSED handling, webhook vs polling tradeoff, Meta vs Sent rejection display, resubmission flow.

Verification

  • bash scripts/validate-skills.sh exits 0.
  • All bundled references/*.md paths in SKILL.md resolve.
  • Negative-trigger phrase present in description.
  • No carrier / Meta / Google docs duplicated — all link to https://docs.sent.dm or top-level Sent-glossary / category / RBM / TCR references.

Test plan

  • CI validator passes
  • Spot-check the three new references render correctly on GitHub
  • Confirm no other skill files modified

- Add negative-trigger sentence to SKILL.md description ruling out generic
  templating engines (Jinja, Handlebars, Mustache, email/HTML).
- New skills/template-builder-ui/references/template-validation-matrix.md:
  per-channel rule table (SMS/WhatsApp/RCS) with UI surface treatment.
- New skills/template-builder-ui/references/template-ui-wireflows.md:
  UX flows (scratch, clone, Meta import, JSON paste, edit, submit, recover).
- New skills/template-builder-ui/references/template-status-handling.md:
  lifecycle state machine, editable fields per state, webhook vs polling,
  Meta vs Sent rejection display, resubmission flow.
- Link the three references from the Validation Strategy section.
- Add top-level sent-glossary pointer to Related Skills.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
sent-dm added a commit that referenced this pull request May 19, 2026
Manually rebased onto dev (worker branched from pre-baseline main).
Per-skill content is identical to PR #5.

- Add template-validation-matrix.md, template-ui-wireflows.md, template-status-handling.md
- Add negative trigger to description (excludes Jinja/Handlebars/email templates)
- Add glossary cross-link to Related skills

Closes #5

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 #5. 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