First iteration over items

module/models/weapon.mjs

@@ -12,13 +12,13 @@ export default class PrismRPGWeapon extends foundry.abstract.TypeDataModel {
     schema.weaponType = new fields.StringField({
       required: true,
       initial: "light",
-      choices: Object.keys(SYSTEM.WEAPON_TYPE || {})
+      choices: SYSTEM.WEAPON_TYPE_CHOICES
     })
 
     schema.weaponGroup = new fields.StringField({
       required: true,
       initial: "longsword",
-      choices: Object.keys(SYSTEM.WEAPON_GROUP || {})
+      choices: SYSTEM.WEAPON_GROUP_CHOICES
     })
 
     // APC (Action Point Cost) - determined by weapon type
@@ -105,37 +105,7 @@ export default class PrismRPGWeapon extends foundry.abstract.TypeDataModel {
       min: 0
     })
 
-    // Legacy properties (kept for backward compatibility)
-    schema.hands = new fields.StringField({
-      required: true,
-      initial: "1",
-      choices: { "1": "1", "2": "2" }
-    })
-
-    schema.isAgile = new fields.BooleanField({
-      required: true,
-      initial: false
-    })
-
-    schema.applyStrengthDamageBonus = new fields.BooleanField({
-      required: true,
-      initial: true
-    })
-
-    schema.defenseMax = new fields.NumberField({
-      ...requiredInteger,
-      required: true,
-      initial: 0,
-      min: 0
-    })
-
-    schema.defense = new fields.NumberField({
-      ...requiredInteger,
-      required: true,
-      initial: 0,
-      min: 0
-    })
-
+    // Bonuses (from magical enhancements, etc.)
     schema.bonuses = new fields.SchemaField({
       attackBonus: new fields.NumberField({ ...requiredInteger, required: true, initial: 0, min: 0 }),
       damageBonus: new fields.NumberField({ ...requiredInteger, required: true, initial: 0, min: 0 }),