Skip to content

Extract one shared command-queue helper for the *Ops runQueued/runCommitted copies #90

Open

Description

@barelyworkingcode

Problem

runCommitted now exists as four identical copies (ProjectOps, LoginOps, CredentialOps, ServiceOps), and six more *Ops types have a runQueued that already calls DoCommitted. The repo rule is to extract shared logic at three uses.

Fix

Add one package-level helper in cmd/relay (for example runOnQueue(q, ctx, fn, committed bool), or two small funcs) and have every *Ops call it. config_queue_structural_test.go matches the helper names in source text, so update it in the same change and check the scan still finds every call.

Found in the review of #87.

Activity

  1. barelyworkingcode commented on Sep 26, 2026

    @barelyworkingcode
    OwnerAuthor

    Parked by the owner on 2026-09-26: the stability pass is refocused on reproduced bugs only. It stays open for a later pass.

  2. barelyworkingcode commented on Sep 26, 2026

    @barelyworkingcode
    OwnerAuthor

    Relabelled as an enhancement: this is a refactor, not a bug.

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

    enhancementNew feature or requestparkedOwner parked: not in the current passpriority: lowWhen there's room

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions