Skip to content

feat: REI Reverse Engineer desktop, AI connections and Rootcastle presentation - #8

Merged
rootcastleco merged 4 commits into
mainfrom
feat/rei-reverse-engineer-desktop
Nov 7, 2026
Merged

rootcastleco merged 4 commits into
mainfrom
feat/rei-reverse-engineer-desktop

Conversation

@rootcastleco

@rootcastleco rootcastleco commented Nov 2, 2026 •

Copy link
Copy Markdown
Owner

Result

REI Reverse Engineer is a native Windows research workbench with Rootcastle branding, a licensed Craft Anything desktop foundation, and local MCP routing to the Python workbench and pinned upstream REA. Discovery registered 248 external routes; Python identification and upstream entropy calls returned through the desktop engine on the same synthetic WASM fixture. Servers retain separate evidence stores.

Settings → AI supports the Experiential Labs gateway with separate EXPLABS_API_KEY routing, streaming and model discovery; official installed Codex/Antigravity CLI sign-in; and a clearly labeled Codex Cloud browser link. CLI login uses fixed commands, cancellation and deadlines without reading browser cookies or credential caches.

The English README includes the official rootcastle.com logo, an original SVG header, an actual application screenshot, setup instructions, architecture, verification boundaries and open-source provenance. DISCLAIMER.md addresses authorized use, warranties, liability limitations, AI output, data handling and third-party costs, subject to applicable law. Original licenses and notices are retained.

Verification

  • Head 524ea4a passed both remote workflows before merge: Python quality gates and Windows desktop.
  • Local Windows 11 x64: Rust 1.97.0, Python 3.14.7, Node 24.22.1.
  • Native app built and opened; real Python/upstream MCP smoke calls passed.
  • 352 AI/UI/CLI Rust tests passed; Python: 64 passed, one optional-tool skip.
  • 12 root skills, formatting, Clippy, layers, locks and source secret checks passed.
  • README links/anchors and SVG rendering checked; official logo identity and private-data-free screenshot hash verified.

Limits

Unsigned development preview. Discovery does not verify every analyzer or route. Experiential tests use a loopback fixture; no paid API call or user OAuth sign-in was performed. Antigravity CLI was absent locally and dedicated login support is version-dependent. Pinned upstream npm dependencies report 11 moderate advisories. Detailed evidence is in docs/DESKTOP-VERIFICATION.md and docs/AI-CONNECTIONS.md.

Merged into main as 78fab82.

Copilot AI balanced review requested due to automatic review settings November 2, 2026 19:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@kilo-code-bot

kilo-code-bot Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Code Review Roast 🔥

Verdict: No Issues Found | Recommendation: Merge

Oh wait, this PR is actually clean. I need to sit down. I had my flamethrower warmed up and everything.

📊 Overall: Like finding a unicorn in production — I didn't think clean PRs existed anymore, but here we are.

Files Reviewed (5 files)
  • DISCLAIMER.md
  • README.md
  • docs/images/README.md
  • docs/images/rootcastle-rei-hero.svg
  • docs/images/rootcastle-rei-official.svg
Previous Review Summaries (2 snapshots, latest commit 1dee4c5)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 1dee4c5)

Verdict: No Issues Found | Recommendation: Merge

Oh wait, this PR is actually clean. I need to sit down. I had my flamethrower warmed up and everything.

📊 Overall: Like finding a unicorn in production — I didn't think clean PRs existed anymore, but here we are.

Files Reviewed (11 files)
  • README.md
  • desktop/apps/craft-anything-cli/src/ai_cmd.rs
  • desktop/crates/ai/src/connections.rs
  • desktop/crates/ai/src/detect.rs
  • desktop/crates/ai/src/keys.rs
  • desktop/crates/ai/src/lib.rs
  • desktop/crates/ai/src/tests.rs
  • desktop/crates/ui-egui/src/lib.rs
  • desktop/crates/ui-egui/src/settings_ui.rs
  • desktop/crates/ui-egui/tests/headless_parity.rs
  • docs/AI-CONNECTIONS.md

Previous review (commit e85f9e8)

Verdict: Unable to complete full review | Recommendation: Split PR before merge

This PR touches 641 files / ~206k insertions — a megapr that dwarfs any sensible review. I can't responsibly roast 206k lines in one sitting, and neither can your reviewers.

Overview

Severity Count
🚨 critical 0
⚠️ warning 0
💡 suggestion 1
🤏 nitpick 0
Issue Details (click to expand)
File Line Roast
(all) — PR is too large to review safely in one pass

🏆 Best part: The verification section is unusually thorough and honest about what's not verified.

💀 Worst part: 641 changed files means no human can actually review this; it's a vibes-based merge waiting to happen.

📊 Overall: Like a 200-page contract handed to you at 4:59 PM on a Friday — technically presentable, practically unreadable.

Files Reviewed (0 of 641 files)
  • No files individually reviewed due to PR size

Reviewed by free · Input: 42.3K · Output: 1.2K · Cached: 198.9K

@rootcastleco
rootcastleco merged commit 78fab82 into main Nov 7, 2026
14 checks passed
@rootcastleco rootcastleco changed the title feat: REI Reverse Engineer native desktop with unified MCP engines feat: REI Reverse Engineer desktop, AI connections and Rootcastle presentation Nov 7, 2026
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.

2 participants