Skip to content

Anchor#4910

Merged
Malkierian merged 1 commit intoHarbourMasters:developfrom
garrettjoecox:anchor-next
Nov 14, 2025
Merged

Anchor#4910
Malkierian merged 1 commit intoHarbourMasters:developfrom
garrettjoecox:anchor-next

Conversation

@garrettjoecox
Copy link
Contributor

@garrettjoecox garrettjoecox commented Jan 19, 2025 •
edited by github-actions bot
Loading

anchor

Please read the known issues & TODO lists below before reporting any bugs or requesting any features.

This is a beta version, be prepared to encounter issues. The stable version can be found here. I am pushing this up now to encourage people to move off of the holiday build, as well as get some broader playtesting on the nightlies in preparation for the 9.0.0 release.

What is this?

Anchor is a co-op mode for Ship of Harkinian. It allows multiple people to play the game together, vanilla or randomizer, sharing inventory items and world flags.

Downloads

Build Artifacts

How To

  • All players should start at the file select screen
  • Open network tab, change scheme to "Anchor"
  • Agree upon a room ID across all players and click "Enable" (Use something unique, this is basically your password)
  • One person should configure the randomizer settings and generate a seed, then share the newly generated JSON spoiler file with other players
  • All players should load the same JSON spoiler file (drag it into SoH window), make sure seed icons match, then create a new file.
  • If using an existing save/seed ensure the player with the most progress loads the file first.
  • All other players should then load their files, and their save state will be synced with the main player's
  • After everyone has loaded in, verify on the network tab that it doesn’t warn about anyone being on a wrong version or seed

TODO/Known Issues

  • If you encounter anything not listed here please report it in the Anchor thread in discord
  • Beans usage/ammo not synced
  • Getting a milk refill will give all other players a bottle with milk
  • Giant's Knife get sent across as BGS
  • Issues saving/loading presets with the network block Upstream issue, will fix later

@garrettjoecox garrettjoecox force-pushed the anchor-next branch 3 times, most recently from ccedfa7 to b86ff22 Compare January 21, 2025 16:03
@lilacLunatic lilacLunatic mentioned this pull request Jan 22, 2025
Open
@mattman107 mattman107 mentioned this pull request Jan 24, 2025
Closed
@garrettjoecox garrettjoecox force-pushed the anchor-next branch 2 times, most recently from 468929e to 5431caf Compare January 31, 2025 01:14
aMannus
aMannus reviewed Feb 3, 2025
View reviewed changes
Copy link
Contributor

@aMannus aMannus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Want to bring up the naming point, and also tagged all the vanilla code changes. Most of these will likely just become a VB_Should. Might help with the effort to bring them over later, just figured it'd be nice to have an indication of how many there are and a list where we can cross them off as they are changed.

soh/soh/Network/Anchor/Anchor.cpp Outdated
Comment on lines +2 to +4

#include "Anchor.h"
#include <nlohmann/json.hpp>
Copy link
Contributor

@aMannus aMannus Feb 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've thrown up this ball on discord recently, but I'd like to put it out there for more public consideration. So far, this has always been called Anchor, which makes sense when you're trying to communicate it being separate from SoH itself. However, now that we're looking to upstream this, I'd personally really like if we rename everything currently called anchor to something more directly descriptive.

For example, my personal recommendation: Online Multiplayer.

I know a lot of people know the name Anchor by now, but I think there's going to be issues with people that download a normal SoH build and seeing "Anchor" in the network tab without having the description from the Anchor PR that people have now. Compare that to them seeing "Online Multiplayer", I don't think I have to explain people are going to understand what that means more quickly.

Just food for thought, I don't think I'll ever be in a position to force a change like this especially with Anchor's history, but I do think this is the right move.

@Fatih120

This comment was marked as off-topic.

Sign in to view
@Malkierian

This comment was marked as off-topic.

Sign in to view
@TheLynk TheLynk mentioned this pull request Jun 29, 2025
Closed
@ThatCodingFrog ThatCodingFrog mentioned this pull request Oct 1, 2025
Closed
@TheLynk TheLynk mentioned this pull request Oct 25, 2025
Draft
@garrettjoecox garrettjoecox force-pushed the anchor-next branch 9 times, most recently from d52690d to ae9136a Compare November 8, 2025 21:57
@garrettjoecox
Copy link
Contributor Author

garrettjoecox commented Nov 8, 2025

This PR has been updated with various fixes, and all vanilla code touch points have been migrated to hooks.

The known remaining issues are listed at the bottom of the PR.

@garrettjoecox garrettjoecox force-pushed the anchor-next branch 4 times, most recently from be95bea to 03fcbeb Compare November 11, 2025 03:21
@garrettjoecox garrettjoecox force-pushed the anchor-next branch 5 times, most recently from bb8f4d3 to e75ad66 Compare November 12, 2025 05:36
Malkierian
Malkierian approved these changes Nov 12, 2025
View reviewed changes
Copy link
Contributor

@Malkierian Malkierian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Honestly, with the footprint into existing code being so miniscule now, and with the playtesting that has been ongoing for so long, I think this can just be merged. I'll give anyone who disagrees 2 days to voice it.

@garrettjoecox garrettjoecox force-pushed the anchor-next branch 9 times, most recently from aa706d4 to 766d5de Compare November 14, 2025 04:19
@garrettjoecox @mattman107
@dcvz @MelonSpeedruns @aMannus @Malkierian @Caladius @Patrick12115 @PurpleHato @balloondude2 @lilacLunatic @flee135 @Sirius902
Anchor
bfa3813 
This is far and away the proudest thing I have been apart of in the Zelda community. Thank you to everyone who either directly or indirectly contributed to this effort. If I forgot to co-author you and you were involved, know that I simply forgot but still appreciated your help! Also thank you to the literal tens of thousands of people who have played Anchor, providing feedback and bug reports. Super thrilled to finally have this merged into the main Ship of Harkinian experience.

Co-authored-by: mattman107 <65982675+mattman107@users.noreply.github.com>
Co-authored-by: David Chavez <david@dcvz.io>
Co-authored-by: MelonSpeedruns <melonspeedruns@outlook.com>
Co-authored-by: aMannus <mannusmenting@gmail.com>
Co-authored-by: Malkierian <malkierian@gmail.com>
Co-authored-by: Caladius <Caladius@users.noreply.github.com>
Co-authored-by: Patrick12115 <115201185+Patrick12115@users.noreply.github.com>
Co-authored-by: PurpleHato <47987542+PurpleHato@users.noreply.github.com>
Co-authored-by: balloondude2 <55861555+balloondude2@users.noreply.github.com>
Co-authored-by: lilacLunatic <8488221+lilacLunatic@users.noreply.github.com>
Co-authored-by: Felix Lee <flee135@users.noreply.github.com>
Co-authored-by: Sirius902 <10891979+Sirius902@users.noreply.github.com>
@Malkierian Malkierian changed the title Anchor Beta Anchor Nov 14, 2025
@Malkierian Malkierian merged commit 060878f into HarbourMasters:develop Nov 14, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aMannus aMannus aMannus left review comments

@Malkierian Malkierian Malkierian approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@garrettjoecox @Fatih120 @Malkierian @aMannus