Conversation
|
All PRs must reference a prior Discord discussion to ensure community alignment before implementation. Please edit the PR description to include a link like: This PR will be automatically closed in 3 days if the link is not added. |
OpenAB PR ScreeningThis is auto-generated by the OpenAB project-screening flow for context collection and reviewer handoff.
Screening report## IntentPR #735 adds a design document intended to explain why OpenAB is architected the way it is, not just how to use it. The operator-visible problem is that contributors, reviewers, deployers, and agent maintainers lack a shared reference for OpenAB’s boundaries, security assumptions, adapter model, and relationship to adjacent systems like OpenClaw. FeatThis is a docs improvement. It adds
Who It ServesPrimary beneficiaries are maintainers and reviewers who need a stable architectural reference when evaluating changes. Secondary beneficiaries are deployers, agent runtime operators, and contributors trying to understand OpenAB’s intended boundaries before adding features. Rewritten PromptAdd a concise architecture/design document for OpenAB that explains the project’s core design principles, execution model, security assumptions, and non-goals. The document should:
Acceptance criteria:
Merge PitchThis is worth advancing because architectural docs reduce reviewer drift and help future contributors avoid adding responsibilities that OpenAB intentionally should not own. The PR appears low implementation risk because it is documentation-only. The main reviewer concern is accuracy: the document may make strong claims about sandboxing, read-only filesystems, Best-Practice ComparisonRelevant OpenClaw principles:
Relevant Hermes Agent principles:
The best-practice takeaway is that the document should use OpenClaw and Hermes as boundary references, not as feature checklists. If OpenAB is not supposed to own durable jobs, retries, scheduled runs, or daemon ticks, the design doc should say that plainly. Implementation OptionsOption 1: Conservative docs-only merge Option 2: Balanced docs plus cross-links Option 3: Docs plus verification checklist Option 4: Ambitious architecture governance pass Comparison Table
RecommendationAdvance with the balanced option: review This keeps the PR mergeable while making the document useful as a shared architectural reference. Any deeper work, such as formalizing security guarantees, adapter contracts, or OpenClaw/Hermes comparisons, should be split into follow-up issues so this docs PR does not become a sprawling architecture redesign. |
What problem does this solve?
OpenAB lacks a document that articulates its design philosophy and architectural decisions. Contributors and users need to understand why OpenAB is built the way it is — not just how to use it.
Summary
Adds
docs/the-design-of-openab.mdcovering the four core design pillars:Also covers: