🐛 URGENT: Fix production crash from bare console.* stripping by clubanderson · Pull Request #4248 · kubestellar/console

clubanderson · 2026-04-02T14:48:33Z

PRODUCTION IS DOWN

PR #4239 added bare console.log/console.debug to Vite's define config. Vite's define does literal text replacement — it replaced console.log(...) with undefined(...) inside vendor dependencies (Tailwind Merge, etc.), causing:

TypeError: (void 0) is not a function
  at bA (vendor-Dj3uwf9i.js:4724:194)

This crashes the entire app on console.kubestellar.io — blank page, React never mounts.

Fix

Remove bare console.* forms from Vite define. Only use globalThis.console.* which is specific enough to target our code without hitting vendor code.

Verified

grep -c "void 0(" dist/assets/vendor-*.js returns 0 after fix (was crashing before)
Build passes

PR #4239 added bare `console.log`/`console.debug` to Vite's define config to strip debug calls. But Vite's define does LITERAL text replacement — it replaced `console.log(...)` with `undefined(...)` in vendor/dependency code (Tailwind Merge, etc.), causing `TypeError: (void 0) is not a function` and crashing the entire app on console.kubestellar.io. Fix: Only use `globalThis.console.*` form which is specific enough to match our code but not vendor dependencies. Signed-off-by: Andrew Anderson <andy@clubanderson.com>

netlify · 2026-04-02T14:48:39Z

✅ Deploy Preview for kubestellarconsole ready!

Name	Link
🔨 Latest commit	`832e2ee`
🔍 Latest deploy log	https://app.netlify.com/projects/kubestellarconsole/deploys/69ce81c4b1ca9d00088e071d
😎 Deploy Preview	https://deploy-preview-4248.console-deploy-preview.kubestellar.io
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

clubanderson · 2026-04-02T14:48:44Z

/lgtm
/approve

github-actions · 2026-04-02T14:48:45Z

👋 Hey @clubanderson — thanks for opening this PR!

🤖 This project is developed exclusively using AI coding assistants.

Please do not attempt to code anything for this project manually.
All contributions should be authored using an AI coding tool such as:

Claude Code (Opus 4.5 / 4.6) — recommended

GitHub Copilot

Cursor

Other AI coding assistants

This ensures consistency in code style, architecture patterns, test coverage,
and commit quality across the entire codebase.

This is an automated message.

kubestellar-prow · 2026-04-02T14:48:47Z

@clubanderson: you cannot LGTM your own PR.

Details

In response to this:

/lgtm
/approve

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

kubestellar-prow · 2026-04-02T14:48:51Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: clubanderson

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~OWNERS~~ [clubanderson]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

github-actions · 2026-04-02T14:49:00Z

Thank you for your contribution! Your PR has been merged.

Check out what's new:

KubeStellar Console — Live multi-cluster dashboard
Marketplace — Community extensions
Knowledge Base — Troubleshooting and how-tos

Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey

Copilot

Pull request overview

Fixes a production outage caused by Vite define doing literal replacements of bare console.*, which impacted vendor bundles and produced runtime TypeError crashes.

Changes:

Removes bare console.log/info/debug/trace replacements from vite.config.ts to avoid altering vendor/dependency code.
Updates the inline documentation to warn against using bare console.* with define because of unintended vendor replacements.

Copilot · 2026-04-02T14:51:12Z

web/vite.config.ts

      'globalThis.console.log': 'undefined',
      'globalThis.console.info': 'undefined',
      'globalThis.console.debug': 'undefined',
      'globalThis.console.trace': 'undefined',


Defining globalThis.console.* to undefined will turn any matching call site into undefined(...), which throws a TypeError at runtime (the same failure mode described in the PR). Use a callable no-op (or another strategy that removes calls without leaving an uncallable identifier) so any surviving globalThis.console.*(...) becomes a safe no-op instead of a crash.

Suggested change

'globalThis.console.log': 'undefined',

'globalThis.console.info': 'undefined',

'globalThis.console.debug': 'undefined',

'globalThis.console.trace': 'undefined',

'globalThis.console.log': '((..._args) => {})',

'globalThis.console.info': '((..._args) => {})',

'globalThis.console.debug': '((..._args) => {})',

'globalThis.console.trace': '((..._args) => {})',

Copilot · 2026-04-02T14:51:12Z

web/vite.config.ts

+    // Strip console calls in production (replaces terser drop_console).
+    // IMPORTANT: Only use globalThis.console.* — bare "console.log" does
+    // literal text replacement that hits vendor/dependency code too,
+    // turning their console.log(...) into undefined(...) which crashes.
    ...(mode === 'production' ? {
      'globalThis.console.log': 'undefined',
      'globalThis.console.info': 'undefined',
      'globalThis.console.debug': 'undefined',
      'globalThis.console.trace': 'undefined',
-      'console.log': 'undefined',
-      'console.info': 'undefined',
-      'console.debug': 'undefined',
-      'console.trace': 'undefined',
    } : {}),


