sync: dev to extern-contrib

[github-actions]

sync-branches: New code has just landed in dev, so let's bring extern-contrib up to speed!

---

Summary by cubic

Allow UpdateVersion to run on edited PRs even if the last commit is from a bot, and gate WebSocket toasts by per-feature popup settings. Bumps version to 3.2.2 with updated release notes.

• Bug Fixes
  • UpdateVersion: exclude all bot actors in the workflow and allow edited events in the script so PR title/body edits still update Update.json without loops.
  • WebSocket toasts: gate bbs_mention/mail_mention by BBSPopup/MessagePopup to match polling behavior.

^{Written for commit d6ecb25. Summary will update on new commits.}

[cloudflare-workers-and-pages]

Deploying xmoj-script-dev-channel with    Cloudflare Pages

Latest commit:	d6ecb25
Status:	✅  Deploy successful!
Preview URL:	https://09f975e3.xmoj-script-dev-channel.pages.dev

View logs

[cubic-dev-ai]

1 issue found across 2 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="Update/UpdateVersion.js">

<violation number="1" location="Update/UpdateVersion.js:18">
P2: The guard still skips for any action other than `edited`, which contradicts the comment that says only synchronize events should be skipped. This can prevent metadata updates on non-synchronize actions (e.g., `opened`), even though they should be allowed per the new behavior.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P2: The guard still skips for any action other than edited, which contradicts the comment that says only synchronize events should be skipped. This can prevent metadata updates on non-synchronize actions (e.g., opened), even though they should be allowed per the new behavior.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At Update/UpdateVersion.js, line 18:

<comment>The guard still skips for any action other than `edited`, which contradicts the comment that says only synchronize events should be skipped. This can prevent metadata updates on non-synchronize actions (e.g., `opened`), even though they should be allowed per the new behavior.</comment>

<file context>
@@ -8,9 +8,14 @@ execSync("gh pr checkout " + PRNumber);
 console.log("Last commit author: " + lastCommitAuthor);
-if (lastCommitAuthor === "github-actions[bot]") {
+console.log("Event action       : " + eventAction);
+if (lastCommitAuthor === "github-actions[bot]" && eventAction !== "edited") {
     console.log("Last commit was made by github-actions[bot]. Skipping to prevent infinite loop.");
     process.exit(0);
</file context>

Suggested change

	if (lastCommitAuthor === "github-actions[bot]" && eventAction !== "edited") {
	if (lastCommitAuthor === "github-actions[bot]" && eventAction === "synchronize") {