[xXPBSXx]

What

The changed entry now reads:

sceNetGetsockopt (libSceNet) reads SO_ERROR from the host socket. Winsock does not clear a socket's pending error when it is read, so on Windows the library reports a pending error once and shows it again only after a new sceNetConnect; a second failure with the same code on the same socket without a sceNetConnect in between would be hidden. One title (PPSA02225) disassembled for this entry has one call site (0x1056d34), with level 0xffff, option 0x1007 (the FreeBSD values of SOL_SOCKET and SO_ERROR; the title holds only the numbers) and a length variable set to 4 beforehand, reached only after a helper called with the value 2 returned true. For a negative result it takes the thread's error code as the value; for a result of 0 or more with a value of 0 it reports the connection as made, and otherwise it turns the value into an error code of its own. The call does not show how often the title reads it per connection attempt or whether it keeps using the socket after an error.

Changed: docs/dev/debt/libSceNet.md. No code change.

Method: Ghidra 12.1.4 headless on the unwrapped executable of the title named; the addresses are the call sites in that executable as Ghidra maps it, read from the instruction listing (static reading only, nothing run on a console). The listing and decompiled code are game code and are not attached.

Fixes: #10096

Tested

Documentation only. tools/check_conventions.py --base upstream/main passes; git merge-tree against main is clean. #9523 and #9931, which edited the same file, are merged; this branch was refreshed from main after that. #9542, which also edited docs/dev/debt/libSceNet.md, is merged; this branch was refreshed from main after that (head 10624a6ad1ae9ec1efc026e4415e6d114f522f94). Every number in the sentence was recounted against the instruction listing. Not built, not run, no title run.

Checklist

• Based on current main; no other open PR implements the same functions
• One topic per PR; follow-ups go in a new PR
• No comments in code except technical debt
• Unimplemented paths throw (NotImplemented_nid_no_patch); silent stubs are listed under Silent stubs in their library's technical debt file
• No notes, investigation .md files or images added to the repository (attach them to this PR)
• New third-party code is a submodule built from source
• Generic behaviour, not specific to one title
• Depends on: none
• Fixes libSceNet debt: note how a title uses the sceNetGetsockopt error value #10096
• Unblocks: none
• GPUs tested: n/a
• AI-assisted: yes

AI-assisted (Claude Code).

🤖 Generated with Claude Code

[github-actions]

Some comparisons could not be completed because a pull request conflicts with the base branch. Review these shared files manually before concluding that there is no overlap.

Pull request	Shared files not compared
PR 5836	docs/dev/debt/libSceNet.md
PR 9932	docs/dev/debt/libSceNet.md
PR 9948	docs/dev/debt/libSceNet.md
PR 11272	docs/dev/debt/libSceNet.md

[MarcZX10]

The shortened entry now stays within the checker’s debt-length guidance and distinguishes the call-site’s visible argument/result handling from behavior that cannot be inferred. The exact-head conventions checker and git diff --check pass; hosted eligibility, conventions and labels pass. This is documentation-only, so runtime tests do not apply; hosted product build jobs are skipped by path policy.

Please refresh this branch from current main and coordinate the debt-file order with #9523, #9542 and #9931. The exact-head sync check reports 2 ahead/111 behind, with docs/dev/debt/libSceNet.md changed on both sides since #9060. Pairwise merges of these three PR heads with this head each conflict in that file. Please agree an order with those authors, add Depends on where appropriate, and re-run the overlap/conventions checks after resolving it. I also checked the uncomparable overlap reports: #9578 merges cleanly with this head; #5836's diff is in peer-socket/setsockopt paths, but its stale pairwise merge conflicts in Export.cpp, GuestSceNet.cpp and the debt file, so please recheck that comparison after refreshing.

[xXPBSXx]

@MarcZX10 thanks. Depends on: none; the conflict in docs/dev/debt/libSceNet.md is on the sceNetGetsockopt entry itself. Order: #9523 (not a draft) should go first and I rebase this PR after it, rewording the sentence against the entry that is current then; #9542 and #9931 are drafts and do not hold this PR. Refreshed from current main (head 3b9a41495f, base main 812ed15875), conventions and git diff --check pass; the changed entry is 979 characters. I rechecked #5836 and #9578 on the new head: both conflict with main in the same files by themselves (Export.cpp, GuestSceNet.cpp and the debt file for #5836; the libkernel.md debt file for #9578), so the pairwise merge conflicts with this head come from those PRs against main, not from this one-line change in libSceNet.md.

AI-assisted (Claude Code).

[xXPBSXx]

@MarcZX10 thanks. The checklist reads Depends on: #9523; the conflict in docs/dev/debt/libSceNet.md is on the sceNetGetsockopt entry itself. Order: #9523 (not a draft) should go first and I rebase this PR after it, rewording the sentence against the entry that is current then; #9542 and #9931 are drafts and do not hold this PR. Refreshed from current main (head 3b9a41495f, base main 812ed15875), conventions and git diff --check pass; the changed entry is 979 characters. I rechecked #5836 and #9578 on the new head: both conflict with main in the same files by themselves (Export.cpp, GuestSceNet.cpp and the debt file for #5836; the libkernel.md debt file for #9578), so the pairwise merge conflicts with this head come from those PRs against main, not from this one-line change in libSceNet.md.

AI-assisted (Claude Code).

[Queuse]

That order works for me: #9523 first, then this documentation update. I refreshed #9523 in 01b7dbaa7f and retained main's current debt wording around it. #9931 is now merged and #9542 is ready for review; neither needs to block this documentation follow-up after #9523 lands.

[xXPBSXx]

Head 295bca045563adf203fe54d372b07d576482ac8b: refreshed from main after #9523 and #9931 merged, so the conflicts with those two are gone and the checklist no longer needs Depends on: #9523; git merge-tree against current main 442c43a574 is clean. #9542 is still open and also edits docs/dev/debt/libSceNet.md; a merge of the two heads reports a content conflict there, and main is clean for each of them alone, so whichever lands second rebases and keeps both entries. Could you re-review that exact head?

AI-assisted (Claude Code).

[xXPBSXx]

Refreshed with a normal merge of main (head 10624a6ad1ae9ec1efc026e4415e6d114f522f94) after #9542 merged; the entry and the order notes are unchanged otherwise. Could you re-review?

AI-assisted (Claude Code).

Dismissing a stale review: the requested changes were addressed after it (new head) and the merge queue blocks on it. Please request changes again if anything is still open.

[adriadam10]

Checked the diff against the description, the conventions/labels/eligibility checks on this head, and that the earlier ordering request (#9523, #9542, #9931) is met now that all three are merged; docs-only, no code change; merged.