Skip to content

Add The Hand in the Sand quest - #1219

Merged
Ebp90 merged 6 commits into
mainfrom
hand-in-the-sand
Sep 1, 2026
Merged

Add The Hand in the Sand quest#1219
Ebp90 merged 6 commits into
mainfrom
hand-in-the-sand

Conversation

@Ebp90

@Ebp90 Ebp90 commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Ones issue: Betty should stop interacting with the player and walk to the nearest tile to the table to place the glass on it. Marked as TODO

@Ebp90

Ebp90 commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

Also Betty needs to sell pink dye after the quest.

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Qodana for JVM

5 new problems were found

Inspection name Severity Problems
Unused symbol 🔶 Warning 4
Redundant 'suspend' modifier 🔶 Warning 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 43.21809% with 427 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.84%. Comparing base (5267f0e) to head (be713de).

Files with missing lines Patch % Lines
...ent/quest/member/hand_in_the_sand/HandInTheSand.kt 34.27% 153 Missing and 10 partials ⚠️
...n/kotlin/content/area/asgarnia/port_sarim/Betty.kt 27.77% 80 Missing and 11 partials ⚠️
.../main/kotlin/content/area/kandarin/yanille/Bert.kt 46.21% 62 Missing and 9 partials ⚠️
...ain/kotlin/content/area/karamja/brimhaven/Sandy.kt 57.44% 31 Missing and 9 partials ⚠️
...ntent/area/kandarin/yanille/GuardCaptainYanille.kt 40.42% 23 Missing and 5 partials ⚠️
...ain/kotlin/content/area/asgarnia/entrana/Mazion.kt 32.43% 23 Missing and 2 partials ⚠️
...lin/content/area/kandarin/yanille/ZavisticRarve.kt 86.36% 4 Missing and 5 partials ⚠️
Additional details and impacted files
Files with missing lines Coverage Δ
...in/kotlin/content/area/asgarnia/entrana/Frincos.kt 33.33% <100.00%> (ø)
game/src/main/kotlin/content/quest/Quest.kt 56.92% <100.00%> (+0.67%) ⬆️
...lin/content/area/kandarin/yanille/ZavisticRarve.kt 44.81% <86.36%> (+19.69%) ⬆️
...ain/kotlin/content/area/asgarnia/entrana/Mazion.kt 37.50% <32.43%> (-5.36%) ⬇️
...ntent/area/kandarin/yanille/GuardCaptainYanille.kt 40.42% <40.42%> (ø)
...ain/kotlin/content/area/karamja/brimhaven/Sandy.kt 57.44% <57.44%> (ø)
.../main/kotlin/content/area/kandarin/yanille/Bert.kt 46.21% <46.21%> (ø)
...n/kotlin/content/area/asgarnia/port_sarim/Betty.kt 29.45% <27.77%> (-0.55%) ⬇️
...ent/quest/member/hand_in_the_sand/HandInTheSand.kt 34.27% <34.27%> (ø)

... and 2 files with indirect coverage changes

@@             Coverage Diff              @@
##               main    #1219      +/-   ##
============================================
+ Coverage     43.79%   43.84%   +0.05%     
- Complexity    10338    10378      +40     
============================================
  Files          2050     2054       +4     
  Lines         91736    92464     +728     
  Branches      14958    15065     +107     
============================================
+ Hits          40173    40540     +367     
- Misses        45797    46104     +307     
- Partials       5766     5820      +54     
Components Coverage Δ
Content 44.13% <43.21%> (+0.06%) ⬆️
Engine 45.77% <ø> (ø)
Network 71.36% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Ebp90 and others added 4 commits September 1, 2026 20:55
Covers the full quest: Bert's severed hand, the drunk Guard Captain,
Zavistic Rarve, Sandy's rota and interrogation, Betty's truth serum
chain, Mazion on Entrana, and the instanced sandpit refill cutscene.

Renames four item definitions to the ids the existing content already
referenced (beer_hand, magic_scroll, magical_orb_active, wizard_head),
names all seventeen quest stages in the varbit so progress is stored as
a string like every other quest, and moves the two server-only flags to
a vars.toml because varbits require an id.

Frincos registered his Talk-to against 'mazion' rather than 'frincos',
stacking a second handler on Mazion and opening a shop he doesn't own.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Ebp90
Ebp90 merged commit 4e0ae09 into main Sep 1, 2026
5 checks passed
@Ebp90
Ebp90 deleted the hand-in-the-sand branch September 1, 2026 20:15
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.

2 participants