fix(FR-2606): replace global.packageVersion with globalThis.packageVersion in WebUISider

[nowgnuesLee]

Resolves #6809(FR-2606)

Summary

Runtime error found after logging into the app with the Vite build:

ReferenceError: global is not defined
  at WebUISider (WebUISider.tsx:349:21)

Triggered the error boundary, blanked the sidebar footer area. One-line fix.

Root cause: WebUISider.tsx:349 read global.packageVersion. Under CRA/Webpack, global was auto-aliased to window in browser builds, so the reference worked by coincidence. Vite intentionally leaves global undefined in the browser (nodePolyfills({ globals: { global: false } }) in vite.config.ts), so the same reference now throws.

The value is actually written to globalThis.packageVersion in the root index.html inline script — switching the read site to globalThis matches the writer and works under any bundler.

Verification

• grep -r 'global\.' react/src → only test-file usages remain (where global === globalThis in node/jsdom env)
• Started dev server, logged in as admin, confirmed sidebar renders 26.4.0-alpha.0.<buildNumber> with no ReferenceError in console
• Error boundary no longer activates on the start page

Stack

Logically belongs to FR-2606 (Vite parity work). Sits above the PWA manifest fix.

[nowgnuesLee]

• fix(FR-2529): include backend.ai-client src in craco TS loader and tsconfig paths #6615
• refactor(FR-2529): extract backend.ai-client as standalone workspace package #6614
• fix(FR-2607): align Electron publicPath patch with Vite output #7189
• fix(FR-2606): clear residual build-time warnings (BUI dts + module directive) #7113
• fix(FR-2606): restore Monaco self-hosting in Vite dev server #7104
• feat(FR-2748): fully migrate test files from jest.* to vi.* and align TS globals #7083
• feat(FR-2746): re-enable 5 tests skipped during Vitest migration #7071
• feat(FR-2744): restore Vitest coverage reporting in CI #7065
• fix(FR-2606): replace webpackIgnore with @vite-ignore in dynamic plugin imports #6879
• fix(FR-2606): replace global.packageVersion with globalThis.packageVersion in WebUISider #6878 👈 (View in Graphite)
• fix(FR-2608): disable vite-plugin-pwa auto-manifest to preserve existing manifest.json #6877
• feat(FR-2611): retire Craco/Webpack — build, Makefile, CI all on Vite/Vitest #6876
• feat: migrate BUI + root tests from Jest to Vitest (follow-up to FR-2609) #6875
• feat(FR-2609): Vitest migration for react/ (100% pass, 856/856 tests) #6874
• feat(FR-2608): Vite production build with vite-plugin-pwa service worker #6873
• feat(FR-2610): Vite dev-reload plugin for i18n / theme / config.toml file changes #6872
• feat(FR-2607): Vite spike answer — es6:// Electron publicPath works via renderBuiltUrl #6871
• feat(FR-2606): restore BUI i18n context isolation under Vite dep optimization #6870
• feat(FR-2606): Vite PoC phase 1 — dev server + HMR + React Compiler parity #6869
• main

---

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• flow:merge-queue - adds this PR to the back of the merge queue
• flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull request overview

Fixes a Vite-only runtime crash in the sidebar footer by reading injected build metadata from globalThis instead of global, aligning with how the values are written in index.html.

Changes:

• Replace global.packageVersion with globalThis.packageVersion in WebUISider to avoid ReferenceError: global is not defined under Vite.

[graphite-app]

Merge activity

• Apr 30, 12:11 PM UTC: yomybaby added this pull request to the Graphite merge queue.
• Apr 30, 12:33 PM UTC: Merged by the Graphite merge queue.