Skip to content

Comments

docs: add ecosystem conventions and event design lessons from PR review#79

Open
bkrabach wants to merge 1 commit intomainfrom
docs/ecosystem-conventions-event-design
Open

docs: add ecosystem conventions and event design lessons from PR review#79
bkrabach wants to merge 1 commit intomainfrom
docs/ecosystem-conventions-event-design

Conversation

@bkrabach
Copy link
Collaborator

Summary

  • Add new Ecosystem Conventions & Event Design subsection to Distilled Lessons (4 entries): metadata naming standard, proactive extensibility slots, raw-data-through pattern, helper promotion threshold
  • Add 3 new entries to PR Review subsection: zero-consumer compat burden, no fragile approaches, exemplar contract docs

Context

Distilled from reviewing PRs #14 (amplifier-core) and #78 (amplifier-foundation) from an external contributor. Discussion with the project owner clarified metadata philosophy, extensibility patterns, and code quality standards for event payloads.

Test plan

  • Markdown renders correctly
  • New entries integrate cleanly into existing Distilled Lessons structure
  • No duplicate or conflicting guidance with existing entries

🤖 Generated with Amplifier

Distill learnings from reviewing PRs #14 (amplifier-core) and #78
(amplifier-foundation) with the project owner. Captures metadata
philosophy, extensibility patterns, and code quality standards.

New entries:
- Ecosystem Conventions & Event Design subsection (4 entries):
  metadata naming, proactive extensibility slots, raw-data-through
  pattern, promotion threshold for helpers
- PR Review subsection additions (3 entries):
  zero-consumer compat burden, no fragile approaches, exemplar
  contract docs

🤖 Generated with [Amplifier](https://github.com/microsoft/amplifier)

Co-Authored-By: Amplifier <240397093+microsoft-amplifier@users.noreply.github.com>
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