Skip to content

Add Jarvis-facing Ollama wrapper CLI with King Wen routing and task-a…#627

Closed
Kbro1989 wants to merge 5 commits into
open-jarvis:mainfrom
Kbro1989:main
Closed

Add Jarvis-facing Ollama wrapper CLI with King Wen routing and task-a…#627
Kbro1989 wants to merge 5 commits into
open-jarvis:mainfrom
Kbro1989:main

Conversation

@Kbro1989

@Kbro1989 Kbro1989 commented Jul 4, 2026

Copy link
Copy Markdown

…ware model selection

What does this PR do?

How was this tested?

Checklist

  • Tests pass (uv run pytest tests/ -v)
  • Linter passes (uv run ruff check src/ tests/)
  • Formatter passes (uv run ruff format --check src/ tests/)
  • New/changed public API has docstrings
  • Follows registry pattern (if adding new component)
  • Documentation updated (if applicable)

@Kbro1989 Kbro1989 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

👍

Kbro1989 added 3 commits July 5, 2026 10:35
- Implement King Wen emotion provider for deterministic emotional state and voice selection.
- Introduce Hermes runtime engine to interface with local Hermes Agent for inference.
- Create Ollama model usage tracking for performance metrics.
- Develop actionable manifest for normalizing tool commands and descriptions.
- Add tests for Ollama launch command and Hermes runtime engine functionality.
- Implemented `actionable_manifest.py` to normalize Jarvis tools and user-facing action words into a serializable manifest.
- Added functions for building, writing, loading, and resolving actions from the manifest.
- Created unit tests for `HermesRuntimeEngine` in `test_hermes_runtime.py` to ensure proper functionality and environment handling.
- Introduced temporary scripts for testing King Wen emotion provider integration and agent behavior in `tmp_kingwen_debug_console.py`, `tmp_kingwen_smoke_agent_tails.py`, `tmp_kingwen_tail_probe.py`.
- Verified agent-layer interactions and state management through smoke tests and focused probes.
@ElliotSlusky

Copy link
Copy Markdown
Collaborator

@Kbro1989 thanks for the effort here, but I'm going to close this as it's the wrong direction for the project.

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.

3 participants