Clean up existing boss content, part 2 (V3 Milestone 5)

- Isshin: the fight now starts with a Genichiro, Way of Tomoe phase
  holding only the moves both wikis agree on (his lightning is left out:
  the wikis conflict). A data migration moves existing Isshin attempts
  up one phase.
- Optional move-level source fields (source_name / source_url, set
  together or not at all), used for moves taken from another page; the
  moveset shows them under the move.
- Both Owls gain Mikiri Counter (Fandom).
- Remove actions that cannot end an attempt: Owl's zig-zag, Guardian
  Ape's roar, Lady Butterfly's rafter jump, Great Shinobi Owl's charm.
- Guardian Ape's diving lunge loses its inferred telegraph; Isshin's
  phase-3 distance slash is no longer called a lightning variant.
- Consistent move types: backflips are combos, lightning attacks are
  lightning, deflectable moves are not marked unblockable.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: CanYouTeachMeHowToCode

backend/alembic/versions/6849ddf003e3_add_move_source.py

@@ -0,0 +1,35 @@
+"""add move source
+
+Revision ID: 6849ddf003e3
+Revises: 4b0d6abd4d80
+Create Date: 2026-10-03 02:32:35.605013
+
+"""
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision: str = '6849ddf003e3'
+down_revision: Union[str, Sequence[str], None] = '4b0d6abd4d80'
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+def upgrade() -> None:
+    """Upgrade schema: an optional per-move source, for a move whose data comes from a
+    different page than its boss's. Null means the move uses the boss's source.
+    """
+    op.add_column('moves', sa.Column('source_name', sa.String(length=200), nullable=True))
+    op.add_column('moves', sa.Column('source_url', sa.Text(), nullable=True))
+    # Autogenerate does not detect check constraints; this mirrors models.Move.
+    op.create_check_constraint('ck_moves_source_complete', 'moves', '(source_name IS NULL) = (source_url IS NULL)')
+
+
+def downgrade() -> None:
+    """Downgrade schema."""
+    op.drop_constraint('ck_moves_source_complete', 'moves', type_='check')
+    op.drop_column('moves', 'source_url')
+    op.drop_column('moves', 'source_name')

backend/alembic/versions/76532dd774e5_shift_isshin_attempts_for_way_of_tomoe_.py

@@ -0,0 +1,35 @@
+"""shift isshin attempts for way of tomoe phase
+
+The Isshin, the Sword Saint fight starts with Genichiro, Way of Tomoe. The seed data
+now records that as phase 1, so Isshin's own phases move from 1-3 to 2-4. Existing
+attempts keep pointing at the same part of the fight by moving up one phase.
+
+Revision ID: 76532dd774e5
+Revises: 6849ddf003e3
+Create Date: 2026-10-03 02:32:51.211107
+
+"""
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision: str = '76532dd774e5'
+down_revision: Union[str, Sequence[str], None] = '6849ddf003e3'
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+ISSHIN = "(SELECT id FROM bosses WHERE slug = 'isshin-sword-saint')"
+
+
+def upgrade() -> None:
+    """Upgrade data: Isshin attempts move up one phase."""
+    op.execute(f"UPDATE attempts SET phase_reached = phase_reached + 1 WHERE boss_id = {ISSHIN}")
+
+
+def downgrade() -> None:
+    """Downgrade data: attempts in the Way of Tomoe phase have no older equivalent; keep them at phase 1."""
+    op.execute(f"UPDATE attempts SET phase_reached = GREATEST(phase_reached - 1, 1) WHERE boss_id = {ISSHIN}")

backend/app/db/models.py

@@ -76,6 +76,7 @@ class Move(Base):
         CheckConstraint(
             "(name_zh_source = 'wiki') = (name_zh_source_url IS NOT NULL)", name="ck_moves_name_zh_source_url"
         ),
+        CheckConstraint("(source_name IS NULL) = (source_url IS NULL)", name="ck_moves_source_complete"),
     )
 
     id: Mapped[int] = mapped_column(primary_key=True)
@@ -92,6 +93,10 @@ class Move(Base):
     telegraph_zh: Mapped[str | None] = mapped_column(Text)
     counter_zh: Mapped[str | None] = mapped_column(Text)
     common_mistakes_zh: Mapped[str | None] = mapped_column(Text)
+    # Only for a move whose data comes from a different page than its boss's;
+    # null means the move uses the boss's source_name / source_url.
+    source_name: Mapped[str | None] = mapped_column(String(200))
+    source_url: Mapped[str | None] = mapped_column(Text)
     # Chinese name: either the name used by a Chinese wiki ('wiki', with the page in
     # name_zh_source_url) or a translation ('translation'), never presented as official.
     name_zh: Mapped[str | None] = mapped_column(String(200))

backend/app/models/boss.py

@@ -16,6 +16,9 @@ class BossMove(BaseModel):
     telegraph_zh: str | None = None
     counter_zh: str | None = None
     common_mistakes_zh: str | None = None
+    # Set only when this move's data comes from a different page than the boss's source.
+    source_name: str | None = None
+    source_url: str | None = None
     # "wiki": the name used by a Chinese wiki, see name_zh_source_url.
     # "translation": translated for this app; not an official name.
     name_zh_source: Literal["wiki", "translation"] | None = None

backend/app/seed.py

@@ -30,6 +30,8 @@
     "telegraph_zh",
     "counter_zh",
     "common_mistakes_zh",
+    "source_name",
+    "source_url",
 )
 
 # Every English text field that has a value needs its Chinese version.

backend/app/services/boss_service.py

@@ -56,6 +56,8 @@ def get_boss(session: Session, boss_slug: str) -> Boss | None:
                         telegraph_zh=link.move.telegraph_zh,
                         counter_zh=link.move.counter_zh,
                         common_mistakes_zh=link.move.common_mistakes_zh,
+                        source_name=link.move.source_name,
+                        source_url=link.move.source_url,
                         name_zh_source=link.move.name_zh_source,
                         name_zh_source_url=link.move.name_zh_source_url,
                     )