Repository navigation
docs(api): describe health and observe-only ingestion in OpenAPI (#44) - #138
Conversation
|
@BasilZafar11 is attempting to deploy a commit to the priyanshu's projects Team on Vercel. A member of the Team first needs to authorize it. |
CodewithJha
left a comment
There was a problem hiding this comment.
I checked the new OpenAPI text against the handlers and ran the ingest paths.
- The change is metadata only (
summaryanddescriptionon four routes inapp.py), so there is no behavior change. - The contract statements are accurate. I confirmed
schema_version=1andredaction.applied=trueare required,attestation=trueplusproject_idorlocal_project_keyare required at open, a repeated identical open is idempotent, and conflicting config returns 409. Size limits return 413, a bearer token is needed only whenMUTINY_API_TOKENis set, and/api/healthstays public. - The text keeps the observe-only point in front of API readers: Hosted stores evidence as data, never executes customer adapters and never opens uploaded paths.
- CI is green.
This satisfies #44. Merging.
|
Merged. Thanks for the contribution. You handled this well. The four route descriptions state the real contract rather than generic text: required This closes #44. If you'd like to continue working on Mutiny, these would be good related areas to look at:
No pressure on any of them. |
Summary
Fixes #44
Add explicit OpenAPI summaries and descriptions to four integration entry points: health, ingest campaign open, event/artifact batch upload, and completion. Descriptions explain public health access, conditional bearer auth, redaction/schema requirements, retry/conflict behavior, and that Hosted does not execute customer adapters or open uploaded paths.
Route handlers, models, status codes, and authentication behavior are unchanged.
Type of change
Checklist
Test plan
/openapi.jsonvia TestClient: all four summaries/descriptions appear, each states the customer execution boundary, and health explicitly states no bearer token is required.uv run --locked pytest tests/integration/test_api_hosted_ingest.py tests/integration/test_api.py -q --tb=short: 66 passed.uv run --locked ruff check: passed.git diff --check: passed.Full backend/reliability, CLI smoke, web build, and isolated artifact checks are left to CI for this description-only change. Branch starts at current upstream main
5ae4b0e, matching fork main.