fix(pkg): publish as memorylens-bench — bare name taken on PyPI - #36
Merged
Conversation
The 'memorylens' name on PyPI belongs to an unrelated project (uploaded 2026-04), so uploads 403'd. Distribution renamed to memorylens-bench; the import namespace and the 'memorylens' CLI command are unchanged. Install hints, README badges/tables, and docs updated to match. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the project’s PyPI distribution name to memorylens-bench because memorylens is already taken on PyPI, while keeping the import namespace (memorylens) and CLI command (memorylens) unchanged.
Changes:
- Renames the published package in
pyproject.tomltomemorylens-bench. - Updates install instructions and pip extra references across docs and user-facing messages.
- Adjusts the FAISS missing-dependency test to assert the new extra name in the ImportError message.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/test_pipeline.py | Updates expected missing-dependency error text to reference memorylens-bench[faiss]. |
| ROADMAP.md | Updates the shipped install command to pip install memorylens-bench. |
| README.md | Updates PyPI badge, install command, and extras table to memorylens-bench[...]. |
| pyproject.toml | Changes distribution name to memorylens-bench while keeping memorylens as the import package and CLI script. |
| memorylens/utils/llm.py | Updates ImportError install hint for the Groq optional dependency. |
| memorylens/memory/vector_faiss.py | Updates FAISS optional-dependency install hints to memorylens-bench[faiss]. |
| memorylens/api.py | Updates REST API install hint to memorylens-bench[server]. |
| main.py | Updates the entry-point docstring install hint to memorylens-bench. |
| dashboard.py | Updates Streamlit UI help text to reference memorylens-bench[faiss]. |
| CONTRIBUTING.md | Updates project layout install hints and optional-dep references to memorylens-bench[...]. |
| CHANGELOG.md | Documents the rename to memorylens-bench and clarifies import/CLI namespace remains memorylens. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
memorylensname on PyPI is owned by an unrelated project (uploaded April 2026), which is whytwine uploadreturned 403 Forbidden.memorylens-bench(pip install memorylens-bench)import memorylens) and thememorylensCLI command are unchangedtwine checkPASSED, wheel ships only thememorylenspackage🤖 Generated with Claude Code