Skip to content

fix: man0u1 gate meetup double-played F'lhaminn's greeting (144-146)#208

Open
swstegall wants to merge 1 commit into
developfrom
fix/man0u1-gate-meetup-double-play
Open

fix: man0u1 gate meetup double-played F'lhaminn's greeting (144-146)#208
swstegall wants to merge 1 commit into
developfrom
fix/man0u1-gate-meetup-double-play

Conversation

@swstegall

Copy link
Copy Markdown
Owner

Summary

Fixes the SEQ_060 Gate of Nald meetup in Man0u1 "Court in the Sands", which spoke F'lhaminn's greeting twice:

  • once as server-side SendGameMessage(144/145/146) on the FLHAMINN_GATE talk (built on a "no client event exists for these" assumption), and
  • again inside the man0u170 cutscene fired on the proximity trigger push.

A 4-playthrough OCR sweep (eZgcq-FMpfw, XbDE5OQ_Y2g, WlKVCvRgQs0, 6eWqFhITeeM) confirms man0u170 IS the gate cutscene that speaks lines 144/145/146. The retail geography is unchanged and now verified:

guild Corguevais scene (man0u160, ending "await me at the Gate of Nald") → travel → man0u170 gate meetup → escort launches from the gateman0u175 is the post-duty Camp Black Brush arrival.

Follow-up to the F'lhaminn dual-spawn scoping in #207 (and the #53 port).

Changes

  • scripts/lua/quests/man/man0u1.lua — drop the duplicate server-side wait-text; man0u170 is now the single pre-duty cutscene (inside startMan0u1Escort). New shared tryStartMan0u1Escort (duty-join confirm → launch) is reached by both F'lhaminn's talk and the proximity trigger, so talking to her starts the escort instead of being a dead click.
  • scripts/tests/quests/uldah_msq2.lua — the SEQ_057..060 case now asserts talk → confirm → decline stays at SEQ_060, and push → confirm → accept launches the escort with man0u170.
  • docs/implementation/quest_man0u1_court_in_the_sands.md — updated the SEQ_060 row and added finding Test Garlemald Server 5 #8.

Test plan

All six CI gates green locally:

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo build --workspace --all-targets --locked
  • cargo test --workspace --locked — 877 passed, 0 failed
  • cargo run --locked -p content-test — 23/23 (incl. the updated SEQ_057..060 case)
  • PROFILE=debug scripts/smoke-local.sh — all four services boot
  • Live-test: at the Gate of Nald, confirm F'lhaminn's greeting (144/145/146) plays once as the man0u170 cutscene, and that talking to her (or walking the trigger) opens the duty-join confirm → escort.

🤖 Generated with Claude Code

https://claude.ai/code/session_012Eokm8V9H6ZTiSvDTMvqVC

The SEQ_060 Gate of Nald meetup spoke F'lhaminn's greeting twice: once as
server-side SendGameMessage(144/145/146) on the FLHAMINN_GATE talk (built
on a "no client event exists for these" assumption), and again inside the
man0u170 cutscene fired on the trigger push.

A 4-playthrough OCR sweep (eZgcq-FMpfw, XbDE5OQ_Y2g, WlKVCvRgQs0,
6eWqFhITeeM) confirms man0u170 IS the gate cutscene that speaks lines
144/145/146. The retail geography is unchanged and verified: one
continuous guild cutscene (man0u160, ending "await me at the Gate of
Nald") -> travel -> the man0u170 gate meetup -> escort launches from the
gate -> man0u175 is the post-duty Camp Black Brush arrival (not pre-duty).

Fix:
- Drop the duplicate server-side wait-text. man0u170 is now the single
  pre-duty cutscene, played inside startMan0u1Escort.
- Route both F'lhaminn's talk and the proximity trigger through a shared
  tryStartMan0u1Escort (duty-join confirm -> launch), so talking to her
  starts the escort instead of being a dead click.
- content-test (uldah_msq2.lua): the SEQ_057..060 case now asserts
  talk -> confirm -> decline stays at SEQ_060, and push -> confirm ->
  accept launches the escort with man0u170.
- design doc: updated the SEQ_060 row and added finding 8.

All six CI gates green (fmt, clippy, build, test, content-test, smoke).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Eokm8V9H6ZTiSvDTMvqVC
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant