PR 8.7: Workspace schema aligned to id/type structure

- Removed sampleId from schema
- Added id + type requirements
- Enforced strict entity shape

Author: DavidQ

docs/dev/codex_commands.md

@@ -2,10 +2,11 @@ MODEL: GPT-5.3-codex
 REASONING: medium
 
 TASK:
-- Replace sampleId → id in workspace.manifest.json
-- Add type field:
-  - existing entries → type: "sample"
-- Ensure uniqueness of id
+- Update workspace.schema.json:
+  - Replace sampleId usage with id
+  - Add type field requirement
+  - Ensure entity array structure matches manifest
+- Ensure no additionalProperties allowed
 - Do NOT modify runtime
 - Do NOT add validators
 - Do NOT modify start_of_day

docs/dev/commit_comment.txt

@@ -1,5 +1,5 @@
-PR 8.6: Manifest entity generalization
+PR 8.7: Workspace schema aligned to id/type structure
 
-- Replaced sampleId with id
-- Added type field
-- Enabled support for games and future entities
+- Removed sampleId from schema
+- Added id + type requirements
+- Enforced strict entity shape

docs/pr/PR_8.7_WORKSPACE_SCHEMA_UPDATE.md

@@ -0,0 +1,42 @@
+# PR 8.7 — Workspace Schema Update (id + type)
+
+## Purpose
+Align workspace.schema.json with new manifest structure using:
+- id
+- type
+
+## Changes
+
+### workspace.schema.json
+
+#### Replace
+- sampleId
+
+#### With
+- id (string, required)
+- type (string, required)
+
+### Entry Shape
+```json
+{
+  "id": "0305",
+  "type": "sample",
+  "tool": "vector-map-editor",
+  "palette": "samples/phase-03/0305/sample.palette.json"
+}
+```
+
+### Rules
+- id must be unique
+- type required
+- tool required
+- palette optional
+- no additionalProperties
+
+## Non-Goals
+- no runtime logic
+- no validators
+
+## Acceptance
+- schema matches manifest structure
+- no references to sampleId remain

tools/schemas/workspace.schema.json

@@ -1,17 +1,61 @@
 {
   "$schema": "https://json-schema.org/draft/2020-12/schema",
-  "$id": "workspace.schema.json",
+  "$id": "tools/schemas/workspace.schema.json",
   "type": "object",
-  "required": ["version", "tools", "palettes"],
+  "required": ["documentKind", "schema", "version", "samples"],
+  "additionalProperties": false,
   "properties": {
-    "version": { "type": "string" },
-    "tools": {
-      "type": "array",
-      "items": { "type": "string" }
+    "documentKind": {
+      "type": "string",
+      "const": "workspace-manifest"
+    },
+    "schema": {
+      "type": "string",
+      "minLength": 1
+    },
+    "version": {
+      "type": "integer",
+      "minimum": 1
     },
-    "palettes": {
+    "samples": {
       "type": "array",
-      "items": { "$ref": "palette.schema.json" }
+      "uniqueItems": true,
+      "items": {
+        "type": "object",
+        "required": ["id", "type", "tool"],
+        "additionalProperties": false,
+        "properties": {
+          "id": {
+            "type": "string",
+            "minLength": 1
+          },
+          "type": {
+            "type": "string",
+            "minLength": 1
+          },
+          "phase": {
+            "type": "string",
+            "minLength": 1
+          },
+          "tool": {
+            "type": "string",
+            "minLength": 1
+          },
+          "tools": {
+            "type": "array",
+            "uniqueItems": true,
+            "items": {
+              "type": "string",
+              "minLength": 1
+            }
+          },
+          "palette": {
+            "type": "string",
+            "minLength": 1
+          }
+        },
+        "description": "id must be unique across samples."
+      }
     }
   }
 }