pmcf 1.24.0 → 1.25.0

package/bin/pmcf-host-defs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-import { writeFile, mkdir, copyFile, glob } from "node:fs/promises";
+import { writeFile, mkdir, copyFile, glob, chmod } from "node:fs/promises";
 import { join } from "node:path";
 import { Host } from "pmcf";
 import { writeLines, sectionLines } from "../src/utils.mjs";
@@ -151,7 +151,9 @@ async function copySshKeys(host, dir) {
   await mkdir(sshDir, { recursive: true });
 
   for await (const file of glob("ssh_host_*", { cwd: host.directory })) {
-    copyFile(join(host.directory, file), join(sshDir, file));
+    const destinationFileName = join(sshDir, file);
+    await copyFile(join(host.directory, file), destinationFileName);
+    await chmod(destinationFileName, destinationFileName.endsWith('.pub') ? 0o0644 : 0o0600);
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pmcf",
-  "version": "1.24.0",
+  "version": "1.25.0",
   "publishConfig": {
     "access": "public"
   },

package/src/base.mjs

@@ -122,8 +122,14 @@ export class Base {
   }
 
   execFinalize() {
+    this.traverse(object => {
+      //console.log(object.toString());
+      object._finalize();
+    });
+  }
+
+  _finalize() {
     if (this.#finalize) {
-      //this.info("finalize");
       let i = 0;
       for (const action of this.#finalize) {
         if (action) {
@@ -135,6 +141,23 @@ export class Base {
     }
   }
 
+  traverse(visitor, ...args) {
+    const visited = new Set();
+    this._traverse(visited, visitor, ...args);
+  }
+
+  _traverse(visited, visitor, ...args) {
+    if (visited.has(this)) {
+      return false;
+    }
+
+    visited.add(this);
+
+    visitor(this, ...args);
+
+    return true;
+  }
+
   error(...args) {
     console.error(`${this.toString()}:`, ...args);
   }

package/src/model.mjs

@@ -41,18 +41,22 @@ export class Owner extends Base {
       }
     }
     Object.assign(this, data);
+  }
 
-    this.finalize(() => {
+  _traverse(...args) {
+    if (super._traverse(...args)) {
       for (const network of this.#networks.values()) {
-        network.execFinalize();
+        network._traverse(...args);
       }
-    });
 
-    this.finalize(() => {
       for (const host of this.#hosts.values()) {
-        host.execFinalize();
+        host._traverse(...args);
       }
-    });
+
+      return true;
+    }
+
+    return false;
   }
 
   get dns() {
@@ -219,6 +223,18 @@ export class World extends Owner {
     this.addObject(this);
   }
 
+  _traverse(...args) {
+    if (super._traverse(...args)) {
+      for (const object of this.#byName.values()) {
+        object._traverse(...args);
+      }
+
+      return true;
+    }
+
+    return false;
+  }
+
   get fullName() {
     return "";
   }
@@ -429,7 +445,6 @@ export class Host extends Base {
   #deployment;
   #chassis;
   #vendor;
-  #location;
 
   static get typeName() {
     return "host";
@@ -452,11 +467,6 @@ export class Host extends Base {
   constructor(owner, data) {
     super(owner, data);
 
-    if (data.location !== undefined) {
-      this.#location = data.location;
-      delete data.location;
-    }
-
     if (data.deployment !== undefined) {
       this.#deployment = data.deployment;
       delete data.deployment;
@@ -520,12 +530,20 @@ export class Host extends Base {
     }
 
     owner.addHost(this);
+  }
 
-    this.finalize(() => {
+  _traverse(...args) {
+    if (super._traverse(...args)) {
       for (const ni of Object.values(this.networkInterfaces)) {
-        ni.execFinalize();
+        ni._traverse(...args);
       }
-    });
+      for (const service of this.services()) {
+        service._traverse(...args);
+      }
+
+      return true;
+    }
+    return false;
   }
 
   get deployment() {
@@ -552,10 +570,6 @@ export class Host extends Base {
     return this.#extends.map(e => this.expand(e));
   }
 
-  get location() {
-    return this.#location || super.location;
-  }
-
   get provides() {
     let provides = new Set(this.#provides);
     this.extends.forEach(h => (provides = provides.union(h.provides)));
@@ -857,6 +871,5 @@ export class Subnet extends Base {
   }
 }
 
-
 const _types = [Location, Network, Subnet, Host, Service, DNSService];
 const _typesByName = Object.fromEntries(_types.map(t => [t.typeName, t]));

package/types/base.d.mts

@@ -22,6 +22,9 @@ export class Base {
     expand(object: any): any;
     finalize(action: any): void;
     execFinalize(): void;
+    _finalize(): void;
+    traverse(visitor: any, ...args: any[]): void;
+    _traverse(visited: any, visitor: any, ...args: any[]): boolean;
     error(...args: any[]): void;
     info(...args: any[]): void;
     toString(): string;

package/types/model.d.mts

@@ -3,6 +3,7 @@ export class Owner extends Base {
     ntp: {
         servers: any[];
     };
+    _traverse(...args: any[]): boolean;
     get dns(): DNSService;
     hosts(): AsyncGenerator<any, void, unknown>;
     addObject(object: any): void;
@@ -58,6 +59,7 @@ export class Host extends Base {
     static prepareData(world: any, data: any): Promise<typeof Host>;
     networkInterfaces: {};
     postinstall: any[];
+    _traverse(...args: any[]): boolean;
     get deployment(): any;
     get chassis(): any;
     get vendor(): any;