|
| 1 | +# Contributing to OEBP |
| 2 | + |
| 3 | +OEBP is trying to become an open, explainable, composable, cross-embodiment |
| 4 | +behavior protocol. Contributions are welcome from robotics researchers, |
| 5 | +robot vendors, middleware maintainers, dataset builders, safety reviewers, and |
| 6 | +application developers. |
| 7 | + |
| 8 | +## Contribution Types |
| 9 | + |
| 10 | +- Specification clarifications and normative wording. |
| 11 | +- JSON Schemas, fixtures, semantic validators, and conformance tests. |
| 12 | +- Core skill, predicate, error, capability, and trace registry entries. |
| 13 | +- Mock, simulation, ROS 2, gRPC, SDK, and robot-specific adapters. |
| 14 | +- Dataset annotation and conversion utilities. |
| 15 | +- Security, safety, threat-model, and governance improvements. |
| 16 | +- Documentation, tutorials, diagrams, examples, and translations. |
| 17 | + |
| 18 | +## Working Rules |
| 19 | + |
| 20 | +- Prefer small, reviewable pull requests. |
| 21 | +- Explain the interoperability problem before proposing a new abstraction. |
| 22 | +- Use RFC-style terms consistently: MUST, MUST NOT, SHOULD, SHOULD NOT, MAY. |
| 23 | +- Keep protocol semantics separate from implementation-specific behavior. |
| 24 | +- Do not embed arbitrary executable code in core protocol documents. |
| 25 | +- Treat model-generated behavior proposals as untrusted data. |
| 26 | +- Add conformance fixtures for semantic changes whenever possible. |
| 27 | +- Keep extension points namespaced under an owned namespace. |
| 28 | + |
| 29 | +## When To Use an RFC |
| 30 | + |
| 31 | +Open an RFC for changes that affect: |
| 32 | + |
| 33 | +- protocol semantics; |
| 34 | +- canonical identifiers; |
| 35 | +- behavior graph execution rules; |
| 36 | +- safety or trust boundaries; |
| 37 | +- schema compatibility; |
| 38 | +- conformance requirements; |
| 39 | +- registry graduation from extension to core. |
| 40 | + |
| 41 | +Small editorial fixes, examples, tests, and non-normative documentation can be |
| 42 | +submitted directly as pull requests. |
| 43 | + |
| 44 | +## Pull Request Checklist |
| 45 | + |
| 46 | +- The problem statement is clear. |
| 47 | +- The change is scoped to one topic. |
| 48 | +- Normative text and schemas agree. |
| 49 | +- Valid and invalid examples are updated when behavior changes. |
| 50 | +- Security and safety implications are described. |
| 51 | +- Compatibility impact is stated. |
| 52 | +- Related RFCs or ADRs are linked. |
| 53 | + |
| 54 | +## Review Culture |
| 55 | + |
| 56 | +OEBP favors precise disagreement over vague approval. Reviewers should identify |
| 57 | +ambiguity, cross-embodiment failure modes, hidden implementation assumptions, |
| 58 | +and missing tests. Authors should expect iteration on terminology and scope. |
| 59 | + |
0 commit comments