Skip to content

Task/integrate cli - #1

Merged
nathanBurg merged 3 commits into
mainfrom
task/integrate-cli
Feb 24, 2026
Merged

Task/integrate cli#1
nathanBurg merged 3 commits into
mainfrom
task/integrate-cli

Conversation

@nathanBurg

@nathanBurg nathanBurg commented Feb 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Rewire the plugin to use the published npm package as its MCP server backend, replacing the previous @githits/mcp that was a placeholder. Overhaul all commands, skills, and documentation to match the CLI's actual tools and authentication flow.

Motivation

The plugin previously referenced placeholder server, @githits/mcp. With the githits CLI now published, the plugin can use npx -y githits mcp start to launch the server. This PR also addresses a seamless auth experience: rather than requiring users to manually authenticate before installing the plugin, the SKILL.md now instructs Claude to automatically run githits login when it encounters an auth error.

Changes

MCP server config

  • .mcp.json — Changed command from npx -y @githits/mcp to npx -y githits mcp start

Plugin manifest

  • .claude-plugin/plugin.json — Updated description to match the CLI; set version to 0.1.2

Slash commands

  • commands/search.md — Rewritten with specific search tool parameters (query, language, license_mode) and feedback follow-up
  • commands/login.md — Replaced generic "use MCP server" instruction with concrete npx -y githits login command, including --no-browser, --force, and --port flags
  • commands/logout.md — Replaced generic instruction with npx -y githits logout
  • commands/status.md — New command; runs npx -y githits auth status and reports auth state
  • commands/help.md — Expanded with MCP tools table, auth instructions, and updated command list

Search skill

  • skills/search/SKILL.md — Complete rewrite:
    • Updated trigger conditions (blocked, uncertain, low-confidence language)
    • Added Authentication section: Claude auto-runs githits login on auth errors, uses --no-browser for headless environments, never asks the user to authenticate manually
    • Updated guidelines for search_language-first, single-query focus, and feedback follow-up

Documentation

  • README.md — Rewritten with prerequisites, updated install path (GitHits-com/githits-claude-code-plugin), tools table, license filtering modes, and full authentication guide
  • CHANGELOG.md — Added initial release entries

Related

Companion PR in githits-cli removes the requireAuth() gate from startMcpServer so the MCP server no longer exits before the handshake when auth is missing — auth errors are instead reported per-tool-call, which the SKILL.md authentication block handles automatically.

@nathanBurg
nathanBurg merged commit 7e46bb9 into main Feb 24, 2026
Comment thread skills/search/SKILL.md
## When to use this skill
- You are blocked, uncertain about an API, or need up-to-date OSS usage.
- The user asks to search for examples or explicitly mentions GitHits.
- You are implementing non-trivial code in languages where confidence is lower.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we should add here a condition: when you retry 2 times and fail, use GitHits to solve the issue or something like that

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good call 👍. I'll create a follow up PR

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.

2 participants