Strip all angle brackets in PF2e attack traits and damage

Broaden stripDiceBrackets to stripAngleBrackets to handle all
PF2e tools angle-bracket formatting (e.g. <10 feet>, <15 feet>),
not just dice notation. Also strip in damage text.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

apps/web/src/adapters/__tests__/bestiary-adapter.test.ts

@@ -6,7 +6,8 @@ import {
 } from "../bestiary-adapter.js";
 
 /** Flatten segments to a single string for simple text assertions. */
-function flatText(trait: TraitBlock): string {
+function flatText(trait: TraitBlock | undefined): string {
+	if (!trait) return "";
 	return trait.segments
 		.map((s) =>
 			s.type === "text"
@@ -88,11 +89,11 @@ describe("normalizeBestiary", () => {
 		expect(c.senses).toBe("Darkvision 60 ft.");
 		expect(c.languages).toBe("Common, Goblin");
 		expect(c.actions).toHaveLength(1);
-		expect(flatText(c.actions![0])).toContain("Melee Attack Roll:");
-		expect(flatText(c.actions![0])).not.toContain("{@");
+		expect(flatText(c.actions?.[0])).toContain("Melee Attack Roll:");
+		expect(flatText(c.actions?.[0])).not.toContain("{@");
 		expect(c.bonusActions).toHaveLength(1);
-		expect(flatText(c.bonusActions![0])).toContain("Disengage");
-		expect(flatText(c.bonusActions![0])).not.toContain("{@");
+		expect(flatText(c.bonusActions?.[0])).toContain("Disengage");
+		expect(flatText(c.bonusActions?.[0])).not.toContain("{@");
 	});
 
 	it("normalizes a creature with legendary actions", () => {
@@ -347,9 +348,9 @@ describe("normalizeBestiary", () => {
 
 		const creatures = normalizeBestiary(raw);
 		const bite = creatures[0].actions?.[0];
-		expect(flatText(bite!)).toContain("Melee Weapon Attack:");
-		expect(flatText(bite!)).not.toContain("mw");
-		expect(flatText(bite!)).not.toContain("{@");
+		expect(flatText(bite)).toContain("Melee Weapon Attack:");
+		expect(flatText(bite)).not.toContain("mw");
+		expect(flatText(bite)).not.toContain("{@");
 	});
 
 	it("handles fly speed with hover condition", () => {
@@ -438,14 +439,15 @@ describe("normalizeBestiary", () => {
 		};
 
 		const creatures = normalizeBestiary(raw);
-		const trait = creatures[0].traits![0];
-		expect(trait.name).toBe("Confusing Burble");
-		expect(trait.segments).toHaveLength(2);
-		expect(trait.segments[0]).toEqual({
+		const trait = creatures[0].traits?.[0];
+		expect(trait).toBeDefined();
+		expect(trait?.name).toBe("Confusing Burble");
+		expect(trait?.segments).toHaveLength(2);
+		expect(trait?.segments[0]).toEqual({
 			type: "text",
 			value: expect.stringContaining("d4"),
 		});
-		expect(trait.segments[1]).toEqual({
+		expect(trait?.segments[1]).toEqual({
 			type: "list",
 			items: [
 				{ label: "1-2", text: "The creature does nothing." },
@@ -498,8 +500,9 @@ describe("normalizeBestiary", () => {
 		};
 
 		const creatures = normalizeBestiary(raw);
-		const trait = creatures[0].traits![0];
-		expect(trait.segments[1]).toEqual({
+		const trait = creatures[0].traits?.[0];
+		expect(trait).toBeDefined();
+		expect(trait?.segments[1]).toEqual({
 			type: "list",
 			items: [
 				{ label: "1", text: "Nothing happens." },

apps/web/src/adapters/__tests__/pf2e-bestiary-adapter.test.ts

@@ -0,0 +1,200 @@
+import { describe, expect, it } from "vitest";
+import { normalizePf2eBestiary } from "../pf2e-bestiary-adapter.js";
+
+function minimalCreature(overrides?: Record<string, unknown>) {
+	return {
+		name: "Test Creature",
+		source: "TST",
+		...overrides,
+	};
+}
+
+describe("normalizePf2eBestiary", () => {
+	describe("weaknesses formatting", () => {
+		it("formats weakness with numeric amount", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						defenses: {
+							weaknesses: [{ name: "fire", amount: 5 }],
+						},
+					}),
+				],
+			});
+			expect(creature.weaknesses).toBe("Fire 5");
+		});
+
+		it("formats weakness without amount (qualitative)", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						defenses: {
+							weaknesses: [{ name: "smoke susceptibility" }],
+						},
+					}),
+				],
+			});
+			expect(creature.weaknesses).toBe("Smoke susceptibility");
+		});
+
+		it("formats weakness with note and amount", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						defenses: {
+							weaknesses: [
+								{ name: "cold iron", amount: 5, note: "except daggers" },
+							],
+						},
+					}),
+				],
+			});
+			expect(creature.weaknesses).toBe("Cold iron 5 (except daggers)");
+		});
+
+		it("formats weakness with note but no amount", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						defenses: {
+							weaknesses: [{ name: "smoke susceptibility", note: "see below" }],
+						},
+					}),
+				],
+			});
+			expect(creature.weaknesses).toBe("Smoke susceptibility (see below)");
+		});
+
+		it("returns undefined when no weaknesses", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [minimalCreature({})],
+			});
+			expect(creature.weaknesses).toBeUndefined();
+		});
+	});
+
+	describe("senses formatting", () => {
+		it("strips tags and includes type and range", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						senses: [
+							{
+								type: "imprecise",
+								name: "{@ability tremorsense}",
+								range: 30,
+							},
+						],
+					}),
+				],
+			});
+			expect(creature.senses).toBe("Tremorsense (imprecise) 30 feet");
+		});
+
+		it("formats sense with only a name", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						senses: [{ name: "darkvision" }],
+					}),
+				],
+			});
+			expect(creature.senses).toBe("Darkvision");
+		});
+
+		it("formats sense with name and range but no type", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						senses: [{ name: "scent", range: 60 }],
+					}),
+				],
+			});
+			expect(creature.senses).toBe("Scent 60 feet");
+		});
+	});
+
+	describe("attack formatting", () => {
+		it("strips angle brackets from traits", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						attacks: [
+							{
+								name: "stinger",
+								range: "Melee",
+								attack: 11,
+								traits: ["deadly <d8>"],
+								damage: "1d6+4 piercing",
+							},
+						],
+					}),
+				],
+			});
+			const attack = creature.attacks?.[0];
+			expect(attack).toBeDefined();
+			expect(attack?.segments[0]).toEqual(
+				expect.objectContaining({
+					type: "text",
+					value: expect.stringContaining("(deadly d8)"),
+				}),
+			);
+		});
+
+		it("strips angle brackets from reach values in traits", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						attacks: [
+							{
+								name: "tentacle",
+								range: "Melee",
+								attack: 18,
+								traits: ["agile", "chaotic", "magical", "reach <10 feet>"],
+								damage: "2d8+6 piercing",
+							},
+						],
+					}),
+				],
+			});
+			const attack = creature.attacks?.[0];
+			expect(attack).toBeDefined();
+			expect(attack?.segments[0]).toEqual(
+				expect.objectContaining({
+					type: "text",
+					value: expect.stringContaining(
+						"(agile, chaotic, magical, reach 10 feet)",
+					),
+				}),
+			);
+		});
+	});
+
+	describe("resistances formatting", () => {
+		it("formats resistance without amount", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						defenses: {
+							resistances: [{ name: "physical" }],
+						},
+					}),
+				],
+			});
+			expect(creature.resistances).toBe("Physical");
+		});
+
+		it("formats resistance with amount", () => {
+			const [creature] = normalizePf2eBestiary({
+				creature: [
+					minimalCreature({
+						defenses: {
+							resistances: [{ name: "fire", amount: 10 }],
+						},
+					}),
+				],
+			});
+			expect(creature.resistances).toBe("Fire 10");
+		});
+	});
+});

