Skip to content

cleanup: remove unused Vue components#584

Open
HexaField wants to merge 2 commits into
devfrom
cleanup/remove-unused-components
Open

cleanup: remove unused Vue components#584
HexaField wants to merge 2 commits into
devfrom
cleanup/remove-unused-components

Conversation

@HexaField

@HexaField HexaField commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Removes 3 Vue components with zero references anywhere in the codebase.

Each deletion verified by grep-searching for:

  • Component name (PascalCase + kebab-case)
  • File path imports
  • Dynamic imports
  • Template usage

Deleted:

  • app/src/components/channel-view-options/ChannelViewOptions.vue
  • app/src/components/expandable-img/expandable-img.vue
  • app/src/views/main/profile/CommunityCard.vue

No functional changes.

Summary by CodeRabbit

Removed Features

  • Channel view options selector has been removed from the interface
  • Image expansion and fullscreen viewer functionality has been removed
  • Community profile card display has been removed

Tests

  • E2E test execution results recorded

Removed after verifying zero references across the codebase:
- app/src/components/channel-view-options/ChannelViewOptions.vue
- app/src/components/expandable-img/expandable-img.vue
- app/src/views/main/profile/CommunityCard.vue
@coderabbitai

coderabbitai Bot commented Apr 8, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@HexaField has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 14 minutes and 26 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 14 minutes and 26 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7def77b2-1c8b-4de5-8b7b-62b3049d84a3

📥 Commits

Reviewing files that changed from the base of the PR and between 9fa211c and f09095a.

📒 Files selected for processing (1)
  • .gitignore
📝 Walkthrough

Walkthrough

Removes three Vue components from the codebase (ChannelViewOptions, expandable-img, and CommunityCard). Adds test result metadata and error context documentation capturing a failed e2e test scenario for community creation.

Changes

Cohort / File(s) Summary
Removed Components
app/src/components/channel-view-options/ChannelViewOptions.vue, app/src/components/expandable-img/expandable-img.vue, app/src/views/main/profile/CommunityCard.vue
Deleted three Vue components including template logic, props, event emissions, and scoped styles. Affected functionality: channel view option selection, image expansion/collapse, and community card rendering.
Test Results & Artifacts
test-results/.last-run.json, test-results/flux-e2e-Flux-E2E-on-SPARQL-create-a-community-chromium/error-context.md
Added auto-generated test result metadata and Playwright error context for a failing e2e test (create a community). Includes failure assertion details, page snapshot, and test source context.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Three components bid farewell with grace,
Images and channels vanish without a trace,
Community cards take their final bow,
Test results whisper the tale right now,
Debugging artifacts hop into place! 🥕✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'cleanup: remove unused Vue components' directly and accurately summarizes the main change: the removal of three unused Vue components from the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cleanup/remove-unused-components

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@test-results/.last-run.json`:
- Around line 1-6: The committed test artifact test-results/.last-run.json
should be removed from version control and the test-results directory ignored;
update .gitignore to include the test-results/ directory (and optionally
.last-run.json), remove the file from the repo with git rm --cached (or delete
then commit) so it no longer appears in history moving forward, and ensure
future CI/dev runs write artifacts locally only; locate the file named
test-results/.last-run.json in the diff and the repository .gitignore to apply
these changes.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 74bf0df8-ca28-41fd-938b-8b034aa229aa

📥 Commits

Reviewing files that changed from the base of the PR and between 412c56e and 9fa211c.

⛔ Files ignored due to path filters (5)
  • e2e/screenshots/after-connect.png is excluded by !**/*.png
  • e2e/screenshots/after-load.png is excluded by !**/*.png
  • e2e/screenshots/community-dialog.png is excluded by !**/*.png
  • e2e/screenshots/create-community-form.png is excluded by !**/*.png
  • test-results/flux-e2e-Flux-E2E-on-SPARQL-create-a-community-chromium/test-failed-1.png is excluded by !**/*.png
📒 Files selected for processing (5)
  • app/src/components/channel-view-options/ChannelViewOptions.vue
  • app/src/components/expandable-img/expandable-img.vue
  • app/src/views/main/profile/CommunityCard.vue
  • test-results/.last-run.json
  • test-results/flux-e2e-Flux-E2E-on-SPARQL-create-a-community-chromium/error-context.md
💤 Files with no reviewable changes (3)
  • app/src/views/main/profile/CommunityCard.vue
  • app/src/components/channel-view-options/ChannelViewOptions.vue
  • app/src/components/expandable-img/expandable-img.vue

Comment thread test-results/.last-run.json Outdated
@netlify

netlify Bot commented Apr 8, 2026

Copy link
Copy Markdown

Deploy Preview for fluxsocial-dev ready!

Name Link
🔨 Latest commit 9fa211c
🔍 Latest deploy log https://app.netlify.com/projects/fluxsocial-dev/deploys/69d5dd51e675e00008b703e6
😎 Deploy Preview https://deploy-preview-584--fluxsocial-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Apr 8, 2026

Copy link
Copy Markdown

Deploy Preview for fluxsocial-dev ready!

Name Link
🔨 Latest commit f09095a
🔍 Latest deploy log https://app.netlify.com/projects/fluxsocial-dev/deploys/69d5df2c189d8b000828690e
😎 Deploy Preview https://deploy-preview-584--fluxsocial-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant