Add query-string-stripper.html tool

[simonw]

Creates a mobile-friendly interface for stripping query strings from URLs. As users paste or type a URL, it instantly displays the URL with everything from ? onwards removed, plus a copy to clipboard button.

🤖 Generated with Claude Code

[claude]

Claude finished @simonw's task —— View job

---

Code Review for Query String Stripper Tool

Reviewing the pull request and providing feedback on code quality, bugs, performance, security, and test coverage.

Todo List:

• Read repository guidelines (CLAUDE.md) - No repo-specific guidelines found
• Analyze the new query-string-stripper.html file
• Review code quality and best practices
• Check for potential bugs or issues
• Evaluate performance considerations
• Assess security concerns
• Review test coverage
• Provide comprehensive feedback

Code Review Summary

The query-string-stripper.html tool is a well-implemented, self-contained HTML utility that safely strips query strings from URLs. Here's my detailed analysis:

✅ Code Quality & Best Practices

• Clean structure: Well-organized HTML with inline CSS and JavaScript
• Semantic HTML: Proper use of labels, input elements, and accessibility considerations
• Mobile-responsive: Good viewport meta tag and responsive CSS with media queries
• Progressive enhancement: Works without JavaScript for basic functionality
• Consistent styling: Clean, modern UI with proper hover/active states

✅ Security Assessment

• No security concerns identified: The tool operates entirely client-side
• Safe string operations: Uses basic string methods (indexOf, substring) without eval or innerHTML
• No external dependencies: Self-contained with no CDN or external script risks
• XSS-resistant: All operations are on input values, not DOM manipulation of untrusted content

⚠️ Minor Issues & Suggestions

query-string-stripper.html:121-122: Mobile clipboard selection could be improved

strippedUrl.setSelectionRange(0, 99999); // For mobile devices

The magic number 99999 works but could use strippedUrl.value.length for better semantics.

query-string-stripper.html:132: Deprecated API usage

document.execCommand('copy');

While this provides good fallback support, consider adding a comment noting this is for legacy browser support.

🚀 Performance Considerations

• Excellent performance: Minimal DOM operations, efficient event handling
• Real-time processing: Input event handler is appropriately lightweight
• Memory efficient: No memory leaks or excessive object creation

🧪 Test Coverage

• No automated tests: This is typical for simple HTML tools in this repository
• Manual testing recommended: Test cases should include:
  • URLs without query strings
  • URLs with multiple query parameters
  • URLs with fragments (#hash)
  • Malformed URLs
  • Very long URLs
  • Empty input handling

📋 Recommendations for Future Enhancement

1. Edge case handling: Consider URLs with fragments (example.com/page?query=1#fragment) - currently strips everything after ?
2. URL validation: Could add basic URL format validation with visual feedback
3. Keyboard shortcuts: Could add Ctrl/Cmd+C shortcut for power users

Overall Assessment

This is a high-quality, secure, and well-implemented tool that follows the patterns established in the repository. The code is production-ready with only minor suggestions for improvement. The tool serves its purpose effectively and safely.

Recommendation: ✅ Approve - Ready to merge