Return generated model instances from connector operations
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 32/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- python
- Domaine
- api, backend-api-design, tooling
Piste de recherche
Start with the CodefulSdkGenerator and the generated connector operation methods, then inspect the pinned managed connector contracts and existing serialization/deserialization tests. Add coverage for modeled, nested, collection, optional, wire-name, binary, text, and empty responses, regenerate affected clients, and verify the artifacts match the paired generator revision and updated API documentation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
Generated connector clients expose response model classes, but operation methods currently deserialize JSON responses to weak dictionary types. For example, Office365Client.get_emails_async returns dict[str, Any] | None rather than an instance of its generated response model. This prevents callers and documentation from relying on typed response properties.
Identified in PR #89 review.
Proposed work
- Update the CodefulSdkGenerator to emit precise operation return annotations when the managed connector contract defines a response schema.
- Deserialize JSON responses into the corresponding generated model classes, including nested models, collections, optional fields, and wire-name mappings.
- Preserve dictionary, binary, text, and empty-response behavior when no generated response model applies.
- Regenerate affected connector clients from pinned contracts and link the paired AzureUX-BPM generator PR.
- Update API documentation after typed responses are available.
Acceptance criteria
- Connector operations with modeled response schemas return generated model instances rather than
dict[str, Any]. - Return annotations distinguish modeled, unmodeled, binary, text, and no-content responses.
- Serialization/deserialization tests cover nested objects, collections, optional fields, and wire-name mappings.
- Existing connector behavior remains compatible for responses without modeled schemas.
- Generated artifacts match the paired generator revision.
- Langage dominant
- Python
- Étoiles
- 0
- Forks
- 3
- Merge moyen
- 4 j 16 h
- PR mergées (30 j)
- 12
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Azure/connectors-python-sdk
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
Azure/connectors-python-sdk#91 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Azure/connectors-python-sdk
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
UKGovernmentBEIS/inspect_ai#5802 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
no-human-ai/no_human#660 ·
Les mainteneurs répondent en général sous 1 jour
-
Add a Security Insights v2 fileOuvertedocumentation good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
documentation need help question
Difficulté 1/5 1-3 heures Accessibilité débutants 66/100
phonology024/babelscribe#26 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100