[PR #276] [MERGED] test(dotnet): verify managed app manifests #1917

New issue

Closed

opened 2026-10-16 21:12:29 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-16 21:12:29 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/276
Author: @N0zoM1z0
Created: 7/16/2026
Status: ✅ Merged
Merged: 7/16/2026
Merged by: @N0zoM1z0

Base: main ← Head: agent/managed-app-manifest-verifier

📝 Commits (1)

0e97375 test(dotnet): verify managed app manifests

📊 Changes

3 files changed (+382 additions, -13 deletions)

View changed files

📝 AGENTS.md (+1 -1)
📝 docs/managed-code-analysis.md (+28 -9)
📝 scripts/verify-managed-conformance.mjs (+353 -3)

📄 Description

Summary

extend npm run verify:managed with a source-owned managed app manifest verifier self-test
add optional REA_MANAGED_APP_MANIFEST_PATH support for operator-local managed app benchmarks without committing private artifacts
document the manifest schema, relative-path handling, compact output boundary, and large-app MethodDef token paging behavior

Verification

npm run verify:managed
npm run format:check
npm run check
npm run knip
npm run jscpd (exit 0; existing clone report remains)
npm run scan:todos
CC=/usr/bin/gcc-13 CXX=/usr/bin/g++-13 npm run verify:package

Notes

Initial npm run verify:package failed because /usr/bin/g++ points to GCC 8.4 and cannot compile node-pty with -std=gnu++20; rerun passed with the already-installed GCC/G++ 13 toolchain selected per-command.
No proprietary managed app artifacts or decompiler output are committed.

_{🔄 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/276 **Author:** [@N0zoM1z0](https://github.com/N0zoM1z0) **Created:** 7/16/2026 **Status:** ✅ Merged **Merged:** 7/16/2026 **Merged by:** [@N0zoM1z0](https://github.com/N0zoM1z0) **Base:** `main` ← **Head:** `agent/managed-app-manifest-verifier` --- ### 📝 Commits (1) - [`0e97375`](https://github.com/morluto/rea/commit/0e97375e68a5fb1a5a90b99a8b1d4254914eda16) test(dotnet): verify managed app manifests ### 📊 Changes **3 files changed** (+382 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `AGENTS.md` (+1 -1) 📝 `docs/managed-code-analysis.md` (+28 -9) 📝 `scripts/verify-managed-conformance.mjs` (+353 -3) </details> ### 📄 Description ## Summary - extend `npm run verify:managed` with a source-owned managed app manifest verifier self-test - add optional `REA_MANAGED_APP_MANIFEST_PATH` support for operator-local managed app benchmarks without committing private artifacts - document the manifest schema, relative-path handling, compact output boundary, and large-app MethodDef token paging behavior ## Verification - `npm run verify:managed` - `npm run format:check` - `npm run check` - `npm run knip` - `npm run jscpd` (exit 0; existing clone report remains) - `npm run scan:todos` - `CC=/usr/bin/gcc-13 CXX=/usr/bin/g++-13 npm run verify:package` ## Notes - Initial `npm run verify:package` failed because `/usr/bin/g++` points to GCC 8.4 and cannot compile `node-pty` with `-std=gnu++20`; rerun passed with the already-installed GCC/G++ 13 toolchain selected per-command. - No proprietary managed app artifacts or decompiler output are committed. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-16 21:12:29 +00:00