Skip to content

Add Grok Bot remote MCP e2e fixture under examples/ - #21

Merged
professorpalmer merged 1 commit into
mainfrom
cursor/grok-bot-remote-e2e-fixture-fc79
Aug 12, 2026
Merged

Add Grok Bot remote MCP e2e fixture under examples/#21
professorpalmer merged 1 commit into
mainfrom
cursor/grok-bot-remote-e2e-fixture-fc79

Conversation

@professorpalmer

Copy link
Copy Markdown
Owner

Summary

  • Add examples/grok-bot-remote-e2e/ as a shippable fixture cwd for Grok Bot Plugins → Puppetmaster remote MCP review/implement validation (follow-up to the scratch workdir that tracked node_modules and left implement markers untracked).
  • Include .gitignore (node_modules, logs, OS junk, Puppetmaster/CodeGraph state, .env), package.json (puppetmaster-grok-bot-e2e, @cursor/sdk ^1.0.26, real main + green npm test), minimal index.js / index.test.js, tracked markers GROK_BOT_E2E.md + GROK_BOT_IMPLEMENT_OK.md, and a short README aligned with docs/GROK_BOT.md.
  • Cross-link from examples/README.md and a single Related-docs line in docs/GROK_BOT.md. No remote MCP runtime changes.

Test plan

  • cd examples/grok-bot-remote-e2e && npm install --package-lock=false --no-audit && npm test exits 0
  • git check-ignore -v examples/grok-bot-remote-e2e/node_modules confirms ignore; no node_modules in the PR diff
  • CI matrix (unittest + existing examples JSON smokes) stays green
  • Optional live: point Grok Bot remote MCP job cwd at this fixture; review uses GROK_BOT_E2E.md, implement may update GROK_BOT_IMPLEMENT_OK.md
Open in Web Open in Cursor 

Replace the scratch workdir pattern with a tracked mini workspace: gitignore, real package main/test, and e2e markers for Plugins review/implement validation.

Co-authored-by: Cary Palmer <professorpalmer@users.noreply.github.com>
@professorpalmer
professorpalmer marked this pull request as ready for review August 12, 2026 05:35
@professorpalmer
professorpalmer merged commit ccd8719 into main Aug 12, 2026
8 checks passed
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