docs(agents): link GitHub issues and pull requests by their full URL

llun commented

2026-11-13 03:26:44 +00:00

Member

Why

Agents write Hookshot references as bare numbers (#2843). On git.magicbean.dev a bare number links to Longshot's own issue or PR of that number, and on Discord to nothing.

What changed

AGENTS.md: one paragraph near the top saying to link GitHub issues and PRs by their full URL.

Flags and settings

None

Measurements

main: b9cdc432 (2026-11-13)
this PR: docs/full-github-links @ 51b55363
Hardware: Ryzen 9 9900X3D, 64 GB, Windows 11 Pro, Python 3.15, llvm-mingw
Scene: no game run (docs only)
Method: python -I -B tests/run_all.py, two runs each
How to reproduce: run the command above on main and on this branch.

	main @ `b9cdc432`	this PR @ `51b55363`
tests/run_all.py passed (run 1 / run 2)	484 / 484	484 / 484

main @ b9cdc432:     484 passed, 6 failed, 40 skipped in 105 s / 108 s
this PR @ 51b55363:  484 passed, 6 failed, 40 skipped in 105 s / 109 s

Testing

Tested: tests/run_all.py on main and on this PR. The same 6 checks fail on both on this PC: test_bake_portability, test_check_third_party, test_finish_output, test_lift_cache, test_release, test_setup_runtime. Forgejo runs no CI, so these runs were local.

Not tested: Nothing: it's text only.

Needs other testers: None

Screenshots

Nothing on screen

Risk and rollback

None; reverting the commit is enough.

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 Agents write Hookshot references as bare numbers (`#2843`). On git.magicbean.dev a bare number links to Longshot's own issue or PR of that number, and on Discord to nothing. ## What changed - AGENTS.md: one paragraph near the top saying to link GitHub issues and PRs by their full URL. ## Flags and settings None ## Measurements main: b9cdc432 (2026-11-13) this PR: docs/full-github-links @ 51b55363 Hardware: Ryzen 9 9900X3D, 64 GB, Windows 11 Pro, Python 3.15, llvm-mingw Scene: no game run (docs only) Method: `python -I -B tests/run_all.py`, two runs each How to reproduce: run the command above on main and on this branch. | | main @ b9cdc432 | this PR @ 51b55363 | |---|---|---| | tests/run_all.py passed (run 1 / run 2) | 484 / 484 | 484 / 484 | ``` main @ b9cdc432: 484 passed, 6 failed, 40 skipped in 105 s / 108 s this PR @ 51b55363: 484 passed, 6 failed, 40 skipped in 105 s / 109 s ``` ## Testing **Tested:** `tests/run_all.py` on main and on this PR. The same 6 checks fail on both on this PC: test_bake_portability, test_check_third_party, test_finish_output, test_lift_cache, test_release, test_setup_runtime. Forgejo runs no CI, so these runs were local. **Not tested:** Nothing: it's text only. **Needs other testers:** None ## Screenshots Nothing on screen ## Risk and rollback None; reverting the commit is enough. ## 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)

llun force-pushed docs/full-github-links from 30ebc6c550

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

Details

CI / Conventional Commit title (pull_request) Has been cancelled

Details

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

Details

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

Details

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

Details

CI / Workflow lint (actionlint) (pull_request) Has been cancelled

Details

Third-party parts / As their notes say (pull_request) Has been cancelled

Details

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

Details

to 51b5536356

CI / Conventional Commit title (pull_request) Blocked by required conditions

Details

CI / Python source checks (ubuntu-24.04, part 1/2) (pull_request) Blocked by required conditions

Details

CI / Python source checks (ubuntu-24.04, part 2/2) (pull_request) Blocked by required conditions

Details

CI / SPIR-V fixtures (global narrow load/store, direct surface atomics, F2I immediate) (pull_request) Blocked by required conditions

Details

CI / Workflow lint (actionlint) (pull_request) Blocked by required conditions

Details

Third-party parts / As their notes say (pull_request) Blocked by required conditions

Details

Third-party parts / The same as upstream (pull_request) Blocked by required conditions

Details

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

Details

2026-11-13 04:09:03 +00:00

Compare

llun commented

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

Author

Member

51b55363 is the shortened version: the paragraph is now three lines, and the description is down to the template's essentials. Still text only; tests/run_all.py gives 484 passed on two runs, and its 6 failures also fail on main on this PC.

51b55363 is the shortened version: the paragraph is now three lines, and the description is down to the template's essentials. Still text only; tests/run_all.py gives 484 passed on two runs, and its 6 failures also fail on main on this PC.