Skip to content

docs: update stale documentation for recent SDK additions#8

Merged
AndrewAltimit merged 1 commit intomainfrom
docs/update-stale-documentation
Feb 15, 2026
Merged

docs: update stale documentation for recent SDK additions#8
AndrewAltimit merged 1 commit intomainfrom
docs/update-stale-documentation

Conversation

@AndrewAltimit
Copy link
Owner

Summary

  • Add psp::audiocodec module (hardware codec decoder for MP3/AAC/ATRAC3/ATRAC3plus) to syscall bindings and Platform SDK tables
  • Add psp::hook module (kernel syscall hooking with inline fallback) to Platform SDK and Kernel Mode tables
  • Add psp::sys::sctrl (SystemCtrlForKernel CFW API) to syscall bindings and Kernel Mode tables
  • Add SrcChannel to psp::audio entry (sample rate conversion channel)
  • Add find_sync() and ID3v2 tag parsing to psp::mp3 entry
  • Fix codec syscall count (5 not 10) after sceAudiocodec* was un-gated from kernel-only
  • Update syscall count (~825 -> ~829) and module count (36+ -> 38+)
  • Update kernel mode description in CLAUDE.md and AGENTS.md

Test plan

  • Verify all documented modules exist in psp/src/lib.rs
  • Verify syscall count via grep
  • Confirm markdown table formatting is correct
  • Visual review of README rendering on GitHub

Generated with Claude Code

Document audiocodec, hook, sctrl modules and SrcChannel/MP3 utilities
added in recent commits. Fix codec syscall count after kernel-gate
removal, update module counts across README, CLAUDE.md, and AGENTS.md.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

Gemini AI Code Review

Issues

(none)

Previous Issues

(none)

Suggestions

  • README.md:137 - The psp::hook module is added to the "Graphics & Rendering" table in the Platform SDK section. Given it deals with kernel syscall hooking and CFW plugins, it would be more appropriately placed in the "Kernel & Hardware" table (which already exists and contains me and hw).
  • README.md:101 - Consider adding skip_id3v2() to the "Key API" column for psp::mp3 to match the level of detail provided for find_sync().

Notes

  • 2 claim(s) were automatically filtered as potential hallucinations (file:line content didn't match claims)

  • Verified that psp::audiocodec exists as both a high-level module (psp/src/audiocodec.rs) and as syscall bindings (psp/src/sys/codec.rs).

  • Confirmed that sceAudiocodec* syscalls are indeed available in user mode (exported in sys/mod.rs without kernel gate), justifying their move from the "Kernel-only" section.

  • The syscall count update (~825 to ~829) is a reasonable approximation of the current bindings.

Reaction


Generated by Gemini AI (gemini-3-flash-preview). Supplementary to human reviews.

@AndrewAltimit
Copy link
Owner Author

Review Response Agent (Iteration 1)

Status: No changes needed

Fixed Issues

  • (none)

Ignored Issues

  • Gemini suggestion to move psp::hook from "Graphics & Rendering" to "Kernel & Hardware" — verified this is incorrect; psp::hook is already in the "Kernel-Only" section at README.md:137, not in "Graphics & Rendering". The line number mapping was wrong.
  • Gemini suggestion to add skip_id3v2() to psp::mp3 Key API column — optional polish, not a bug. The description column already mentions "ID3v2 tag parsing" which covers this functionality.
  • Codex review — failed to execute due to OpenAI rate limit (HTTP 429). No feedback produced.

Deferred to Human

  • (none)

Notes

  • This is a docs-only PR (3 markdown files, +16/-10 lines). No code changes to review for bugs.
  • Both Gemini suggestions were either based on incorrect line mapping or were minor polish items, not actionable issues.

The agent reviewed feedback but determined no code changes were required.

@AndrewAltimit AndrewAltimit merged commit 711b599 into main Feb 15, 2026
7 checks passed
@AndrewAltimit AndrewAltimit deleted the docs/update-stale-documentation branch February 15, 2026 12:08
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.

1 participant