[Claude PR] Fix HTTP connection issue for schedule triggers

[claude]

Summary

Fixed the issue where schedule triggers couldn't be created when connecting to Windmill over HTTP. The problem was caused by crypto.randomUUID() requiring a secure context (HTTPS), which broke functionality for users on internal HTTP networks.

Changes Made

• Replaced crypto.randomUUID() with generateRandomString() in triggers.svelte.ts
• Added generateRandomString import from $lib/utils
• The generateRandomString() function uses Math.random() which works in any context

Issue Reference

Fixes #5847

Test Plan

• Verify the change compiles correctly
• Test schedule trigger creation on HTTP connection
• Ensure generated IDs are still unique and functional

Generated with Claude Code

---

Important

Replaces crypto.randomUUID() with generateRandomString() in triggers.svelte.ts to fix schedule trigger creation over HTTP.

• Behavior:
  • Replaces crypto.randomUUID() with generateRandomString() in triggers.svelte.ts to fix schedule trigger creation over HTTP.
  • generateRandomString() uses Math.random() to work in any context.
• Imports:
  • Adds generateRandomString import from $lib/utils in triggers.svelte.ts.
• Testing:
  • Verify compilation and test schedule trigger creation on HTTP.
  • Ensure generated IDs are unique and functional.

^{This description was created by for d0a4456. You can customize this summary. It will automatically update as commits are pushed.}

[cloudflare-workers-and-pages]

Deploying windmill with    Cloudflare Pages

Latest commit:	d0a4456
Status:	✅  Deploy successful!
Preview URL:	https://156340ec.windmill.pages.dev
Branch Preview URL:	https://claude-issue-5847-20250601-0.windmill.pages.dev

View logs

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to d0a4456 in 28 seconds. Click for details.

• Reviewed 22 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. frontend/src/lib/components/triggers/triggers.svelte.ts:23

• Draft comment:
  Added generateRandomString to imports. Ensure that generateRandomString reliably produces unique IDs to avoid collisions, since it now replaces crypto.randomUUID().
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

2. frontend/src/lib/components/triggers/triggers.svelte.ts:97

• Draft comment:
  Replacing crypto.randomUUID() with generateRandomString() fixes HTTPS requirement. Confirm that the alternative provides sufficient uniqueness for draft trigger IDs.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_50wDSyVOqfd3Q9gt

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}