mirror of
https://github.com/rennf93/roboco.git
synced 2026-08-03 07:23:24 +02:00
219 lines
7.5 KiB
Python
219 lines
7.5 KiB
Python
"""Periscope (Board Program) engine route coverage — CEO-only, read-only.
|
|||
|
|
Mirrors test_pest_control_routes.py minus the approve/reject routes: a
|
||
|
|
market brief is a report, not a queue item."""
|
||
|
|
|
||
|
|
from __future__ import annotations
|
||
|
|
|
||
|
|
from http import HTTPStatus
|
||
|
|
from typing import TYPE_CHECKING
|
||
|
|
from uuid import UUID, uuid4
|
||
|
|
|
||
|
|
import pytest
|
||
|
|
import pytest_asyncio
|
||
|
|
from fastapi import FastAPI
|
||
|
|
from httpx import ASGITransport, AsyncClient
|
||
|
|
from roboco.api.deps import get_agent_context, get_db
|
||
|
|
from roboco.api.routes.periscope import router as periscope_router
|
||
|
|
from roboco.db.tables import AgentTable, TaskTable
|
||
|
|
from roboco.foundation import identity as _foundation
|
||
|
|
from roboco.foundation.policy.content import markers
|
||
|
|
from roboco.models import AgentRole, AgentStatus, Team
|
||
|
|
from roboco.models.base import Complexity, TaskNature, TaskStatus, TaskType
|
||
|
|
from roboco.models.permissions import AgentContext
|
||
|
|
from roboco.services.task import PERISCOPE_SOURCE
|
||
|
|
from sqlalchemy import update
|
||
|
|
|
||
|
|
CEO_UUID = _foundation.AGENTS["ceo"].uuid
|
||
|
|
|
||
|
|
if TYPE_CHECKING:
|
||
|
|
from collections.abc import AsyncIterator
|
||
|
|
|
||
|
|
from sqlalchemy.ext.asyncio import AsyncSession
|
||
|
|
|
||
|
|
|
||
|
|
async def _seed_agent(session: AsyncSession, role: AgentRole, slug: str) -> AgentTable:
|
||
|
|
agent = AgentTable(
|
||
|
|
id=uuid4(),
|
||
|
|
name=slug,
|
||
|
|
slug=f"{slug}-{uuid4().hex[:6]}",
|
||
|
|
role=role,
|
||
|
|
team=None,
|
||
|
|
status=AgentStatus.ACTIVE,
|
||
|
|
model_config={},
|
||
|
|
system_prompt="x",
|
||
|
|
capabilities=[],
|
||
|
|
permissions={},
|
||
|
|
metrics={},
|
||
|
|
)
|
||
|
|
session.add(agent)
|
||
|
|
await session.flush()
|
||
|
|
return agent
|
||
|
|
|
||
|
|
|
||
|
|
async def _seed_ceo(session: AsyncSession) -> None:
|
||
|
|
if await session.get(AgentTable, CEO_UUID) is not None:
|
||
|
|
return
|
||
|
|
session.add(
|
||
|
|
AgentTable(
|
||
|
|
id=CEO_UUID,
|
||
|
|
name="ceo",
|
||
|
|
slug=f"ceo-{uuid4().hex[:6]}",
|
||
|
|
role=AgentRole.CEO,
|
||
|
|
team=None,
|
||
|
|
status=AgentStatus.ACTIVE,
|
||
|
|
model_config={},
|
||
|
|
system_prompt="x",
|
||
|
|
capabilities=[],
|
||
|
|
permissions={},
|
||
|
|
metrics={},
|
||
|
|
)
|
||
|
|
)
|
||
|
|
await session.flush()
|
||
|
|
|
||
|
|
|
||
|
|
async def _seed_completed_brief(session: AsyncSession) -> TaskTable:
|
||
|
|
system = await _seed_agent(session, AgentRole.SYSTEM, "system")
|
||
|
|
hom = await _seed_agent(session, AgentRole.HEAD_MARKETING, "head-marketing")
|
||
|
|
await _seed_ceo(session)
|
||
|
|
task = TaskTable(
|
||
|
|
id=uuid4(),
|
||
|
|
title="Periscope market-research cycle",
|
||
|
|
description="Research the market and file ONE brief.",
|
||
|
|
acceptance_criteria=["propose_market_brief() called once"],
|
||
|
|
status=TaskStatus.COMPLETED,
|
||
|
|
priority=2,
|
||
|
|
task_type=TaskType.ADMINISTRATIVE,
|
||
|
|
nature=TaskNature.NON_TECHNICAL,
|
||
|
|
estimated_complexity=Complexity.LOW,
|
||
|
|
created_by=system.id,
|
||
|
|
assigned_to=hom.id,
|
||
|
|
team=Team.BOARD,
|
||
|
|
source=PERISCOPE_SOURCE,
|
||
|
|
confirmed_by_human=False,
|
||
|
|
)
|
||
|
|
session.add(task)
|
||
|
|
await session.flush()
|
||
|
|
markers.set_market_brief(
|
||
|
|
task,
|
||
|
|
{
|
||
|
|
"headline": "A rival tool shipped agentic PR review",
|
||
|
|
"findings": [
|
||
|
|
{
|
||
|
|
"id": "finding-0",
|
||
|
|
"claim": "Competitor X launched an autonomous reviewer",
|
||
|
|
"source_url": "https://example.com/x-launch",
|
||
|
|
"relevance": "Overlaps our pr_reviewer role",
|
||
|
|
}
|
||
|
|
],
|
||
|
|
"threats": ["Feature parity gap"],
|
||
|
|
"opportunities": ["Lean into structured findings"],
|
||
|
|
"positioning_note": "Emphasize the findings ledger",
|
||
|
|
"injection_hits": [],
|
||
|
|
},
|
||
|
|
)
|
||
|
|
await session.flush()
|
||
|
|
return task
|
||
|
|
|
||
|
|
|
||
|
|
def _build_app(db_session: AsyncSession, role: AgentRole, agent_id: UUID) -> FastAPI:
|
||
|
|
app = FastAPI()
|
||
|
|
app.include_router(periscope_router, prefix="/api/periscope")
|
||
|
|
|
||
|
|
async def _override_db() -> AsyncIterator[AsyncSession]:
|
||
|
|
yield db_session
|
||
|
|
|
||
|
|
async def _override_agent() -> AgentContext:
|
||
|
|
return AgentContext(agent_id=agent_id, role=role, team=None)
|
||
|
|
|
||
|
|
app.dependency_overrides[get_db] = _override_db
|
||
|
|
app.dependency_overrides[get_agent_context] = _override_agent
|
||
|
|
return app
|
||
|
|
|
||
|
|
|
||
|
|
@pytest_asyncio.fixture
|
||
|
|
async def ceo_client(db_session: AsyncSession) -> AsyncIterator[AsyncClient]:
|
||
|
|
await _seed_ceo(db_session)
|
||
|
|
app = _build_app(db_session, AgentRole.CEO, CEO_UUID)
|
||
|
|
transport = ASGITransport(app=app)
|
||
|
|
async with AsyncClient(transport=transport, base_url="http://test") as client:
|
||
|
|
yield client
|
||
|
|
app.dependency_overrides.clear()
|
||
|
|
# This route only reads — nothing here writes/commits — but a sibling
|
||
|
|
# real-DB suite could still leave a stray board_periscope task behind
|
||
|
|
# from an earlier failed run; keep the cleanup unconditional and cheap.
|
||
|
|
await db_session.execute(
|
||
|
|
update(TaskTable)
|
||
|
|
.where(TaskTable.source == PERISCOPE_SOURCE)
|
||
|
|
.values(status=TaskStatus.CANCELLED)
|
||
|
|
)
|
||
|
|
await db_session.commit()
|
||
|
|
|
||
|
|
|
||
|
|
@pytest.mark.asyncio
|
||
|
|
async def test_list_briefs_empty_when_none_filed(ceo_client: AsyncClient) -> None:
|
||
|
|
resp = await ceo_client.get("/api/periscope/briefs")
|
||
|
|
assert resp.status_code == HTTPStatus.OK
|
||
|
|
assert resp.json() == []
|
||
|
|
|
||
|
|
|
||
|
|
@pytest.mark.asyncio
|
||
|
|
async def test_list_briefs_returns_filed_brief(
|
||
|
|
db_session: AsyncSession, ceo_client: AsyncClient
|
||
|
|
) -> None:
|
||
|
|
task = await _seed_completed_brief(db_session)
|
||
|
|
resp = await ceo_client.get("/api/periscope/briefs")
|
||
|
|
assert resp.status_code == HTTPStatus.OK
|
||
|
|
body = resp.json()
|
||
|
|
assert len(body) == 1
|
||
|
|
row = body[0]
|
||
|
|
assert row["task_id"] == str(task.id)
|
||
|
|
assert row["headline"] == "A rival tool shipped agentic PR review"
|
||
|
|
assert len(row["findings"]) == 1
|
||
|
|
assert row["findings"][0]["source_url"] == "https://example.com/x-launch"
|
||
|
|
assert row["threats"] == ["Feature parity gap"]
|
||
|
|
assert row["opportunities"] == ["Lean into structured findings"]
|
||
|
|
assert row["positioning_note"] == "Emphasize the findings ledger"
|
||
|
|
|
||
|
|
|
||
|
|
@pytest.mark.asyncio
|
||
|
|
async def test_list_briefs_omits_non_terminal_exploration(
|
||
|
|
db_session: AsyncSession, ceo_client: AsyncClient
|
||
|
|
) -> None:
|
||
|
|
"""A still-open (PENDING) exploration cycle carries no brief yet — it
|
||
|
|
must never render as an empty/blank brief row."""
|
||
|
|
system = await _seed_agent(db_session, AgentRole.SYSTEM, "system")
|
||
|
|
hom = await _seed_agent(db_session, AgentRole.HEAD_MARKETING, "head-marketing")
|
||
|
|
db_session.add(
|
||
|
|
TaskTable(
|
||
|
|
id=uuid4(),
|
||
|
|
title="Periscope market-research cycle",
|
||
|
|
description="x",
|
||
|
|
acceptance_criteria=["x"],
|
||
|
|
status=TaskStatus.PENDING,
|
||
|
|
priority=2,
|
||
|
|
task_type=TaskType.ADMINISTRATIVE,
|
||
|
|
nature=TaskNature.NON_TECHNICAL,
|
||
|
|
estimated_complexity=Complexity.LOW,
|
||
|
|
created_by=system.id,
|
||
|
|
assigned_to=hom.id,
|
||
|
|
team=Team.BOARD,
|
||
|
|
source=PERISCOPE_SOURCE,
|
||
|
|
confirmed_by_human=False,
|
||
|
|
)
|
||
|
|
)
|
||
|
|
await db_session.flush()
|
||
|
|
resp = await ceo_client.get("/api/periscope/briefs")
|
||
|
|
assert resp.status_code == HTTPStatus.OK
|
||
|
|
assert resp.json() == []
|
||
|
|
|
||
|
|
|
||
|
|
@pytest.mark.asyncio
|
||
|
|
async def test_non_ceo_is_forbidden(db_session: AsyncSession) -> None:
|
||
|
|
await _seed_completed_brief(db_session)
|
||
|
|
app = _build_app(db_session, AgentRole.DEVELOPER, uuid4())
|
||
|
|
transport = ASGITransport(app=app)
|
||
|
|
async with AsyncClient(transport=transport, base_url="http://test") as client:
|
||
|
|
resp = await client.get("/api/periscope/briefs")
|
||
|
|
assert resp.status_code == HTTPStatus.FORBIDDEN
|
||
|
|
app.dependency_overrides.clear()
|