Skip to content

fix(setup-instructions): check aws installation before running installer and remove agent rules/skills folder location table - #280

Merged
arnewouters merged 2 commits into
mainfrom
fix/setup-instruction
Sep 1, 2026
Merged

fix(setup-instructions): check aws installation before running installer and remove agent rules/skills folder location table#280
arnewouters merged 2 commits into
mainfrom
fix/setup-instruction

Conversation

@MingdaMa

@MingdaMa MingdaMa commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Description

Two bugs in agent toolkit setup file

Bug 1

Agent would try to install AWS CLI even when user has it installed prior to setting up agent toolkit.

Bug 2

Rules file and skills file folder location lookup tables only contain an incomplete list of agents, which might result in agents not placing rules file and skills file into agents that are not listed in the tables.

Fix

  • Adding AWS CLI detection by telling agent to run aws --version before running AWS CLI installer
  • Removing rules file and skills file location lookup tables and adjust wording around where to place those files

Type of Change

  • New plugin
  • New skill
  • Bug fix
  • Documentation update
  • CI/CD change
  • Other

Checklist

  • I have read the CONTRIBUTING guide
  • My changes pass python3 tools/validate.py
  • I have updated relevant documentation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

…ler and remove agent rules/skills folder location table
@MingdaMa
MingdaMa requested a review from a team as a code owner August 28, 2026 22:28
@arnewouters
arnewouters added this pull request to the merge queue Sep 1, 2026
Merged via the queue into main with commit 85cb699 Sep 1, 2026
9 checks passed
@arnewouters
arnewouters deleted the fix/setup-instruction branch September 1, 2026 05:16
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.

3 participants