SEP-2061: Route task-plugin actor lookups through resolve_actor

[Copilot]

System-originated tasks displayed raw UUIDs in Archives, Checksums, MySQL Backups, and Alters, while other task surfaces showed readable labels. This aligns actor rendering across those APIs.

Changes

• Route the three response builders through resolve_actor:

  "created_by": resolve_actor(task.created_by, mapping)

• Resolve both created_by and last_updated_by, preserving raw IDs for unknown actors.
• Update actor-field documentation to include system labels.
• Cover list, detail, create, and update responses for all four affected plugins.
• Add framework-level coverage for system-label precedence and retain provider-less fallback coverage.
• Add a changelog fragment for the API display-value correction.

[Copilot]

🟡 Changes recommended

Refresh generated API artifacts and update the remaining stale actor descriptions.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Updates task-plugin APIs to display readable system actor labels instead of raw UUIDs, while preserving fallback behavior.

Changes:

• Routes actor fields through resolve_actor.
• Adds coverage across affected plugin endpoints.
• Updates documentation and changelog.
• Requires refreshed OpenAPI artifacts and remaining stale descriptions.

File summaries

File	Summary
tests/app/sep/apps/mysql_backups/test_api_routes.py	Tests actor rendering across routes.
tests/app/sep/apps/framework/test_apps.py	Tests label precedence and fallbacks.
tests/app/sep/apps/framework/kit.py	Extends task actor fixtures.
tests/app/sep/apps/checksums/test_route_args.py	Tests Checksums actor rendering.
tests/app/sep/apps/archives/test_api.py	Tests Archives actor rendering.
tests/app/sep/apps/alters/test_api_routes.py	Tests Alters actor rendering.
changelog.d/SEP-2061.fixed.md	Documents the display correction.
app/sep/apps/mysql_backups/deps.py	Resolves MySQL Backup actors.
app/sep/apps/framework/responses.py	Documents actor display values; generated artifacts remain stale.
app/sep/apps/framework/apps.py	Resolves actors in default responses; related descriptions need updating.
app/sep/apps/alters/deps.py	Resolves Alters actors.

Review details

Suppressed comments (1)

app/sep/apps/framework/responses.py:162

• The actor field descriptions now mention system labels, but build_default_task_response still describes the created_by / last_updated_by override as a "username remap" (around line 286). Since the default builder now injects system labels too, that helper contract is stale and should be updated to describe actor display-value resolution rather than only usernames.

    :param created_by: Display name for the user who initiated the task (system
        label or provider username when resolvable, otherwise the stored user id).
    :param last_updated_by: Display name for the user who last modified the task
        record (system label or provider username when resolvable, otherwise the
        stored user id).

• Files reviewed: 11/11 changed files
• Comments generated: 2
• Review effort level: Lite (auto)

Note

Copilot is running an experiment and ran this review at Lite.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.