node-app-build 5.26.2

Mini app developer CLI: scaffold, validate, package node-app-* Debian packages
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
//! Lightweight manifest.json parser used by `validate` and `package`.
//!
//! The full manifest type model lives in `core/domain/src/models/app_manifest.rs`
//! (T012). To avoid coupling the CLI to the full domain crate during early
//! development (and to keep `node-app` cheap to build for first-time
//! contributors), we re-implement the small subset of fields the CLI needs.
//! When the domain crate stabilises, this module will switch to depending on
//! `node-domain` and re-exporting its types.

use anyhow::{anyhow, bail, Context, Result};
use regex::Regex;
use serde::{Deserialize, Serialize};
use std::path::Path;

#[derive(Debug, Clone, Serialize, Deserialize)]
pub struct AppManifest {
    #[serde(default = "default_manifest_version")]
    pub manifest_version: u8,
    pub name: String,
    pub version: String,
    pub app_type: AppType,
    #[serde(default)]
    pub abi: Option<String>,
    #[serde(default)]
    pub entrypoint: Option<String>,
    #[serde(default)]
    pub hot_reload: Option<String>,
    #[serde(default)]
    pub description: Option<String>,
    #[serde(default)]
    pub has_ui: bool,
    #[serde(default)]
    pub ui_path: Option<String>,
    #[serde(default)]
    pub capabilities: Option<ManifestCapabilities>,
    /// Standalone-app block: socket path the daemon binds. Required when
    /// `app_type == "standalone"` AND the manifest declares any capability
    /// providers. Validated by `validate_manifest`.
    #[serde(default)]
    pub standalone: Option<StandaloneConfig>,
    /// TCP block (feature 470). When `preferred_port` is set, the deb's
    /// postinst/prerm hook into the port registry via `node-ctl`.
    #[serde(default)]
    pub tcp: Option<TcpConfig>,
}

#[derive(Debug, Clone, Serialize, Deserialize)]
pub struct StandaloneConfig {
    pub socket_path: String,
}

#[derive(Debug, Clone, Serialize, Deserialize)]
pub struct TcpConfig {
    #[serde(default)]
    pub preferred_port: Option<u16>,
}

fn default_manifest_version() -> u8 {
    1
}

#[derive(Debug, Copy, Clone, PartialEq, Eq, Serialize, Deserialize)]
#[serde(rename_all = "lowercase")]
pub enum AppType {
    Native,
    Bun,
    Standalone,
}

#[derive(Debug, Clone, Default, Serialize, Deserialize)]
pub struct ManifestCapabilities {
    #[serde(default)]
    pub requires: Vec<String>,
    #[serde(default)]
    pub provides: Vec<String>,
}

pub fn parse_manifest(path: &Path) -> Result<AppManifest> {
    let raw = std::fs::read_to_string(path)
        .with_context(|| format!("read {}", path.display()))?;
    let manifest: AppManifest = serde_json::from_str(&raw)
        .with_context(|| format!("parse {} as manifest.json", path.display()))?;
    Ok(manifest)
}

/// Run all v2-aware validations on a manifest. Returns Ok(()) on full pass,
/// or an error describing the first failure.
///
/// `is_apt_install_target` indicates whether the package is intended to be
/// distributed at the apt path (`/usr/lib/node/apps/<name>/`). When true,
/// native (cdylib) manifests are rejected (T118 — path-based tier rule, R8).
pub fn validate_manifest(m: &AppManifest, is_apt_install_target: bool) -> Result<()> {
    // Name pattern (lowercase alphanumeric + hyphens, with optional publisher prefix).
    static NAME_RE: once_cell::sync::Lazy<Regex> = once_cell::sync::Lazy::new(|| {
        Regex::new(r"^[a-z][a-z0-9-]*(/([a-z][a-z0-9-]*))?$").unwrap()
    });
    if !NAME_RE.is_match(&m.name) {
        bail!(
            "manifest name '{}' is invalid (expected lowercase + hyphens, optional publisher/ prefix)",
            m.name
        );
    }
    if m.name.starts_with("node-app-") {
        bail!("manifest name must not start with 'node-app-' (the .deb script adds the prefix)");
    }

    // SemVer-ish.
    static SEMVER_RE: once_cell::sync::Lazy<Regex> = once_cell::sync::Lazy::new(|| {
        Regex::new(r"^[0-9]+\.[0-9]+\.[0-9]+([-+][0-9A-Za-z.+-]+)?$").unwrap()
    });
    if !SEMVER_RE.is_match(&m.version) {
        bail!("manifest version '{}' is not a valid SemVer", m.version);
    }

    // v2 manifests must declare an ABI in {v1}.
    if m.manifest_version >= 2 {
        let abi = m.abi.as_deref().ok_or_else(|| {
            anyhow!("manifest_version=2 requires an `abi` field")
        })?;
        if abi != "v1" {
            bail!("unsupported abi '{}'; only 'v1' is recognised by the runtime", abi);
        }
    }

    // hot_reload values.
    if let Some(hr) = m.hot_reload.as_deref() {
        match hr {
            "supported" | "experimental" | "unsupported" => {}
            other => bail!(
                "hot_reload '{}' is invalid (expected supported|experimental|unsupported)",
                other
            ),
        }
    }

    // Path safety on entrypoint and ui_path. Same rule both fields:
    // - Match path-component regex.
    // - No '..' segments.
    // - No leading '/'.
    static PATH_RE: once_cell::sync::Lazy<Regex> = once_cell::sync::Lazy::new(|| {
        Regex::new(r"^[a-zA-Z0-9_][a-zA-Z0-9_./-]*$").unwrap()
    });
    let check_path = |label: &str, p: &str| -> Result<()> {
        if !PATH_RE.is_match(p) {
            bail!("{} path '{}' has illegal characters", label, p);
        }
        if p.split('/').any(|seg| seg == "..") {
            bail!("{} path '{}' contains '..' segment", label, p);
        }
        if p.starts_with('/') {
            bail!("{} path '{}' must be relative", label, p);
        }
        Ok(())
    };
    if let Some(ep) = m.entrypoint.as_deref() {
        check_path("entrypoint", ep)?;
    }
    if m.has_ui {
        let ui = m.ui_path.as_deref().ok_or_else(|| {
            anyhow!("has_ui=true but ui_path is missing")
        })?;
        check_path("ui_path", ui)?;
    }

    // Capability strings parse correctly.
    if let Some(caps) = &m.capabilities {
        for c in &caps.requires {
            parse_capability_requirement(c)
                .with_context(|| format!("invalid capability requirement: '{}'", c))?;
        }
        for c in &caps.provides {
            parse_capability_provides(c)
                .with_context(|| format!("invalid capability provides: '{}'", c))?;
        }
    }

    // Tier check: native (cdylib) cannot install at the apt path.
    if is_apt_install_target && m.app_type == AppType::Native {
        bail!(
            "native (cdylib) apps cannot ship at the apt-install path \
             (/usr/lib/node/apps/). Native apps must be bundled with `node`. \
             Switch app_type to 'bun' or coordinate with the project to add a \
             bundled-app slot."
        );
    }

    // Standalone-app rules (mirror core/domain/src/models/app_manifest.rs):
    //   - app_type == Standalone with any `provides` requires `standalone.socket_path`,
    //     absolute, under /run/, no `..` segments.
    //   - Non-standalone manifests MUST NOT carry a `standalone` block.
    let provides_count = m
        .capabilities
        .as_ref()
        .map(|c| c.provides.len())
        .unwrap_or(0);
    match m.app_type {
        AppType::Standalone => {
            if provides_count > 0 {
                let cfg = m.standalone.as_ref().ok_or_else(|| {
                    anyhow!(
                        "standalone apps that declare 'provides' require a \
                         'standalone.socket_path' field"
                    )
                })?;
                validate_standalone_socket_path(&cfg.socket_path)
                    .context("standalone.socket_path invalid")?;
            }
        }
        AppType::Native | AppType::Bun => {
            if m.standalone.is_some() {
                bail!(
                    "'standalone' block is only valid when app_type == 'standalone' \
                     (found app_type='{:?}')",
                    m.app_type
                );
            }
        }
    }

    Ok(())
}

/// Validate a standalone socket path string.
///
/// Rules:
///   1. Absolute path (starts with `/`).
///   2. Lives under `/run/` (rejects `/etc/...`, `/tmp/...`, etc.).
///   3. No `..` segments.
fn validate_standalone_socket_path(p: &str) -> Result<()> {
    if !p.starts_with('/') {
        bail!("socket_path '{}' must be absolute", p);
    }
    if !p.starts_with("/run/") {
        bail!("socket_path '{}' must live under /run/", p);
    }
    if p.split('/').any(|seg| seg == "..") {
        bail!("socket_path '{}' contains '..' segment", p);
    }
    Ok(())
}

/// Tiny capability-requirement parser used during validate. Mirrors the
/// grammar in `core/domain/src/models/capability.rs::parse_capability` —
/// we keep this in sync manually for now (single source of truth lives in
/// the domain crate; CI cross-checks the two parsers in T128).
fn parse_capability_requirement(s: &str) -> Result<()> {
    let mut parts = s.split(':');
    let ns = parts
        .next()
        .ok_or_else(|| anyhow!("empty capability"))?
        .trim();
    if ns.is_empty() {
        bail!("capability namespace is empty");
    }
    static NS_RE: once_cell::sync::Lazy<Regex> = once_cell::sync::Lazy::new(|| {
        Regex::new(r"^[a-z][a-z0-9-]*(\.[a-z][a-z0-9-]*)+$").unwrap()
    });
    if !NS_RE.is_match(ns) {
        bail!(
            "capability namespace '{}' must be dotted lowercase (e.g. core.lightning.payment.send)",
            ns
        );
    }
    let mut seen_daily = false;
    for part in parts {
        if let Some(rest) = part.strip_prefix("max=") {
            // <N>(sat|msat)/(day|tx)
            let (num_unit, period) = rest
                .split_once('/')
                .ok_or_else(|| anyhow!("max constraint missing /period: '{}'", part))?;
            let (num, unit) = num_unit
                .strip_suffix("msat")
                .map(|n| (n, "msat"))
                .or_else(|| num_unit.strip_suffix("sat").map(|n| (n, "sat")))
                .ok_or_else(|| anyhow!("max value must end in 'sat' or 'msat': '{}'", num_unit))?;
            let _: u64 = num
                .parse()
                .with_context(|| format!("max value '{}' must be an integer", num))?;
            match (unit, period) {
                ("sat", "day") | ("msat", "day") => {
                    if seen_daily {
                        bail!("duplicate max=Nsat/day constraint on '{}'", ns);
                    }
                    seen_daily = true;
                }
                ("sat", "tx") | ("msat", "tx") => {}
                _ => bail!(
                    "unsupported constraint period '{}' (expected day or tx)",
                    period
                ),
            }
        } else {
            // Plain `:scope` token — accepted, no semantic validation here.
            if part.is_empty() {
                bail!("empty constraint segment in '{}'", s);
            }
        }
    }
    Ok(())
}

fn parse_capability_provides(s: &str) -> Result<()> {
    static NS_RE: once_cell::sync::Lazy<Regex> = once_cell::sync::Lazy::new(|| {
        Regex::new(r"^[a-z][a-z0-9-]*(\.[a-z][a-z0-9-]*)+$").unwrap()
    });
    if !NS_RE.is_match(s) {
        bail!("capability provides '{}' must be dotted lowercase", s);
    }
    Ok(())
}

#[cfg(test)]
mod tests {
    use super::*;

    fn min_v2(name: &str, app_type: AppType) -> AppManifest {
        AppManifest {
            manifest_version: 2,
            name: name.into(),
            version: "1.0.0".into(),
            app_type,
            abi: Some("v1".into()),
            entrypoint: Some(match app_type {
                AppType::Native => "app.so".into(),
                AppType::Bun => "dist/index.js".into(),
                AppType::Standalone => "app".into(),
            }),
            hot_reload: None,
            description: None,
            has_ui: false,
            ui_path: None,
            capabilities: None,
            standalone: None,
            tcp: None,
        }
    }

    #[test]
    fn accepts_v1_manifest_without_abi() {
        let mut m = min_v2("foo", AppType::Bun);
        m.manifest_version = 1;
        m.abi = None;
        validate_manifest(&m, true).unwrap();
    }

    #[test]
    fn rejects_v2_without_abi() {
        let mut m = min_v2("foo", AppType::Bun);
        m.abi = None;
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains("requires an `abi`"));
    }

    #[test]
    fn rejects_invalid_name() {
        let mut m = min_v2("FOO", AppType::Bun);
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains("invalid"));
        m.name = "node-app-bar".into();
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains("must not start with 'node-app-'"));
    }

    #[test]
    fn rejects_traversal_in_entrypoint() {
        let mut m = min_v2("foo", AppType::Bun);
        m.entrypoint = Some("../etc/passwd".into());
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains(".."));
    }

    #[test]
    fn rejects_native_at_apt_path() {
        let m = min_v2("foo", AppType::Native);
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains("native"));
    }

    #[test]
    fn accepts_native_when_bundled() {
        let m = min_v2("foo", AppType::Native);
        validate_manifest(&m, false).unwrap();
    }

    #[test]
    fn accepts_standalone_at_apt_path() {
        // Standalone apps are distributed via apt as binaries — allowed at apt path.
        let m = min_v2("my-service", AppType::Standalone);
        validate_manifest(&m, true).unwrap();
    }

    #[test]
    fn standalone_with_provides_requires_socket_path() {
        let mut m = min_v2("led", AppType::Standalone);
        m.capabilities = Some(ManifestCapabilities {
            requires: vec![],
            provides: vec!["led.event.set".into()],
        });
        // No `standalone` block → rejected.
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains("socket_path"));
    }

    #[test]
    fn standalone_with_provides_and_socket_path_passes() {
        let mut m = min_v2("led", AppType::Standalone);
        m.capabilities = Some(ManifestCapabilities {
            requires: vec![],
            provides: vec!["led.event.set".into()],
        });
        m.standalone = Some(StandaloneConfig {
            socket_path: "/run/node-app-led.sock".into(),
        });
        validate_manifest(&m, true).unwrap();
    }

    #[test]
    fn rejects_socket_path_outside_run() {
        let mut m = min_v2("led", AppType::Standalone);
        m.capabilities = Some(ManifestCapabilities {
            requires: vec![],
            provides: vec!["led.event.set".into()],
        });
        m.standalone = Some(StandaloneConfig {
            socket_path: "/tmp/led.sock".into(),
        });
        let err = validate_manifest(&m, true).unwrap_err();
        // `.context()` wraps the underlying message; format with `{:#}` to
        // get the full chain in one string.
        let msg = format!("{:#}", err);
        assert!(msg.contains("/run/"), "expected /run/ in chain: {msg}");
    }

    #[test]
    fn rejects_relative_socket_path() {
        let mut m = min_v2("led", AppType::Standalone);
        m.capabilities = Some(ManifestCapabilities {
            requires: vec![],
            provides: vec!["led.event.set".into()],
        });
        m.standalone = Some(StandaloneConfig {
            socket_path: "led.sock".into(),
        });
        let err = validate_manifest(&m, true).unwrap_err();
        let msg = format!("{:#}", err);
        assert!(msg.contains("absolute"), "expected 'absolute' in chain: {msg}");
    }

    #[test]
    fn rejects_standalone_block_on_non_standalone_app() {
        let mut m = min_v2("foo", AppType::Bun);
        m.standalone = Some(StandaloneConfig {
            socket_path: "/run/foo.sock".into(),
        });
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains("standalone"));
    }

    #[test]
    fn standalone_without_provides_skips_socket_check() {
        // A standalone app that only consumes capabilities (no provides) can
        // omit the standalone.socket_path field entirely.
        let m = min_v2("consumer", AppType::Standalone);
        validate_manifest(&m, true).unwrap();
    }

    #[test]
    fn has_ui_requires_ui_path_for_standalone_bun_fullstack() {
        // Bun standalone-fullstack: has_ui=true requires ui_path. Already
        // enforced by the generic check, but assert it explicitly.
        let mut m = min_v2("bun-fs", AppType::Standalone);
        m.has_ui = true;
        m.ui_path = None;
        let err = validate_manifest(&m, true).unwrap_err();
        assert!(err.to_string().contains("ui_path"));

        m.ui_path = Some("ui-dist".into());
        validate_manifest(&m, true).unwrap();
    }

    #[test]
    fn parses_constraint_strings() {
        parse_capability_requirement("core.storage.kv").unwrap();
        parse_capability_requirement("core.lightning.payment.send:max=500sat/day").unwrap();
        parse_capability_requirement("core.lightning.payment.send:max=1000msat/tx").unwrap();
        parse_capability_requirement(
            "core.lightning.payment.send:max=500sat/day:max=10000msat/tx",
        )
        .unwrap();
        assert!(parse_capability_requirement("CORE.bad").is_err());
        assert!(parse_capability_requirement("core.bad:max=foo/day").is_err());
        assert!(parse_capability_requirement("core.bad:max=10sat/year").is_err());
        // duplicate daily caps rejected
        assert!(parse_capability_requirement(
            "core.bad:max=10sat/day:max=20sat/day"
        )
        .is_err());
    }
}