重み付きプランナープールとしてダンジョンを選択できるようにする (#355)

Author: kijimad

docs/design/8.md

@@ -64,11 +64,9 @@ var PlannerTypeCave = PlannerType{
 package dungeon
 
 type DungeonDefinition struct {
-    ID             string
     Name           string
     Description    string
     TotalFloors    int              // 総階層数
-    MinLevel       int              // 解放条件
     EnemyTableName string           // 敵テーブル
     ItemTableName  string           // アイテムテーブル
     PlannerPool    []PlannerWeight  // 使用するステージ種類
@@ -77,11 +75,6 @@ type DungeonDefinition struct {
 type PlannerWeight struct {
     PlannerType mapplanner.PlannerType
     Weight      int
-    Config      PlannerConfig  // Planner固有の設定
-}
-
-type PlannerConfig struct {
-    WallType  WallType  // 壁種類
 }
 ```
 
@@ -93,7 +86,6 @@ type PlannerConfig struct {
 
 ```go
 var DungeonForest = DungeonDefinition{
-    ID:             "forest",
     Name:           "迷いの森",
     Description:    "木々が生い茂る危険な森",
     TotalFloors:    10,
@@ -104,29 +96,19 @@ var DungeonForest = DungeonDefinition{
         {
             PlannerType: PlannerTypeForest,
             Weight:      5,
-            Config: PlannerConfig{
-                WallType: WallTypeTree,
-            },
         },
         {
             PlannerType: PlannerTypeSmallRoom,
             Weight:      2,
-            Config: PlannerConfig{
-                WallType: WallTypeTree,  // 森ダンジョンでも木の壁
-            },
         },
         {
             PlannerType: PlannerTypeBigRoom,
             Weight:      1,
-            Config: PlannerConfig{
-                WallType: WallTypeTree,
-            },
         },
     },
 }
 
 var DungeonCave = DungeonDefinition{
-    ID:             "cave",
     Name:           "暗い洞窟",
     Description:    "光の届かない深い洞窟",
     TotalFloors:    15,
@@ -137,29 +119,19 @@ var DungeonCave = DungeonDefinition{
         {
             PlannerType: PlannerTypeCave,
             Weight:      6,
-            Config: PlannerConfig{
-                WallType: WallTypeRock,
-            },
         },
         {
             PlannerType: PlannerTypeSmallRoom,
             Weight:      1,
-            Config: PlannerConfig{
-                WallType: WallTypeRock,
-            },
         },
         {
             PlannerType: PlannerTypeBigRoom,
             Weight:      2,
-            Config: PlannerConfig{
-                WallType: WallTypeRock,
-            },
         },
     },
 }
 
 var DungeonRuins = DungeonDefinition{
-    ID:             "ruins",
     Name:           "古代遺跡",
     Description:    "かつて栄えた文明の跡",
     TotalFloors:    20,
@@ -170,23 +142,14 @@ var DungeonRuins = DungeonDefinition{
         {
             PlannerType: PlannerTypeSmallRoom,
             Weight:      4,
-            Config: PlannerConfig{
-                WallType: WallTypeBrick,
-            },
         },
         {
             PlannerType: PlannerTypeRuins,
             Weight:      3,
-            Config: PlannerConfig{
-                WallType: WallTypeStone,
-            },
         },
         {
             PlannerType: PlannerTypeBigRoom,
             Weight:      2,
-            Config: PlannerConfig{
-                WallType: WallTypeBrick,
-            },
         },
     },
 }
@@ -260,20 +223,8 @@ internal/states/
 └─────────────────────────────────┘
 ```
 
-### 7. データ保存
-
-```go
-// セーブデータに含める
-type DungeonProgress struct {
-    DungeonID    string  // 現在挑戦中のダンジョン
-    CurrentFloor int     // 現在階層
-    Seed         uint64  // マップ生成シード（再現性のため）
-}
-```
-
 ### 8. 今後の拡張案
 
-- FloorConfig追加（階層ごとに壁/敵テーブルを変えたい場合）
 - ダンジョン固有のギミック（毒沼、溶岩など）
 - ランダムイベント（宝箱部屋、商人など）
 - 難易度選択

docs/design/9.md

@@ -0,0 +1,64 @@
+# シード管理設計
+
+## 概要
+
+ゲームの乱数を管理し、再現可能なゲームプレイを実現する。
+
+- **ステージ生成**: stageSeed で再現可能（深度に依存しない）
+- **ゲームプレイ**: world.Config.RNG を使用（同じ操作なら再現可能）
+
+## 初期化フロー
+
+### 1. Config読み込み時 (`config.Load()`)
+
+```go
+// 環境変数 RUINS_SEED が設定されている場合
+if os.Getenv("RUINS_SEED") != "" {
+    cfg.Seed = 環境変数の値
+} else {
+    // 未設定の場合はランダム生成
+    cfg.Seed = rand.Uint64()
+}
+
+// SeedからRNGを生成
+cfg.RNG = rand.New(rand.NewPCG(cfg.Seed, 0))
+```
+
+### 2. ステージ生成時 (`DungeonState.OnStart()`)
+
+```go
+// ステージ用シードをRNGから生成
+stageSeed := world.Config.RNG.Uint64()
+
+// SelectPlanner用のRNGはstageSeedから生成
+stageRNG := rand.New(rand.NewPCG(stageSeed, 0))
+builderType, err := dungeon.SelectPlanner(def, stageRNG)
+
+// PlanにはstageSeedを渡す（内部で独自にRNGを作成）
+plan, err := mapplanner.Plan(world, ..., stageSeed, builderType)
+```
+
+### 3. ゲームプレイ中
+
+```go
+// 戦闘、ドロップなどは world.Config.RNG を使用
+weaponName, err := commandTable.SelectByWeight(world.Config.RNG)
+materialName, err := dropTable.SelectByWeight(world.Config.RNG)
+```
+
+## 再現性
+
+| シナリオ | 再現性 |
+|---------|--------|
+| 同じ RUINS_SEED + 同じ操作 | 完全再現可能 |
+| 同じ stageSeed | 同じステージを再現可能 |
+| 異なる操作 | ステージは同じ、以降は異なる |
+
+## RNG使用箇所
+
+| 用途 | 使用するRNG |
+|------|------------|
+| プランナー選択 | stageSeed から生成した RNG |
+| マップ生成 | stageSeed（Plan内部でRNG作成） |
+| 攻撃選択 | world.Config.RNG |
+| ドロップ選択 | world.Config.RNG |

internal/cmd/generate_enemy_doc.go

@@ -137,12 +137,7 @@ func calculateEnemyProbabilities(table raw.EnemyTable, depth int) map[string]flo
 	// 深度範囲内のエントリをフィルタリング
 	validEntries := make([]raw.EnemyTableEntry, 0, len(table.Entries))
 	for _, entry := range table.Entries {
-		// MinDepthチェック（0は制限なし）
-		if entry.MinDepth > 0 && depth < entry.MinDepth {
-			continue
-		}
-		// MaxDepthチェック（0は制限なし）
-		if entry.MaxDepth > 0 && depth > entry.MaxDepth {
+		if depth < entry.MinDepth || depth > entry.MaxDepth {
 			continue
 		}
 		validEntries = append(validEntries, entry)

internal/cmd/generate_item_doc.go

@@ -137,12 +137,7 @@ func calculateProbabilities(table raw.ItemTable, depth int) map[string]float64 {
 	// 深度範囲内のエントリをフィルタリング
 	validEntries := make([]raw.ItemTableEntry, 0, len(table.Entries))
 	for _, entry := range table.Entries {
-		// MinDepthチェック（0は制限なし）
-		if entry.MinDepth > 0 && depth < entry.MinDepth {
-			continue
-		}
-		// MaxDepthチェック（0は制限なし）
-		if entry.MaxDepth > 0 && depth > entry.MaxDepth {
+		if depth < entry.MinDepth || depth > entry.MaxDepth {
 			continue
 		}
 		validEntries = append(validEntries, entry)

internal/cmd/play.go

@@ -32,7 +32,10 @@ var CmdPlay = &cli.Command{
 
 func runPlay(_ context.Context, _ *cli.Command) error {
 	// 設定を読み込み
-	cfg := config.Get()
+	cfg, err := config.Load()
+	if err != nil {
+		return fmt.Errorf("設定の読み込みに失敗: %w", err)
+	}
 
 	// ログ設定を読み込み
 	logger.LoadFromConfig(cfg.LogLevel, cfg.LogCategories)
@@ -90,7 +93,7 @@ func runPlay(_ context.Context, _ *cli.Command) error {
 		}()
 	}
 
-	world, err := maingame.InitWorld(cfg.WindowWidth, cfg.WindowHeight)
+	world, err := maingame.InitWorld(cfg)
 	if err != nil {
 		return err
 	}

internal/cmd/screenshot.go

@@ -27,29 +27,21 @@ func runScreenshot(_ context.Context, cmd *cli.Command) error {
 		return fmt.Errorf("引数が不足している。ステート名が必要")
 	}
 
-	// 固定seed値を使用したtown dungeon state
-	const townSeedVal = 1
-	townStateFactory := gs.NewTownState(gs.WithSeed(townSeedVal))
+	townStateFactory := gs.NewTownState()
 
 	switch mode {
 	case gs.CraftMenuState{}.String():
 		return vrt.RunTestGame(mode, townStateFactory(), &gs.CraftMenuState{})
 	case "DebugMenu":
 		return vrt.RunTestGame(mode, townStateFactory(), gs.NewDebugMenuState())
 	case gs.DungeonState{}.String():
-		// 固定seed値を使用する
-		seedVal := uint64(1)
 		return vrt.RunTestGame(mode, &gs.DungeonState{
 			Depth:       1,
-			Seed:        &seedVal,
 			BuilderType: mapplanner.PlannerTypeSmallRoom,
 		})
 	case gs.FieldInfoState{}.String():
-		// 固定seed値を使用したダンジョンの上に視界情報画面を重ねる
-		seedVal := uint64(1)
 		return vrt.RunTestGame(mode, &gs.DungeonState{
 			Depth:       1,
-			Seed:        &seedVal,
 			BuilderType: mapplanner.PlannerTypeSmallRoom,
 		}, &gs.FieldInfoState{})
 	case gs.EquipMenuState{}.String():

internal/config/config.go

@@ -1,9 +1,8 @@
 package config
 
 import (
+	"math/rand/v2"
 	"os"
-
-	"github.com/caarlos0/env/v11"
 )
 
 // Profile は設定プロファイルを表す
@@ -41,6 +40,13 @@ type Config struct {
 	StartingState    string `env:"RUINS_STARTING_STATE"`
 	DisableAnimation bool   `env:"RUINS_DISABLE_ANIMATION"`
 
+	// 乱数シード。環境変数で指定すると再現可能になる
+	// 未指定の場合は自動生成される
+	Seed uint64 `env:"RUINS_SEED"`
+
+	// 乱数生成器。Seedから生成される
+	RNG *rand.Rand
+
 	// パフォーマンス設定
 	TargetFPS     int    `env:"RUINS_TARGET_FPS"`
 	ProfileMemory bool   `env:"RUINS_PROFILE_MEMORY"`
@@ -50,36 +56,13 @@ type Config struct {
 	ProfilePath   string `env:"RUINS_PROFILE_PATH"`
 }
 
-// load は環境変数から設定を読み込む
-func load() (*Config, error) {
-	cfg := &Config{}
-
-	// プロファイルを最初に決定(デフォルトはproduction)
-	profile := os.Getenv("RUINS_PROFILE")
-	if profile == "" {
-		cfg.Profile = ProfileProduction
-	} else {
-		cfg.Profile = Profile(profile)
-	}
-
-	// プロファイルに基づくデフォルト値を設定
-	cfg.applyProfileDefaults()
-
-	// 環境変数で明示的に設定された値で上書き
-	if err := env.Parse(cfg); err != nil {
-		return nil, err
-	}
-
-	return cfg, nil
-}
-
-// applyProfileDefaults はプロファイルに基づいてデフォルト値を設定する
-func (c *Config) applyProfileDefaults() {
+// ApplyProfileDefaults はプロファイルに基づいてデフォルト値を設定する
+func (c *Config) ApplyProfileDefaults() {
 	switch c.Profile {
-	case ProfileDevelopment:
-		c.applyDevelopmentDefaults()
 	case ProfileProduction:
 		c.applyProductionDefaults()
+	case ProfileDevelopment:
+		c.applyDevelopmentDefaults()
 	default:
 		// デフォルトは本番設定
 		c.applyProductionDefaults()