Skip to content

feat(mcp): add TypeWhisper catalog entry - #64016

Open
SeoFood wants to merge 1 commit into
NousResearch:mainfrom
SeoFood:seofood/typewhisper-mcp-catalog
Open

feat(mcp): add TypeWhisper catalog entry#64016
SeoFood wants to merge 1 commit into
NousResearch:mainfrom
SeoFood:seofood/typewhisper-mcp-catalog

Conversation

@SeoFood

@SeoFood SeoFood commented Jul 13, 2026

Copy link
Copy Markdown

What does this PR do?

Adds TypeWhisper to the Nous-approved MCP catalog so users can install its local speech-to-text tools with:

hermes mcp install typewhisper

The catalog entry launches the published @typewhisper/mcp@0.1.2 stdio server. TypeWhisper discovers its authenticated local API automatically on macOS and Windows; remote hosts can be configured through the MCP server's env block.

The default tool selection enables status, model listing, file transcription, history search, and read-only dictionary access. Dictionary writes and deletes remain available in the install checklist but are disabled by default.

Type of Change

  • New feature

Changes Made

  • Add optional-mcps/typewhisper/manifest.yaml.
  • Pin the reviewed npm server version for reproducible installs.
  • Document local discovery, remote token configuration, host-side file paths, and tool reconfiguration.

The branch is rebased onto current main, including #68217. Hermes no longer supports pip/PyPI wheel or sdist distribution; catalog data now comes directly from the checkout for source/Docker installs and from the bundled optional-mcps store path for Nix installs.

How to Test

scripts/run_tests.sh tests/hermes_cli/test_mcp_catalog.py tests/hermes_cli/test_dashboard_admin_endpoints.py

The targeted suite passes all 129 tests on the current PR head.

The real install path was also exercised against a running TypeWhisper instance with an isolated HERMES_HOME:

HERMES_HOME="$(mktemp -d)" .venv/bin/hermes mcp install typewhisper

Hermes discovered all 10 tools and persisted the six manifest defaults.

Checklist

  • Read the contributing guide
  • Conventional commit used
  • Searched existing issues and PRs for TypeWhisper
  • Focused change with no unrelated refactors
  • Current catalog and dashboard contract tests cover the new manifest
  • Tested on macOS 27.0
  • Cross-platform behavior considered: the MCP package supports macOS and Windows discovery, plus explicit remote URL/token configuration
  • Documentation/config examples updated in the manifest; no core config keys or architecture changed

@SeoFood
SeoFood marked this pull request as ready for review July 13, 2026 22:50
@alt-glitch alt-glitch added type/feature New feature or request tool/mcp MCP client and OAuth comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have labels Jul 13, 2026

@tonydwb tonydwb 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.

Code Review Summary

Verdict: Comment

30 PRs reviewed (batch: 64044-63999). See aggregate summary at PR 64044.


Reviewed by Hermes Agent

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the focused catalog contribution. The manifest uses the existing MCP-catalog route and its exact @typewhisper/mcp@0.1.2 pin matches the current catalog contract.

Problems

  • Blocking: the new manifest is not declared in wheel data files. pyproject.toml:323-336 requires one [tool.setuptools.data-files] target per optional-mcps/<name>/manifest.yaml, but it currently lists only linear and n8n. Because hermes_cli/mcp_catalog.py:136-140 resolves the catalog from the installed distribution, packaged installs would not contain this entry even though it works from a source checkout.

Suggested changes

  • Add "optional-mcps/typewhisper" = ["optional-mcps/typewhisper/manifest.yaml"] to pyproject.toml and add a regression guard for manifest/data-files parity.

Automated hermes-sweeper review.

Comment thread optional-mcps/typewhisper/manifest.yaml
@teknium1 teknium1 added sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users labels Jul 16, 2026
@SeoFood
SeoFood force-pushed the seofood/typewhisper-mcp-catalog branch from aa24e74 to 5f0a355 Compare July 16, 2026 09:44
@SeoFood
SeoFood requested a review from a team July 16, 2026 09:44
@SeoFood
SeoFood force-pushed the seofood/typewhisper-mcp-catalog branch from 5f0a355 to a0e141e Compare July 21, 2026 18:39
@SeoFood
SeoFood force-pushed the seofood/typewhisper-mcp-catalog branch from a0e141e to 6f7904c Compare July 27, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades tool/mcp MCP client and OAuth type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants