Skip to content

feat: Use semantic method instead of POST by default - #315

Merged
razor-x merged 1 commit into
mainfrom
claude/blueprint-update-tests-fb0p3h
Aug 19, 2026
Merged

feat: Use semantic method instead of POST by default#315
razor-x merged 1 commit into
mainfrom
claude/blueprint-update-tests-fb0p3h

Conversation

@razor-x

@razor-x razor-x commented Aug 19, 2026

Copy link
Copy Markdown
Member

Summary

This PR adds three new unmanaged model classes to the Seam C# SDK and corrects HTTP method routing across multiple API endpoints. The unmanaged models represent access grants, access methods, and user identities that do not have client sessions, instant keys, customization profiles, or keys.

Key Changes

New Models Added

  • UnmanagedAccessGrant: Represents unmanaged access grants with support for errors, pending mutations, and requested access methods
  • UnmanagedAccessMethod: Represents unmanaged access methods with error handling and pending mutations
  • UnmanagedUserIdentity: Represents unmanaged user identities with ACS user integration and error tracking

HTTP Method Corrections

Fixed incorrect HTTP method routing in multiple API endpoints:

  • Changed Post to Get for read operations: /devices/get, /devices/unmanaged/get, /locks/get, /access_methods/get, /events/get, /acs/systems/get, /acs/entrances/get, /acs/encoders/get, /action_attempts/get
  • Changed Post to Delete for delete operations: /user_identities/delete, /access_codes/delete, /thermostats/delete_climate_preset, /acs/credentials/delete, /spaces/delete, /acs/access_groups/delete, /acs/users/delete, /client_sessions/delete, /noise_thresholds_noise_sensors/delete, /schedules_thermostats/delete, /access_grants/delete, /connected_accounts/delete, /instant_keys/delete, /phones/deactivate, /unmanaged_access_codes/delete, /webhooks/delete, /daily_programs_thermostats/delete
  • Changed Post to Get for list operations: /user_identities/list

Model Enhancements

  • Added new device types: yacan_lock and aqara_lock to Device and UnmanagedDevice enums
  • Added new provider: yacan to ConnectWebviews and DeviceProvider
  • Added insufficient_permissions error type to Device, AcsSystem, AccessCode, and UnmanagedAccessCode models
  • Added akilesMetadata field to AcsCredential, AcsEntrance, and ActionAttemptScanCredentialResultAcsCredentialOnSeam
  • Added displayStatus field to AccessGrant and AccessMethod models
  • Added mergedUserIdentityIds and mergedUserIdentityKeys fields to UserIdentity
  • Added deepLink parameter to CreatePortalRequest in Customers API
  • Added akiles_member_group to AcsAccessGroup access group types
  • Made acsCredentialOnSeam nullable in ActionAttemptScanCredentialResult
  • Marked EventAccessCodeDelayInRemovingFromDevice as obsolete

API Cleanup

  • Removed deprecated customMetadataHas parameter from Locks and NoiseSensors list requests
  • Removed deprecated connectedAccountIds, createdBefore, deviceIds, limit, manufacturer, pageCursor, search, spaceId, and unstableLocationId parameters from NoiseSensors list request
  • Removed connectedAccountType field from EventConnectedAccountDeleted

Dependencies

  • Updated @seamapi/blueprint from ^1.0.0 to ^1.8.0
  • Updated @seamapi/types from 1.975.0 to ^1.1026.0

https://claude.ai/code/session_012uVXgF6ytTSoXRSkv1Cd1P

Bumps @seamapi/blueprint 1.0.0 -> 1.8.0 and @seamapi/types 1.975.0 ->
1.1026.0, then regenerates the C# SDK. The blueprint update changes most
`get` endpoints from POST to GET and adds new resources/fields (e.g.
unmanaged access grants/methods/user identities, Akiles credential
metadata, new boolean warning fields).

Fixes RequestTransportTests.SendsPostParamsAsAJsonBody, which asserted
that /devices/get sends a POST; that endpoint (like other `get`
endpoints) is now GET under the updated blueprint, so the test was
switched to exercise a still-POST endpoint (/connected_accounts/sync).
@razor-x razor-x changed the title Add unmanaged access models and fix HTTP method routing feat: Use semantic method instead of POST by default Aug 19, 2026
@razor-x
razor-x marked this pull request as ready for review August 19, 2026 04:35
@razor-x
razor-x merged commit 6981756 into main Aug 19, 2026
14 checks passed
@razor-x
razor-x deleted the claude/blueprint-update-tests-fb0p3h branch August 19, 2026 04:35
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.

2 participants