Skip to content

fix: remove stale URN-prefixed alias files breaking Zeitwerk#466

Merged
gjtorikian merged 1 commit intomainfrom
gjtorikian/remove-stale-urn-files
Apr 27, 2026
Merged

fix: remove stale URN-prefixed alias files breaking Zeitwerk#466
gjtorikian merged 1 commit intomainfrom
gjtorikian/remove-stale-urn-files

Conversation

@gjtorikian
Copy link
Copy Markdown
Contributor

These 5 files were left over from a previous codegen run. The current oagen-emitters strips URN prefixes via stripUrnPrefix, so these alias files are no longer generated and are not tracked in the oagen manifest. Their presence causes Zeitwerk autoload errors.

Fixes #463

These 5 files were left over from a previous codegen run. The current
oagen-emitters strips URN prefixes via `stripUrnPrefix`, so these alias
files are no longer generated and are not tracked in the oagen manifest.
Their presence causes Zeitwerk autoload errors because the `OAuth`
casing in the constant name doesn't match Zeitwerk's default inflection
(`Oauth`), and no inflection override was ever emitted for them.

Fixes #463

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@gjtorikian gjtorikian requested review from a team as code owners April 27, 2026 20:56
@gjtorikian gjtorikian requested a review from tribble April 27, 2026 20:56
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

This PR removes 5 stale auto-generated alias files that define deprecated URN-prefixed constant names (e.g. UrnIetfParamsOAuthGrantTypeDeviceCodeSessionAuthenticateRequest) pointing to their modern equivalents. Because the current oagen codegen no longer produces these files and they are absent from the oagen manifest, Zeitwerk raises autoload errors when it encounters them, so deletion is the correct fix.

Confidence Score: 5/5

Purely a deletion of stale auto-generated alias files; safe to merge.

All 5 changes are simple file deletions of deprecated, auto-generated constant aliases. No logic is introduced or modified, and the PR description clearly explains the root cause (Zeitwerk autoload failures from orphaned codegen artifacts). No security, correctness, or breaking-change concerns were found.

No files require special attention.

Important Files Changed

Filename Overview
lib/workos/user_management/urn_ietf_params_oauth_grant_type_device_code_session_authenticate_request.rb Deleted stale auto-generated alias mapping UrnIetfParamsOAuthGrantTypeDeviceCodeSessionAuthenticateRequest → DeviceCodeSessionAuthenticateRequest; removal fixes Zeitwerk autoload error.
lib/workos/user_management/urn_workos_oauth_grant_type_email_verification_code_session_authenticate_request.rb Deleted stale auto-generated alias mapping UrnWorkosOAuthGrantTypeEmailVerificationCodeSessionAuthenticateRequest → EmailVerificationCodeSessionAuthenticateRequest.
lib/workos/user_management/urn_workos_oauth_grant_type_magic_auth_code_session_authenticate_request.rb Deleted stale auto-generated alias mapping UrnWorkosOAuthGrantTypeMagicAuthCodeSessionAuthenticateRequest → MagicAuthCodeSessionAuthenticateRequest.
lib/workos/user_management/urn_workos_oauth_grant_type_mfa_totp_session_authenticate_request.rb Deleted stale auto-generated alias mapping UrnWorkosOAuthGrantTypeMfaTotpSessionAuthenticateRequest → MFATotpSessionAuthenticateRequest.
lib/workos/user_management/urn_workos_oauth_grant_type_organization_selection_session_authenticate_request.rb Deleted stale auto-generated alias mapping UrnWorkosOAuthGrantTypeOrganizationSelectionSessionAuthenticateRequest → OrganizationSelectionSessionAuthenticateRequest.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[oagen codegen run] -->|stripUrnPrefix applied| B[Generates canonical class files\ne.g. DeviceCodeSessionAuthenticateRequest]
    A -->|stale artifact — no longer generated| C[URN-prefixed alias files\ne.g. UrnIetfParams...Request]
    C -->|Zeitwerk encounters file| D[Autoload error:\nconstant name mismatch]
    B -->|This PR: deletes alias files| E[Zeitwerk autoloads cleanly]
    D -.->|fixed by PR #466| E
Loading

Reviews (1): Last reviewed commit: "fix: remove stale URN-prefixed alias fil..." | Re-trigger Greptile

@gjtorikian gjtorikian merged commit 92b2aa5 into main Apr 27, 2026
7 checks passed
@gjtorikian gjtorikian deleted the gjtorikian/remove-stale-urn-files branch April 27, 2026 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

zeitwerk

1 participant