Skip to content

feat(cli): manage annotation queues and reviewed additions - #314

Closed
liam-langchain wants to merge 1 commit into
codex/cli-diagnosticsfrom
codex/cli-queues
Closed

liam-langchain wants to merge 1 commit into
codex/cli-diagnosticsfrom
codex/cli-queues

Conversation

@liam-langchain

@liam-langchain liam-langchain commented Sep 15, 2026

Copy link
Copy Markdown

Summary

Adds queue create, queue list, queue delete, queue items, and queue add for annotation queues and reviewed run/thread selection plans.

Depends only on shared foundation #309, not feedback. After #309 merges, retarget to main and run CI; do not merge into the foundation branch.

Manual testing

Use an authenticated profile and disposable resources; replace placeholders:

make build
bin/langsmith queue create --name cli-review --dataset DATASET_ID
bin/langsmith queue list --limit 20
bin/langsmith queue add QUEUE_ID --project-id PROJECT_ID --trace-id TRACE_ID --dry-run --output plan.json
bin/langsmith queue add QUEUE_ID --project-id PROJECT_ID --plan plan.json
bin/langsmith queue items QUEUE_ID --limit 20
bin/langsmith queue delete QUEUE_ID --yes

Review the frozen plan before applying. Re-adding an item can reopen its review.

Test Plan

  • Thread request shape, dry-run without writes, and scope/duplicate validation.
  • Returned identity checks, empty write responses, and deletion confirmation.
  • Live thread and mixed-success failure-injection coverage.
  • Main-targeted CI after the foundation merges.

No API or SDK changes. Corridor was blocked by session policy.

@liam-langchain

Copy link
Copy Markdown
Author

Review the combined draft #326 for the current implementation, tests, and ADLC findings. This capability PR is retained as a reference; do not merge it separately alongside #326. No branch history was rewritten.

@liam-langchain

Copy link
Copy Markdown
Author

Superseded by the consolidated implementation in #326, which is now the single review and merge target. Closing this split PR to avoid duplicate reviews or merges. The branch and discussion are retained for reference.

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.

1 participant