Skip to content

Add Claude Desktop boot jarvOS prompt - #37

Merged
levineam merged 1 commit into
mainfrom
feat/claude-desktop-boot-jarvos
May 13, 2026
Merged

Add Claude Desktop boot jarvOS prompt#37
levineam merged 1 commit into
mainfrom
feat/claude-desktop-boot-jarvos

Conversation

@levineam

@levineam levineam commented May 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a Boot jarvOS MCP prompt for Claude Desktop manual hydration
  • tune jarvos_hydrate discovery so the natural phrase “boot jarvOS” maps to hydration
  • document the Claude Desktop manual boot flow while keeping startup auto-hydration unsupported

Validation

  • npm test (modules/jarvos-agent-context)
  • node modules/jarvos-runtime-kit/scripts/jarvos-runtime-kit.js check all --json
  • node tests/modules-smoke-test.js
  • npm test
  • bash scripts/smoke-test.sh
  • direct MCP prompts/list and prompts/get smoke
  • direct jarvos_hydrate smoke with maxChars 9000

Notes

  • No related public issue was found for this narrow Claude Desktop MCP prompt UX.
  • This does not add Claude Desktop startup auto-hydration; Desktop remains a manual MCP hydration flow.

Summary by CodeRabbit

  • New Features

    • Added "Boot jarvOS" prompt for compatible MCP clients.
  • Documentation

    • Added manual boot workflow instructions for Claude Desktop.
    • Documented new "Boot jarvOS" prompt capabilities and initialization process.
  • Tests

    • Added test coverage for prompt functionality, capability advertisement, and error handling.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dabcf96d-1b2d-4b43-8e32-df0d33471243

📥 Commits

Reviewing files that changed from the base of the PR and between 54c6a4f and 249c0cc.

📒 Files selected for processing (4)
  • modules/jarvos-agent-context/README.md
  • modules/jarvos-agent-context/scripts/jarvos-mcp.js
  • modules/jarvos-agent-context/test/agent-context.test.js
  • runtimes/claude/README.md

📝 Walkthrough

Walkthrough

This PR adds MCP "prompt" support to the jarvOS MCP server, introducing a boot_jarvos prompt that guides users through bounded hydration via the jarvos_hydrate tool. The change includes prompt registry, JSON-RPC handlers, updated documentation, and comprehensive test coverage.

Changes

MCP Prompt Feature and Documentation

Layer / File(s) Summary
User-facing documentation for boot_jarvos prompt
modules/jarvos-agent-context/README.md, runtimes/claude/README.md
Added Prompts section to agent-context README documenting the boot_jarvos prompt and its expected client behavior. Added "Boot jarvOS in Claude Desktop" manual workflow section to Claude runtime README with instructions for using the prompt, and extended the Available Tools list with prompt documentation.
MCP prompt server implementation
modules/jarvos-agent-context/scripts/jarvos-mcp.js
Updated jarvos_hydrate tool description, introduced BOOT_JARVOS_PROMPT_TEXT constant and PROMPTS registry, added promptResult(name, args) function to generate MCP prompt responses with maxChars argument substitution, extended initialize response to advertise prompts capability, added JSON-RPC handlers for prompts/list and prompts/get, improved error handling to prefer error.code and error.message when present, and exported new prompt-related entities.
MCP prompt capability and error handling tests
modules/jarvos-agent-context/test/agent-context.test.js
Added spawnSync import and test helper mcpRequest to invoke jarvos-mcp.js via spawned Node process. Added tests verifying initialize advertises prompts capability, prompts/list returns boot_jarvos metadata, prompts/get returns formatted boot instructions with argument echoing, and unknown prompts produce JSON-RPC error code -32602.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • levineam/jarvOS#30: Both modify jarvos-mcp.js to extend the JSON-RPC MCP server; this PR adds prompt support alongside existing tool support from #30.
  • levineam/jarvOS#31: This PR extends the jarvos_hydrate tool integration by updating its description and wiring it into the new boot_jarvos prompt handlers, building on tool additions from #31.

Poem

🐰 A prompt to boot and hydrate with grace,
Bounded budgets now have their place,
MCP speaks, the server listens true,
With prompts and tools, the jarvOS flies through! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/claude-desktop-boot-jarvos

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


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 and usage tips.

@levineam
levineam merged commit 8afc5af into main May 13, 2026
6 checks passed
@levineam
levineam deleted the feat/claude-desktop-boot-jarvos branch May 13, 2026 20:10
levineam added a commit that referenced this pull request May 13, 2026
Adds the merged Claude Desktop boot jarvOS prompt work from PR #37 to the v0.1.0 changelog and release notes, including the manual hydration limitation.
@levineam

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown
✅ Actions performed

Review triggered.

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.

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