chore(deps): bump playwright and @playwright/test in /toboggan-web#11
Open
dependabot[bot] wants to merge 64 commits intomainfrom
Open
chore(deps): bump playwright and @playwright/test in /toboggan-web#11dependabot[bot] wants to merge 64 commits intomainfrom
dependabot[bot] wants to merge 64 commits intomainfrom
Conversation
Add comprehensive WASM implementation with major security and performance improvements: ## Security Enhancements - Eliminate all unsafe code using safe Rc<RefCell<T>> pattern - Add HTML sanitization to prevent XSS attacks - Implement URL validation for iframe content with sandbox attributes - Add proper input escaping for all dynamic content ## Performance Optimizations - Use pre-allocated string buffers for HTML generation - Optimize memory allocations in rendering hot paths - Implement aggressive build size optimization (opt-level="z", lto="fat") - Remove deprecated wee_alloc dependency for better performance ## Error Handling & Reliability - Add comprehensive error types with categorization - Implement WebSocket retry logic with exponential backoff - Add proper error boundaries with user feedback - Improve connection state management ## API Design & Configuration - Add TobogganConfig for WebSocket URL and retry settings - Implement proper cleanup lifecycle for memory management - Add comprehensive test coverage for core functionality - Support both default and custom configuration ## Code Quality - Fix all clippy lint issues with modern Rust idioms - Add proper Default trait implementation - Convert tests to regular unit tests + WASM-specific tests - Comprehensive documentation with security and performance details 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Enhanced error handling with typed responses (ApiResponse, ErrorResponse) - Comprehensive configuration management with validation and env vars - WebSocket resilience with client limits, cleanup, and heartbeat - Security enhancements with configurable CORS origins - Performance optimizations using lock-free DashMap for clients - Observability with structured logging and request metrics - Graceful shutdown with signal handling and timeouts - Fixed all clippy warnings and formatting issues - Updated tests to work with new response structures 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
After removing the flaky test_two_clients_navigation_sync, several helper functions became unused and were causing clippy warnings treated as errors: - receive_next_non_pong_notification - coordinated_command_execution - receive_recent_non_pong_notification This cleanup resolves all clippy dead_code warnings and ensures the test suite passes all linting checks. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
refactor(client): Remove sync feature and keep only async mode
This major refactoring removes the SlideId type from toboggan-core and replaces it with simple usize indices throughout the codebase, simplifying the API and improving performance. Major changes: - Remove SlideId type and related functionality from toboggan-core - Update State enum to use UInt32 indices instead of SlideId strings - Modify all API endpoints to use slide indices instead of IDs - Update WebSocket communication to use index-based navigation - Fix iOS app compilation by updating UniFFI Swift bindings - Update TUI, server, and WASM clients to work with index-based slides - Maintain backward compatibility in REST API responses Breaking changes: - WebSocket API now uses indices instead of SlideId strings - State enum variants updated to use UInt32 parameters - Client libraries must use getSlide(index) instead of getSlide(slideId) Fixes iOS compilation issues after the core refactoring. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
After the SlideId → index refactoring, the updateSlideIndexFromState method became redundant since the State enum now directly provides slide indices. This method was: - Never called after the refactoring - Contained complex inference logic no longer needed - Included fallback guessing that could introduce bugs The State enum variants now directly contain the slide index: - .running(previous, current, next, duration) where current IS the index - No need to infer or calculate indices from navigation state This simplifies the codebase by removing ~30 lines of dead code and eliminates potential bugs from the arbitrary fallback logic. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
refactor: Remove SlideId from core types and replace with indices
feat(desktop): Upgrade iced framework to v0.13.1 and improve UI
Feat/esp32
dependabot
bot
added
dependencies
dependabot
bot
force-pushed
the
dependabot/npm_and_yarn/toboggan-web/multi-a50d7f32cf
branch
from
52b041f to
246402a
Compare
feat: Export static html
fix: Cleanup the code
Dependabot couldn't find the original pull request head commit, 246402a.
dependabot
bot
force-pushed
the
dependabot/npm_and_yarn/toboggan-web/multi-a50d7f32cf
branch
from
246402a to
d5f64c7
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Bumps playwright to 1.56.1 and updates ancestor dependency @playwright/test. These dependencies need to be updated together.
Updates
playwrightfrom 1.55.0 to 1.56.1Release notes
Sourced from playwright's releases.
... (truncated)
Commits
54c7115chore: revert "minimal vscode version notice" (#37892)7d45eb3chore: mark v1.56.1 (#37784)e6ef697cherry-pick(#37871): chore: allow local-network-access permission in chromium932542ccherry-pick(#37891): fix(agents): remove workspaceFolder ref from vscode mcp0662dd2cherry-pick(#37759): chore: rename agents to test agents919549echerry-pick(#37758): docs: mention VS Code insiders in the agents docse593c64cherry-pick(#37757): chore(mcp): fallback to cwd when resolving test configa8a6e10cherry-pick(#37755): chore(mcp): minimal vscode version noticef36b2eecherry-pick(#37731): docs: add agents video to agents page (#37733)b6af258cherry-pick(#37727): devops: fix NPM release step (#37728)Maintainer changes
This version was pushed to npm by [GitHub Actions](https://www.npmjs.com/~GitHub Actions), a new releaser for playwright since your current version.
Updates
@playwright/testfrom 1.55.0 to 1.56.1Release notes
Sourced from
@playwright/test's releases.... (truncated)
Commits
54c7115chore: revert "minimal vscode version notice" (#37892)7d45eb3chore: mark v1.56.1 (#37784)e6ef697cherry-pick(#37871): chore: allow local-network-access permission in chromium932542ccherry-pick(#37891): fix(agents): remove workspaceFolder ref from vscode mcp0662dd2cherry-pick(#37759): chore: rename agents to test agents919549echerry-pick(#37758): docs: mention VS Code insiders in the agents docse593c64cherry-pick(#37757): chore(mcp): fallback to cwd when resolving test configa8a6e10cherry-pick(#37755): chore(mcp): minimal vscode version noticef36b2eecherry-pick(#37731): docs: add agents video to agents page (#37733)b6af258cherry-pick(#37727): devops: fix NPM release step (#37728)Maintainer changes
This version was pushed to npm by [GitHub Actions](https://www.npmjs.com/~GitHub Actions), a new releaser for
@playwright/testsince your current version.You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.