support harmony remote by mjpeg

[lixf6]

No description provided.

[Copilot]

Pull Request Overview

This PR adds support for HarmonyOS remote screen streaming via MJPEG and integrates it into the application. Key changes include updating scrcpy server path formatting, introducing a new HTML page for video display and associated WebSocket handling, implementing the HarmonyMjpegServer class for HarmonyOS, and updating dependency management and documentation.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
uiautodev/remote/scrcpy.py	Adjusted code formatting for scrcpy server JAR path initialization
uiautodev/remote/html/harmony-video.html	Added a new HTML page for MJPEG video stream with WebSocket logic
uiautodev/remote/harmony_scrcpy.py	Introduced HarmonyMjpegServer for handling HarmonyOS screen streaming
uiautodev/app.py	Added a new WebSocket endpoint and helper function for HarmonyOS
pyproject.toml	Updated dependencies and extras for HarmonyOS support
README.md	Updated installation instructions for the HarmonyOS extras package

Comments suppressed due to low confidence (1)

uiautodev/remote/html/harmony-video.html:124

• The coordinate logging indicates a value of 0 for startX (and similarly for other coordinates), which may signal an issue with the coordinate calculation logic or event processing. Please verify that the calculateCoordinates() function correctly translates canvas coordinates, or remove these debug logs if they are no longer required.

console.log(startX); // 输出 0

[Copilot]

[nitpick] Consider removing or cleaning up commented-out code blocks in the mouse event handlers to enhance code clarity and maintainability, particularly if they are no longer needed for debugging or future functionality.

Suggested change

	// isMouseDown = true;
	downCoordinates = calculateCoordinates(event)
	// 按下即触发点击事件
	// sendMouseEvent('normal', downCoordinates);
	// endX = startX;
	// endY = startY;
	// Long press detection
	// longPressTimer = setTimeout(function() {
	// if (isMouseDown) {
	// sendMouseEvent('long', { clientX: startX, clientY: startY });
	// }
	// }, 500); // Long press time threshold in milliseconds
	});
	// canvas.addEventListener('mousemove', function(event) {
	// console.log('move', event.clientX, event.clientY)
	// if (isMouseDown) {
	downCoordinates = calculateCoordinates(event);
	});
	// Future implementation for mousemove event can be added here if needed.

[Copilot]

Consider setting a timeout on the socket (e.g., using settimeout) to avoid potential blocking during long network reads in production environments.

Suggested change

	self.channel = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
	self.channel = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
	self.channel.settimeout(10) # Set a timeout of 10 seconds for socket operations