Skip to content

Rename mcp-vs-api-calls to mcp-vs-api to match the tutorial slug - #819

Closed
stephengruppetta wants to merge 1 commit into
masterfrom
rename-mcp-vs-api-slug
Closed

stephengruppetta wants to merge 1 commit into
masterfrom
rename-mcp-vs-api-slug

Conversation

@stephengruppetta

Copy link
Copy Markdown
Contributor

The tutorial's slug is now mcp-vs-api, and materials folders match the post slug (python-mcp, python-mcp-client, python-copy, python-json). This renames the folder to match.

Two README fixes while in here:

  • The tutorial link pointed at realpython.com/mcp-vs-api-calls/, which is a slug that never existed. Now points at realpython.com/mcp-vs-api/.
  • The install command left anthropic unpinned, so it had drifted from the tutorial after today's tech-review fix. Now "anthropic>=1,<2", matching the article.

ruff format --check, ruff check, and dircheck.py all pass on the renamed folder.

🤖 Opened by Claude Code (RPBot) from Stephen's account, with his permission, after a discussion with him.

The post slug is mcp-vs-api, and materials folders match the post slug
(python-mcp, python-mcp-client, python-copy). Also fixes the README's
tutorial link, which pointed at a slug that never existed, and pins
anthropic in the install command to match the tutorial.
@stephengruppetta

Copy link
Copy Markdown
Contributor Author

🤖 Posted by Claude Code (RPBot) from Stephen's account, with his permission.

@bartosz-zaczynski flagging for review when you get a chance — Stephen can't approve it since it was opened from his own account.

Follow-up needed after this merges: the opt-in for this tutorial (mcp-vs-api-code, CMS Optin 1283) has its download_url pointing at .../tree/master/mcp-vs-api-calls/. That's correct today and becomes a dead link the moment this rename lands, so it needs switching to .../tree/master/mcp-vs-api/. Ping Stephen once it's in and it'll get done.

realpython-bot pushed a commit that referenced this pull request Sep 16, 2026
Consolidates the two open PRs on this folder into one.

- Renames mcp-vs-api-calls/ to mcp-vs-api/ to match the tutorial slug
  (supersedes #819, minus its stray root-level uv.lock).
- Syncs client.py and client_api.py with the tech-review changes in
  tutorial-drafts#1184: collect every tool_use block and append the
  assistant turn once, select the text block by type rather than
  position, and raise max_tokens to 2048 (supersedes #832).
- Repairs the leftover fragments in #832: client_api.py did not parse,
  and client.py ran its __main__ block twice.
- Points the README at the correct tutorial URL and the pinned install
  command.

Code verified line-for-line against the article. ruff check and
ruff format pass.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@realpython-bot

Copy link
Copy Markdown
Collaborator

Superseded by #850, which includes this rename alongside the client-code sync from #832, so the folder moves once rather than twice.

One thing deliberately left out: this branch also added a root-level uv.lock (3 lines — version, revision, requires-python). master has no root lockfile, so that looked accidental and isn't carried over. Say the word if it was intentional and I'll add it back.


🤖 Posted by Claude Code (RPBot) on Stephen's behalf, with his permission.

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