From 810ad5ee64c7e57dc6e1df1e90f3809a6f748f79 Mon Sep 17 00:00:00 2001
From: Tetardtek <kvnn64@gmail.com>
Date: Tue, 24 Mar 2026 18:27:36 +0100
Subject: [PATCH] fix: quests respect zone locking + level ordering fixes
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Quêtes d'arcs filtrées par zones débloquées — pas de quête Égouts
visible tant que l'arc Marais n'est pas complété.
Gardien des marais: level 5→4. Dératisation: level 4→5.
Exterminateur: description corrigée (kill_any x30, pas "chaque espèce").
---
 src/quest/quest.service.ts | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/quest/quest.service.ts b/src/quest/quest.service.ts
index 5acefbc..7277130 100644
--- a/src/quest/quest.service.ts
+++ b/src/quest/quest.service.ts
@@ -8,6 +8,7 @@ import { QuestArc } from './quest-arc.entity';
 import { PlayerQuest } from './player-quest.entity';
 import { PlayerQuestArc } from './player-quest-arc.entity';
 import { Character } from '../character/entities/character.entity';
+import { getUnlockedZones } from '../common/zone-access';
 
 const MAX_ACTIVE_QUESTS = 3;
 
@@ -39,6 +40,9 @@ export class QuestService {
     });
     const questStatusMap = new Map(playerQuests.map((pq) => [pq.questId, pq.status]));
 
+    // Zone locking — only show quests from unlocked zones
+    const unlockedZones = await getUnlockedZones(characterId, this.arcRepo, this.playerArcRepo);
+
     const quests = await this.questRepo.find({
       relations: ['arc'],
       order: { arcOrder: 'ASC' },
@@ -47,10 +51,11 @@ export class QuestService {
     return quests.filter((q) => {
       if (q.minLevel > character.level) return false;
 
+      // Zone filter: if quest belongs to an arc with a zone, check zone is unlocked
+      if (q.arc?.zone && !unlockedZones.includes(q.arc.zone)) return false;
+
       const status = questStatusMap.get(q.id);
-      // Already active or completed (waiting claim) → not available
       if (status === 'active' || status === 'completed') return false;
-      // Already claimed and not repeatable → not available
       if (status === 'claimed' && !q.repeatable) return false;
 
       return true;