Skip to content

Prompt the user to choose whether to use coding agent#143

Closed
Cedric / ViaDézo1er (viadezo1er) wants to merge 9 commits into
mainfrom
cedric/bt-choose-using-coding-agent
Closed

Prompt the user to choose whether to use coding agent#143
Cedric / ViaDézo1er (viadezo1er) wants to merge 9 commits into
mainfrom
cedric/bt-choose-using-coding-agent

Conversation

@viadezo1er

@viadezo1er Cedric / ViaDézo1er (viadezo1er) commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

? Use coding agent to instrument the code? (y/n) › yes default is yes

Also merge the code behind bt setup and bt setup -i.

ask the user if they want to launch the agent to instrument the code
@viadezo1er Cedric / ViaDézo1er (viadezo1er) changed the title feat, chore: merge bt setup and bt setup -i paths Prompt the user to choose wether to use coding agent Apr 23, 2026
@viadezo1er Cedric / ViaDézo1er (viadezo1er) changed the title Prompt the user to choose wether to use coding agent Prompt the user to choose whether to use coding agent Apr 23, 2026
`bt setup --json --agent claude --mcp` failed since claude fails when
trying to add an already configured mcp
@viadezo1er Cedric / ViaDézo1er (viadezo1er) force-pushed the cedric/bt-choose-using-coding-agent branch from 783c42a to 65ace0c Compare April 23, 2026 23:56
@github-actions

github-actions Bot commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e2f65d9576f7:

Available artifact names
  • ``artifacts-build-global
  • ``artifacts-build-local-x86_64-pc-windows-msvc
  • ``artifacts-build-local-x86_64-apple-darwin
  • ``artifacts-build-local-x86_64-unknown-linux-musl
  • ``artifacts-build-local-x86_64-unknown-linux-gnu
  • ``artifacts-build-local-aarch64-apple-darwin
  • ``artifacts-build-local-aarch64-unknown-linux-gnu
  • ``artifacts-plan-dist-manifest
  • ``cargo-dist-cache

@viadezo1er Cedric / ViaDézo1er (viadezo1er) force-pushed the cedric/bt-choose-using-coding-agent branch from a07e076 to 37d06bb Compare April 24, 2026 00:10
bt setup --json --no-skills --no-mcp --no-instrument --no-input reported a scope even though none was chosen
@viadezo1er Cedric / ViaDézo1er (viadezo1er) marked this pull request as draft April 24, 2026 01:16
@viadezo1er

Cedric / ViaDézo1er (viadezo1er) commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

Changed my mind about the position of the prompt, not ready to review.

@viadezo1er Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review April 27, 2026 17:58
@AbhiPrasad

Copy link
Copy Markdown
Member

I merged in #146 which will cause merge conflicts for this PR. please rebase (and honestly you might just wanna close this because of the changes in 146 as well)

@viadezo1er

Cedric / ViaDézo1er (viadezo1er) commented Apr 27, 2026

Copy link
Copy Markdown
Contributor Author

I merged in #146 which will cause merge conflicts for this PR. please rebase (and honestly you might just wanna close this because of the changes in 146 as well)

? Use coding agent to instrument the code? (y/n) › yes This doesn't exist in #146 so I'll rebase

Edit: Run instrumentation agent to set up tracing in this repo? is not asked by default in bt setup, will just change that

@viadezo1er

Copy link
Copy Markdown
Contributor Author

mis merged

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