Skip to content

Clean up example templates for guidelines - #25

Merged
MiguelsPizza merged 1 commit into
mainfrom
claude/clean-example-templates-01DSWcjjFLLXmyeJzwfoT9Ev
Nov 23, 2025
Merged

Clean up example templates for guidelines#25
MiguelsPizza merged 1 commit into
mainfrom
claude/clean-example-templates-01DSWcjjFLLXmyeJzwfoT9Ev

Conversation

@MiguelsPizza

Copy link
Copy Markdown
Contributor

Remove all code-level inline comments from React and Vanilla templates to adhere to contribution guidelines which require clean, self-documenting code with JSDoc for public APIs only.

Changes:

  • React template: Remove inline comments from App.tsx, main.tsx, worker files
  • Vanilla template: Remove inline comments from index.html, worker files
  • All code remains functionally identical
  • Type checking and builds pass successfully

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test addition/improvement

Related Issues

Fixes #
Related to #

Changes Made

  • Change 1
  • Change 2
  • Change 3

Testing

Test Commands Run

pnpm typecheck  # ✓ Passed
pnpm lint       # ✓ Passed
pnpm build      # ✓ Passed
pnpm test       # ✓ Passed - E2E tests run locally

Manual Testing

  • Tested in development mode (pnpm dev)
  • Tested production build (pnpm build && preview)
  • Tested on mobile/tablet (if UI changes)
  • Tested WebMCP integration (if applicable)
  • Tested MCP tool execution (if applicable)

Screenshots

Checklist

  • My code follows the style guidelines in CONTRIBUTING.md
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have added tests that prove my fix is effective or that my feature works
  • I have run E2E tests locally (pnpm test) and they pass
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Breaking Changes

None

Additional Notes

Remove all code-level inline comments from React and Vanilla templates
to adhere to contribution guidelines which require clean, self-documenting
code with JSDoc for public APIs only.

Changes:
- React template: Remove inline comments from App.tsx, main.tsx, worker files
- Vanilla template: Remove inline comments from index.html, worker files
- All code remains functionally identical
- Type checking and builds pass successfully
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
mcp-ui-with-webmcp-my-mcp-server 0482501 Nov 22 2025, 11:58 PM

@MiguelsPizza
MiguelsPizza merged commit 6fcc257 into main Nov 23, 2025
4 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