Skip to content

docs(debt): list the unimplemented AJM codec 23 of PPSA21480 - #6672

Merged
oneandonlydean merged 1 commit into
boykopovar:mainfrom
xXPBSXx:docs/debt-ajm-codec-23
Oct 17, 2026
Merged

oneandonlydean merged 1 commit into
boykopovar:mainfrom
xXPBSXx:docs/debt-ajm-codec-23

Conversation

@xXPBSXx

@xXPBSXx xXPBSXx commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

What

Broken Sword - Shadow of the Templars: Reforged (PPSA21480) stops at AJM: decoding codec 23 is not implemented (PPSA21480_rZ1/run_233643.err line 8, main 31cbb55d7; the earlier stop sceCoredumpWriteUserString is gone since #5834). Ajm.cpp knows the codec ids MP3 0, ATRAC9 1, M4AAC 2 and Opus 24 and throws for every other id when decoding; the technical debt file does not list that. This adds one entry under Unknown function info saying so, with the title and the unknown codec. Issue #6671 has the log lines.

Tested

Documentation only. The entry restates the codec ids from Ajm.cpp (CODEC_MP3, CODEC_AT9, CODEC_M4AAC, CODEC_OPUS) and the throw at the decode path on main; the log lines were read with Python; not tested: a run with this change (no code changed).

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 code changed)
  • No notes, investigation .md files or images added to the repository
  • New third-party code is a submodule built from source (none)
  • Generic behaviour, not specific to one title
  • Depends on: none
  • Unblocks: none (documentation; title PPSA21480, AJM: decoding codec 23 is not implemented (Broken Sword - Shadow of the Templars: Reforged, PPSA21480) #6671)
  • GPUs tested: n/a
  • AI-assisted: yes

AI-assisted (Claude Code).

🤖 Generated with Claude Code

Broken Sword - Shadow of the Templars: Reforged stops at AJM decoding codec 23, which is not one of the codec ids known in Ajm.cpp (MP3 0, ATRAC9 1, M4AAC 2, Opus 24).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@xXPBSXx

xXPBSXx commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

@adriadam10 a short request for review of this documentation PR (one entry in docs/dev/debt/libSceAjm.native.md: AJM decoding throws for codecs other than MP3, ATRAC9, M4AAC and Opus; Broken Sword, PPSA21480, stops at codec 23, see #6671). Head 4dc23f9c1 is MERGEABLE / CLEAN, the Conventions, Eligibility and Intake checks are green.

AI-assisted (Claude Code).

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Other open pull requests touch the same code. Merging one will break the other, or both implement the same thing. If this one builds on them, list them in Depends on.

Pull request Conflicting files (lines on main) Same exports Same new files
PR 6841 docs/dev/debt/libSceAjm.native.md L9

@MarcZX10

Copy link
Copy Markdown
Contributor

The overlap bot flags #6841 on the same AJM debt entry. #6841 adds a trace-backed codec-23 implementation and updates this entry to describe the observed behavior and remaining assumptions. The two changes are separate in scope, but if #6841 lands, this debt-only PR should not reintroduce “codec 23 unimplemented.” Please coordinate merge order with #6841’s author; that PR has been asked to rebase and resolve the shared entry.

@oneandonlydean
oneandonlydean merged commit 2993eef into boykopovar:main Oct 17, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants