Skip to content

Remove editable source override from everyrow-mcp#135

Merged
CallumMcMahon merged 1 commit intomainfrom
fix/mcp-editable-source
Feb 11, 2026
Merged

Remove editable source override from everyrow-mcp#135
CallumMcMahon merged 1 commit intomainfrom
fix/mcp-editable-source

Conversation

@CallumMcMahon
Copy link
Member

Summary

  • Remove [tool.uv.sources] with relative editable path from everyrow-mcp/pyproject.toml
  • Regenerate uv.lock to resolve everyrow from PyPI instead of local path

The editable source override (everyrow = { path = "../", editable = true }) breaks Claude Desktop extension installs. When the extension is unpacked to ~/Library/Application Support/Claude/Claude Extensions/, the relative ../ path resolves to a directory with no Python project, causing:

error: Failed to generate package metadata for `everyrow==0.2.1 @ editable+../`
Caused by: .../Claude Extensions does not appear to be a Python project

The regular dependency everyrow>=0.2.1 resolves from PyPI correctly for production. For local dev, [tool.uv.sources] can be added back locally (just don't commit it) or use uv pip install -e ../ after sync.

Related: #120

🤖 Generated with Claude Code

The `[tool.uv.sources]` entry with a relative editable path breaks
Claude Desktop extension installs — the relative `../` path doesn't
resolve correctly when the extension is unpacked to ~/Library/Application
Support/Claude/Claude Extensions/. The regular PyPI dependency
`everyrow>=0.2.1` is sufficient for production.

Fixes #120 (comment)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

@jackwildman jackwildman left a comment

Choose a reason for hiding this comment

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

👍

@CallumMcMahon CallumMcMahon merged commit 21b8452 into main Feb 11, 2026
5 checks passed
@CallumMcMahon CallumMcMahon deleted the fix/mcp-editable-source branch February 11, 2026 16:29
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