Skip to content

Fix Neutralino resource path so release loads OpenWhisper UI#1

Open
weroperking wants to merge 1 commit intomainfrom
codex/diagnose-tool-not-working-after-build
Open

Fix Neutralino resource path so release loads OpenWhisper UI#1
weroperking wants to merge 1 commit intomainfrom
codex/diagnose-tool-not-working-after-build

Conversation

@weroperking
Copy link
Copy Markdown
Owner

@weroperking weroperking commented Feb 26, 2026

Motivation

  • Packaged Neutralino apps were resolving resources to an absolute /resources path which can cause the default Neutralino starter page to be shown instead of the project's UI, so the resources path must be project-relative to ensure resources/index.html is found in releases.

Description

  • Updated neutralino.config.json to set cli.resourcesPath from /resources to ./resources/ and normalized the nativeAllowList formatting so packaged builds resolve the app's resources/ folder correctly.

Testing

  • Executed npx neu build --release to validate packaging; the build run in this environment emitted a CLI warning (Cannot read properties of undefined (reading 'window')) but the config file now contains the updated cli.resourcesPath and the packaging step reports the application package was generated (resources path fix applied).

Codex Task

Summary by CodeRabbit

  • Chores
    • Configuration file formatting improvements for better maintainability.
    • Updated resource path configuration to use a more flexible directory structure.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fdba365 and 0c146f7.

📒 Files selected for processing (1)
  • neutralino.config.json

📝 Walkthrough

Walkthrough

neutralino.config.json configuration file updated with two modifications: the native allow list was reformatted into a multi-line array structure, and the CLI resources path was changed from an absolute path to a relative path reference.

Changes

Cohort / File(s) Summary
Configuration Updates
neutralino.config.json
Native allow list reformatted from single-line to multi-line array (no semantic change). CLI resources path updated from absolute path "/resources" to relative path "./resources/" for resource lookup.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A config file so neat and tidy,
Paths now relative, not rigid,
Native entries line by line,
Resources found with paths divine,
Changes simple, clean, and fine! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically identifies the main change: fixing the Neutralino resource path configuration to enable the OpenWhisper UI to load in releases.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/diagnose-tool-not-working-after-build

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant