Skip to content

fix: remove invalid tool aliases from Ember agent#1355

Merged
codemillmatt merged 1 commit intogithub:stagedfrom
jennyf19:fix/ember-tools-field
Apr 10, 2026
Merged

fix: remove invalid tool aliases from Ember agent#1355
codemillmatt merged 1 commit intogithub:stagedfrom
jennyf19:fix/ember-tools-field

Conversation

@jennyf19
Copy link
Copy Markdown
Contributor

@jennyf19 jennyf19 commented Apr 9, 2026

Problem

The Ember agent (agents/ember.agent.md) specifies tools using outdated VS Code-era names:

yaml tools: ["codebase", "terminalCommand", "fetch_webpage"]

Per the GitHub docs on custom agent tools, unrecognized tool names are silently ignored. The valid aliases are: read, edit, search, execute/shell, web, agent, todo.

Since none of the three specified names are valid, the platform ignores all of them — effectively giving Ember zero tools. This makes Ember unable to read files, run commands, search, or do anything beyond conversation.

Fix

Remove the tools field entirely. When omitted, the agent gets all available tools by default — which is the correct behavior for Ember as a general-purpose AI partner.

Verification

  • Compared with the working droid agent which uses valid aliases: tools: ["read", "search", "edit", "shell"]
  • Confirmed against the docs that omitting tools enables all tools
  • The change is a single-line deletion from YAML frontmatter

Note

The CONTRIBUTING.md example also uses the same outdated tool names (codebase, terminalCommand). That may warrant a separate fix.

The tools field used non-standard names (codebase, terminalCommand,
fetch_webpage) that are not recognized as valid tool aliases. Per the
custom agents configuration docs, unrecognized tool names are silently
ignored, which effectively gave Ember zero tools.

Removing the tools field entirely enables all available tools by default,
which is the correct behavior for a general-purpose AI partner agent.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 9, 2026 22:27
@jennyf19 jennyf19 requested a review from aaronpowell as a code owner April 9, 2026 22:27
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

🔍 Skill Validator Results

2 resource(s) checked | ✅ All checks passed

Full output
No agents found in the specified paths: "/home/runner/work/awesome-copilot/awesome-copilot/agents/ember.agent.md"

Note: Errors were found. These are currently reported as warnings and do not block merge. Please review and address when possible.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes outdated/invalid tool aliases from the Ember agent definition so the agent is not left without usable tools due to unrecognized tool names.

Changes:

  • Deleted the tools frontmatter entry from agents/ember.agent.md to rely on default tool availability.

Copy link
Copy Markdown

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Thanks @jennyf19

@codemillmatt codemillmatt merged commit 429a075 into github:staged Apr 10, 2026
13 of 14 checks passed
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.

4 participants