refactor: Removed unused header components from components-temp#26741
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
…to refactor/rm-unused-headers
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: My investigation confirmed:
Since no functional application code depends on these components, no user-facing features are affected, and no E2E tests need to be run. This is a safe cleanup of dead code from the component library. Performance Test Selection: |
Description
This PR removes four unused header components from
app/component-library/components-temp/: HeaderCollapsible, HeaderCollapsibleStandard, HeaderCollapsibleSubpage, and HeaderStackedSubpage.Reason for change: These components are not used anywhere in the app. Removing them reduces maintenance surface and dead code.
What changed:
Deleted component directories (21 files total)
useHeaderCollapsiblehook, test, stories, indexStorybook
require()entries from.storybook/storybook.requires.jsfor the removed components.No app code imported these components; removal is self-contained to
components-tempand Storybook registration.Changelog
This PR is not end-user-facing; it removes unused internal header components.
CHANGELOG entry: null
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/DSYS-495
Manual testing steps
Screenshots/Recordings
Before
N/A – code removal; components were unused.
After
N/A – same app behavior; removed components were not referenced.
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Low risk dead-code cleanup that deletes unused
components-tempheaders and their Storybook registrations; main risk is only if an out-of-tree or dynamic import relied on these removed exports.Overview
Removes unused header components from
components-temp. DeletesHeaderCollapsible,HeaderCollapsibleStandard,HeaderCollapsibleSubpage, andHeaderStackedSubpage(including associated types, hooks, tests, and Storybook stories/indices).Updates Storybook registration. Drops the corresponding
require()entries from.storybook/storybook.requires.jsso Storybook no longer attempts to load the removed stories.Written by Cursor Bugbot for commit ed2be56. This will update automatically on new commits. Configure here.