@rpg-engine/long-bow 0.6.82 → 0.6.83

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rpg-engine/long-bow",
-  "version": "0.6.82",
+  "version": "0.6.83",
   "license": "MIT",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",

package/src/components/Quests/QuestList.tsx

@@ -32,14 +32,18 @@ export const QuestList: React.FC<IQuestListProps> = ({ quests }) => {
               <Label>Description:</Label>
               <Value>{quest.description}</Value>
             </QuestItem>
-            <QuestItem>
-              <Label>Objectives:</Label>
-              <Value>{formatObjectives(quest.objectives)}</Value>
-            </QuestItem>
-            <QuestItem>
-              <Label>Rewards:</Label>
-              <Value>{formatRewards(quest.rewards)}</Value>
-            </QuestItem>
+            {quest.objectives && quest.objectives.length > 0 && (
+              <QuestItem>
+                <Label>Objectives:</Label>
+                <Value>{formatObjectives(quest.objectives)}</Value>
+              </QuestItem>
+            )}
+            {quest.rewards && quest.rewards.length > 0 && (
+              <QuestItem>
+                <Label>Rewards:</Label>
+                <Value>{formatRewards(quest.rewards)}</Value>
+              </QuestItem>
+            )}
           </QuestCard>
         ))
       ) : (
@@ -104,34 +108,35 @@ const formatObjectives = (
   objectives: (IQuestObjectiveKill | IQuestObjectiveInteraction)[]
 ) => {
   try {
-    if (!objectives || !Array.isArray(objectives)) return 'No objectives';
+    if (!objectives || !Array.isArray(objectives)) return '';
     return objectives
       .map(objective => {
         if ('killCountTarget' in objective) {
           const killObjective = objective as IQuestObjectiveKill;
           return `Kill ${formatText(
-            killObjective.creatureKeys?.join(', ') ?? 'Unknown'
+            killObjective.creatureKeys?.join(', ') ?? ''
           )}: ${killObjective.killCount ?? 0}/${killObjective.killCountTarget ??
             0}`;
         } else if ('targetNPCkey' in objective) {
           const interactionObjective = objective as IQuestObjectiveInteraction;
           return `Interact with NPC: ${formatText(
-            interactionObjective.targetNPCkey ?? 'Unknown'
+            interactionObjective.targetNPCkey ?? ''
           )}`;
         } else {
-          return 'Unknown objective';
+          return '';
         }
       })
+      .filter(Boolean)
       .join('; ');
   } catch (error) {
     console.error('Error formatting objectives:', error);
-    return 'Error formatting objectives';
+    return '';
   }
 };
 
 const formatRewards = (rewards: IQuest['rewards']) => {
   try {
-    if (!rewards || !Array.isArray(rewards)) return 'No rewards';
+    if (!rewards || !Array.isArray(rewards)) return '';
 
     return rewards
       .map(reward => {
@@ -158,7 +163,7 @@ const formatRewards = (rewards: IQuest['rewards']) => {
             : `Spells: ${formatText(spellKeysFormatted)}`;
         }
 
-        return formattedReward || 'No rewards';
+        return formattedReward || '';
       })
       .filter(Boolean)
       .join('; ');
@@ -167,12 +172,12 @@ const formatRewards = (rewards: IQuest['rewards']) => {
       `Error formatting rewards: ${JSON.stringify(rewards)}:`,
       error
     );
-    return 'Error on rewards';
+    return '';
   }
 };
 
 const formatText = (text: string) => {
-  if (!text) return 'Unknown';
+  if (!text) return '';
   return text
     .split('-')
     .map(word => word.charAt(0).toUpperCase() + word.slice(1))
@@ -193,7 +198,7 @@ const getStatusColor = (status?: QuestStatus) => {
 };
 
 const formatStatus = (status?: QuestStatus) => {
-  if (!status) return 'Unknown';
+  if (!status) return '';
   return status
     .split(/(?=[A-Z])/)
     .join(' ')