-
Notifications
You must be signed in to change notification settings - Fork 28
Extension v2.0 #207
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
0xtsukino
wants to merge
139
commits into
main
Choose a base branch
from
new-ext-2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Extension v2.0 #207
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Add WindowManager for independent multi-window state tracking - Implement window.tlsn.open(url) client API with validation - Add OPEN_WINDOW message handler in background script - Add request interception and overlay updates per window - Add automatic cleanup of closed windows - Add URL protocol validation (http/https only) - Add comprehensive test coverage (72 tests passing) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Add showOverlayWhenReady flag to ManagedWindow for lazy overlay display - Implement persistent tabs.onUpdated listener to show overlay when tab is ready - WindowManager.registerWindow no longer shows overlay immediately - Overlay shown when tab status becomes 'complete' via tabs.onUpdated - Add backward compatibility handler for TLSN_CONTENT_TO_EXTENSION - Legacy handler opens x.com window using new WindowManager system - Update tests to verify showOverlayWhenReady behavior - All 72 tests passing This fixes race condition where overlay was shown before content script was ready. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Task 4.2: Integration test HTML page - Interactive test page with 6 test sections - Basic window opening with predefined URLs - Custom URL testing with input field - Window options testing (dimensions, overlay toggle) - Multiple windows test (3, 5, 10 windows) - Error handling tests (invalid URLs, protocols) - Legacy API backward compatibility test - Real-time statistics tracking - Styled UI with instructions and status messages Task 4.3: Manual testing checklist - 12 comprehensive test categories - 50+ individual test cases with pass/fail checkboxes - Tests cover: basic operations, custom URLs, options, multiple windows, request interception, error handling, cleanup, backward compatibility, overlay functionality, edge cases, console logs - Performance observation section - Sign-off and reporting format - Acceptance criteria for each test Task 4.4: Performance testing guidelines - 8 structured performance test procedures - Memory usage, CPU usage, and request processing metrics - Baseline performance targets and thresholds - Memory leak detection methodology - High-traffic site testing protocol - Request tracking overhead measurement - Cleanup efficiency verification - Long-running window test (30 minutes) - Periodic cleanup verification - Tools and commands reference - Performance issue detection checklist - Reporting template All 72 unit tests passing ✅ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Test flow diagram - Quick start guide - File descriptions and usage instructions - Testing best practices checklist - Common issues and troubleshooting - Issue reporting guidelines - CI/CD future considerations 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
…ndMessage from client API 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
…date test example 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
…pened windows 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
… and DevConsole UI enhancements
…ndering, and DevConsole UI enhancements" This reverts commit 730ce17.
+ fixed some warnings in verifier
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.