Add optimized four-in-one prompt strategy - #3
Closed
bstee615 wants to merge 1 commit into
Closed
Conversation
Add a standalone PatchBased02OptimizedFourInOne strategy that selects the established aggressionHighLow eagerness prompt, including default-model retry handling. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 13bd697b-ebe3-4c55-a34e-58dcf027cad6
bstee615
pushed a commit
that referenced
this pull request
Aug 28, 2026
* Let the setup banner reload an agent's configuration A user who finishes setup outside the app — `claude login` in a terminal, an exported key — leaves no signal the app can see, so the banner kept asking them to sign in to something they had already signed in to. Give them a way to say "look again", and rename the docs link to "learn more" now that it is one of two links rather than the only one. The re-look is the tail of a download promoted to its own gesture: restart chat discovery, then refresh models. `AgentSdkSetupChannel` grows a second request key rather than per-agent code, so agent #3 still needs no edit here — one consumed nonce per key, cleared as it is claimed, so a repeat press still lands. The reload clause folds into each of the four `noAccount` sentences rather than trailing them: it is unconditional, so the table stays at four branches and no localized string is assembled from fragments. * Rank the no-account copy as the buttons rank it, and harden its links Read the sentence in the order the routes are weighted: GitHub sign-in leads, as the primary button; the provider sign-in follows; reload and docs trail, being the copy's only links rather than buttons. Reload and docs become their own sentences — kept as trailing clauses they would have fallen under the "if you already set up Claude elsewhere" conditional, which does not scope docs. Addresses review feedback: build both `command:` hrefs through `createCommandUri` instead of by hand (`encodeURIComponent` leaves `)` alone, so an agent id containing one closed the markdown link destination early), and escape the host-supplied display name and sign-in provider before interpolating them into markdown this banner trusts for two commands. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * Rewrite the no-account copy, and point Claude at its integrations docs The four sentences now put every sign-in route and the reload into one "or" list, ranked as the buttons rank them, and give the docs their own trailing sentence. Claude's docs URL moves to the third-party integrations page, which is what "other ways to set up Claude" actually means: Console, Bedrock, Vertex, Foundry, Teams and Enterprise. "Set up" is the verb, two words, as the rest of the string already had it. Both agents' URL constants still described the workbench as labelling a button. It has been a link since docs stopped being an action. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
bstee615
force-pushed
the
bstee615-add-four-in-one-strategy
branch
from
August 28, 2026 23:15
af91c1b to
4cefbbd
Compare
bstee615
changed the base branch from
bstee615-include-patchbased02optimized
to
main
August 28, 2026 23:18
Owner
Author
|
Replaced by upstream draft PR microsoft#333302. Closing this fork-only draft. |
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.
Summary
PatchBased02OptimizedFourInOneprompting strategyeagernessPrompt: 'aggressionHighLow'option without requiring Proxy/modelsschema supportBehavior
The strategy uses optimized PatchBased02 prompt/response handling. For high and low eagerness, it prepends the existing
<|aggression|>...<|/aggression|>tag before the unchanged PatchBased02 postscript. Medium leaves the existing postscript unchanged.Stack
Alternative to the Proxy-backed integration option. This PR depends only on base draft PR #1 and is based on
bstee615-include-patchbased02optimized.Validation
npm run typechecknpm run precommithygiene