Skip to content

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

Description

@enyst

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.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestready-for-devScoped for contribution; managed by repository readiness checks.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions