Skip to content

python(feat): annotations resource - #792

Open
Brandon-Shippy wants to merge 3 commits into
mainfrom
python/annotations-resource
Open

Brandon-Shippy wants to merge 3 commits into
mainfrom
python/annotations-resource

Conversation

@Brandon-Shippy

@Brandon-Shippy Brandon-Shippy commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Description

Adds an annotations resource to sift_client, covering data reviews, phases, and the review history.

  • AnnotationsAPIAsync with the usual get, list_, find, create, update, archive, unarchive, plus batch archive. Wired up as client.annotations
  • AnnotationLogsAPIAsync nested as client.annotations.logs
  • AnnotationsLowLevelClient wraps AnnotationService; the high-level client has zero proto imports
  • Pydantic models in sift_types/annotation.py
  • create_review(...) and create_phase(...) alongside create. Passing channels derives the asset
  • resolve(), flag(), reopen(), assign(), comment() on the resource and the Annotation
  • Sync wrappers and type stubs registered

Validation

  • End to End testing with manual script written to test each new API feature.
  • Verified against a local stack, including the annotation rendering in Explore

@github-actions

github-actions Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Python docs preview: https://sift-stack.github.io/sift/python/pr-792/

Deployed from 6bda0aa. The link may take up to a minute to become live as GitHub Pages propagates.

@Brandon-Shippy
Brandon-Shippy marked this pull request as ready for review September 17, 2026 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant