Skip to content

Remove or migrate legacy ConfigFragment permission stubs #47

Closed

Description

@intel352

During multisite/admin plugin cleanup, stale local branch fix/replace-misc-stubs was found. It changed only internal/config.yaml, replacing /api/v1/admin/workflows/{id}/permissions stub routes with SQLite-backed queries against workflow_permissions.

Current state:

  • internal/config.yaml is still embedded by ConfigFragment().
  • The permissions routes still return [] and 501 permission management not yet implemented.
  • No workflow_permissions table exists in current config, so the old branch cannot be ported as-is.
  • docs/plans/2026-05-26-admin-dashboard-revival-design.md rejected extending the old config-injection dashboard in favor of admin.dashboard contribution contracts and composed auth/authz.
  • Current embedded UI does not call /api/v1/admin/workflows/{id}/permissions.

Needed outcome:
Decide whether internal/config.yaml legacy admin routes are still a supported surface. If yes, add the missing schema and tests before replacing the stub. If no, remove or clearly mark the stale permissions routes so operators do not see a partially implemented admin API.

Activity

  1. intel352 commented on Jun 20, 2026

    @intel352
    ContributorAuthor

    Opened PR #48 to remove the legacy ConfigFragment permission stubs and add a regression test so the stale endpoint is not advertised again: #48. CI is green; the PR is waiting on required human review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions