Skip to content

Remove unused chat directory and related components#2479

Merged
sawka merged 2 commits intomainfrom
copilot/remove-unused-chat-files
Oct 26, 2025
Merged

Remove unused chat directory and related components#2479
sawka merged 2 commits intomainfrom
copilot/remove-unused-chat-files

Conversation

Copy link
Contributor

Copilot AI commented Oct 26, 2025

Removes the entire frontend/app/view/chat/ directory and its dependencies—avatar and collapsiblemenu components—which were only referenced within the chat code itself.

Removed:

  • frontend/app/view/chat/ directory (13 files: channels, chat, chatbox, chatmessages, userlist + their styles/stories)
  • frontend/app/element/avatar.{tsx,scss,stories.tsx} (only used by chat/userlist)
  • frontend/app/element/collapsiblemenu.{tsx,scss,stories.tsx} (only used by chat/channels)
  • avatar-dims-mixin from frontend/app/mixins.scss (only used by avatar.scss)

Total: 19 files, ~1300 lines removed.

Original prompt

Can you verify that everything in frontend/app/view/chat is not used? if so remove all the files. i also think avatar.tsx is then unused... if so, remove that too... and then i also thnk collapsibleMenu is unused? finally if we can remove avatar, i think we can remove avatar-dims-mixin from mixins.scss...

(do not modify any go.mod go.sum files -- e.g. in the tsunami directory as those files are not part of the frontend or this task)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

- Removed entire frontend/app/view/chat/ directory (13 files)
- Removed avatar.tsx, avatar.scss, avatar.stories.tsx
- Removed collapsiblemenu.tsx, collapsiblemenu.scss, collapsiblemenu.stories.tsx
- Removed avatar-dims-mixin from mixins.scss

All files were verified to be unused outside their own directory.

Co-authored-by: sawka <2722291+sawka@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove unused files from frontend/app/view/chat Remove unused chat directory and related components Oct 26, 2025
Copilot AI requested a review from sawka October 26, 2025 05:20
@sawka sawka marked this pull request as ready for review October 26, 2025 05:29
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 26, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@sawka sawka merged commit e277174 into main Oct 26, 2025
8 of 10 checks passed
@sawka sawka deleted the copilot/remove-unused-chat-files branch October 26, 2025 05:33
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.

3 participants