feat: bdt-status Claude mod, bdt pr info, bdt issue take (#68) - #69
Merged
Merged
Conversation
Closed
aersam
changed the title
aersam
marked this pull request as ready for review
October 3, 2026 07:54
This was referenced Oct 3, 2026
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
- bdt pr info [--json]: PR link/state/aggregate build state/closed issues - mods/bdt-status: shows that above the prompt in Claude Code (mods, v2.1.287+) - bdt issue take [NUMBER]: 'Taken by <user>' comment with the agent session link - marketplace manifest so the mod installs via 'claude plugin install' Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
- pr info: closed issues from GitHub's closingIssuesReferences; all-skipped checks are 'none'; cancelled checks don't fail the build (same as pr status); PR title stripped of control chars - issue take: sanitised, timeout-bounded user name; says which issue it took - get_pr takes the field list (no near-duplicate get_pr_info) - mod: keeps the last PR on a failed refresh, one timer, validates bdt's output - docs: SKILL triggers, accurate session-link wording, ADO limits Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Latest build per pipeline: failed result -> failing, unfinished -> pending (waiting when a stage needs approval), none/passing otherwise; unknown if ADO can't be asked. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
One DEFAULT_TARGET_BRANCH for pr create/status/info/retry/publish/update/comment/watch-deploy, issue take and commit; pass --target main to keep the old behaviour. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Posts 'Taken by <user>' naming the session it generates (--session-id) so the comment links the right session even when bdt issue do itself runs inside an agent; prompt tells the agent not to take it again. comment helpers get agent_note=False for this. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… is a claim Reads the issue's last comment (GitHub GraphQL last:1, ADO comments order=desc) and skips when it starts with 'Taken by'; applies to issue do too, which still starts the agent. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
pr info --json stays here as its data source. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Issue #68 ("Use Claude Mods"):
bdt pr info [--json]– PR link, state, aggregate build state and the issue(s) it closes in one call; build state is judged likebdt pr statuson GitHub and Azure DevOps. This is the data source for the Claude Code mod, which now lives in Add bdt-status Claude Code mod (PR/issue link + build state above the prompt) skills#61 (moved out of this PR).bdt issue take [NUMBER]– commentsTaken by <user>plus the agent session. No parameters needed: without a number it uses the issue the branch's PR closes. Does nothing if the issue's newest comment already is a claim.bdt issue donow takes the issue first, naming the session it starts (--session-id), and tells the agent not to take it again.Breaking
--target/--target-branchoption is nowdev(wasmain):pr create/status/info/retry/publish/update/comment/watch-deploy,issue take,commit. Repos whose PRs go intomainmust pass--target main. This repo has nodevbranch, so its ownbdt pr createneeds--target main.Notes / limits
--target-branch. Documented in the README.CLAUDE_CODE_BRIDGE_SESSION_IDis set (existing behaviour ofdetect_agent_session); otherwise the bare id.bdt issue take), docs: document bdt find-repo #61 (thebdt-statusmod)./bms-code-review --full(correctness, security, docs/duplication); findings fixed.Fixes #68
🤖 Generated with Claude Code
Claude Session: 668a4561-5525-55c6-a144-9b70cfb1fb34