[PR #858] [MERGED] refactor(cli): split core analysis command registrar #2296

New issue

Closed

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

bsakaguchi commented

2026-10-16 21:16:54 +00:00

Owner

📋 Pull Request Information

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

Base: main ← Head: cursor/split-core-analysis-commands-6404

📝 Commits (1)

c7ea919 refactor(cli): split core analysis command registrar

📊 Changes

3 files changed (+798 additions, -765 deletions)

View changed files

📝 src/cli/coreAnalysisCommands.ts (+5 -765)
➕ src/cli/coreBinaryCommands.ts (+344 -0)
➕ src/cli/coreNativeCommands.ts (+449 -0)

📄 Description

Summary

Extract binary overview/procedure CLI commands into coreBinaryCommands.ts.
Extract native load-image / memory / annotation CLI commands into coreNativeCommands.ts.
Keep registerCoreAnalysisCommands as a thin public registrar.

Verification

npm run check:fast
Focused CLI coverage/catalog/dispatcher tests (19 passed)
Real Hopper/Ghidra verification was not run

_{🔄 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/858 **Author:** [@morluto](https://github.com/morluto) **Created:** 10/7/2026 **Status:** ✅ Merged **Merged:** 10/7/2026 **Merged by:** [@morluto](https://github.com/morluto) **Base:** `main` ← **Head:** `cursor/split-core-analysis-commands-6404` --- ### 📝 Commits (1) - [`c7ea919`](https://github.com/morluto/rea/commit/c7ea919ed4b8cd176626fc8d1a6d4474442fe42f) refactor(cli): split core analysis command registrar ### 📊 Changes **3 files changed** (+798 additions, -765 deletions) <details> <summary>View changed files</summary> 📝 `src/cli/coreAnalysisCommands.ts` (+5 -765) ➕ `src/cli/coreBinaryCommands.ts` (+344 -0) ➕ `src/cli/coreNativeCommands.ts` (+449 -0) </details> ### 📄 Description  ## Summary - Extract binary overview/procedure CLI commands into `coreBinaryCommands.ts`. - Extract native load-image / memory / annotation CLI commands into `coreNativeCommands.ts`. - Keep `registerCoreAnalysisCommands` as a thin public registrar. ## Verification - `npm run check:fast` - Focused CLI coverage/catalog/dispatcher tests (19 passed) - Real Hopper/Ghidra verification was **not** run  <div><a href="https://cursor.com/agents/bc-44a88d98-6d9b-4da2-a464-22f0c2a36404?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a href="https://cursor.com/background-agent?bcId=bc-44a88d98-6d9b-4da2-a464-22f0c2a36404&cursor_ref=pr_footer&cursor_cta=open_in_cursor"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a> </div> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-16 21:16:54 +00:00