OpenHands / OpenHands/extensions

Define design-doc expectations for deep, high-risk PRs

Open
#549 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement ready-for-dev
Dominant language
Python
Stars
148
Forks
90
Avg merge
1d 17h
Merged PRs (30d)
36

Description

Desired Behavior

The extensions repository's code-review guide should ask for concise design context when a deep, high-risk extension, skill, plugin, or integration change cannot be evaluated efficiently from the diff, while avoiding unnecessary documentation for small or self-explanatory changes.

Because this repository has the .pr/ artifact workflow but does not currently include the repository-local pr-design-doc authoring skill, the guidance should permit either an available design-doc tool or an equivalent PR-description write-up and must not promise unavailable local tooling.

Acceptance Criteria
  • Deep, high-risk PRs without adequate design context receive a COMMENT review instead of automated approval.
  • Low-risk, trivial, generated, or self-explanatory changes are not blocked solely by design-doc absence or line count.
  • Equivalent design context states intent, important before/after behavior or contract shape, compatibility/risk, and grounded code references.
  • Guidance does not claim that .agents/skills/pr-design-doc/ exists in this repository.
  • A .pr/ artifact is not the sole required context at the point automated approval triggers cleanup.

This issue was created by an AI agent (OpenHands) on behalf of @enyst.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the repository's code-review guide and the .pr artifact and automated-approval workflow, then read how context is evaluated and cleanup is triggered. Update the guidance so deep, high-risk changes require adequate design context, while low-risk or self-explanatory changes remain unblocked; verify it does not claim unavailable local tooling or make the artifact the sole required context.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
66/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.