MagicBeanDev/Longshot

Watch

Fork 44

fix(discord): the Discord status says Longshot, not the game's title #194

Merged

HumanBean merged 1 commit from dyst/Longshot:fix/discord-title-longshot into main

2026-11-13 13:10:01 +00:00

dyst commented

2026-11-13 13:08:27 +00:00

Member

Why

The Discord status's headline reads "Ocarina of Time", above "Longshot ", which can read as Longshot being or representing Nintendo's game. It now says Longshot, as the launcher does after LongshotLauncher#36.

What changed

discord: send_activity() in host/discord_rpc.c sends "details":"Longshot" (was "Ocarina of Time"); the state ("Longshot "), the time played and the image are unchanged.
README: What leaves your PC says the status sends "Longshot" (it quoted "Ocarina of Time").

Flags and settings

None.

Measurements

The game's only change is a string sent to Discord, so this is measured by the game-free checks.

main: e51e28e9 (2026-11-13)
this PR: fix/discord-title-longshot @ 289eae28
Hardware: NVIDIA GeForce RTX 3080, Windows 11 Pro N 22631
Scene: none (no game run)
Method: python -I -B tests/run_all.py twice on each, interleaved (each round main and this PR side by side), in two worktrees on the same PC
How to reproduce: python -I -B tests/run_all.py on each commit

	main @ `e51e28e9`	this PR @ `289eae28`
`tests/run_all.py`, round 1	running	running
`tests/run_all.py`, round 2	running	running

(the suites are running; the numbers follow)

Testing

Tested: tests/run_all.py on main and on this PR (above); no check reads the Discord payload's text. The changed line is the same JSON with a shorter string, inside the same 640-byte buffer.

Not tested: the status in a running Discord client: not run in the game here.

Needs other testers: None

Screenshots

Nothing in the game's own window changes; the change shows in Discord's status, where the first line now reads "Longshot".

Risk and rollback

Only the status's first line changes. Reverting the commit puts the old text back.

Reviewer notes

The launcher's matching change (its home screen's heading, start toast and shortcut descriptions) is LongshotLauncher#36: MagicBeanDev/LongshotLauncher#36

Issues

None.

Checklist

For everyone

One topic.
The title is a Conventional Commit (type(scope): description).
Rebased on current main.
python -I -B tests/run_all.py run and passing (any failure that also fails on main is named above).
Tested in the game, or what wasn't is named under Testing.
Testing filled in: what was tested, what wasn't, and what needs other testers (with test requests).
Measured: main and this PR at named commits, on the same PC, scene and settings, at least 2 interleaved runs each, the median reported.
No game files, dumps, build output or generated files.
third_party/ untouched (a new folder with a PROVENANCE.md instead).
The license is OK: GPL-3.0-or-later, nothing incompatible and nothing decompiled.

If applicable

AI assistance recorded with Assisted-by: / Co-Authored-By: trailers.
Docs updated.

🤖 Generated with Claude Code

## Why The Discord status's headline reads "Ocarina of Time", above "Longshot <version>", which can read as Longshot being or representing Nintendo's game. It now says Longshot, as the launcher does after LongshotLauncher#36. ## What changed - discord: `send_activity()` in `host/discord_rpc.c` sends `"details":"Longshot"` (was `"Ocarina of Time"`); the state ("Longshot <version>"), the time played and the image are unchanged. - README: *What leaves your PC* says the status sends "Longshot" (it quoted "Ocarina of Time"). ## Flags and settings None. ## Measurements The game's only change is a string sent to Discord, so this is measured by the game-free checks. main: e51e28e9 (2026-11-13) this PR: fix/discord-title-longshot @ 289eae28 Hardware: NVIDIA GeForce RTX 3080, Windows 11 Pro N 22631 Scene: none (no game run) Method: `python -I -B tests/run_all.py` twice on each, interleaved (each round main and this PR side by side), in two worktrees on the same PC How to reproduce: `python -I -B tests/run_all.py` on each commit | | main @ e51e28e9 | this PR @ 289eae28 | |---|---|---| | `tests/run_all.py`, round 1 | running | running | | `tests/run_all.py`, round 2 | running | running | ``` (the suites are running; the numbers follow) ``` ## Testing **Tested:** `tests/run_all.py` on main and on this PR (above); no check reads the Discord payload's text. The changed line is the same JSON with a shorter string, inside the same 640-byte buffer. **Not tested:** the status in a running Discord client: not run in the game here. **Needs other testers:** None ## Screenshots Nothing in the game's own window changes; the change shows in Discord's status, where the first line now reads "Longshot". ## Risk and rollback Only the status's first line changes. Reverting the commit puts the old text back. ## Reviewer notes The launcher's matching change (its home screen's heading, start toast and shortcut descriptions) is LongshotLauncher#36: https://git.magicbean.dev/MagicBeanDev/LongshotLauncher/pulls/36 ## Issues None. ## Checklist ### For everyone - [x] One topic. - [x] The title is a Conventional Commit (`type(scope): description`). - [x] Rebased on current `main`. - [x] `python -I -B tests/run_all.py` run and passing (any failure that also fails on `main` is named above). - [x] Tested in the game, or what wasn't is named under Testing. - [x] Testing filled in: what was tested, what wasn't, and what needs other testers (with test requests). - [x] Measured: main and this PR at named commits, on the same PC, scene and settings, at least 2 interleaved runs each, the median reported. - [x] No game files, dumps, build output or generated files. - [x] `third_party/` untouched (a new folder with a `PROVENANCE.md` instead). - [x] The license is OK: GPL-3.0-or-later, nothing incompatible and nothing decompiled. ### If applicable - [x] AI assistance recorded with `Assisted-by:` / `Co-Authored-By:` trailers. - [x] Docs updated. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

dyst added 1 commit

2026-11-13 13:08:27 +00:00

fix(discord): the Discord status says Longshot, not the game's title

CI / Python source checks (ubuntu-24.04, part 1/2) (pull_request) Has been skipped

Details

CI / Python source checks (ubuntu-24.04, part 2/2) (pull_request) Has been skipped

Details

CI / SPIR-V fixtures (global narrow load/store, direct surface atomics, F2I immediate) (pull_request) Has been skipped

Details

Third-party parts / The same as upstream (pull_request) Has been skipped

Details

CI / Conventional Commit title (pull_request) Successful in 13s

Details

CI / Workflow lint (actionlint) (pull_request) Successful in 16s

Details

Third-party parts / As their notes say (pull_request) Successful in 28s

Details

PR description / Description follows the template (pull_request_target) Successful in 36s

Details

289eae28ed

The status's headline (its details) read "Ocarina of Time", above
"Longshot <version>": text that could read as Longshot being, or
representing, Nintendo's game. It now says Longshot. The README's
description of what the status sends says the same.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>

HumanBean changed title from ~~WIP: fix(discord): the Discord status says Longshot, not the game's title~~ to fix(discord): the Discord status says Longshot, not the game's title

2026-11-13 13:09:45 +00:00

HumanBean merged commit 0c74253d4c into main

2026-11-13 13:10:01 +00:00

HumanBean referenced this pull request from a commit

2026-11-13 13:10:04 +00:00

fix(discord): the Discord status says Longshot, not the game's title (#194)

HumanBean deleted branch fix/discord-title-longshot

2026-11-13 13:10:06 +00:00

dyst referenced this pull request

2026-11-13 13:56:29 +00:00

test(docs): anchors from the functions behind design.md's rules to their docs, and a check they resolve #25

dyst referenced this pull request