Centralize location LLM assets

internal/modules/dnd/extract/locationoccurrences/assets.go

@@ -1,6 +0,0 @@
-package locationoccurrences
-
-import "embed"
-
-//go:embed assets/prompts/*.yaml assets/prompts/*.md assets/schemas/*.json
-var embeddedAssets embed.FS

internal/modules/dnd/extract/locationoccurrences/assets/prompts/dnd.location_occurrences.yaml

@@ -1,47 +0,0 @@
-id: dnd.location_occurrences
-version: "v1"
-default_profile: dnd-extraction
-inputs:
-  - name: transcript
-    required: true
-    content_type: application/json
-  - name: players
-    required: false
-    content_type: text/plain
-  - name: party
-    required: false
-    content_type: text/plain
-  - name: glossary
-    required: false
-    content_type: text/plain
-  - name: locations
-    required: true
-    content_type: application/json
-messages:
-  - role: system
-    content_file: ./sharedassets/common-dnd-system.md
-  - role: user
-    content_file: ./sharedassets/common-dnd-identity.md
-  - role: user
-    content_file: ./sharedassets/common-dnd-references.md
-    cache_control:
-      type: ephemeral
-  - role: user
-    content_file: ./sharedassets/common-dnd-transcript.md
-    cache_control:
-      type: ephemeral
-  - role: user
-    content_file: ./sharedassets/common-dnd-extraction-evidence.md
-  - role: user
-    content_file: ./locations.md
-  - role: user
-    content_file: ./task.md
-  - role: user
-    content_file: ./instructions.md
-    cache_control:
-      type: ephemeral
-output:
-  format: json
-  validation_mode: json_schema
-  schema_path: dnd_location_occurrences_llm.v1.json
-  repair_attempts: 0

internal/modules/dnd/extract/locationoccurrences/assets/prompts/instructions.md

@@ -1,8 +0,0 @@
-Return the occurrences array even when no occurrence is established. Every
-record must contain location_id, name, kind, and source_refs. Copy location_id
-and name from one supplied registry record, and cite only narrow transcript
-ranges that support both that location and its classified occurrence.
-
-Do not summarize location descriptions, infer a missing registry record, or
-use registry context as evidence. Omit source_id; Notarius assigns the current
-transcript source identity.

internal/modules/dnd/extract/locationoccurrences/assets/prompts/locations.md

@@ -1,9 +0,0 @@
-A normalized location registry is provided below for identity grounding. It may
-be empty. Each record contains the exact location ID and canonical display name
-to copy when the transcript establishes an occurrence of that place.
-
-Registry content is context, not occurrence evidence. Do not derive an
-occurrence or a source range from the registry, and do not infer a location
-that is absent from it.
-
-{{ input "locations" }}

internal/modules/dnd/extract/locationoccurrences/assets/prompts/task.md

@@ -1,29 +0,0 @@
-Extract Dungeons & Dragons location occurrences from the supplied transcript.
-
-Include an occurrence only when the transcript establishes one supplied
-location, one occurrence kind, and a coherent passage supporting both. Use
-only the exact ID and name pair from the supplied location registry. Return an
-empty occurrences array when no supplied location has an evidenced occurrence
-in this transcript passage.
-
-Use exactly one kind per occurrence:
-
-- visited: party members are physically present, arrive, remain, or depart;
-- planned: the party explicitly proposes, intends, or agrees to future travel;
-- recalled: the transcript explicitly recounts an earlier party visit; or
-- mentioned: the location is explicitly referenced without stronger support,
-  including non-actionable speculation or a mere hypothetical reference.
-
-A mere hypothetical or speculative reference is not planned unless the
-transcript also establishes an actual proposal, intention, or agreement to
-travel. When the hypothetical itself explicitly names a supplied registry
-location, it may be mentioned using the narrow passage that supports that
-reference.
-
-For overlapping support, visited outranks planned, recalled, and mentioned;
-planned outranks recalled and mentioned; recalled outranks mentioned. A passage
-may produce multiple records when it independently establishes separate facts,
-such as recalling an earlier visit while planning a return. Omit inferred,
-unstated, uncertain, or unsupported places and occurrences. Do not infer a
-location or occurrence from surrounding events when the transcript does not
-state it.

