Skip to content

ref(integrations): Reorganize pipeline components into per-provider folders#116334

Merged
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-integrations-reorganize-pipeline-components-into-per-provider-folders
May 28, 2026
Merged

ref(integrations): Reorganize pipeline components into per-provider folders#116334
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-integrations-reorganize-pipeline-components-into-per-provider-folders

Conversation

@evanpurkhiser
Copy link
Copy Markdown
Member

Each pipelineIntegrationX.tsx (and its spec) moves into a per-provider folder under static/app/components/pipeline/ as integrationX/index.tsx. pipelineDummyProvider.tsx similarly moves to dummyProvider/index.tsx. Internal imports in registry.tsx and the two external imports of InstallationInfo are updated.

The motivation is to give each provider room to grow sibling modules — custom hooks, helpers, sub-components — without polluting the shared pipeline directory. The first such sibling (useDiscordAppDirectoryInstall) lands in a follow-up PR; this one is a pure rename with no behavior change.

shared/, registry.tsx, modal, usePipeline, types, testUtils, and index.stories stay flat. Exported symbol names are unchanged.

Verified: pnpm typecheck, pnpm run lint:js, and pnpm test-ci components/pipeline (140/140) all pass.

…olders

Move each pipelineIntegrationX.tsx (and its spec) into integrationX/index.tsx
under static/app/components/pipeline/, plus pipelineDummyProvider.tsx into
dummyProvider/index.tsx. Update internal imports in registry.tsx and the two
external imports of InstallationInfo.

This gives each provider room to grow sibling modules (hooks, helpers,
sub-components) without polluting the shared pipeline directory, and mirrors
the per-provider layout used by the backend integrations packages.

Pure rename; no behavior change.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.58%

@evanpurkhiser evanpurkhiser marked this pull request as ready for review May 27, 2026 19:58
@evanpurkhiser evanpurkhiser requested review from a team as code owners May 27, 2026 19:58
@evanpurkhiser evanpurkhiser enabled auto-merge (squash) May 27, 2026 20:01
@evanpurkhiser evanpurkhiser merged commit de03ba9 into master May 28, 2026
76 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/ref-integrations-reorganize-pipeline-components-into-per-provider-folders branch May 28, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants