Skip to content

Setup: the servers ship in eic-shell — drop the clone/PATH/bootstrap steps - #1

Merged
aprozo merged 9 commits into
mainfrom
feat/simplified-setup
Sep 30, 2026
Merged

aprozo merged 9 commits into
mainfrom
feat/simplified-setup

Conversation

@aprozo

@aprozo aprozo commented Aug 31, 2026 •

Copy link
Copy Markdown
Collaborator

Setup: ./eic-shell → eic-mcp up → eic-mcp config opencode; assistant inside eic-shell or on your machine; macOS: DOCKER_OPTIONS="$(./eic-shell -- eic-mcp docker-args)" (eic/eic-shell#39). Draft until eic_xl includes the servers (eic/containers#357).

@github-actions

github-actions Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Thank you!

Thank you for your pull request 😃

🤖 This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

  • 🎯 correct output
  • 🖼️ correct figures
  • ❓ new warnings
  • ‼️ new errors

Rendered Changes

🔍 Inspect the changes: https://github.com/eic/tutorial-mcp/compare/md-outputs..md-outputs-PR-1

The following changes were observed in the rendered markdown documents:

 01-why-genai-for-physics.md   | 175 +++++++++++------------------
 02-your-ai-coding-setup.md    |  28 ++---
 03-mcp-servers.md             | 249 ++++++++++--------------------------------
 04-skills.md                  |  91 +++++----------
 05-end-to-end-agents.md       |  81 ++++++--------
 06-eic-mcp-servers.md         | 111 +++++--------------
 about-the-physics.md (gone)   | 128 ----------------------
 analysis-approaches.md (gone) |  82 --------------
 config.yaml                   |   3 -
 discuss.md (gone)             |  30 -----
 index.md                      |   8 +-
 instructor-notes.md           |  40 +++----
 md5sum.txt                    |  23 ++--
 setup.md                      | 159 +++++++++------------------
 14 files changed, 298 insertions(+), 910 deletions(-)
What does this mean?

If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible.

This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

⏱️ Updated at 2026-09-30 14:04:00 +0000

The eic_xl image now ships eic-mcp and the three servers (native streamable
HTTP), so setup shrinks to: ./eic-shell → eic-mcp up → eic-mcp config
<client>.  Documents both assistant modes explicitly — (a) opencode/claude/
copilot inside eic-shell, (b) an assistant on the host over the shared
network (macOS keeps the port-publishing step, minus the launcher mount).
…rap flow

Also drops the vendored-launcher row from the README (bin/eic-mcp was
removed with the previous commit: its only consumer was the macOS -v mount,
gone now that the image ships the launcher).
@aprozo
aprozo force-pushed the feat/simplified-setup branch from d1ddfa0 to f1cf854 Compare September 7, 2026 07:33
github-actions Bot pushed a commit that referenced this pull request Sep 7, 2026
@aprozo
aprozo marked this pull request as ready for review September 7, 2026 08:05
"installed in eic-shell" instead of describing image contents;
shorter phrasing elsewhere. Commands unchanged.
github-actions Bot pushed a commit that referenced this pull request Sep 7, 2026
github-actions Bot pushed a commit that referenced this pull request Sep 7, 2026
@aprozo
aprozo force-pushed the feat/simplified-setup branch from e0a938d to f55f72e Compare September 30, 2026 12:21
github-actions Bot pushed a commit that referenced this pull request Sep 30, 2026
github-actions Bot pushed a commit that referenced this pull request Sep 30, 2026
github-actions Bot pushed a commit that referenced this pull request Sep 30, 2026
@aprozo
aprozo requested a balanced review from Copilot September 30, 2026 13:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The setup contains two broken directory/context instructions and does not implement the advertised macOS workflow.

Review effort: Balanced
Findings: 3 Medium severity

Open (3)
What changed in this PR

Simplifies the tutorial now that eic-shell ships the MCP servers and opencode.

Changes:

  • Removes cloning, PATH, and server bootstrap steps.
  • Streamlines setup and lesson text.
  • Removes redundant reference pages and bundled launcher.
File Description
README.md Simplifies setup and repository overview
learners/​setup.md Reworks eic-shell and assistant setup
learners/​discuss.md Removes discussion page
learners/​analysis-approaches.md Removes alternative-analysis page
learners/​about-the-physics.md Removes physics reference page
instructors/​instructor-notes.md Updates workshop preparation
index.md Refines introductory wording
files/​skills/​AGENTS.md Clarifies no-download policy
files/​mcp-config/​vscode-mcp.json Updates server-start guidance
files/​mcp-config/​opencode.jsonc Shortens configuration comments
files/​mcp-config/​copilot-mcp-config.json Updates server-start guidance
extras/​README.md Removes deleted-page reference
episodes/​06-eic-mcp-servers.md Condenses the server catalog
episodes/​05-end-to-end-agents.md Replaces cloned resources with downloads
episodes/​04-skills.md Updates skill installation workflow
episodes/​03-mcp-servers.md Simplifies MCP setup and explanations
episodes/​02-your-ai-coding-setup.md Streamlines physics setup
episodes/​01-why-genai-for-physics.md Condenses agent concepts
config.yaml Removes deleted learner pages
bin/​eic-mcp Removes the bundled launcher

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread learners/setup.md Outdated
Comment thread learners/setup.md
Comment thread learners/setup.md
aprozo and others added 2 commits September 30, 2026 16:00
Clarify usage of eic-shell and opencode on macOS.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated setup instructions to include changing directory before downloading opencode.jsonc.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@aprozo
aprozo merged commit 1e0c939 into main Sep 30, 2026
4 checks passed
github-actions Bot pushed a commit that referenced this pull request Sep 30, 2026
Auto-generated via `{sandpaper}`
Source  : 1e0c939
Branch  : main
Author  : Alexandr Prozorov <alexandr.a.prozorov@gmail.com>
Time    : 2026-09-30 14:02:32 +0000
Message : Merge pull request #1 from eic/feat/simplified-setup

Setup: the servers ship in eic-shell — drop the clone/PATH/bootstrap steps
github-actions Bot pushed a commit that referenced this pull request Sep 30, 2026
Auto-generated via `{sandpaper}`
Source  : 590f3a0
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2026-09-30 14:04:10 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 1e0c939
Branch  : main
Author  : Alexandr Prozorov <alexandr.a.prozorov@gmail.com>
Time    : 2026-09-30 14:02:32 +0000
Message : Merge pull request #1 from eic/feat/simplified-setup

Setup: the servers ship in eic-shell — drop the clone/PATH/bootstrap steps
github-actions Bot pushed a commit that referenced this pull request Oct 6, 2026
Auto-generated via `{sandpaper}`
Source  : 590f3a0
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2026-09-30 14:04:10 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 1e0c939
Branch  : main
Author  : Alexandr Prozorov <alexandr.a.prozorov@gmail.com>
Time    : 2026-09-30 14:02:32 +0000
Message : Merge pull request #1 from eic/feat/simplified-setup

Setup: the servers ship in eic-shell — drop the clone/PATH/bootstrap steps
github-actions Bot pushed a commit that referenced this pull request Oct 13, 2026
Auto-generated via `{sandpaper}`
Source  : 590f3a0
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2026-09-30 14:04:10 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 1e0c939
Branch  : main
Author  : Alexandr Prozorov <alexandr.a.prozorov@gmail.com>
Time    : 2026-09-30 14:02:32 +0000
Message : Merge pull request #1 from eic/feat/simplified-setup

Setup: the servers ship in eic-shell — drop the clone/PATH/bootstrap steps
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