CXP-756 Attach ChildResourceType to emitted org/OU resource#136
CXP-756 Attach ChildResourceType to emitted org/OU resource#136JavierCarnelli-ConductorOne wants to merge 2 commits into
Conversation
…ces (CXP-756) The SDK's syncer schedules child-resource crawls by reading a ChildResourceType annotation off each emitted resource instance, not off the registered ResourceType. organizationResource() and organizationalUnitResource() only declared it on the ResourceType, so the OU crawl never fired: organizational_unit sync count was always 0, and accounts re-parented under an OU ended up with a dangling parent pointing at a resource that was never emitted. Attach the annotation to each emitted org/OU resource instance, mirroring the working account.go -> permission_set_assignment pattern, and drop the now-dead (and misleading) annotation from the ResourceType definitions. Add regression tests asserting the annotation the syncer actually checks is present on emitted resources. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Connector PR Review: CXP-756 Attach ChildResourceType to emitted org/OU resourceBlocking Issues: 0 Suggestions: 0 Threads Resolved: 0 Review Summary Security Issues Correctness Issues Suggestions |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
…this affects the capabilities metadata
3 participants
The SDK's syncer schedules child-resource crawls by reading a ChildResourceType annotation off each emitted resource instance, not off the registered ResourceType. organizationResource() and organizationalUnitResource() only declared it on the ResourceType, so the OU crawl never fired: organizational_unit sync count was always 0, and accounts re-parented under an OU ended up with a dangling parent pointing at a resource that was never emitted.
Attach the annotation to each emitted org/OU resource instance, mirroring the working account.go -> permission_set_assignment pattern, and drop the now-dead (and misleading) annotation from the ResourceType definitions. Add regression tests asserting the annotation the syncer actually checks is present on emitted resources.