Skip to content

Docs: Overhaul Docker usage guide#163

Merged
rhennigan merged 4 commits into
mainfrom
feature/docker-docs-update
Apr 18, 2026
Merged

Docs: Overhaul Docker usage guide#163
rhennigan merged 4 commits into
mainfrom
feature/docker-docs-update

Conversation

@rhennigan
Copy link
Copy Markdown
Member

Summary

  • Restructures docs/docker.md with a clearer setup flow, updated licensing paths for the non-root container user (/home/wolframengine/.cache/Wolfram/WolframScript), Windows-specific guidance, improved entitlement ID instructions, and refreshed MCP client configuration examples.
  • Adds a Features bullet and an Installation subsection to the root README.md pointing users to docs/docker.md.

Test plan

  • Verify docs/docker.md renders correctly on GitHub
  • Verify README.md renders correctly on GitHub and the link to docs/docker.md resolves
  • Follow the updated node-locked license instructions end-to-end on Linux
  • Follow the updated setup on Windows (Docker Desktop) to confirm the Windows notes are accurate
  • Confirm the updated MCP client configuration JSON snippets work in Claude Desktop / Claude Code

🤖 Generated with Claude Code

rhennigan and others added 2 commits April 18, 2026 09:59
Restructure the Docker documentation with clearer setup flow, updated
licensing paths for the non-root container user, Windows-specific
guidance, improved entitlement ID instructions, and refreshed MCP client
configuration examples.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Add a Features bullet and an Installation subsection pointing users to
docs/docker.md for containerized MCP server setup.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 18, 2026 14:04
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

This PR overhauls the Docker usage documentation for running the Wolfram MCP Server, and adds a short “Docker Image” entry in the root README to point users at the Docker guide.

Changes:

  • Restructures docs/docker.md into a clearer setup flow with updated licensing instructions, Windows notes, and refreshed MCP client config examples.
  • Updates workspace mounting guidance and troubleshooting notes.
  • Adds a Docker image feature bullet and an “Installation → Docker Image” subsection in README.md linking to the Docker guide.

Reviewed changes

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

File Description
docs/docker.md Reworked Docker setup/licensing flow, updated MCP config examples, and refreshed troubleshooting/tag guidance.
README.md Adds a Docker image mention and a short install subsection linking to docs/docker.md.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/docker.md Outdated
Comment thread docs/docker.md
Comment thread docs/docker.md
Comment thread docs/docker.md
Comment thread docs/docker.md
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/docker.md Outdated
Relative paths like ./Licensing break when MCP clients launch docker
with a working directory other than the project directory.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rhennigan rhennigan merged commit f7985d3 into main Apr 18, 2026
5 checks passed
@rhennigan rhennigan deleted the feature/docker-docs-update branch April 18, 2026 14:56
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