[PR #6794] [CLOSED] docs(desktop): list all supported IL2CPP metadata versions #7415

New issue

Closed

opened 2026-11-01 21:47:42 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-11-01 21:47:42 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/6794
Author: @ShuaixinHuang
Created: 10/23/2026
Status: ❌ Closed

Base: main ← Head: codex/docs-desktop-il2cpp-versions

📝 Commits (1)

2a0f591 docs(desktop): list all supported IL2CPP metadata versions

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 docs/desktop-applications.md (+2 -2)

📄 Description

Summary

Fixes #6672. The desktop application guide lists only IL2CPP metadata versions 29, 31 and 39 for follow-up inspection, omitting already-supported versions 24 and 27. List all five versions consistently with the shared admission constant and dedicated IL2CPP guide.

Change and scope

One Markdown file, two lines changed. Documentation only. Preserve the warning that a magic/version prefix does not establish complete metadata validity or native-binary correspondence; the linked IL2CPP guide retains version-24 layout restrictions. No code, tests, schema, provider, package or dependency changes. Separate maintainer scope decision: not applicable.

Documentation or usability only; no target interaction.

Validation performed

Base bd4f5e9c; head 2a0f5914. Windows x64, pinned Node 24.22.1 and locked dependencies:

npm run check: all six tasks, module boundaries and architecture guards passed.
npm run docs:check: six tasks and nine README consistency tests passed.
Executed compiled DesktopApplicationService/Reader probe on generated magic/version prefixes: 24,27,29,31,39 each offered inspect_il2cpp_metadata; version12 and truncated version did not. All seven assertions passed. Temporary fixture directories were removed.
Oxfmt, git diff --check, commit hooks and push-hook typecheck/lint passed.
Source comparison: desktop follow-up admission uses IL2CPP_METADATA_VERSIONS [24,27,29,31,39]; dedicated IL2CPP guide agrees.
No new tests for the wording-only edit. Prefix fixtures qualify directory triage only, not complete metadata decoding, a real Unity installation or CLI/MCP transport. No native provider or target execution was used.
Online CI required passed for head 2a0f5914: https://github.com/morluto/rea/actions/runs/39976732053 .

Author readiness

Human owner: @ShuaixinHuang

I reviewed the final diff and relevant existing code, understand the change and its edge cases, and take responsibility for the complete PR, including AI-assisted work.
I checked the evidence above, confirmed it supports the current change, and recorded meaningful validation gaps.
I declare this PR ready for initial maintainer review.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/6794 **Author:** [@ShuaixinHuang](https://github.com/ShuaixinHuang) **Created:** 10/23/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `codex/docs-desktop-il2cpp-versions` --- ### 📝 Commits (1) - [`2a0f591`](https://github.com/morluto/rea/commit/2a0f59144c5673791c99da11f74e13ae26ca1802) docs(desktop): list all supported IL2CPP metadata versions ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `docs/desktop-applications.md` (+2 -2) </details> ### 📄 Description ## Summary Fixes #6672. The desktop application guide lists only IL2CPP metadata versions 29, 31 and 39 for follow-up inspection, omitting already-supported versions 24 and 27. List all five versions consistently with the shared admission constant and dedicated IL2CPP guide. ## Change and scope One Markdown file, two lines changed. Documentation only. Preserve the warning that a magic/version prefix does not establish complete metadata validity or native-binary correspondence; the linked IL2CPP guide retains version-24 layout restrictions. No code, tests, schema, provider, package or dependency changes. Separate maintainer scope decision: not applicable. - [x] Documentation or usability only; no target interaction. ## Validation performed Base bd4f5e9c; head 2a0f5914. Windows x64, pinned Node 24.22.1 and locked dependencies: - npm run check: all six tasks, module boundaries and architecture guards passed. - npm run docs:check: six tasks and nine README consistency tests passed. - Executed compiled DesktopApplicationService/Reader probe on generated magic/version prefixes: 24,27,29,31,39 each offered inspect_il2cpp_metadata; version12 and truncated version did not. All seven assertions passed. Temporary fixture directories were removed. - Oxfmt, git diff --check, commit hooks and push-hook typecheck/lint passed. - Source comparison: desktop follow-up admission uses IL2CPP_METADATA_VERSIONS [24,27,29,31,39]; dedicated IL2CPP guide agrees. - No new tests for the wording-only edit. Prefix fixtures qualify directory triage only, not complete metadata decoding, a real Unity installation or CLI/MCP transport. No native provider or target execution was used. - Online CI required passed for head 2a0f5914: https://github.com/morluto/rea/actions/runs/39976732053 . ## Author readiness Human owner: @ShuaixinHuang - [x] I reviewed the final diff and relevant existing code, understand the change and its edge cases, and take responsibility for the complete PR, including AI-assisted work. - [x] I checked the evidence above, confirmed it supports the current change, and recorded meaningful validation gaps. - [x] I declare this PR ready for initial maintainer review. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-11-01 21:47:42 +00:00