Address PR #152 review feedback

- SituationRestImpl.forwardSituation: wrap body in try/catch Exception
  and return ALECRestUtils.somethingWentWrong(e) to preserve the
  structured 5xx response that the previous forwardAndStoreSituation
  helper provided.
- SituationDetailTab.vue: drop the dangling useUserStore() call left
  over after the consent gate was removed (the import was already
  deleted, the variable was unused).

Note: the shipped Karaf UI bundle under
features/ui/src/main/resources/ui-ext/ still contains the removed
welcome/configuration flow and needs a rebuild. That rebuild is
deferred to a follow-up - pre-existing case-sensitivity bugs in the
ui/ sources (axiosinstances.ts vs ./axiosInstances import,
TimeLine.vue vs Timeline import) prevent yarn build from succeeding
on a case-sensitive filesystem, and resolving those is out of scope
for this PR.

Assisted-by: Claude Code:Opus 4.7

Author: marshallmassengill

features/ui/src/main/java/org/opennms/alec/rest/SituationRestImpl.java

@@ -248,8 +248,12 @@ private boolean alarmIsNotInAnotherSituation(String reductionKey) throws Interru
     }
 
     private Response forwardSituation(Situation oldSituation, Set<Alarm> alarms) {
-        Situation newSituation = ImmutableSituation.newBuilderFrom(oldSituation).setAlarms(alarms).build();
-        situationDatasource.forwardSituation(newSituation);
-        return Response.ok().build();
+        try {
+            Situation newSituation = ImmutableSituation.newBuilderFrom(oldSituation).setAlarms(alarms).build();
+            situationDatasource.forwardSituation(newSituation);
+            return Response.ok().build();
+        } catch (Exception e) {
+            return ALECRestUtils.somethingWentWrong(e);
+        }
     }
 }

ui/src/components/SituationDetailTab.vue

@@ -23,8 +23,6 @@ const situationStore = useSituationsStore()
 
 const REJECTED = CONST.REJECTED
 
-const userStore = useUserStore()
-
 const props = defineProps<{
 	situationInfo: TSituation
 }>()