Skip to content

delete runtime mapping, normalizations, revalidations#355

Open
Yostra wants to merge 4 commits intomainfrom
runtime_map_and_downstream
Open

delete runtime mapping, normalizations, revalidations#355
Yostra wants to merge 4 commits intomainfrom
runtime_map_and_downstream

Conversation

@Yostra
Copy link
Copy Markdown
Collaborator

@Yostra Yostra commented May 5, 2026

Summary

Cleanup follow-ups from PR #352:

  • Drop static alias maps. OPENAPI_RESOURCE_TABLE_ALIASES,
    STRIPE_OBJECT_TO_SYNC_OBJECT_ALIASES, and normalizeStripeObjectName
    removed. resolveTableName no longer merges a hidden default; tables derive
    from x-resourceId (._, lowercase).
    • radar.early_fraud_warningradar_early_fraud_warning
    • entitlements.active_entitlemententitlements_active_entitlement
    • entitlements.featureentitlements_feature (was colliding with
      active_entitlement)
  • Remove webhook revalidation. revalidate_objects config,
    REVALIDATE_ENTITIES, RevalidateEntityName, ResourceConfig.isFinalState,
    and the re-fetch branch in processStripeEvent are gone. processStripeEvent
    no longer takes Config. Events polling + destination _updated_at guards
    cover the original race.
  • Delete dead prefix helpers. PREFIX_RESOURCE_MAP,
    getResourceFromPrefix, getResourceConfigFromId, and getTableName had no
    callers; events route via event.data.object.object, not ID prefixes.
  • Conformance test: accept top-level anyOf config schemas
    (source-postgres, destination-stripe).

Base automatically changed from schema_update to main May 5, 2026 00:10
@Yostra Yostra force-pushed the runtime_map_and_downstream branch from 0b2d9a6 to 13c7fbd Compare May 5, 2026 00:14
@Yostra Yostra force-pushed the runtime_map_and_downstream branch from bff44a2 to 5c6d9c3 Compare May 6, 2026 02:26
@Yostra Yostra marked this pull request as ready for review May 6, 2026 04:13
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