Skip to content

chore: split heavy ML stack into optional mmory extras#42

Merged
rosspeili merged 3 commits into
ARPAHLS:mainfrom
AaineeSinha:optimize-deps
Jun 8, 2026
Merged

chore: split heavy ML stack into optional mmory extras#42
rosspeili merged 3 commits into
ARPAHLS:mainfrom
AaineeSinha:optimize-deps

Conversation

@AaineeSinha

@AaineeSinha AaineeSinha commented Jun 5, 2026

Copy link
Copy Markdown

Closes #35

This PR separates the heavy machine learning packages (chromadb, sentence-transformers, torch) out of the main requirements.txt file and moves them into a new requirements-memory.txt file.

This ensures that the core CLI installation remains lightweight and fast for new users and CI environments while preserving an explicit installation path for future vector memory features.

@rosspeili

Copy link
Copy Markdown
Contributor

Thanks for this as well @AaineeSinha.

Before merge, could you fix a couple small README things in the same PR?

  • Close the optional pip install -r requirements-memory.txt code block before ### 2. Configure defaults (looks like the fence is missing right now).
  • Tidy the project structure line for requirements-memory.txt (duplicate comment on one line).
  • Optional: link to feat: long-term agent memory across sessions (sidecar storage) #13 where you mention long-term / vector memory.

CI should be fine as-is since it only installs requirements.txt. Thanks again!

@AaineeSinha

Copy link
Copy Markdown
Author

Hi @rosspeili, I have updated the README.md to resolve the layout and reference issues. The code block closing fence has been fixed, the trailing text typo in the project structure tree has been removed, and a reference to issue #13 has been added to the long-term memory section. Ready for another look when you have a moment!

@rosspeili

Copy link
Copy Markdown
Contributor

Thanks for the update @AaineeSinha, project structure and the #13 mention look good.

The optional install code block still needs a closing ``` after pip install -r requirements-memory.txt (before ### 2. Configure defaults). Right now that heading is inside the bash fence on GitHub.

Once that’s in, good to merge from my side. Thanks again!

@AaineeSinha

Copy link
Copy Markdown
Author

Hi @rosspeili, I've closed the code block fence right after the optional memory install step so the ### 2. Configure defaults (optional) heading renders correctly now.

@rosspeili

Copy link
Copy Markdown
Contributor

Confirmed, the code fence is fixed and Quick Start renders properly now. Core vs memory split looks good, also, merging after CI is green. Thanks again!

@rosspeili rosspeili merged commit e1664a7 into ARPAHLS:main Jun 8, 2026
1 check passed
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.

feat: slim core install — move unused ML stack to optional extras

2 participants