25 lines
1.2 KiB
Markdown
25 lines
1.2 KiB
Markdown
Status: ready-for-agent
|
|
|
|
# 19 — DOC: Cryptography dependency + test environment note
|
|
|
|
## What to build
|
|
|
|
Document and verify test/dev environment setup for wallet crypto paths. `packages/node/meshnet_node/wallet.py` uses `cryptography`; failures occur when `.venv` lacks deps. `cryptography>=41` is already declared in `packages/node/pyproject.toml`, so this issue should focus on documenting the editable-install path and only add root/dev extras if tests still import the node wallet without installing the node package.
|
|
|
|
**Code refs:**
|
|
|
|
- `packages/node/pyproject.toml` — `cryptography>=41` (verify declared)
|
|
- `packages/node/meshnet_node/wallet.py`
|
|
- Handoff: tests fail without `cryptography`, `openai`, `langchain` in `.venv`
|
|
|
|
## Acceptance criteria
|
|
|
|
- [ ] Confirm `cryptography>=41` remains in node package deps; add to root/dev extras only if tests import wallet without node install
|
|
- [ ] Add short **Test environment** section to `docs/dev/test-env.md` (or `CONTRIBUTING.md` if created): use `.venv/Scripts/python.exe`, `pip install -e packages/node ...`, optional dep skips
|
|
- [ ] Note which tests require optional deps (`--ignore=test_openai_gateway,...`)
|
|
- [ ] No unrelated production code changes
|
|
|
|
## Blocked by
|
|
|
|
None
|