You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This PR renames HeaderCompactSearch to HeaderSearch in app/component-library/components-temp/.
Reason for change: Simplifying the component name; "HeaderSearch" better reflects its purpose (a header with an integrated search field) without the "Compact" qualifier.
What changed:
Component rename
Created app/component-library/components-temp/HeaderSearch/ with:
HeaderSearch.tsx – main component (screen and inline variants unchanged)
Feature: HeaderCompactSearch renamed to HeaderSearchScenario: Lint and type check passGiven the branch is checked out
When the author runs yarn lint and yarn lint:tsc
Then both complete without errors
Scenario: No references to HeaderCompactSearchGiven the codebase is searched for HeaderCompactSearch
Then no imports or usages remain
Scenario: HeaderSearch unit tests passGiven the branch is checked out
When the author runs yarn jest app/component-library/components-temp/HeaderSearch/HeaderSearch.test.tsx
Then all tests pass
Scenario: HeaderSearch story rendersGiven Storybook is running
When the user opens "Components Temp / HeaderSearch"Then Screen and Inline stories render as before
Screenshots/Recordings
Before
N/A – component behavior unchanged; only name and file paths changed.
After
N/A – same header-with-search appearance; component is now HeaderSearch.
I've completed the PR template to the best of my ability
I've included tests if applicable
I've documented my code using JSDoc format if applicable
I've applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
Pre-merge reviewer checklist
I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.
Note
Low Risk
Low risk rename confined to component-library temp components and Storybook; main risk is missed import/path updates or changed testIDs impacting tests or downstream consumers.
Overview
Renames the temp header-with-search component from HeaderCompactSearch to HeaderSearch, including updating the exported types (Header*Variant/props) and default export.
Updates Storybook registration and stories to the new component name, and adjusts unit tests (including testIDs like header-search/header-search-inline) to match the rename.
Written by Cursor Bugbot for commit 3be9bec. This will update automatically on new commits. Configure here.
Selected Performance tags: None (no tests recommended)
Risk Level: low
AI Confidence: 95%
click to see 🤖 AI reasoning details
E2E Test Selection:
This PR is a pure component rename from HeaderCompactSearch to HeaderSearch within the component library's temporary components folder (app/component-library/components-temp/).
Key findings:
The component has NO external importers - grep searches for both HeaderCompactSearch and HeaderSearch show usage only within the component's own directory (test files, stories, and internal imports)
This is a self-contained component in components-temp that is not yet integrated into any app screens or features
The changes are purely cosmetic/organizational - renaming files and updating internal type names
No functional logic changes were made to the component
The component's own unit tests are included and updated with the rename
Since this component is not used anywhere in the application, no E2E tests can be affected by this change. The storybook.requires.js update is auto-generated and only affects Storybook documentation, not the app itself.
Performance Test Selection:
This is a pure component rename with no functional changes. The HeaderSearch component is not used anywhere in the app (no external importers found), so there is no impact on app performance. The changes are limited to file/type renaming within the component library's temporary components folder.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR renames HeaderCompactSearch to HeaderSearch in
app/component-library/components-temp/.Reason for change: Simplifying the component name; "HeaderSearch" better reflects its purpose (a header with an integrated search field) without the "Compact" qualifier.
What changed:
Component rename
app/component-library/components-temp/HeaderSearch/with:HeaderSearch.tsx– main component (screen and inline variants unchanged)HeaderSearch.types.ts–HeaderSearchVariant,HeaderSearchProps,HeaderSearchScreenProps,HeaderSearchInlinePropsHeaderSearch.test.tsx– tests updated for new name and testIDs (header-search,header-search-inline)HeaderSearch.stories.tsx– Storybook entry "Components Temp / HeaderSearch"index.ts– exports default and typesHeaderCompactSearch*→HeaderSearch*(e.g.HeaderSearchVariant,HeaderSearchProps).Removed HeaderCompactSearch
app/component-library/components-temp/HeaderCompactSearch/(index, types, component, stories, test)..storybook/storybook.requires.jsto requireHeaderSearch.stories.tsxinstead ofHeaderCompactSearch.stories.tsx.No other app code used HeaderCompactSearch; the rename is limited to the component folder and Storybook.
Changelog
This PR is not end-user-facing; it renames an internal header component.
CHANGELOG entry: null
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/DSYS-497
Manual testing steps
Screenshots/Recordings
Before
N/A – component behavior unchanged; only name and file paths changed.
After
N/A – same header-with-search appearance; component is now HeaderSearch.
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Low risk rename confined to component-library temp components and Storybook; main risk is missed import/path updates or changed
testIDs impacting tests or downstream consumers.Overview
Renames the temp header-with-search component from
HeaderCompactSearchtoHeaderSearch, including updating the exported types (Header*Variant/props) and default export.Updates Storybook registration and stories to the new component name, and adjusts unit tests (including
testIDs likeheader-search/header-search-inline) to match the rename.Written by Cursor Bugbot for commit 3be9bec. This will update automatically on new commits. Configure here.