ci: install coherence-membrane spine so tests can import studio_engine - #5
Merged
Merged
Conversation
studio_engine.certify composes the coherence-membrane spine (an internal peer flagship, not third-party and not on PyPI). engine.py imports certify eagerly, so importing studio_engine at all requires the spine. CI only ran 'pip install -e .', so every test that imports studio_engine died with ModuleNotFoundError: No module named 'coherence_membrane'. Install the spine from source before running the suite. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
HarperZ9
added a commit
that referenced
this pull request
Jul 27, 2026
#5) studio_engine.certify composes the coherence-membrane spine (an internal peer flagship, not third-party and not on PyPI). engine.py imports certify eagerly, so importing studio_engine at all requires the spine. CI only ran 'pip install -e .', so every test that imports studio_engine died with ModuleNotFoundError: No module named 'coherence_membrane'. Install the spine from source before running the suite. Co-authored-by: Zain Dana Harper <zain@aurora-framework.dev> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Problem
CI on
mainis red.python -m unittest discover -s testsfails at import time:across every test that imports
studio_engine(test_engine, test_compose, test_criteria, test_native_render, test_certify, ...).Root cause
studio_engine/__init__.py->.engine-> (line 22)from .certify import world_certificate->certify.pyimportscoherence_membrane. So anyimport studio_engineneeds the spine. CI only ranpip install -e ., which installs studio-engine but not coherence-membrane. coherence-membrane is an internal peer flagship ("the spine"), not a third-party dependency, and is not published to PyPI, so it was never installed.Fix
Add a CI step to install coherence-membrane from source (
pip install git+https://github.com/HarperZ9/coherence-membrane.git) before running the suite. This is intended internal composition; "zero-dependency" means no external deps, which still holds.Verified locally (with the spine present)
python -m unittest discover -s tests-> 169 OKpython test_forward_delivery_contract.py-> 4 OKnode --test showcase/tests/*.test.mjs-> 18 pass, 0 fail