Development

[pladisdev]

No description provided.

[Copilot]

Pull request overview

This PR bumps the app to v1.3.2 and focuses on improving Edge TTS reliability, along with some build/CI updates and small frontend routing/config tweaks.

Changes:

• Update Edge TTS dependency and add runtime retry/update logic for Edge TTS 403 failures.
• Adjust frontend default route to redirect / to /settings, and fix Vite config __dirname usage in ESM.
• Update Windows CI to install WiX Toolset v5 and reduce/disable Docker build steps in CI.

Reviewed changes

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

Show a summary per file

File	Description
requirements.txt	Bumps edge-tts dependency version.
README.md	Updates changelog for v1.3.2.
frontend/vite.config.js	Adds ESM-compatible __dirname resolution for path aliases/output dir.
frontend/src/app.jsx	Adds a default route redirect from / to /settings.
frontend/package-lock.json	Updates lockfile entries from dependency refresh.
deployment/ChatYapper.wxs	Bumps MSI product version to 1.3.2.
deployment/build_msi.py	Extends WiX extension detection to include global extensions.
backend/version.py	Bumps backend version to 1.3.2.
backend/modules/tts.py	Adds Edge TTS 403 retry and attempts to update edge-tts at runtime.
.github/workflows/build-and-release.yml	Moves CI to WiX v5 and disables Docker build/push to speed up CI.

Files not reviewed (1)

• frontend/package-lock.json: Language not supported

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

subprocess is imported but never used in this module; it should be removed to avoid lint warnings and keep imports minimal.

Suggested change

import subprocess

[Copilot]

Auto-updating dependencies at runtime via pip install --upgrade edge-tts has significant security and operational risks (downloads arbitrary code, requires network + write permissions, and will likely fail in the PyInstaller-built executable environment). Prefer surfacing a clear error with upgrade instructions (or pin/ship a compatible version) rather than attempting an in-app upgrade.

Suggested change

	# Track edge-tts update attempts to avoid repeated updates
	_edge_tts_update_attempted = False
	async def try_update_edge_tts():
	"""Attempt to update edge-tts package when API compatibility issues occur"""
	global _edge_tts_update_attempted
	if _edge_tts_update_attempted:
	logger.info("edge-tts update already attempted this session, skipping")
	return False
	_edge_tts_update_attempted = True
	logger.info("Attempting to update edge-tts package to fix API compatibility...")
	try:
	# Run pip upgrade in subprocess
	python_exe = sys.executable
	result = await asyncio.create_subprocess_exec(
	python_exe, "-m", "pip", "install", "--upgrade", "edge-tts",
	stdout=asyncio.subprocess.PIPE,
	stderr=asyncio.subprocess.PIPE
	)
	stdout, stderr = await result.communicate()
	if result.returncode == 0:
	logger.info(f"edge-tts successfully updated: {stdout.decode()}")
	# Reload the edge_tts module
	try:
	import importlib
	global edge_tts
	if edge_tts:
	importlib.reload(edge_tts)
	logger.info("edge-tts module reloaded successfully")
	else:
	import edge_tts as new_edge_tts
	edge_tts = new_edge_tts
	logger.info("edge-tts module imported successfully")
	return True
	except Exception as e:
	logger.warning(f"edge-tts updated but module reload failed: {e}")
	logger.info("Restart the application to use the updated edge-tts")
	return False
	else:
	logger.error(f"edge-tts update failed: {stderr.decode()}")
	return False
	except Exception as e:
	logger.error(f"Failed to update edge-tts: {e}")
	return False
	# Track edge-tts update attempts to avoid repeated update guidance logs
	_edge_tts_update_attempted = False
	async def try_update_edge_tts():
	"""Log manual remediation guidance instead of attempting a runtime package update."""
	global _edge_tts_update_attempted
	if _edge_tts_update_attempted:
	logger.info("edge-tts upgrade guidance already provided this session, skipping")
	return False
	_edge_tts_update_attempted = True
	python_exe = sys.executable or "python"
	logger.error(
	"Automatic runtime updates for edge-tts are disabled for security and reliability "
	"reasons. Please upgrade the dependency outside the application and restart it."
	)
	logger.info(
	f"Manual upgrade command: {python_exe} -m pip install --upgrade edge-tts"
	)
	return False

[Copilot]

New 403-retry path (including the edge-tts update attempt) is untested. There are existing pytest tests for TTS providers; add a unit test that mocks edge_tts.Communicate.save to raise the 403 error and asserts the retry behavior (and that the updater is called at most once).

[Copilot]

The Docker build/push step is commented out, but the docker-build job still proceeds to generate release files and posts a PR comment claiming the image was built successfully. Either re-enable the build/push step or guard/disable the downstream steps so CI doesn't report a Docker image/tag that doesn't exist.

[github-actions]

✅ Windows Build Successful

Executable: ChatYapper.exe (67.00 MB)
MSI Installer: ChatYapper-1.3.2.msi (66.32 MB)
App Version: 1.3.2
Build ID: build-v1.3.2-2026.04.05-93efe05
Commit: 93efe05

Build Status

• ✅ Windows executable & MSI installer
• 🔄 Linux build (check separate job)
• 🔄 Docker image (check separate job)

Download the artifacts from the workflow run to test before merging.

Once merged to main, an official release will be created automatically with tag v1.3.2.

[github-actions]

🐳 Docker Image Built Successfully

Image: ghcr.io/pladisdev/chat-yapper:pr-93efe05
Tag: pr-93efe05

Test this PR with Docker:

docker pull ghcr.io/pladisdev/chat-yapper:pr-93efe05

docker run -d \
  --name chat-yapper-pr \
  -p 8069:8008 \
  -e TWITCH_CLIENT_ID=your_id \
  -e TWITCH_CLIENT_SECRET=your_secret \
  ghcr.io/pladisdev/chat-yapper:pr-93efe05

Access at: http://localhost:8069

The Docker image will be published to the GitHub Container Registry when merged to main.