Skip to content

Remove pre-existing dead code surfaced by the automations removal sweep #334

Description

@paulocastellano

While removing the automations module (#333) an orphan sweep of resources/js and app/ turned up dead code that predates automations. It is out of scope for that PR and tracked here so it is not lost.

Everything below has zero importers/references on main (verified by grepping for the import path or class name across resources/js, app, config, routes, tests, resources/views).

Frontend components and layouts (starter kit, Jan 2026)

  • resources/js/components/AlertError.vue
  • resources/js/components/AppContent.vue
  • resources/js/components/AppShell.vue
  • resources/js/components/Breadcrumbs.vue
  • resources/js/components/Heading.vue
  • resources/js/components/Icon.vue
  • resources/js/components/NavFooter.vue
  • resources/js/components/PlaceholderPattern.vue
  • resources/js/components/UserInfo.vue
  • resources/js/components/WorkspaceSwitcher.vue
  • resources/js/components/posts/previews/LinkCard.vue (Bluesky link card preview, Jul 2026)
  • resources/js/composables/useDateMaska.ts
  • resources/js/layouts/GuestLayout.vue
  • resources/js/layouts/auth/AuthCardLayout.vue
  • resources/js/layouts/auth/AuthSimpleLayout.vue

components/ui/* directories with no importer outside ui/

  • alert-dialog, aspect-ratio, button-group, carousel, context-menu, drawer, empty, hover-card, input-group, input-otp, kbd, native-select, navigation-menu, progress, radio-group, range-calendar, resizable, scroll-area, sheet, tags-input
  • After removing them, re-check package.json for packages that only they used (embla-carousel-vue, vue-input-otp, vaul-vue, maska are candidates).

PHP

  • app/Concerns/PasswordValidationRules.php
  • app/Enums/Ai/Orientation.php
  • app/Exceptions/Ai/QuotaExhaustedException.php

npm

  • axios: no import anywhere in resources/js. Inertia v3 no longer depends on it.

Notes

  • The import/order eslint errors on main (32 at the time of writing) are auto-fixed by npm run lint in CI, but each PR that runs lint locally picks up unrelated import reorders. Worth one dedicated lint-fix commit.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions