Skip to content

Conversation

@dianed-square
Copy link
Contributor

@dianed-square dianed-square commented Feb 5, 2026

Summary

Documents the --container flag for running extensions in Docker containers.

Documentation updates:

  • documentation/docs/getting-started/using-extensions.md:
    • Add "Extensions in Containers" section for discoverability
  • documentation/docs/guides/goose-cli-commands.md:
    • Add --container flag to session and run commands
  • documentation/docs/mcp/container-use-mcp.md:
    • Add link to "Isolated Development Environments" topic
  • documentation/docs/tutorials/goose-in-docker.md:
    • Expand scope to "goose and Docker" and remove "Building" from title
    • Add crosslink for alt scenario
    • Add "Running Extensions in a Container" section
  • documentation/docs/tutorials/isolated-development-environments.md:
    • Add crosslink for alt scenario
    • Replace configuration instructions with link to container-use-mcp tutorial

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

Manual testing


@dianed-square dianed-square requested a review from a team as a code owner February 5, 2026 01:42
Copilot AI review requested due to automatic review settings February 5, 2026 01:42
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-02-05 17:21 UTC

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 documents the --container flag that allows running goose extensions inside Docker containers, expanding the documentation to cover devcontainer workflows.

Changes:

  • Adds comprehensive documentation for the --container flag across CLI command references
  • Creates new section on running extensions in Docker containers with usage examples and requirements
  • Refactors documentation structure to separate container-use MCP extension setup from general Docker usage
  • Adds cross-references between related Docker and container documentation topics

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
documentation/docs/tutorials/isolated-development-environments.md Removes detailed installation instructions (delegated to container-use-mcp tutorial) and adds cross-reference for devcontainer users
documentation/docs/tutorials/goose-in-docker.md Expands scope from "Building goose in Docker" to cover both running goose in Docker and running extensions in containers; adds new section with --container flag documentation
documentation/docs/mcp/container-use-mcp.md Adds cross-reference link to isolated development environments tutorial
documentation/docs/guides/goose-cli-commands.md Documents --container flag for both session and run commands
documentation/docs/getting-started/using-extensions.md Adds "Extensions in Containers" section with cross-reference to detailed documentation

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 5, 2026 01:51
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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 5, 2026 01:56
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

@dianed-square dianed-square added this pull request to the merge queue Feb 5, 2026
Merged via the queue into main with commit 57033b7 Feb 5, 2026
20 checks passed
@dianed-square dianed-square deleted the docs/container-flag branch February 5, 2026 17:17
kuccello pushed a commit to kuccello/goose that referenced this pull request Feb 7, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

2 participants