DRAFT - Work in Process - Initial layout of new doc structure#284
DRAFT - Work in Process - Initial layout of new doc structure#284jpelletier1 wants to merge 18 commits intomainfrom
Conversation
This is an initial layout that roughly mimics Devin docs. It attempts to separate Documentation and Use Cases.
- incident-triage.mdx: Add content based on Datadog debugging blog post - Features automated error analysis with OpenHands SDK - Includes GitHub Actions workflow for Datadog integration - Links to blog: https://openhands.dev/blog/debugging-production-issues-with-ai-agents-automating-datadog-error-analysis - cobol-modernization.mdx: Add content based on COBOL-to-Java refactoring blog post - Features iterative refinement approach with multiple agents - Includes code transformation examples (COBOL to Java) - Links to blog: https://openhands.dev/blog/20251218-cobol-to-java-refactoring - vulnerability-remediation.mdx: Add content based on vulnerability fixer documentation - Explains automated vulnerability remediation with AI agents - Links to example implementation at github.com/OpenHands/vulnerability-fixer - Covers integration with security scanners (Snyk, Dependabot, CodeQL) - dependency-upgrades.mdx: Remove redundant security scanning section - Links to Vulnerability Remediation page for security-related upgrades - Focuses on general dependency upgrade workflows Co-authored-by: openhands <openhands@all-hands.dev>
- Rename from 'TODO-Tutorial Library' to 'Tutorial Library' - Remove 'Getting Started with Tutorials' section - Add Note linking to Use Cases section for in-depth guidance - Add Code Review cross-references: - Link to Code Review Use Case page - Link to PR Review SDK Guide (sdk/guides/github-workflows/pr-review) - Add Bug Fixing cross-reference to Incident Triage Use Case Co-authored-by: openhands <openhands@all-hands.dev>
Replace tabs-based layout with a card-based design: - OpenHands Cloud (recommended) - links to https://app.all-hands.dev - Local GUI - links to /openhands/usage/run-openhands/local-setup - Terminal / CLI - links to /openhands/usage/cli/installation Each card includes relevant icons and bullet points highlighting key features. Co-authored-by: openhands <openhands@all-hands.dev>
|
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
|
@OpenHands please update this branch to use this same file on main: |
|
I'm on it! jpelletier1 can track my progress at all-hands.dev |
Sync openhands/usage/use-cases/code-review.mdx with the updated version from main that includes the Software Agent SDK-based PR review workflow documentation.
SummaryI have successfully completed the requested update to PR #284. What was done:
Changes made:The file was replaced from a TODO stub version with the complete documentation from main that includes:
Verification:
|
|
@OpenHands Please remove the following links in the nav as they are redundant / already used in other places
|
|
I'm on it! jpelletier1 can track my progress at all-hands.dev |
These links were duplicated in the Additional Documentation section when they already exist under V0 Reference in the same navigation. Co-authored-by: openhands <openhands@all-hands.dev>
SummaryI have successfully completed the requested changes to PR #284. Checklist
DetailsThe removed navigation items were redundant because they already exist under the "V0 Reference" group within the same "Additional Documentation" section (lines 159-183 in the original file). The commit ( |
|
@OpenHands Let's create a new section below "Additional Documentation" called "OpenHands Community". Move all pages under "Additional Documentation > Community" to that new section. Also move "Additional Documentation > Developers" to that new section/ |
|
I'm on it! jpelletier1 can track my progress at all-hands.dev |
|
@OpenHands check and reflect the comments by |
|
I'm on it! neubig can track my progress at all-hands.dev |
- Created new 'OpenHands Community' section below 'Additional Documentation' - Moved 'Developers' group from Additional Documentation to OpenHands Community - Moved 'Community' group from Additional Documentation to OpenHands Community Co-authored-by: openhands <openhands@all-hands.dev>
SummaryI have successfully completed the requested changes to PR #284. Checklist
Resulting Navigation Structure |
- Update SDLC integration intro text - Add Linear task management reference - Add codecov module reference - Add codecov re-run guidance - Add Continuous Integration section with links to integrations - Remove empty Note block at end of file - Add detailed guidance on when to use OpenHands with specific task types Co-authored-by: openhands <openhands@all-hands.dev>
SummaryI have successfully completed the requested task to check and reflect neubig's comments on PR #284. Checklist
Changes Summary
Conciseness CheckAll changes directly correspond to neubig's specific suggestions - no extraneous modifications were made. Each edit was a direct implementation of a review comment suggestion. The branch has been successfully pushed and all review threads have been marked as resolved through the GraphQL API. |
- Updated Dependency Updates use case - Removed 'TODO-' on some pages - Updated Community section - Removed $10 references
all-hands-bot
left a comment
There was a problem hiding this comment.
Taste Rating: 🟡 Acceptable - Good content and improved structure, but some organizational issues and incomplete references need attention.
Verdict: ✅ Worth merging after addressing the critical TODO and clarifying the GitHub Actions examples.
| But it can be particularly useful for certain types of tasks. For instance: | ||
|
|
||
| - **Clearly Specified Tasks:** Generally, if the task has a very clear success criterion, OpenHands will do better. It is especially useful if you can define it in a way that can be verified programmatically, like making sure that all of the tests pass or test coverage gets above a certain value using a particular program. But even when you don't have something like that, you can just provide a checklist of things that need to be done. | ||
| - **Highly Repetitive Tasks:** These are tasks that need to be done over and over again, but nobody really wants to do them. Some good examples include code review, improving test coverage, upgrading dependency libraries. In addition to having clear success criteria, you can create "[skills](TODO)" that clearly describe your policies about how to perform these tasks, and improve the skills over time. |
There was a problem hiding this comment.
🔴 Critical: TODO placeholder needs to be replaced with an actual link.
| - **Highly Repetitive Tasks:** These are tasks that need to be done over and over again, but nobody really wants to do them. Some good examples include code review, improving test coverage, upgrading dependency libraries. In addition to having clear success criteria, you can create "[skills](TODO)" that clearly describe your policies about how to perform these tasks, and improve the skills over time. | |
| - **Highly Repetitive Tasks:** These are tasks that need to be done over and over again, but nobody really wants to do them. Some good examples include code review, improving test coverage, upgrading dependency libraries. In addition to having clear success criteria, you can create "[skills](/overview/skills)" that clearly describe your policies about how to perform these tasks, and improve the skills over time. |
| types: [opened, labeled] | ||
| pull_request: | ||
| types: [opened, synchronize] | ||
| issue_comment: | ||
| types: [created] | ||
|
|
||
| jobs: | ||
| auto-review: | ||
| if: github.event_name == 'pull_request' | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - name: OpenHands PR Review | ||
| uses: openhands/review-action@v1 | ||
| with: | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| issue-triage: | ||
| if: github.event_name == 'issues' && contains(github.event.issue.labels.*.name, 'needs-analysis') | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: openhands/triage-action@v1 | ||
| with: | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| ``` |
There was a problem hiding this comment.
🟠 Important: These GitHub Actions (openhands/review-action@v1, openhands/triage-action@v1) appear to be hypothetical examples rather than published actions.
Add a note clarifying this is an example workflow pattern, or link to actual working actions if they exist. Users might try to use these directly and encounter "action not found" errors.
| "openhands/usage/key-features", | ||
| "overview/model-context-protocol", | ||
| { | ||
| "group": "Integrations", | ||
| "group": "Skills", | ||
| "pages": [ | ||
| "openhands/usage/cloud/bitbucket-installation", | ||
| "openhands/usage/cloud/github-installation", | ||
| "openhands/usage/cloud/gitlab-installation", | ||
| "openhands/usage/cloud/slack-installation", | ||
| { | ||
| "group": "Project Management Tools", | ||
| "pages": [ | ||
| "openhands/usage/cloud/project-management/jira-integration" | ||
| ] | ||
| } | ||
| "overview/skills", |
There was a problem hiding this comment.
🟡 Suggestion: The "Onboarding OpenHands" section only has 2 items, and one (overview/skills/repo) is already nested under the Skills section in Product Guides.
Consider either:
- Expanding this section with more onboarding content, or
- Merging it into "Get Started" or "Essential Guidelines"
This would reduce navigation duplication and make the structure clearer.
This is an initial layout that roughly mimics Devin docs. It attempts to separate Documentation and Use Cases.
Some background:
TODO:
Pages marked TODO- are stubbed out but need more content
Content is needed under the "Use Cases" tab
Redesign the "Quick Start" page
I have read and reviewed the documentation changes to the best of my ability.
If the change is significant, I have run the documentation site locally and confirmed it renders as expected.
Summary of changes