Skip to content

test(task): lock in markdown subagent descriptions#29377

Closed
SpencerJung wants to merge 1 commit into
anomalyco:devfrom
SpencerJung:fix/27831-subagent-description
Closed

test(task): lock in markdown subagent descriptions#29377
SpencerJung wants to merge 1 commit into
anomalyco:devfrom
SpencerJung:fix/27831-subagent-description

Conversation

@SpencerJung
Copy link
Copy Markdown

Summary

  • add a regression test for markdown-defined subagents loaded from .opencode/agents/*.md
  • verify the frontmatter description survives into Agent.list()
  • verify the Task tool description renders that description instead of any generic fallback text

Why

Issue #27831 reports that markdown-defined subagents can fall back to a generic manual-only description in the Task tool even when description is present in frontmatter. On current dev, the production code path already preserves that field, but there was no regression coverage for the markdown-defined agent path.

Closes #27831

Verification

  • lsp_diagnostics on packages/opencode/test/tool/task.test.ts -> no diagnostics
  • �un run test test/tool/task.test.ts (from packages/opencode) -> 16 pass, 0 fail
  • attempted �un install in the worktree, but it failed on ree-sitter-powershell because this Windows environment does not have the required Visual Studio C++ toolchain; no lockfiles were modified

Add a regression case for anomalyco#27831 so markdown-defined subagents keep their frontmatter description when the Task tool builds its subagent list. This protects the already-correct config plumbing on dev from falling back to the generic manual-only text again.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label May 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions
Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Potential Related PR Found:

Why it's related: This PR appears to cover similar territory—testing markdown frontmatter description propagation for agents. PR #29377 focuses on the Task tool description rendering, while #28086 appears to test the Agent.list() path. They may be addressing complementary aspects of the same issue (#27831) or one could be a predecessor/related fix.

I recommend checking if PR #28086 is already merged and whether it addresses the regression test needs or if they cover different test scenarios.

@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window.

Feel free to open a new pull request that follows our guidelines.

@github-actions github-actions Bot removed the needs:compliance This means the issue will auto-close after 2 hours. label May 26, 2026
@github-actions github-actions Bot closed this May 26, 2026
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.

Custom markdown-defined subagents show "should only be called manually" in Task tool despite having description in frontmatter

1 participant