Add an agent skill for driving the proxy - #12
Merged
Merged
Conversation
The proxy leaves the agent two things it cannot work out from the tool schemas: which profile to pass, and what to do when a call is denied. Profile names are absent by design, so an agent that does not list them first either guesses or fails; and an AccessDenied under --sso-role is a decision, which an agent will otherwise try to route around. skills/awsmcproxy states both, along with not falling back to the aws CLI on failure. It sits at skills/ so `npx skills add winebarrel/awsmcproxy` finds it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
winebarrel
enabled auto-merge
August 9, 2026 06:00
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #12 +/- ##
=======================================
Coverage 98.40% 98.40%
=======================================
Files 5 5
Lines 377 377
=======================================
Hits 371 371
Misses 3 3
Partials 3 3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
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
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.
The proxy leaves the agent two things it cannot work out from the tool schemas:
list_profilesrereads~/.aws/configon every call. An agent that does not list them first either guesses or fails.AccessDeniedunder--sso-roleis a decision. Left unsaid, an agent treats it as an obstacle and looks for another profile, another role, or theawsCLI — which carries noprofile, is not signed by the proxy, and drops the narrowing entirely.skills/awsmcproxy/SKILL.mdstates both, plus the SSO-login case that cannot be completed on the user's behalf and the absence of a per-call region argument.It sits at
skills/rather than.claude/skills/so it is installable by any agent:Verified with
npx skills add . -l, which discovers the skill and prints its description.