vism

Вобщем в spatie/laravel-short-schedule тоже отказались люди от скрипта такого и делают так
Я вот задумался и тоже наверное проще отказаться и для пары джобов делать так

//Yes, we also decided not to use the package. We want to try use a task scheduling to run queue jobs with delay =)

//For example (command):

// run the job now
dispatch(new ExampleJob);

// and every 10 seconds this minute
for ($i = 1; $i <= 5; $i++ ) {
dispatch(new ExampleJob)->delay($i * 10);
}

А внутри джобы у меня unique интерфейс, чтоб не стакались они в случае чего

учитывая код, просто фигачь $_POST['email'] прямо во view :) Хуже не станет

Называется, услышал звон, да не знаю где он.

1. Сервисный слой ты превратил в контроллер.
Т.е. у тебя есть контроллер, а ты создал ещё один, т.к. где-то слышал, что нужны сервисы.
Но, сервисы не шлют ответы, это внутренний слой для отделения логики.
Ты можешь те методы вызвать через консоль, другие сервисы и т.п.
Там должен просто генерироваться return, а сам response уже в контроллере (чтоб контроллер своб функцию выполнял). Валидацию реквеста кстати тоже надо делать не в сервесе.

2. CategoryQuery - это вобще дичь дикая.
Ты опять создаёшь дубликат, потому что услышал о репозиториях, и даже интерфейс. Вот я угараю с людей, кто везде пихает интерфейсы. Интерфейсы нужны там, где они нужны. Где будет 2+ наследователей интерфейса.
Откуда у тебя тут наследники, если ты завязан на элокуент?
А ноги от индусов репозитаристов. Не нужен репозитарий, Query и интерфейсы при работе в элокуент. Он сам в себе это всё уже содержит и все повторяющиеся вещи обёрнуты уже. find, first, firstOrNew и т.д.
Так что используй элокуент и не переусложняй проект напрасно. Не надо делать сложно, делай просто.

3. CategoryAction.
Опять интерфейс...
Вот это Экшн по сути у тебя сервис. Тут ты имеено перенёс бизнес логику создания и редактирования и возвращаешь ответ данного метода. Вот удали свой CategoryAction и перенеси методы в сервис.

Это потому что вы пытаетесь делать то, что не понимаете, но прочитали в умной книжке/статье, увидели на ютубе.
джедай режим / Не нужно поля приватными делать в модели твоей.

Не пытайтесь делать то, что не понимаете, через лет 5 будет понятно, а пока делайте как в доках написано, про простому.
А бизнес логику в отдельные классы выносите. Пока ПО для боингов писать не начнёте, вам хватит.

А если уж так важно всё тру вей писать, то лару сразу надо удалить, качать симфони и пытаться превозмогать там.

Это задание, а не вопрос.
Ну, а если вопрос, то переписать код, чтоб формировался 1 запрос к базе и уже из него доставать данные.
Никаких проблем, много раз так делал.
Ну а если задание, то $200/h и за несколько часов составим нужный запрос:)

создаёте папку services и внутрь пихаете бизнес логику.

пользователей много?
Сессии в файлах хранятся?
Вангую я, что сессии у вам очищаются в это время.

Тут слишком мало, чтоб говорить прям вот точно. Но в целом верно.
Едиственное, если вам нужен exception чтоб сообщать пользователю, то лучше создать свой Exception и отлавливать его.
Другими словами, если мы не хотите возвращать в качестве ответа внутреннюю ошибку, что лучше разгранисить

нужно писать уникальный ИД и по нему выбирать строку.
Т.е.
1. в скрипте генерируете хэш
2. делаете апдейт с этим хэшем
3. по этому хэшу уже берете строку и отрабатываете.

любая, да миллион причин от железа до софта и искать можете от пары часов до пары лет и не найти

Поменяйте архитектуру, чтоб скрипт работал не 12 часов, а много мелких скриптов, которые будут добавлять пачки задач, выполняться, писать логи в базу

через крон например

А вы в курсе, чтоб из репозитария нельзя возвращать никакой Элокуент ВОБЩЕ
Вы уверены, что слово роепозитари вам нужно, если не понимаете зачем, значит не нужно.

Оберни свой говнокод в документ рэди и всё заработает.

Вобщем вы за ответом пришли туда, где 95% евда мидл-джуны.

Варианты ранее предложеные либо сильно меняют структуру, либо переусложняют.

Просто создаете второй метод и общее тело (формирование запроса) выносите в третий приватный/протектид метод. (вот vista1x не поленился и даже код накидал примерный)
В созданном методе заодно оставляете аргумент для сортировки.
Нужен будет еще - создаете третий.

Понимаете, что таких появится еще много - тогда уже меняете структуру, рефакторите.

array($request->all());

Заходите в книжный, выбираете максимально толстую и свежую редакцию книги по ПХП и читаете.