Как сделать это более грамотно в Laravel

1.Сразу предлагаю Вам вынести валидацию в отдельный класс, поскольку здесь Вы получаете все данные из $request, который сейчас даже нигде не описан. Примерно так:

public function addNews(StoreNewsRequest $request)
{
...
}

2.Историю лучше вести с привязкой через полиморфические связи. Таким образом, в БД всегда будет храниться история взаимодействий с данной новостью, а Вы сможете легко посмотреть, кто и что делал с этой новостью на протяжении всего времени. Больше про полиморфические связи в Eloquent здесь.

3.Писать в коде прямым текстом вот так: $history->action = "Добавил новость"; - зло. Лучше это вынести в константу модели History:

class History 
{
   const USER_ADDED_NEWS = "Добавил новость";
}    
...
$history->action = History::USER_ADDED_NEWS;

В заключении посоветую почитать эти best practices, явно много нового и полезного для себя найдете :)