Skip to content

feat: npx sem-skill installer (one-command agent setup)#392

Merged
rs545837 merged 1 commit into
mainfrom
feat/sem-skill-npx
Jun 25, 2026
Merged

feat: npx sem-skill installer (one-command agent setup)#392
rs545837 merged 1 commit into
mainfrom
feat/sem-skill-npx

Conversation

@rs545837

Copy link
Copy Markdown
Member

Rides the agent ecosystem , the cheapest way to turn agent users into sem users.

npx @ataraxy-labs/sem-skill

One command installs the sem skill into ~/.claude/skills/ and registers the sem MCP server, so a coding agent (Claude Code) picks up sem , impact / context / orient / diff over grep for structural questions , with no manual MCP wiring. Idempotent.

Verified the installer end to end (skill installed, MCP detected, idempotent re-run).

Not published to npm yet , that's a separate npm publish once you approve.

…t/hops

Adds @ataraxy-labs/sem-skill: 'npx @ataraxy-labs/sem-skill' installs the sem
skill into ~/.claude/skills/ and registers the sem MCP server, so any coding
agent picks up sem (impact/context/orient/diff over grep) with no manual setup.
This is the 'ride the agent ecosystem' distribution: every agent user can adopt
sem in one command.

The package distributes the canonical skill from #376 (a prepack copies
skills/sem/SKILL.md, single source of truth, credit to @linhlban150612). The
skill is extended with sem orient / sem_entities query and sem context --hops
so it reflects current capabilities.

@inspect-review inspect-review Bot 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.

inspect review

Triage: 35 entities analyzed | 0 critical, 0 high, 0 medium, 35 low
Verdict: standard_review

Findings (2)

  1. [low] install.mjs line 55: Regex pattern /^sem[:\s]/m will fail to match 'sem' if it appears at the start of a line without a colon or space immediately after. The MCP list output likely shows 'sem' as a standalone word or with different formatting, causing the installer to incorrectly attempt re-registration even when sem is already registered.
  2. [low] Race condition: execFileSync('claude', ['mcp', 'list']) output is checked with regex /^sem[:\s]/m which only matches if 'sem' appears at the start of a line. If the MCP server is named 'sem' but appears mid-line in the output (e.g., 'user: sem'), the regex will fail to match and the code will attempt to re-register an already registered server, potentially causing an error.

Reviewed by inspect | Entity-level triage found 0 high-risk changes

@rs545837 rs545837 merged commit 7e5b555 into main Jun 25, 2026
3 checks passed
@rs545837 rs545837 deleted the feat/sem-skill-npx branch June 25, 2026 00:42
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