internal/modules/dnd/extract/locationoccurrences/assets/schemas/dnd_location_occurrences_llm.v1.json

@@ -1,34 +0,0 @@
-{
-  "$schema": "https://json-schema.org/draft/2020-12/schema",
-  "$id": "notarius.dnd.location_occurrences.llm",
-  "type": "object",
-  "additionalProperties": false,
-  "required": ["occurrences"],
-  "properties": {
-    "occurrences": {
-      "type": "array",
-      "items": {
-        "type": "object",
-        "additionalProperties": false,
-        "required": ["location_id", "name", "kind", "source_refs"],
-        "properties": {
-          "location_id": {"type": "string"},
-          "name": {"type": "string"},
-          "kind": {"enum": ["visited", "planned", "recalled", "mentioned"]},
-          "source_refs": {
-            "type": "array",
-            "items": {
-              "type": "object",
-              "additionalProperties": false,
-              "required": ["start_unit_id", "end_unit_id"],
-              "properties": {
-                "start_unit_id": {"type": "integer"},
-                "end_unit_id": {"type": "integer"}
-              }
-            }
-          }
-        }
-      }
-    }
-  }
-}

internal/modules/dnd/extract/locationoccurrences/prompt_assets.go

@@ -2,8 +2,10 @@ package locationoccurrences
 
 import (
 	"fmt"
+	"io/fs"
 	"sync"
 
+	rootassets "gitea.maximumdirect.net/eric/notarius/assets"
 	"gitea.maximumdirect.net/eric/notarius/internal/framework/llm"
 	"gitea.maximumdirect.net/eric/notarius/internal/framework/promptfs"
 	"gitea.maximumdirect.net/eric/notarius/internal/modules/dnd/shared"
@@ -14,10 +16,10 @@ const promptAssetRoot = "assets/prompts"
 var promptAssetManifest = shared.PromptAssetManifest{
 	ModuleDir: PromptID,
 	ModuleFiles: []promptfs.ModulePromptFile{
-		{Name: "dnd.location_occurrences.yaml", Path: "assets/prompts/dnd.location_occurrences.yaml"},
-		{Name: "locations.md", Path: "assets/prompts/locations.md"},
-		{Name: "task.md", Path: "assets/prompts/task.md"},
-		{Name: "instructions.md", Path: "assets/prompts/instructions.md"},
+		{Name: "dnd.location_occurrences.yaml", Path: "prompts/dnd.location_occurrences.yaml"},
+		{Name: "locations.md", Path: "prompts/locations.md"},
+		{Name: "task.md", Path: "prompts/task.md"},
+		{Name: "instructions.md", Path: "prompts/instructions.md"},
 	},
 	SharedFiles: []string{
 		"common-dnd-system.md",
@@ -28,20 +30,41 @@ var promptAssetManifest = shared.PromptAssetManifest{
 	},
 }
 
+func moduleAssetFS() (fs.FS, error) {
+	assets, err := fs.Sub(rootassets.FS(), "dnd/location-occurrences")
+	if err != nil {
+		return nil, fmt.Errorf("scope location-occurrence assets: %w", err)
+	}
+	return assets, nil
+}
+
 func RegisterPromptAssets(registry *llm.AssetRegistry) error {
-	promptFS, err := promptAssetManifest.PromptFS(embeddedAssets)
+	assets, err := moduleAssetFS()
+	if err != nil {
+		return err
+	}
+	promptFS, err := promptAssetManifest.PromptFS(assets)
 	if err != nil {
 		return fmt.Errorf("prepare location-occurrence prompt assets: %w", err)
 	}
 	if err := registry.RegisterPromptFS(promptFS, promptAssetRoot); err != nil {
 		return err
 	}
-	return registry.RegisterSchemaFS(embeddedAssets, "assets/schemas")
+	schemas, err := fs.Sub(assets, "schemas")
+	if err != nil {
+		return fmt.Errorf("scope location-occurrence schemas: %w", err)
+	}
+	return registry.RegisterSchemaFS(schemas, ".")
 }
 
 func promptAssetMetadata() (string, error) {
 	promptAssetHashOnce.Do(func() {
-		promptAssetHash, promptAssetHashErr = promptAssetManifest.Hash(embeddedAssets)
+		assets, err := moduleAssetFS()
+		if err != nil {
+			promptAssetHashErr = err
+			return
+		}
+		promptAssetHash, promptAssetHashErr = promptAssetManifest.Hash(assets)
 	})
 	return promptAssetHash, promptAssetHashErr
 }

internal/modules/dnd/extract/locationoccurrences/schema.go

@@ -11,11 +11,15 @@ const (
 )
 
 func loadResponseSchema() (llm.ResponseSchema, error) {
-	return llm.LoadResponseSchema(embeddedAssets, llm.ResponseSchemaDefinition{
+	assets, err := moduleAssetFS()
+	if err != nil {
+		return llm.ResponseSchema{}, err
+	}
+	return llm.LoadResponseSchema(assets, llm.ResponseSchemaDefinition{
 		Key:       ResponseSchemaKey,
 		ID:        ResponseSchemaID,
 		Version:   SchemaVersion,
 		Name:      ResponseSchemaName,
-		AssetPath: "assets/schemas/dnd_location_occurrences_llm.v1.json",
+		AssetPath: "schemas/dnd_location_occurrences_llm.v1.json",
 	})
 }

internal/modules/dnd/extract/locations/assets.go

@@ -1,6 +0,0 @@
-package locations
-
-import "embed"
-
-//go:embed assets/prompts/*.yaml assets/prompts/*.md assets/schemas/*.json
-var embeddedAssets embed.FS

internal/modules/dnd/extract/locations/assets/prompts/dnd.locations.yaml

@@ -1,42 +0,0 @@
-id: dnd.locations
-version: "v1"
-default_profile: dnd-extraction
-inputs:
-  - name: transcript
-    required: true
-    content_type: application/json
-  - name: players
-    required: false
-    content_type: text/plain
-  - name: party
-    required: false
-    content_type: text/plain
-  - name: glossary
-    required: false
-    content_type: text/plain
-messages:
-  - role: system
-    content_file: ./sharedassets/common-dnd-system.md
-  - role: user
-    content_file: ./sharedassets/common-dnd-identity.md
-  - role: user
-    content_file: ./sharedassets/common-dnd-references.md
-    cache_control:
-      type: ephemeral
-  - role: user
-    content_file: ./sharedassets/common-dnd-transcript.md
-    cache_control:
-      type: ephemeral
-  - role: user
-    content_file: ./sharedassets/common-dnd-extraction-evidence.md
-  - role: user
-    content_file: ./task.md
-  - role: user
-    content_file: ./instructions.md
-    cache_control:
-      type: ephemeral
-output:
-  format: json
-  validation_mode: json_schema
-  schema_path: dnd_locations_llm.v1.json
-  repair_attempts: 0

internal/modules/dnd/extract/locations/assets/prompts/instructions.md

@@ -1,6 +0,0 @@
-Return only observed location display names and narrow transcript source ranges.
-
-Exclude people, creatures, objects, organizations, abstract concepts, and
-places merely inferred from an event. Omit uncertain or unsupported places.
-Campaign references may clarify terms already present in the transcript, but
-they are not evidence and must never supply a source range.

internal/modules/dnd/extract/locations/assets/prompts/task.md

@@ -1,8 +0,0 @@
-Extract physical places established by the provided Dungeons & Dragons
-transcript and cite where each place is identified.
-
-Include planes, regions, settlements, districts, buildings, rooms, landmarks,
-routes, and geographic features. A generic label such as "the tavern" is
-allowed only when the transcript uses it for a specific place. Keep aliases and
-nested places when the transcript identifies them; do not merge or invent
-qualifiers for similarly named places.

internal/modules/dnd/extract/locations/assets/schemas/dnd_locations_llm.v1.json

@@ -1,32 +0,0 @@
-{
-  "$schema": "https://json-schema.org/draft/2020-12/schema",
-  "$id": "notarius.dnd.locations.llm",
-  "type": "object",
-  "additionalProperties": false,
-  "required": ["locations"],
-  "properties": {
-    "locations": {
-      "type": "array",
-      "items": {
-        "type": "object",
-        "additionalProperties": false,
-        "required": ["name", "source_refs"],
-        "properties": {
-          "name": {"type": "string"},
-          "source_refs": {
-            "type": "array",
-            "items": {
-              "type": "object",
-              "additionalProperties": false,
-              "required": ["start_unit_id", "end_unit_id"],
-              "properties": {
-                "start_unit_id": {"type": "integer"},
-                "end_unit_id": {"type": "integer"}
-              }
-            }
-          }
-        }
-      }
-    }
-  }
-}

internal/modules/dnd/extract/locations/prompt_assets.go

@@ -2,8 +2,10 @@ package locations
 
 import (
 	"fmt"
+	"io/fs"
 	"sync"
 
+	rootassets "gitea.maximumdirect.net/eric/notarius/assets"
 	"gitea.maximumdirect.net/eric/notarius/internal/framework/llm"
 	"gitea.maximumdirect.net/eric/notarius/internal/framework/promptfs"
 	"gitea.maximumdirect.net/eric/notarius/internal/modules/dnd/shared"
@@ -14,9 +16,9 @@ const promptAssetRoot = "assets/prompts"
 var promptAssetManifest = shared.PromptAssetManifest{
 	ModuleDir: PromptID,
 	ModuleFiles: []promptfs.ModulePromptFile{
-		{Name: "dnd.locations.yaml", Path: "assets/prompts/dnd.locations.yaml"},
-		{Name: "task.md", Path: "assets/prompts/task.md"},
-		{Name: "instructions.md", Path: "assets/prompts/instructions.md"},
+		{Name: "dnd.locations.yaml", Path: "prompts/dnd.locations.yaml"},
+		{Name: "task.md", Path: "prompts/task.md"},
+		{Name: "instructions.md", Path: "prompts/instructions.md"},
 	},
 	SharedFiles: []string{
 		"common-dnd-system.md",
@@ -27,20 +29,41 @@ var promptAssetManifest = shared.PromptAssetManifest{
 	},
 }
 
+func moduleAssetFS() (fs.FS, error) {
+	assets, err := fs.Sub(rootassets.FS(), "dnd/locations/extract")
+	if err != nil {
+		return nil, fmt.Errorf("scope location extraction assets: %w", err)
+	}
+	return assets, nil
+}
+
 func RegisterPromptAssets(registry *llm.AssetRegistry) error {
-	promptFS, err := promptAssetManifest.PromptFS(embeddedAssets)
+	assets, err := moduleAssetFS()
+	if err != nil {
+		return err
+	}
+	promptFS, err := promptAssetManifest.PromptFS(assets)
 	if err != nil {
 		return fmt.Errorf("prepare location prompt assets: %w", err)
 	}
 	if err := registry.RegisterPromptFS(promptFS, promptAssetRoot); err != nil {
 		return err
 	}
-	return registry.RegisterSchemaFS(embeddedAssets, "assets/schemas")
+	schemas, err := fs.Sub(assets, "schemas")
+	if err != nil {
+		return fmt.Errorf("scope location extraction schemas: %w", err)
+	}
+	return registry.RegisterSchemaFS(schemas, ".")
 }
 
 func promptAssetMetadata() (string, error) {
 	promptAssetHashOnce.Do(func() {
-		promptAssetHash, promptAssetHashErr = promptAssetManifest.Hash(embeddedAssets)
+		assets, err := moduleAssetFS()
+		if err != nil {
+			promptAssetHashErr = err
+			return
+		}
+		promptAssetHash, promptAssetHashErr = promptAssetManifest.Hash(assets)
 	})
 	return promptAssetHash, promptAssetHashErr
 }

internal/modules/dnd/extract/locations/schema.go

@@ -11,11 +11,15 @@ const (
 )
 
 func loadResponseSchema() (llm.ResponseSchema, error) {
-	return llm.LoadResponseSchema(embeddedAssets, llm.ResponseSchemaDefinition{
+	assets, err := moduleAssetFS()
+	if err != nil {
+		return llm.ResponseSchema{}, err
+	}
+	return llm.LoadResponseSchema(assets, llm.ResponseSchemaDefinition{
 		Key:       ResponseSchemaKey,
 		ID:        ResponseSchemaID,
 		Version:   SchemaVersion,
 		Name:      ResponseSchemaName,
-		AssetPath: "assets/schemas/dnd_locations_llm.v1.json",
+		AssetPath: "schemas/dnd_locations_llm.v1.json",
 	})
 }