Skip to content

fix: drop stale --web flag from generated .mcp.json#20

Merged
zzet merged 1 commit into
zzet:mainfrom
pzartem:fix/init-mcp-drop-web-flag
May 8, 2026
Merged

fix: drop stale --web flag from generated .mcp.json#20
zzet merged 1 commit into
zzet:mainfrom
pzartem:fix/init-mcp-drop-web-flag

Conversation

@pzartem
Copy link
Copy Markdown
Contributor

@pzartem pzartem commented May 8, 2026

Summary

Commit 7208e77 removed --web from gortex mcp and gortex server, but the Claude Code init template still hardcoded it, so freshly init'd projects produced an .mcp.json that crashed on launch with an unknown-flag error. Also fixes the matching example in docs/onboarding.md.

Changes

  • internal/agents/claudecode/content.go — the ProjectMCPJSON template still hardcoded --web, plus a stale comment claiming the flag was intentional.
  • docs/onboarding.md — onboarding example still showed gortex server --index . --web --watch.

Testing

  • All tests pass (go test -race ./...)
  • New tests added for new functionality
  • Benchmarks run if performance-relevant

Checklist

  • Code follows existing patterns in the codebase
  • No unnecessary abstractions added
  • Language extractor includes Meta["methods"] for interfaces (if applicable)
  • Methods have EdgeMemberOf edges to their containing type (if applicable)

Commit 7208e77 removed --web from `gortex mcp` and `gortex server`,
but the Claude Code init template still hardcoded it, so freshly
init'd projects produced an .mcp.json that crashed on launch with
an unknown-flag error. Also fixes the matching example in
docs/onboarding.md.
@zzet zzet self-assigned this May 8, 2026
@zzet zzet added the bug Something isn't working label May 8, 2026
@zzet zzet merged commit c727b3e into zzet:main May 8, 2026
9 checks passed
@zzet
Copy link
Copy Markdown
Owner

zzet commented May 8, 2026

@pzartem good catch, thank you for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants