-
Notifications
You must be signed in to change notification settings - Fork 191
Open
Labels
invariantsthe design invariants of the codebasethe design invariants of the codebase
Description
Problem
We have API compliance tests, but they are currently optional/manual:
- label-triggered or manually dispatched
- not a default enforcement path for API-touching PRs
- configured with
continue-on-error
So today they are a useful tool, but not a reliable part of public API enforcement.
What we may want
Automatically run at least a reduced API compliance suite for PRs that likely affect public APIs, for example:
- PRs touching
openhands-sdk/ - PRs touching
openhands-agent-server/ - PRs labeled
api-change - release PRs
Possible rollout options:
- Run a smoke subset automatically for likely API changes.
- Keep the full suite manual/label-triggered.
- Fail loudly or at least post a very visible result summary.
Why
Structural diff checks are necessary, but runtime/API compliance coverage gives extra confidence that behavior remains usable for real clients.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
invariantsthe design invariants of the codebasethe design invariants of the codebase