Skip to content

docs: document notification-routing skill#70

Closed
diberry wants to merge 1 commit intodevfrom
squad/68-config-model-docs
Closed

docs: document notification-routing skill#70
diberry wants to merge 1 commit intodevfrom
squad/68-config-model-docs

Conversation

@diberry
Copy link
Copy Markdown
Owner

@diberry diberry commented Mar 27, 2026

Closes #67

Documents the notification-routing skill (pub-sub channel routing to prevent alert fatigue).

Source: bradygaster#625 by Tamir Dresher (commit \�9c06b09)

Working as PAO (DevRel)

Changes

  • New docs page: \ eatures/notification-routing.md\ — channel config schema, CHANNEL: tag convention, routing dispatcher, anti-patterns, distributed systems context
  • Navigation entry added in
    avigation.ts\ (Features section, before Notifications)
  • Test assertion updated in \docs-build.test.ts\ (\EXPECTED_FEATURES\ array)

Quality checks

  • Microsoft Style Guide: sentence-case headings, active voice, second person, present tense
  • Scannability: tables for config and anti-patterns, bullets for symptoms, code blocks for examples
  • Deep linking: cross-references to Notifications, Skills, and Distributed Mesh pages
  • Single commit, docs-test sync

⚠️ Awaiting Flight + FIDO review before merge.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@diberry
Copy link
Copy Markdown
Owner Author

diberry commented Mar 27, 2026

Closing — pushed to wrong branch. Reopening from correct branch.

@diberry diberry closed this Mar 27, 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.

2 participants