Skip to content

Added DemoTime Browser Extension (dev) #374

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
arijsdijk wants to merge 1 commit into
base: dev
Choose a base branch
from

Added DemoTime Browser Extension (dev)

f2d6ea4
Select commit
Loading
Failed to load commit list.
Open

Added DemoTime Browser Extension (dev) #374

Added DemoTime Browser Extension (dev)
f2d6ea4
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Correctness Check completed Jan 18, 2026 in 1m 51s

4 issues identified (11 code objects reviewed).

• Merge Base: 659efb3
• Head: f2d6ea4

Details

✅ File Path Comments Posted
✅ apps/browser-extension/extension.config.js 0
❌ apps/browser-extension/scripts/home.js 1
❌ apps/browser-extension/scripts/scenes.js 1
❌ apps/browser-extension/scripts/settings.js 1
❌ apps/browser-extension/scripts/theme.js 1

Filtered Issues Details

apps/browser-extension/extension.config.js
  • line 4: If config is null or undefined, accessing config.plugins on line 4 will throw a TypeError. While the extension framework likely guarantees a valid config object, there's no defensive check. [ Low confidence ]
apps/browser-extension/scripts/theme.js
  • line 43: In main(), calling loadTheme() on line 43 returns a Promise that is neither awaited nor has a .catch() handler. While loadTheme has internal error handling, if an unexpected error occurs outside the try/catch (e.g., in applyTheme), it would result in an unhandled promise rejection. [ Low confidence ]
View more details on MacroscopeApp