Skip to content

Commit b782511

Browse files
committed
chore(desktop): remove local neu dependency in favor of npx
- Remove `npm install` requirements from README, Dockerfile, and CI. - Update package.json scripts to use `npx -y` for all neutralino commands. - Simplify build instructions to rely on npm scripts abstraction.
1 parent 7b12698 commit b782511

6 files changed

Lines changed: 34 additions & 1128 deletions

File tree

.github/workflows/desktop-build.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,6 @@ jobs:
2121
with:
2222
node-version: "lts/*"
2323

24-
- name: Install dependencies
25-
working-directory: desktop-app
26-
run: npm install
27-
2824
- name: Build all binaries (embedded + portable)
2925
working-directory: desktop-app
3026
run: npm run build:all

desktop-app/Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ WORKDIR /app/desktop-app
1717
# Copy only the desktop-app directory with prepared resources
1818
COPY --from=prepare /app/desktop-app .
1919

20-
# Install pinned neu CLI from package.json, then build both embedded + portable
21-
RUN npm install --ignore-scripts && npm run build:all
20+
# Build both embedded + portable
21+
RUN npm run build:all
2222

2323
# Final stage: Export the dist artifacts
2424
FROM alpine:lts

desktop-app/README.md

Lines changed: 3 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -20,26 +20,15 @@ Desktop-only files (not generated):
2020

2121
### Setup
2222

23-
```bash
24-
npm install
25-
```
23+
No installation is required. The app is built and run using `npx` (via npm scripts).
2624

2725
### Running the app
2826

2927
```bash
3028
npm run dev
3129
```
3230

33-
<details>
34-
<summary>npx alternative (no install required)</summary>
35-
36-
```bash
37-
node prepare.js && npx @neutralinojs/neu run
38-
```
39-
40-
</details>
41-
42-
This automatically runs `prepare.js` (via the `predev` hook) before starting the app. Hot-reload is enabled by default. Enable the browser inspector by setting `"enableInspector": true` in `neutralino.config.json`.
31+
This automatically runs `prepare.js` before starting the app. Hot-reload is enabled by default. Enable the browser inspector by setting `"enableInspector": true` in `neutralino.config.json`.
4332

4433
For more information, see the [Neutralinojs documentation](https://neutralino.js.org/docs/cli/neu-cli#installation).
4534

@@ -51,45 +40,25 @@ For more information, see the [Neutralinojs documentation](https://neutralino.js
5140
npm run build
5241
```
5342

54-
<details>
55-
<summary>npx alternative</summary>
56-
57-
```bash
58-
node prepare.js && npx @neutralinojs/neu build --embed-resources
59-
```
60-
61-
</details>
62-
6343
**Portable** — ZIP bundle with separate `resources.neu` file:
6444

6545
```bash
6646
npm run build:portable
6747
```
6848

69-
<details>
70-
<summary>npx alternative</summary>
71-
72-
```bash
73-
node prepare.js && npx @neutralinojs/neu build --release
74-
```
75-
76-
</details>
77-
7849
**Both** — Build embedded + portable in one step:
7950

8051
```bash
8152
npm run build:all
8253
```
8354

84-
> **Note:** All build commands auto-run `prepare.js` via the `prebuild` hook — no need to run it manually.
85-
8655
Build output is placed in `dist/`.
8756

8857
For more information, see the [Neutralinojs documentation](https://neutralino.js.org/docs/cli/neu-cli#neu-build).
8958

9059
### Building with Docker
9160

92-
Build binaries without installing Node.js or the `neu` CLI locally:
61+
Build binaries without installing Node.js locally:
9362

9463
```bash
9564
docker compose up --build

0 commit comments

Comments
 (0)