With the bare console.* defines removed, this config no longer strips typical console.log/info/debug/trace usage in the app (only the rare globalThis.console.* form, which appears unused). Either adjust the comment/intent to reflect that console calls are no longer stripped, or update the stripping approach so it actually targets the app code without touching vendor dependencies.

clubanderson · 2026-04-02T14:51:23Z

🔄 Auto-Applying Copilot Code Review

Copilot code review found 1 code suggestion(s) and 1 general comment(s).

@copilot Please apply all of the following code review suggestions:

web/vite.config.ts (line 58): 'globalThis.console.log': '((..._args) => {})', 'globalThis.console.info':...

Also address these general comments:

web/vite.config.ts (line 59): With the bare console.* defines removed, this config no longer strips typical console.log/info/debug/trace usage in

Push all fixes in a single commit. Run cd web && npm run build && npm run lint before committing.

Auto-generated by copilot-review-apply workflow.

Adds a postbuild script that scans vendor bundles for `void 0(` — the minified form of `undefined()` — which indicates Vite's define config replaced `console.*` calls in dependency code. This would have caught the production crash from PR #4239 / #4248 before deployment. Also removes unused DEV_BAR_HEIGHT_PX import blocking tsc. Signed-off-by: Andrew Anderson <andy@clubanderson.com>

tsc fails on this unused import, preventing Netlify from deploying the production crash fix (#4248). This import was removed in #4244 but re-introduced by #4245. Signed-off-by: Andrew Anderson <andy@clubanderson.com>

…ld (#4253) tsc fails on this unused import, preventing Netlify from deploying the production crash fix (#4248). This import was removed in #4244 but re-introduced by #4245. Signed-off-by: Andrew Anderson <andy@clubanderson.com>

Adds a postbuild script that scans vendor bundles for `void 0(` — the minified form of `undefined()` — which indicates Vite's define config replaced `console.*` calls in dependency code. This would have caught the production crash from PR #4239 / #4248 before deployment. Also removes unused DEV_BAR_HEIGHT_PX import blocking tsc. Signed-off-by: Andrew Anderson <andy@clubanderson.com>

#4252) Adds a postbuild script that scans vendor bundles for `void 0(` — the minified form of `undefined()` — which indicates Vite's define config replaced `console.*` calls in dependency code. This would have caught the production crash from PR #4239 / #4248 before deployment. Also removes unused DEV_BAR_HEIGHT_PX import blocking tsc. Signed-off-by: Andrew Anderson <andy@clubanderson.com>

Copilot AI review requested due to automatic review settings April 2, 2026 14:48

kubestellar-prow bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Apr 2, 2026

kubestellar-prow bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 2, 2026

clubanderson merged commit d1181e1 into main Apr 2, 2026
13 of 16 checks passed

kubestellar-prow bot deleted the fix/production-crash branch April 2, 2026 14:48

kubestellar-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 2, 2026

Copilot started reviewing on behalf of clubanderson April 2, 2026 14:49 View session

Copilot AI reviewed Apr 2, 2026

View reviewed changes

clubanderson assigned Copilot Apr 2, 2026

clubanderson mentioned this pull request Apr 2, 2026

✨ Add post-build vendor safety check to prevent Vite define corruption #4252

Merged

12 tasks

clubanderson mentioned this pull request Apr 2, 2026

🐛 URGENT: Remove unused import blocking Netlify deploy #4253

Merged

clubanderson mentioned this pull request Apr 2, 2026

✨ Expand post-build safety checks: chunks, MSW, bundle size #4272

Merged

10 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🐛 URGENT: Fix production crash from bare console.* stripping#4248

🐛 URGENT: Fix production crash from bare console.* stripping#4248
clubanderson merged 1 commit intomainfrom
fix/production-crash

clubanderson commented Apr 2, 2026

Uh oh!

netlify bot commented Apr 2, 2026 •

edited

Loading

Uh oh!

clubanderson commented Apr 2, 2026

Uh oh!

github-actions bot commented Apr 2, 2026

Uh oh!

kubestellar-prow bot commented Apr 2, 2026

Uh oh!

Uh oh!

kubestellar-prow bot commented Apr 2, 2026

Uh oh!

github-actions bot commented Apr 2, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 2, 2026

Uh oh!

Copilot AI Apr 2, 2026

Uh oh!

clubanderson commented Apr 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

clubanderson commented Apr 2, 2026

PRODUCTION IS DOWN

Fix

Verified

Uh oh!

netlify bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for kubestellarconsole ready!

Uh oh!

clubanderson commented Apr 2, 2026

Uh oh!

github-actions bot commented Apr 2, 2026

Uh oh!

kubestellar-prow bot commented Apr 2, 2026

Uh oh!

Uh oh!

kubestellar-prow bot commented Apr 2, 2026

Uh oh!

github-actions bot commented Apr 2, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 2, 2026

Choose a reason for hiding this comment

Uh oh!

clubanderson commented Apr 2, 2026

🔄 Auto-Applying Copilot Code Review

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

netlify bot commented Apr 2, 2026 •

edited

Loading