merge(main): squash integration of 3 branches#22
Conversation
Includes: - feat/soulfield-user-services - feat/e2e-demo - fix/newelle-launcher Summary: - Add systemd user services + docs - Add E2E demo script + doc (research→spec→dry→apply) - Add Newelle launcher + troubleshooting; fix Anthropic headers
There was a problem hiding this comment.
Codex Review: Here are some suggestions.
https://github.com/mrhpython/Soulfield/blob/af88f0f36ebd11c0d6f040228e5fd478fbb19629/ .venv/pyvenv.cfg#L1-L5
[P1] Remove committed virtual environment
The commit adds an entire Python virtualenv under .venv/ including platform‑specific binaries and compiled site packages with hard‑coded paths (for example the venv was created at /home/michael/soulfield/.venv). Checking this directory into git balloons the repo by hundreds of thousands of lines and produces artefacts that won’t work on other systems. Dependency environments should be recreated from requirements.txt instead of being versioned. Consider deleting .venv from the tree and adding it to .gitignore.
Reply with @codex fix comments to fix any unresolved comments.
About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".
…dar with all-day + window), provider-agnostic local embeddings (MiniLM) with OpenAI fallback, Pinecone adapter refactor, CLI context tools, health endpoint, retention + pruning + backfill, and docs updates
|
Closing - this PR is based on a very old repository snapshot and contains 100+ files. The changes are no longer relevant to current main. |
Includes:
Summary:
Title:
Summary
Checklist
Validation
1.
2.
Links