Skip to content

review: anrokx open PR review (796, 791, 790, 788)#797

Merged
web3dev1337 merged 3 commits intomainfrom
review/anrokx-pr-review
Mar 8, 2026
Merged

review: anrokx open PR review (796, 791, 790, 788)#797
web3dev1337 merged 3 commits intomainfrom
review/anrokx-pr-review

Conversation

@web3dev1337
Copy link
Owner

@web3dev1337 web3dev1337 commented Mar 5, 2026

Summary

Review of AnrokX open PRs (#796, #791, #790, #788) plus launch readiness assessment and README rewrite.

PR Review (REVIEWS/2026-03-06-anrokx-open-prs.md)

PR Title Verdict
#788 feat: add serverOnlyFileWatching user setting Merge as-is
#790 test: fix windows CI recovery cleanup tests Merge as-is
#791 fix(windows): prevent updater plugin startup panic Merge as-is
#796 fix: clean windows onboarding changes against main Address 10 issues (6 medium, 4 low) first

Launch Readiness Assessment

Tauri Desktop App: Fully on main. Installed on Windows at C:\Users\AB\AppData\Local\Claude Orchestrator\. Desktop shortcuts exist. Bundles Node.js + all deps — no dev tools needed for end users.

What's ready:

  • App builds, installs, runs on Windows (self-contained installer)
  • MIT license, v1.0.0, full release tooling (audit scripts, snapshot scripts, history rewrite tools)
  • Feature-complete: workspaces, tabs, worktrees, kanban, tiers, diagnostics, onboarding wizard
  • CI: Windows unit tests + Tauri build in GitHub Actions
  • Auto-updater implemented

What's blocking open-source launch:

  1. 6 open PRs — 3 ready to merge now (feat: add serverOnlyFileWatching user setting #788, test: fix windows CI recovery cleanup tests #790, fix(windows): prevent updater plugin startup panic in packaged app #791)
  2. PR fix: clean windows onboarding changes against main #796 Windows onboarding has medium issues (dead code, memory leak, empty updater pubkey)
  3. UI cleanup + feature flags (9-step plan in PLANS/) — not started, current UI cluttered
  4. Security audit (npm run audit:public-release) not yet run

Verdict: Ready for internal use / testers. Not yet ready for public open-source launch (UI cleanup + audit needed first).

README Rewrite

Rewrote README.md to reflect actual repository state:

  • Removed marketing fluff and personal project references
  • Added prerequisites section for new users
  • Documented Tauri app, diff viewer, commander, tabs, kanban board
  • Updated architecture to reflect actual scale (83 server modules, 24 client, 116 tests)
  • Added proper install instructions for web + Windows desktop
  • Documented cascaded configs, CI, dual-instance setup

Test plan

🤖 Generated with Claude Code

web3dev1337 and others added 3 commits March 6, 2026 09:02
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove marketing fluff ("Revolutionary Feature") and personal project references
- Add accurate prerequisites section for new users
- Document the Tauri desktop app, diff viewer, commander, tabs, kanban board
- Update architecture section to reflect actual scale (83 server modules, 24 client modules, 116 tests)
- Add proper install instructions for both web and Windows desktop
- Document cascaded config system, CI workflows, dual-instance setup
- Replace stale internal PLANS/ references with user-facing content

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Move 31 .md files out of root into organized docs/ subdirectories:
- docs/historical/ — 25 archived plans, implementation notes, old audits
- docs/windows/ — Windows install and build guides
- docs/diff-viewer/ — Diff viewer setup and features
- docs/ — Commander API ref, latest security audit

Root now has only 5 essential files:
  README.md, CLAUDE.md, CODEBASE_DOCUMENTATION.md, CONTRIBUTING.md, SECURITY.md

Update references in CLAUDE.md and scripts/public-release-audit.js
to point to new file locations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@web3dev1337 web3dev1337 merged commit 9e192fd into main Mar 8, 2026
1 of 3 checks passed
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.

1 participant