Skip to content

docs: add native Windows Hermes recovery guide - #812

Merged
dplush merged 1 commit into
mnemosyne-oss:mainfrom
dplush:docs/native-windows-hermes-recovery
Aug 19, 2026
Merged

dplush merged 1 commit into
mnemosyne-oss:mainfrom
dplush:docs/native-windows-hermes-recovery

Conversation

@dplush

@dplush dplush commented Aug 19, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add a focused native-Windows local Hermes installation and recovery section to the canonical Hermes guide
  • document the pip-absent uv pip --python path, native Scripts/python.exe discovery, scoped --no-profile-links installs, and WinError 1314 wrapper retry
  • clarify wrapper timeout/status behavior and managed-venv update limits

Why this is narrow

This is documentation only. It records the current contracts from #805, #808, and #811 without changing installer behavior, plugin loading, packaging, or CI.

Non-goals

  • no Docker/Compose procedure changes
  • no junction workaround, automatic mode switch, config hand-editing, or privileged built-in-tool override
  • no unverified hermes plugins doctor or direct runtime smoke instructions
  • no changes to the legacy LLM installation guide or release notes

Verification

  • python scripts/verify-docs.py
  • source/parser help check for mnemosyne-hermes install (--mode, --python, --import-timeout, --no-profile-links) and status
  • live Hermes help check for plugin enable (without tool override), config set, and memory status
  • git diff --check
  • independent current-head diff review: PASS

Summary

  • Adds native Windows Hermes installation and recovery guidance to the canonical guide.
  • Documents local interpreter discovery, uv pip --python, scoped profile installation, WinError 1314 retries, wrapper timeouts, status checks, and managed-environment limits.
  • Improves Hermes agent integration for Windows without changing installer behavior, plugin loading, packaging, CLI behavior, MCP integration, or CI.

Architecture and guarantees

  • Does not change working, episodic, or BEAM memory tiers.
  • Does not change retrieval, consolidation, veracity, sync, or benchmark methodology.
  • Preserves the local-first design because the documented Hermes setup remains local.
  • Does not change the privacy posture or introduce a remote dependency.
  • Improves long-term maintainability by centralizing Windows procedures in the canonical Hermes guide.
  • This is the right call for documentation coverage. It avoids platform-specific installer changes while making local recovery behavior explicit.

@coderabbitai

coderabbitai Bot commented Aug 19, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 9fa0ab9c-ff98-4706-8055-54f1fadbe660

📥 Commits

Reviewing files that changed from the base of the PR and between d24483e and 70a060d.

📒 Files selected for processing (1)
  • docs/hermes-integration.md

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Walkthrough

Added native Windows Hermes installation and recovery documentation. The guide covers environment selection, dependency installation, symlink and wrapper modes, plugin activation, status verification, and common failure recovery.

Changes

Windows Hermes documentation

Layer / File(s) Summary
Windows installation setup
docs/hermes-integration.md
Documents Hermes home and interpreter selection, dependency installation with pip or uv, and symlink installation.
Symlink and wrapper recovery
docs/hermes-integration.md
Documents WinError 1314 recovery and explicit wrapper validation timeout handling.
Plugin activation and troubleshooting
docs/hermes-integration.md
Adds activation and verification commands, plus recovery guidance for common installation and environment issues.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: 🔵 Low · up to 70a06

The Windows recovery guide does not change runtime behavior, but it may omit local-first, network-access, and offline-fallback guidance that shapes user expectations and privacy setup. The PR is mergeable with explicit owner awareness or a small documentation follow-up.

Possibly related PRs

Suggested reviewers: axdsan, paoloantinori

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main documentation change: adding a native Windows Hermes recovery guide.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@dplush

dplush commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 19, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@dplush
dplush marked this pull request as ready for review August 19, 2026 19:27
@dplush
dplush requested a review from AxDSan as a code owner August 19, 2026 19:27
@dplush
dplush merged commit 7ec6f4c into mnemosyne-oss:main Aug 19, 2026
9 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