fix(release): v1.3.5 prevent duplicate +10 Arcade Bonus Milestone

ARCHITECTURE.md

@@ -134,6 +134,8 @@ Current behavior:
 - Requires explicit user confirmation.
 - Keeps Bonus Milestone separate from the normal Facilitator milestone bonus.
 - Avoids double counting when the API says the bonus is already included.
+- The claim chip is status-only: it triggers a signed API refresh and never adjusts the headline DOM total itself.
+- API point snapshots remain unmodified in extension storage; derived totals are only for presentation and toolbar badges.
 - Persists confirmation using a profile/period-scoped storage key.
 - Supports undo.
 - Ignores stale confirmation for periods where the bonus is unavailable.

CHANGELOG.md

@@ -5,6 +5,24 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.3.5] - 2026-10-30
+
+### :bug: Bug Fixes
+
+- Prevent double counting of the self-reported `+10` Bonus Milestone in the Arcade popup headline.
+- Preserve raw Arcade API point totals in active-account and legacy storage instead of persisting locally adjusted totals.
+- Respect the API's `facilitator.bonusIncludedInTotal` flag when deriving Facilitator bonus for the popup, account cards, and extension badge.
+
+### :white_check_mark: Tests
+
+- Add regression coverage for the API-confirmed 130-point score remaining unchanged after Bonus Milestone confirmation.
+- Cover Facilitator/Arcade countdown Remote Config behavior, active timers, and single-card fallback after Facilitator expiry.
+
+### :wrench: Maintenance
+
+- Refactor countdown UI to reduce repeated code, remove empty exception handlers, and resolve DeepSource JavaScript findings.
+- Bump the extension version to 1.3.5 in preparation for the `dev → main` release. The release tag is created only after approval and merge.
+
 ## [1.3.4] - 2026-10-06
 
 ### :lock: Security

components/BonusMilestoneControl.tsx

@@ -54,46 +54,7 @@ function renderedManualBonus(state: BonusMilestoneControlState): number {
   return state.appliedPoints > 0 ? state.appliedPoints : CLAIM_BONUS_POINTS;
 }
 
-/** Parse a rendered point total into a safe numeric value. */
-function parseDisplayedTotal(value: string | null | undefined): number {
-  if (!value) return 0;
-  const normalized = value.replace(/\s/gu, "").replace(/,/gu, "");
-  const parsed = Number(normalized);
-  return Number.isFinite(parsed) ? parsed : 0;
-}
-
-/** Add only the self-reported bonus to the main total, without double counting it. */
-function syncDisplayedTotal(state: BonusMilestoneControlState): void {
-  const total = document.getElementById("arcade-points");
-  if (!total) return;
-
-  if (total.dataset.bonusMilestoneProfile !== state.profileUrl) {
-    delete total.dataset.bonusMilestoneApplied;
-    delete total.dataset.bonusMilestoneRenderedTotal;
-    total.dataset.bonusMilestoneProfile = state.profileUrl;
-  }
-
-  const currentTotal = parseDisplayedTotal(total.textContent);
-  const previousApplied =
-    Number(total.dataset.bonusMilestoneApplied || "0") || 0;
-  const previousRendered = Number(
-    total.dataset.bonusMilestoneRenderedTotal || "NaN",
-  );
-  const stillContainsPreviousBonus =
-    Number.isFinite(previousRendered) && currentTotal === previousRendered;
-  const baseTotal = stillContainsPreviousBonus
-    ? Math.max(0, currentTotal - previousApplied)
-    : currentTotal;
-  const nextApplied = state.bonusIncludedInTotal
-    ? 0
-    : renderedManualBonus(state);
-  const nextTotal = baseTotal + nextApplied;
-
-  total.textContent = formatPoints(nextTotal);
-  total.dataset.bonusMilestoneApplied = String(nextApplied);
-  total.dataset.bonusMilestoneRenderedTotal = String(nextTotal);
-}
-
+/** The signed Arcade refresh owns the total; the claim chip never edits score DOM. */
 /** Render the self-report disclaimer and the official task link before confirmation. */
 function BonusMilestoneInfo({ completed }: BonusMilestoneInfoProps) {
   return (
@@ -147,13 +108,6 @@ function BonusMilestoneControl() {
     [reload],
   );
 
-  useEffect(
-    function keepMainTotalInSync() {
-      syncDisplayedTotal(state);
-    },
-    [state],
-  );
-
   const manualBonus = renderedManualBonus(state);
   const canConfirm = state.participating && state.enabled && state.points > 0;
 

package.json

@@ -2,7 +2,7 @@
   "name": "google-cloud-skills-boost-helper",
   "description": "__MSG_extDescription__",
   "private": true,
-  "version": "1.3.4",
+  "version": "1.3.5",
   "author": "dev@eplus.dev",
   "license": "MIT",
   "homepage_url": "https://eplus.dev",

services/optionsService.ts

@@ -131,9 +131,11 @@ const OptionsService = {
     const nickname = account.nickname;
     const arcadePoints = account.arcadeData?.arcadePoints?.totalPoints || 0;
     // If facilitator program is enabled, compute facilitator bonus and add to total
-    const facilitatorBonus = account.facilitatorProgram
-      ? calculateFacilitatorBonus(account.arcadeData?.faciCounts)
-      : 0;
+    const facilitatorBonus =
+      account.facilitatorProgram &&
+      account.arcadeData?.facilitator?.bonusIncludedInTotal !== true
+        ? calculateFacilitatorBonus(account.arcadeData?.faciCounts)
+        : 0;
     const finalPoints = (arcadePoints || 0) + (facilitatorBonus || 0);
     const activeLabel = browser.i18n.getMessage("statusActive");
     const facilitatorLabel = browser.i18n.getMessage("labelFacilitatorProgram");