docs(#55): add OpenHands entry to landscape analysis - #7108
Conversation
Add an OpenHands section to docs/landscape.md in the production agent orchestration systems category. Documents the platform's architecture, the PolyForm licensing constraint on the enterprise directory (only the MIT-licensed resolver and CLI are viable for open-source use), known prompt injection vulnerabilities disclosed by Johann Rehberger in 2025, and relevance to fullsend's problem areas including sandboxing, injection defense, and observability. Cross-references issue #260 which tracks concrete experiments against the resolver. Closes #55
|
🤖 Review · Commit: |
Site previewPreview: https://2de7ebee-site.fullsend-ai.workers.dev Commit: |
|
🤖 Finished Review · ✅ Success · Started 5:27 PM UTC · Completed 5:43 PM UTC Commit: Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.68 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Risk Assessment: low (1/5) DetailsRoutine documentation-only PR by a bot adding 23 lines to a landscape analysis file, with no protected paths, no security or CI concerns, and an issue scope that directly matches the change; all signals unchanged from prior assessment, anchoring to prior score of 1. Previous runRisk Assessment: low (1/5) DetailsRoutine documentation-only PR by a bot adding 22 lines to a landscape analysis file, with no protected paths, no security or CI concerns, and an issue scope that directly matches the change. |
|
Looks good to me Previous runReviewFindingsHigh
Medium
Next steps:
|
![fullsend-ai-review[bot]](https://avatars.githubusercontent.com/in/3478501?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
|
🤖 Finished Fix · ✅ Success · Started 5:45 PM UTC · Completed 5:51 PM UTC Commit: Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $1.02 |
Update the OpenHands landscape entry to reflect changes since the original evaluation: - Licensing: rewrite to reflect the project's current MIT-only license, noting the historical PolyForm constraint has been resolved and the enterprise/ directory removed. - Architecture: document the Agent Canvas pivot and Agent Client Protocol (ACP) support, which makes the platform agent-agnostic at the coding-agent layer while still requiring the Agent Server for orchestration. - Relevance: add a fourth observation covering the orchestration-vs- agent-runtime trade-off identified in issue #55's updated triage. Remove the now-unnecessary "MIT-licensed" qualifier from the resolver reference. Addresses #7108
🔧 Fix agent — iteration 1 (bot-triggered)Updated the OpenHands landscape entry to reflect three significant changes identified in the review: (1) rewrote the Licensing section to reflect the project's current MIT-only license, (2) documented the Agent Canvas / ACP architectural pivot in the Architecture section, and (3) added a fourth relevance observation covering the orchestration-vs-agent-runtime trade-off. All changes are sourced from the updated triage evaluation in issue #55. Fixed (3):
Tests: passed Decision points
Next steps:
|
|
🤖 Finished Review · ✅ Success · Started 5:53 PM UTC · Completed 6:06 PM UTC Commit: Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.09 |
Superseded by updated review
fullsend-ai-review
Bot
added
the
requires-manual-review
|
🤖 Finished Retro · ✅ Success · Started 7:46 PM UTC · Completed 7:59 PM UTC Commit: Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.34 |
Retro: PR #7108 — docs(#55): add OpenHands entry to landscape analysisTimeline
Workflow quality assessmentReview quality: Excellent. The review agent correctly identified all three factual errors the code agent introduced — stale PolyForm licensing, missing ACP/Agent Canvas architecture pivot, and an incomplete relevance analysis. These were genuine errors in the content, not false positives. Fix agent: Efficient. All 3 findings addressed in a single commit at $1.02 cost. Re-review confirmed everything was resolved. Code agent: Missed available context. The Redundant dispatches: A second triage ($0.74) and second code run ($0.68, no PR created) were triggered by ralphbean's follow-up comment. Total redundant cost: ~$1.42. Total pipeline cost: ~$12.02 (Triage $1.43, Code $2.80, Review $6.77, Fix $1.02) Evidence for existing issues
Proposals filed
|
1 participant
Summary
Adds an OpenHands section to
docs/landscape.mdunder "Production agent orchestration systems," documenting the platform's architecture, licensing constraints, security history, and relevance to fullsend.Related Issue
Closes #55
Changes
docs/landscape.mdfollowing the format of existing entries in the production agent orchestration systems sectionTesting
make lintpasses (stage changes first, then run)Checklist
!for breaking changes)docstype — documentation-only change does not appear in release notesCloses #55
Post-script verification
agent/55-add-openhands-landscape)695da0ac8c3f989dcf587e381ec7c5d83f541d23..HEAD)