apps/web/src/adapters/pf2e-bestiary-adapter.ts

@@ -15,7 +15,7 @@ interface RawPf2eCreature {
 	level?: number;
 	traits?: string[];
 	perception?: { std?: number };
-	senses?: { name?: string; type?: string }[];
+	senses?: { name?: string; type?: string; range?: number }[];
 	languages?: { languages?: string[] };
 	skills?: Record<string, { std?: number }>;
 	abilityMods?: Record<string, number>;
@@ -40,8 +40,8 @@ interface RawDefenses {
 	};
 	hp?: { hp?: number }[];
 	immunities?: (string | { name: string })[];
-	resistances?: { amount: number; name: string; note?: string }[];
-	weaknesses?: { amount: number; name: string; note?: string }[];
+	resistances?: { amount?: number; name: string; note?: string }[];
+	weaknesses?: { amount?: number; name: string; note?: string }[];
 }
 
 interface RawAbility {
@@ -79,6 +79,10 @@ function capitalize(s: string): string {
 	return s.charAt(0).toUpperCase() + s.slice(1);
 }
 
+function stripAngleBrackets(s: string): string {
+	return s.replaceAll(/<([^>]+)>/g, "$1");
+}
+
 function makeCreatureId(source: string, name: string): CreatureId {
 	const slug = name
 		.toLowerCase()
@@ -119,13 +123,19 @@ function formatSkills(
 }
 
 function formatSenses(
-	senses: readonly { name?: string; type?: string }[] | undefined,
+	senses:
+		| readonly { name?: string; type?: string; range?: number }[]
+		| undefined,
 ): string | undefined {
 	if (!senses || senses.length === 0) return undefined;
 	return senses
 		.map((s) => {
-			const label = s.name ?? s.type ?? "";
-			return label ? capitalize(label) : "";
+			const label = stripTags(s.name ?? s.type ?? "");
+			if (!label) return "";
+			const parts = [capitalize(label)];
+			if (s.type && s.name) parts.push(`(${s.type})`);
+			if (s.range != null) parts.push(`${s.range} feet`);
+			return parts.join(" ");
 		})
 		.filter(Boolean)
 		.join(", ");
@@ -150,31 +160,35 @@ function formatImmunities(
 
 function formatResistances(
 	resistances:
-		| readonly { amount: number; name: string; note?: string }[]
+		| readonly { amount?: number; name: string; note?: string }[]
 		| undefined,
 ): string | undefined {
 	if (!resistances || resistances.length === 0) return undefined;
 	return resistances
-		.map((r) =>
-			r.note
-				? `${capitalize(r.name)} ${r.amount} (${r.note})`
-				: `${capitalize(r.name)} ${r.amount}`,
-		)
+		.map((r) => {
+			const base =
+				r.amount == null
+					? capitalize(r.name)
+					: `${capitalize(r.name)} ${r.amount}`;
+			return r.note ? `${base} (${r.note})` : base;
+		})
 		.join(", ");
 }
 
 function formatWeaknesses(
 	weaknesses:
-		| readonly { amount: number; name: string; note?: string }[]
+		| readonly { amount?: number; name: string; note?: string }[]
 		| undefined,
 ): string | undefined {
 	if (!weaknesses || weaknesses.length === 0) return undefined;
 	return weaknesses
-		.map((w) =>
-			w.note
-				? `${capitalize(w.name)} ${w.amount} (${w.note})`
-				: `${capitalize(w.name)} ${w.amount}`,
-		)
+		.map((w) => {
+			const base =
+				w.amount == null
+					? capitalize(w.name)
+					: `${capitalize(w.name)} ${w.amount}`;
+			return w.note ? `${base} (${w.note})` : base;
+		})
 		.join(", ");
 }
 
@@ -249,9 +263,11 @@ function normalizeAttacks(
 		const attackMod = a.attack == null ? "" : ` +${a.attack}`;
 		const traits =
 			a.traits && a.traits.length > 0
-				? ` (${a.traits.map((t) => stripTags(t)).join(", ")})`
+				? ` (${a.traits.map((t) => stripAngleBrackets(stripTags(t))).join(", ")})`
+				: "";
+		const damage = a.damage
+			? `, ${stripAngleBrackets(stripTags(a.damage))}`
 			: "";
-		const damage = a.damage ? `, ${stripTags(a.damage)}` : "";
 		return {
 			name: capitalize(stripTags(a.name)),
 			segments: [

packages/domain/src/conditions.ts

@@ -77,7 +77,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description5e:
 			"Can't see. Auto-fail sight checks. Attacks have Disadvantage; attacks against have Advantage.",
 		descriptionPf2e:
-			"Can't see. All terrain is difficult terrain. –4 status penalty to Perception checks involving sight. Immune to visual effects. Auto-fail checks requiring sight. Off-guard.",
+			"Can't see. All terrain is difficult terrain. Auto-fail checks requiring sight. Immune to visual effects. Overrides dazzled.",
 		iconName: "EyeOff",
 		color: "neutral",
 	},
@@ -98,7 +98,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description: "Can't hear. Auto-fail hearing checks.",
 		description5e: "Can't hear. Auto-fail hearing checks.",
 		descriptionPf2e:
-			"Can't hear. –2 status penalty to Perception checks and Initiative. Auto-fail hearing checks. Immune to auditory effects.",
+			"Can't hear. Auto-critically-fail hearing checks. –2 status penalty to Perception. Auditory actions require DC 5 flat check. Immune to auditory effects.",
 		iconName: "EarOff",
 		color: "neutral",
 	},
@@ -166,7 +166,8 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 			"Incapacitated. Can't move or speak. Auto-fail Str/Dex saves. Attacks against have Advantage. Hits within 5 ft. are critical.",
 		description5e:
 			"Incapacitated. Can't move or speak. Auto-fail Str/Dex saves. Attacks against have Advantage. Hits within 5 ft. are critical.",
-		descriptionPf2e: "Can't act. Off-guard. –4 status penalty to AC.",
+		descriptionPf2e:
+			"Can't act. Off-guard. Can only Recall Knowledge or use mental actions.",
 		iconName: "ZapOff",
 		color: "yellow",
 	},
@@ -243,7 +244,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description5e:
 			"Incapacitated. Can't move. Can speak only falteringly. Auto-fail Str/Dex saves. Attacks against have Advantage.",
 		descriptionPf2e:
-			"Can't act. –X value to actions per turn while the value counts down.",
+			"Can't act. Lose X total actions across turns, then the condition ends. Overrides slowed.",
 		iconName: "Sparkles",
 		color: "yellow",
 		valued: true,
@@ -256,7 +257,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description5e:
 			"Incapacitated. Speed 0. Can't move or speak. Unaware of surroundings. Drops held items, falls Prone. Auto-fail Str/Dex saves. Attacks against have Advantage. Hits within 5 ft. are critical.",
 		descriptionPf2e:
-			"Can't act. Off-guard. –4 status penalty to AC. –3 to Perception. Fall prone, drop items.",
+			"Can't act. Off-guard. Blinded. –4 status penalty to AC, Perception, and Reflex saves. Fall prone, drop items.",
 		iconName: "Moon",
 		color: "indigo",
 	},
@@ -290,7 +291,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description: "",
 		description5e: "",
 		descriptionPf2e:
-			"Off-guard. Can't Delay, Ready, or use reactions. GM determines targets randomly. Flat check DC 11 to act normally each turn.",
+			"Off-guard. Can't Delay, Ready, or use reactions. Must Strike or cast offensive cantrips at random targets. DC 11 flat check when damaged to end.",
 		iconName: "CircleHelp",
 		color: "pink",
 		systems: ["pf2e"],
@@ -335,7 +336,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description: "",
 		description5e: "",
 		descriptionPf2e:
-			"–X status penalty to Con-based checks and DCs. Lose X × Hit Die in max HP. Decreases by 1 on full night's rest.",
+			"–X status penalty to Con-based checks and DCs. Lose X × level in max HP. Decreases by 1 on full night's rest.",
 		iconName: "Droplets",
 		color: "red",
 		systems: ["pf2e"],
@@ -359,7 +360,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description: "",
 		description5e: "",
 		descriptionPf2e:
-			"–X status penalty to Str-based rolls, including melee attack and damage rolls.",
+			"–X status penalty to Str-based rolls and DCs, including melee attack and damage rolls and Athletics checks.",
 		iconName: "TrendingDown",
 		color: "amber",
 		systems: ["pf2e"],
@@ -371,7 +372,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description: "",
 		description5e: "",
 		descriptionPf2e:
-			"–2 status penalty to all checks. Can't use hostile actions. Ends if hostile action is used against you.",
+			"–2 status penalty to Perception and skill checks. Can't use concentrate actions unless related to the fascination. Ends if hostile action is used against you or allies.",
 		iconName: "Eye",
 		color: "violet",
 		systems: ["pf2e"],
@@ -404,7 +405,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description: "",
 		description5e: "",
 		descriptionPf2e:
-			"Immobilized. Off-guard. Can't use actions with the move trait unless to Break Grapple.",
+			"Off-guard. Immobilized. Manipulate actions require DC 5 flat check or are wasted.",
 		iconName: "Hand",
 		color: "neutral",
 		systems: ["pf2e"],
@@ -415,7 +416,7 @@ export const CONDITION_DEFINITIONS: readonly ConditionDefinition[] = [
 		description: "",
 		description5e: "",
 		descriptionPf2e:
-			"Known location but can't be seen. DC 11 flat check to target. Can use Seek to find.",
+			"Known location but can't be seen. Off-guard to that creature. DC 11 flat check to target or miss.",
 		iconName: "EyeOff",
 		color: "slate",
 		systems: ["pf2e"],
@@ -521,5 +522,7 @@ export function getConditionsForEdition(
 ): readonly ConditionDefinition[] {
 	return CONDITION_DEFINITIONS.filter(
 		(d) => d.systems === undefined || d.systems.includes(edition),
-	);
+	)
+		.slice()
+		.sort((a, b) => a.label.localeCompare(b.label));
 }