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>
94 lines
4.2 KiB
Python
94 lines
4.2 KiB
Python
from unittest.mock import AsyncMock, patch
|
|
|
|
import pytest
|
|
from tests.helpers import fake_task
|
|
|
|
|
|
pytestmark = pytest.mark.usefixtures("_bind_user")
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
def _no_plan_gate():
|
|
"""The plan gate reads the database; the tests that are about it re-patch it."""
|
|
with patch("grimoire.mcp.tools.tasks.dedup_svc.plan_gate", AsyncMock(return_value=None)):
|
|
yield
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_start_planning_tool_delegates_to_service():
|
|
payload = {"milestone": {"id": 5}, "applicable_rules": [], "project_rules": [],
|
|
"applicable_rules_truncated": False, "project_goal": "", "open_task_count": 0}
|
|
with patch("grimoire.mcp.tools.tasks.planning_svc.start_planning",
|
|
AsyncMock(return_value=payload)) as mock:
|
|
from grimoire.mcp.tools.tasks import start_planning
|
|
out = await start_planning(project_id=3, title="Plan it")
|
|
assert out["milestone"]["id"] == 5
|
|
# No body and no steps reach the service as None, so it seeds the template
|
|
# and takes the single-milestone path.
|
|
assert mock.call_args.kwargs == {
|
|
"user_id": 7, "project_id": 3, "title": "Plan it", "body": None, "steps": None,
|
|
}
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_start_planning_returns_the_plan_that_already_covers_it():
|
|
"""A session that cannot see the existing plan makes a second one. It is
|
|
handed the first instead, and nothing is created."""
|
|
match = {"duplicate": True, "existing_id": 12}
|
|
svc = AsyncMock()
|
|
with patch("grimoire.mcp.tools.tasks.dedup_svc.plan_gate",
|
|
AsyncMock(return_value=match)) as gate, \
|
|
patch("grimoire.mcp.tools.tasks.dedup_svc.find_duplicate_note", AsyncMock()) as steps, \
|
|
patch("grimoire.mcp.tools.tasks.planning_svc.start_planning", svc):
|
|
from grimoire.mcp.tools.tasks import start_planning
|
|
out = await start_planning(project_id=3, title="Metadata", body="design",
|
|
steps=[{"title": "Resolve editions", "body": "via providers"}])
|
|
assert out is match
|
|
svc.assert_not_awaited()
|
|
steps.assert_not_awaited()
|
|
# The candidate is judged by its steps as well as its design.
|
|
assert gate.call_args.args[:3] == (7, 3, "Metadata")
|
|
assert "design" in gate.call_args.args[3]
|
|
assert "Resolve editions\nvia providers" in gate.call_args.args[3]
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_force_creates_the_plan_without_asking_the_gate():
|
|
gate = AsyncMock(return_value={"duplicate": True})
|
|
with patch("grimoire.mcp.tools.tasks.dedup_svc.plan_gate", gate), \
|
|
patch("grimoire.mcp.tools.tasks.planning_svc.start_planning",
|
|
AsyncMock(return_value={"milestone": {"id": 5}})) as svc:
|
|
from grimoire.mcp.tools.tasks import start_planning
|
|
out = await start_planning(project_id=3, title="Metadata", force=True)
|
|
assert out["milestone"]["id"] == 5
|
|
svc.assert_awaited_once()
|
|
gate.assert_not_awaited()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_task_augments_plan_with_rules():
|
|
applicable = {"rules": [{"id": 1, "title": "r"}], "truncated": False,
|
|
"project_rules": [{"id": 3, "title": "own"}]}
|
|
with patch("grimoire.mcp.tools.tasks.notes_svc.get_note_for_user",
|
|
AsyncMock(return_value=(fake_task(task_kind="plan", id=9, project_id=3), "owner"))), \
|
|
patch("grimoire.mcp.tools.tasks.rulebooks_svc.get_applicable_rules",
|
|
AsyncMock(return_value=applicable)):
|
|
from grimoire.mcp.tools.tasks import get_task
|
|
out = await get_task(task_id=9)
|
|
assert out["applicable_rules"] == [{"id": 1, "title": "r"}]
|
|
assert out["project_rules"] == [{"id": 3, "title": "own"}]
|
|
assert "subscribed_rulebooks" not in out
|
|
assert out["applicable_rules_truncated"] is False
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_task_work_kind_has_no_rules():
|
|
with patch("grimoire.mcp.tools.tasks.notes_svc.get_note_for_user",
|
|
AsyncMock(return_value=(fake_task(task_kind="work", id=9, project_id=3), "owner"))), \
|
|
patch("grimoire.mcp.tools.tasks.rulebooks_svc.get_applicable_rules",
|
|
AsyncMock()) as mock_rules:
|
|
from grimoire.mcp.tools.tasks import get_task
|
|
out = await get_task(task_id=9)
|
|
assert "applicable_rules" not in out
|
|
assert not mock_rules.called
|