Skip to content

Nest tag and bulk under qn endpoint#10

Merged
johnpmitsch merged 1 commit into
mainfrom
bulk_tags_move
Jun 8, 2026
Merged

Nest tag and bulk under qn endpoint#10
johnpmitsch merged 1 commit into
mainfrom
bulk_tags_move

Conversation

@johnpmitsch

Copy link
Copy Markdown
Collaborator

Summary

  • Move top-level qn tag and qn bulk under qn endpoint to match the existing nesting pattern (endpoint security, endpoint rate-limit).
  • Hard break, no aliases (pre-1.0).
  • Split bulk status into bulk pause / bulk resume verbs, mirroring the single-endpoint commands; drop the BulkStatus enum.
  • Rename tag delete positional idtag_id for consistency with tag rename.

New surface:

qn endpoint tag {list,rename,delete,add,remove}
qn endpoint bulk {pause,resume}
qn endpoint bulk tag {add,remove}

Linear: DX-5619

Test plan

  • cargo build
  • cargo clippy --all-targets -- -D warnings
  • cargo fmt --check
  • cargo test — 173 passed, 0 failed
  • qn tag list and qn bulk status return "unrecognized subcommand"
  • qn endpoint tag --help lists all five verbs; qn endpoint bulk --help lists pause/resume/tag

Top-level `qn tag` and `qn bulk` only ever operated on endpoints. Move
them under `qn endpoint` to match the existing `endpoint security` and
`endpoint rate-limit` nesting pattern.

Hard break (pre-1.0): no aliases for the old paths.

  qn endpoint tag {list,rename,delete,add,remove}
  qn endpoint bulk {pause,resume}
  qn endpoint bulk tag {add,remove}

Bulk status is split into `pause`/`resume` verbs (was a single `status`
verb with `--status active|paused`) to mirror the single-endpoint
commands. The `BulkUpdateEndpointStatusRequest.status` mapping is
hardcoded in the two verb arms; a header comment documents the
divergence from the SDK's free-string field. `tag delete`'s positional
is renamed `id` -> `tag_id` to match `tag rename`.

173 tests pass; clippy + fmt clean.

DX-5619
@johnpmitsch johnpmitsch merged commit 1a36ebb into main Jun 8, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants