Skip to content

Conversation

@dianed-square
Copy link
Contributor

@dianed-square dianed-square commented Jan 16, 2026

Summary

This PR corrects the global agents directory path for the current implementation and notes that it will change in v1.20.0.

Documentation updates:

  • `documentation/docs/guides/context-engineering/using-skills.md
    • change "agents" to "agent" (current) and note pending change to "agents"

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

None

Copilot AI review requested due to automatic review settings January 16, 2026 17:18
@dianed-square dianed-square requested a review from a team as a code owner January 16, 2026 17:18
Copy link
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

This PR attempts to correct the global agents directory path in the skills documentation, claiming that the current implementation uses ~/.config/agent/skills/ (singular) and will change to ~/.config/agents/skills/ (plural) in v1.20.0.

Changes:

  • Updated the skills location documentation to change agents to agent (singular) for the current path with a note about a future change to plural form


1. `~/.claude/skills/` — Global, shared with Claude Desktop
2. `~/.config/agents/skills/` — Global, portable across AI coding agents
2. `~/.config/agent/skills/` — Global, portable across AI coding agents (`~/.config/agents/skills` in goose v1.20.0 and later)
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The documentation change is incorrect. The actual implementation in crates/goose/src/agents/skills_extension.rs line 83 uses ~/.config/agents/skills/ (plural "agents"), not ~/.config/agent/skills/ (singular "agent"). This change would mislead users about the correct directory path.

Suggested change
2. `~/.config/agent/skills/` — Global, portable across AI coding agents (`~/.config/agents/skills` in goose v1.20.0 and later)
2. `~/.config/agents/skills/` — Global, portable across AI coding agents

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

current release version uses "agent"

@github-actions
Copy link
Contributor

github-actions bot commented Jan 16, 2026

PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-01-16 17:24 UTC

@dianed-square dianed-square merged commit 87e52b5 into main Jan 16, 2026
27 checks passed
@dianed-square dianed-square deleted the docs/agent-directory branch January 16, 2026 17:24
zanesq added a commit that referenced this pull request Jan 16, 2026
* 'main' of github.com:block/goose: (28 commits)
  chore(deps): bump aiohttp from 3.13.0 to 3.13.3 in /scripts/provider-error-proxy (#6539)
  chore(deps): bump brotli from 1.1.0 to 1.2.0 in /scripts/provider-error-proxy (#6538)
  docs: temp correction for agent directory (#6544)
  chore: upgrade rmcp (#6516)
  docs: clarify directory in /documentation readme (#6541)
  Release 1.20.0
  Standalone mcp apps (#6458)
  don't add escaping to the command field (#6519)
  Fix popular topics not starting chat when clicked (#6508)
  fix[desktop]: deeplink ui repeat on refresh (#6469)
  fixed test compilation on main branch (#6512)
  fix: correctly parse extension name from tool call for MCP apps (#6482)
  fixed 0 token in openrouter steaming (#6493)
  feat(goose-acp): enable parallel sessions with isolated agent state (#6392)
  copilot instruction to flag prelease docs (#6504)
  docs: acp mcp support (#6491)
  feat: add flatpak support for linux (#6387)
  fix(code_execution): serialize record_result output as JSON (#6495)
  perf(google): avoid accumulating thoughtSignatures across conversation history (#6462)
  fix(openai): make tool_call arguments optional and fix silent stream termination (#6309)
  ...
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