bvandeusenandClaude Opus 5.5
0f5ef0be63
CI & Build / Python lint (push) Successful in 3s
CI & Build / Plugin hooks (push) Successful in 12s
CI & Build / TypeScript typecheck (push) Successful in 56s
CI & Build / integration (push) Successful in 1m4s
CI & Build / Python tests (push) Successful in 1m55s
CI & Build / Build & push image (push) Successful in 44s
Step 2 of the Scribe -> Grimoire rename. - src/scribe -> src/grimoire, and every import, patch target, module path, hypercorn entry point and `python -m grimoire.restore` with it; pyproject, uv.lock and the frontend package are named grimoire. - Image git.fabledsword.com/bvandeusen/fabledgrimoire; CI's Postgres service runs as grimoire / grimoire_test. - Compose files default the DB user, database and (prod) volume to grimoire, each overridable from .env, so an install created as Scribe keeps its data by setting POSTGRES_USER, POSTGRES_DB and PGDATA_VOLUME. - Backups are written as grimoire-<ts>.tar with format grimoire-backup-tar. Archives named scribe-* with the scribe-backup-tar manifest still restore and are still counted at the destination (folder and S3), with tests. - Exports, the MCP auth realm, the ASGI scope key and the persistent logger carry the new name. Kept on purpose: the secret-sealing key label stays "scribe.secrets.v1", because every stored secret was derived under it. Plugin identity (plugin name, hooks, env vars, .scribe marker, skills) is step 3. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
26 lines
780 B
Python
26 lines
780 B
Python
"""Tests for MCP per-request user_id context."""
|
|
import pytest
|
|
|
|
from grimoire.mcp._context import current_user_id, _user_id_ctx
|
|
|
|
|
|
def test_current_user_id_raises_when_unset():
|
|
"""No active context — must raise so tools don't silently fall back to a default."""
|
|
_user_id_ctx.set(None)
|
|
with pytest.raises(RuntimeError, match="no MCP user context"):
|
|
current_user_id()
|
|
|
|
|
|
def test_current_user_id_returns_set_value():
|
|
_user_id_ctx.set(42)
|
|
assert current_user_id() == 42
|
|
|
|
|
|
def test_current_user_id_isolation_via_contextvar_token():
|
|
"""Reset via the token returned by set() restores prior state."""
|
|
_user_id_ctx.set(1)
|
|
token = _user_id_ctx.set(2)
|
|
assert current_user_id() == 2
|
|
_user_id_ctx.reset(token)
|
|
assert current_user_id() == 1
|