npm-pkgbuild 22.1.18 → 22.1.19

package/README.md

@@ -97,49 +97,49 @@ See [mf-hosting](https://www.npmjs.com/package/mf-hosting) or [mf-hosting-fronte
     *   [Properties](#properties)
 *   [extractFromPackage](#extractfrompackage)
     *   [Parameters](#parameters-1)
-*   [NPMPackContentProvider](#npmpackcontentprovider)
+*   [BUILDAH](#buildah)
+*   [ContentProvider](#contentprovider)
     *   [Parameters](#parameters-2)
     *   [Properties](#properties-1)
-    *   [name](#name)
-*   [NodeModulesContentProvider](#nodemodulescontentprovider)
-    *   [Parameters](#parameters-3)
-    *   [Properties](#properties-2)
+    *   [propertiesFor](#propertiesfor)
+        *   [Parameters](#parameters-3)
     *   [asyncIterator](#asynciterator)
-    *   [name](#name-1)
 *   [FileContentProvider](#filecontentprovider)
     *   [Parameters](#parameters-4)
     *   [asyncIterator](#asynciterator-1)
-    *   [name](#name-2)
-*   [NFTContentProvider](#nftcontentprovider)
+    *   [name](#name)
+*   [NodeModulesContentProvider](#nodemodulescontentprovider)
     *   [Parameters](#parameters-5)
+    *   [Properties](#properties-2)
     *   [asyncIterator](#asynciterator-2)
-    *   [name](#name-3)
-*   [ContentProvider](#contentprovider)
+    *   [name](#name-1)
+*   [NPMPackContentProvider](#npmpackcontentprovider)
     *   [Parameters](#parameters-6)
     *   [Properties](#properties-3)
-    *   [propertiesFor](#propertiesfor)
-        *   [Parameters](#parameters-7)
+    *   [name](#name-2)
+*   [NFTContentProvider](#nftcontentprovider)
+    *   [Parameters](#parameters-7)
     *   [asyncIterator](#asynciterator-3)
+    *   [name](#name-3)
 *   [DEBIAN](#debian)
     *   [hookMapping](#hookmapping)
     *   [attributes](#attributes)
     *   [prepare](#prepare)
         *   [Parameters](#parameters-8)
-*   [pkgKeyValuePairOptions](#pkgkeyvaluepairoptions)
-*   [attributes](#attributes-1)
 *   [quoteFile](#quotefile)
     *   [Parameters](#parameters-9)
 *   [RPM](#rpm)
     *   [hookMapping](#hookmapping-1)
-    *   [attributes](#attributes-2)
+    *   [attributes](#attributes-1)
     *   [prepare](#prepare-1)
         *   [Parameters](#parameters-10)
+*   [pkgKeyValuePairOptions](#pkgkeyvaluepairoptions)
+*   [attributes](#attributes-2)
 *   [OCI](#oci)
 *   [DOCKER](#docker)
     *   [attributes](#attributes-3)
     *   [prepare](#prepare-2)
         *   [Parameters](#parameters-11)
-*   [BUILDAH](#buildah)
 *   [Packager](#packager)
     *   [Parameters](#parameters-12)
     *   [packageName](#packagename)
@@ -257,39 +257,35 @@ Extract package definition from package.json.
 
 Returns **AsyncIterable<[PackageDefinition](#packagedefinition)>**&#x20;
 
-## NPMPackContentProvider
+## BUILDAH
 
-**Extends ContentProvider**
+**Extends DOCKER**
 
-Content from npm pack.
+Use buildah @see <https://buildah.io>
+
+## ContentProvider
+
+Source of package content.
 
 ### Parameters
 
 *   `definitions` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**&#x20;
-*   `entryProperties` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** to be set for each entry
 
 ### Properties
 
 *   `dir` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**&#x20;
+*   `destination` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**&#x20;
+*   `defaultProperties` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**&#x20;
+*   `permissions` **([Map](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Map)<[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object), [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)> | [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object))**&#x20;
 
-### name
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** name of the content provider
-
-## NodeModulesContentProvider
-
-**Extends ContentProvider**
-
-Content from node\_modules.
-Requires .npmrc or NPM\_TOKEN environment
-
-### Parameters
+### propertiesFor
 
-*   `definitions` &#x20;
+#### Parameters
 
-### Properties
+*   `name` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**&#x20;
+*   `isCollection` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)**&#x20;
 
-*   `withoutDevelpmentDependencies` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)**&#x20;
+Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**&#x20;
 
 ### asyncIterator
 
@@ -297,10 +293,6 @@ List all entries.
 
 Returns **AsyncIterable<(ContentEntry | CollectionEntry)>** all entries
 
-### name
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** name of the content provider
-
 ## FileContentProvider
 
 **Extends ContentProvider**
@@ -319,53 +311,67 @@ Returns **AsyncIterable<(ContentEntry | CollectionEntry)>** all entries
 
 Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** name of the content provider
 
-## NFTContentProvider
+## NodeModulesContentProvider
 
 **Extends ContentProvider**
 
-Content provided form the file system.
+Content from node\_modules.
+Requires .npmrc or NPM\_TOKEN environment
 
 ### Parameters
 
-*   `definitions` **([Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object) | [string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String))**&#x20;
+*   `definitions` &#x20;
+
+### Properties
+
+*   `withoutDevelpmentDependencies` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)**&#x20;
 
 ### asyncIterator
 
-Returns **AsyncIterable\<ContentEntry>** all entries
+List all entries.
+
+Returns **AsyncIterable<(ContentEntry | CollectionEntry)>** all entries
 
 ### name
 
 Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** name of the content provider
 
-## ContentProvider
+## NPMPackContentProvider
 
-Source of package content.
+**Extends ContentProvider**
+
+Content from npm pack.
 
 ### Parameters
 
 *   `definitions` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**&#x20;
+*   `entryProperties` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** to be set for each entry
 
 ### Properties
 
 *   `dir` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**&#x20;
-*   `destination` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**&#x20;
-*   `defaultProperties` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**&#x20;
-*   `permissions` **([Map](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Map)<[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object), [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)> | [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object))**&#x20;
 
-### propertiesFor
+### name
 
-#### Parameters
+Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** name of the content provider
 
-*   `name` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)**&#x20;
-*   `isCollection` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)**&#x20;
+## NFTContentProvider
 
-Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**&#x20;
+**Extends ContentProvider**
+
+Content provided form the file system.
+
+### Parameters
+
+*   `definitions` **([Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object) | [string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String))**&#x20;
 
 ### asyncIterator
 
-List all entries.
+Returns **AsyncIterable\<ContentEntry>** all entries
 
-Returns **AsyncIterable<(ContentEntry | CollectionEntry)>** all entries
+### name
+
+Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** name of the content provider
 
 ## DEBIAN
 
@@ -393,15 +399,6 @@ Map install hook named from default (arch) to deb.
 
 Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Promise)<[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)>**&#x20;
 
-## pkgKeyValuePairOptions
-
-Type: [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)
-
-## attributes
-
-well known package properties
-<https://www.archlinux.org/pacman/PKGBUILD.5.html>
-
 ## quoteFile
 
 ### Parameters
@@ -435,6 +432,15 @@ Check for rpmbuild presence.
 
 Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Promise)<[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)>** true when rpmbuild executable is present
 
+## pkgKeyValuePairOptions
+
+Type: [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)
+
+## attributes
+
+well known package properties
+<https://www.archlinux.org/pacman/PKGBUILD.5.html>
+
 ## OCI
 
 **Extends Packager**
@@ -464,12 +470,6 @@ Check for docker presence.
 
 Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Promise)<[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)>** true when docker executable is present
 
-## BUILDAH
-
-**Extends DOCKER**
-
-Use buildah @see <https://buildah.io>
-
 ## Packager
 
 ### Parameters

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-pkgbuild",
-  "version": "22.1.18",
+  "version": "22.1.19",
   "publishConfig": {
     "access": "public",
     "provenance": false
@@ -52,16 +52,15 @@
     "aggregate-async-iterator": "^1.2.5",
     "commander": "^15.0.0",
     "compare-versions": "^6.1.1",
-    "content-entry": "^15.0.2",
-    "content-entry-filesystem": "^9.1.2",
-    "content-entry-transform": "^1.7.6",
+    "content-entry": "^15.0.3",
+    "content-entry-filesystem": "^9.1.3",
+    "content-entry-transform": "^1.7.8",
     "execa": "^10.0.1",
     "ini": "^7.0.0",
-    "iterable-string-interceptor": "^3.0.8",
     "key-value-transformer": "^3.3.2",
     "npm-package-walker": "^8.0.11",
     "npm-packlist": "^11.3.0",
-    "pacc": "^11.13.0",
+    "pacc": "^11.15.0",
     "package-directory": "^8.2.0",
     "pacote": "^22.0.0",
     "picomatch": "^4.0.7",
@@ -69,7 +68,7 @@
     "uti": "^8.11.3"
   },
   "devDependencies": {
-    "@types/node": "^26.6.1",
+    "@types/node": "^26.6.2",
     "ava": "^8.0.1",
     "c8": "^12.0.0",
     "documentation": "^14.0.3",

package/src/output/alpm.mjs

@@ -15,13 +15,14 @@ import {
   equalSeparatedKeyValuePairOptions,
   Uint8ArraysToLines
 } from "key-value-transformer";
+import { Packager } from "./packager.mjs";
 import {
-  Packager,
   pkgbuild_version_attribute,
   pkgbuild_description_attribute,
   pkgbuild_name_attribute,
-  dependency_attribute_collection_writable
-} from "./packager.mjs";
+  dependency_attribute_collection_writable,
+  arch_attribute_writable
+} from "../types.mjs";
 import {
   copyEntries,
   fieldProvider,
@@ -88,88 +89,154 @@ export class ALPM extends Packager {
    * https://www.archlinux.org/pacman/PKGBUILD.5.html
    */
   static attributes = {
-    Maintainer: {
+    name: {
+      ...pkgbuild_name_attribute,
+      collection: true,
+      externalName: "pkgname"
+    },
+    description: { ...pkgbuild_description_attribute, externalName: "pkgdesc" },
+    version: { ...pkgbuild_version_attribute, externalName: "pkgver" },
+    maintainer: {
       ...string_collection_attribute_writable,
-      name: "Maintainer",
-      alias: "maintainer",
-      prefix: "# "
+      externalName: "Maintainer",
+      name: "maintainer",
+      prefix: "# ",
+      skipEmpty: true
     },
     packager: {
       ...string_collection_attribute_writable,
       name: "packager",
-      alias: "maintainer"
+      alias: "maintainer",
+      skipEmpty: true
     },
-    pkgname: { ...pkgbuild_name_attribute, name: "pkgname", collection: true },
-    pkgver: { ...pkgbuild_version_attribute, name: "pkgver" },
-    pkgrel: {
+    release: {
       ...integer_attribute,
-      name: "pkgrel",
-      alias: "release",
+      name: "release",
+      externalName: "pkgrel",
       default: 1,
       mandatory: true
     },
     epoch: { ...integer_attribute, name: "epoch", default: 0 },
-    pkgdesc: { ...pkgbuild_description_attribute, name: "pkgdesc" },
-    url: { ...string_attribute, name: "url", alias: "homepage" },
+    url: {
+      ...string_attribute,
+      name: "url",
+      alias: "homepage",
+      skipEmpty: true
+    },
     license: {
       ...string_collection_attribute_writable,
+      separator: undefined,
       name: "license",
-      mandatory: true
+      skipEmpty: true
+    },
+    install: { ...string_attribute, name: "install", skipEmpty: true },
+    changelog: { ...string_attribute, name: "changelog", skipEmpty: true },
+    source: {
+      ...string_collection_attribute_writable,
+      name: "source",
+      separator: undefined,
+      skipEmpty: true
     },
-    install: { ...string_attribute, name: "install" },
-    changelog: { ...string_attribute, name: "changelog" },
-    source: { ...string_collection_attribute_writable, name: "source" },
     validpgpkeys: {
       ...string_collection_attribute_writable,
-      name: "validpgpkeys"
+      name: "validpgpkeys",
+      separator: undefined,
+      skipEmpty: true
     },
-    noextract: { ...default_attribute, name: "noextract" },
-    cksums: { ...string_collection_attribute_writable, name: "cksums" },
-    md5sums: { ...string_collection_attribute_writable, name: "md5sums" },
-    sha1sums: { ...string_collection_attribute_writable, name: "sha1sums" },
-    sha256sums: { ...string_collection_attribute_writable, name: "sha256sums" },
-    sha384sums: { ...string_collection_attribute_writable, name: "sha384sums" },
-    sha512sums: { ...string_collection_attribute_writable, name: "sha512sums" },
-    groups: { ...string_collection_attribute_writable, name: "groups" },
-    arch: {
+    noextract: { ...default_attribute, name: "noextract", skipEmpty: true },
+    cksums: {
       ...string_collection_attribute_writable,
-      name: "arch",
-      default: ["any"],
-      mandatory: true
+      name: "cksums",
+      skipEmpty: true
+    },
+    md5sums: {
+      ...string_collection_attribute_writable,
+      name: "md5sums",
+      separator: undefined,
+      default: ["SKIP"]
+    },
+    sha1sums: {
+      ...string_collection_attribute_writable,
+      name: "sha1sums",
+      separator: undefined,
+      skipEmpty: true
+    },
+    sha256sums: {
+      ...string_collection_attribute_writable,
+      name: "sha256sums",
+      separator: undefined,
+      skipEmpty: true
+    },
+    sha384sums: {
+      ...string_collection_attribute_writable,
+      name: "sha384sums",
+      separator: undefined,
+      skipEmpty: true
+    },
+    sha512sums: {
+      ...string_collection_attribute_writable,
+      name: "sha512sums",
+      separator: undefined,
+      skipEmpty: true
+    },
+    groups: {
+      ...string_collection_attribute_writable,
+      name: "groups",
+      separator: undefined,
+      skipEmpty: true
+    },
+    arch: {
+      ...arch_attribute_writable,
+      default: ["any"]
     },
     backup: {
       ...string_collection_attribute_writable,
       name: "backup",
+      separator: undefined,
       skipEmpty: true
     },
-    depends: {
-      ...dependency_attribute_collection_writable /*, alias: "dependencies" */,
-      name: "depends"
+    dependencies: {
+      ...dependency_attribute_collection_writable,
+      name: "dependencies",
+      externalName: "depends",
+      skipEmpty: true
     },
     makedepends: {
       ...dependency_attribute_collection_writable,
-      name: "makedepends"
+      name: "makedepends",
+      skipEmpty: true
     },
     checkdepends: {
       ...dependency_attribute_collection_writable,
-      name: "checkdepends"
+      name: "checkdepends",
+      skipEmpty: true
     },
     optdepends: {
       ...dependency_attribute_collection_writable,
-      name: "optdepends"
+      name: "optdepends",
+      skipEmpty: true
     },
     conflicts: {
       ...dependency_attribute_collection_writable,
       name: "conflicts",
       skipEmpty: true
     },
-    provides: { ...dependency_attribute_collection_writable, name: "provides" },
+    provides: {
+      ...dependency_attribute_collection_writable,
+      name: "provides",
+      skipEmpty: true
+    },
     replaces: {
       ...dependency_attribute_collection_writable,
       name: "replaces",
       skipEmpty: true
     },
-    options: { ...default_attribute, name: "options" }
+    options: {
+      ...default_attribute,
+      name: "options",
+      separator: undefined,
+      skipEmpty: true
+    }
   };
 
   static async prepare(options = {}, variant = {}) {
@@ -209,8 +276,8 @@ export class ALPM extends Packager {
   }
 
   get packageFileName() {
-    const p = this.properties;
-    return `${p.name}-${p.version}-${p.release}-${p.arch}${this.fileNameExtension}`;
+    const p = this.externalProperties;
+    return `${p.pkgname}-${p.pkgver}-${p.pkgrel}-${p.arch}${this.fileNameExtension}`;
   }
 
   dependencyExpression(name, expression) {
@@ -220,9 +287,6 @@ export class ALPM extends Packager {
   async create(sources, transformer, publishingDetails, options, expander) {
     const { properties, staging, destination } = await this.prepare(options);
 
-    if (properties.source) {
-      properties.md5sums = ["SKIP"];
-    }
     if (properties.hooks) {
       properties.install = `${properties.name}.install`;
 
@@ -239,7 +303,15 @@ export class ALPM extends Packager {
       out.end();
     }
 
-    const depends = this.makeDepends(properties.dependencies).join(" ");
+    if (properties.backup?.[0] === "/") {
+      properties.backup = properties.backup.replace(/\//, "");
+    }
+
+    const dependencies = properties.dependencies;
+    delete properties.dependencies;
+    const fp = fieldProvider(properties, this.attributes);
+
+    const depends = this.makeDepends(dependencies).join(" ");
     const dependsStatement = depends.length ? `depends=(${depends})` : "";
     const verbose = options.verbose ? 'ls -laR "$pkgdir"' : "";
 
@@ -259,12 +331,6 @@ package() {
 `;
     }
 
-    if (properties.backup?.[0] === "/") {
-      properties.backup = properties.backup.replace(/\//, "");
-    }
-
-    const fp = fieldProvider(properties, this.attributes);
-
     transformer.push({
       name: PKGBUILD,
       match: entry => entry.name === PKGBUILD,
@@ -287,18 +353,18 @@ package() {
       join(staging, "src"),
       expander
     )) {
-      if(file.destination !== '../PKGBUILD') {
-      if (file.owner || file.group) {
-        permission.push(
-          `    chown ${[file.owner ?? "", file.group ?? ""].join(":")} \"$pkgdir/${file.destination}\"`
-        );
-      }
-      const mode = (await file.mode) & 0o7777;
-      if (mode) {
-        permission.push(
-          `    chmod ${Number(mode).toString(8)} \"$pkgdir/${file.destination}\"`
-        );
-      }
+      if (file.destination !== "../PKGBUILD") {
+        if (file.owner || file.group) {
+          permission.push(
+            `    chown ${[file.owner ?? "", file.group ?? ""].join(":")} \"$pkgdir/${file.destination}\"`
+          );
+        }
+        const mode = (await file.mode) & 0o7777;
+        if (mode) {
+          permission.push(
+            `    chmod ${Number(mode).toString(8)} \"$pkgdir/${file.destination}\"`
+          );
+        }
       }
 
       if (options.verbose) {

package/src/output/debian.mjs

@@ -6,7 +6,7 @@ import {
   yesno_attribute_writable,
   string_attribute_writable,
   string_collection_attribute_writable,
-  toExternal
+  types
 } from "pacc";
 import { ContentEntry, IteratorContentEntry } from "content-entry";
 import {
@@ -14,15 +14,15 @@ import {
   createPropertiesTransformer
 } from "content-entry-transform";
 import { keyValueTransformer, Uint8ArraysToLines } from "key-value-transformer";
+import { Packager } from "./packager.mjs";
+import { copyEntries, fieldProvider, aggregate } from "../util.mjs";
 import {
-  Packager,
   pkgbuild_version_attribute,
   pkgbuild_description_attribute,
   pkgbuild_name_attribute,
   dependency_attribute_collection_writable,
-  architectureType
-} from "./packager.mjs";
-import { copyEntries, fieldProvider, aggregate } from "../util.mjs";
+  arch_attribute_writable
+} from "../types.mjs";
 
 const debian_dependency_attribute_collection_writable = {
   ...dependency_attribute_collection_writable,
@@ -52,85 +52,126 @@ export class DEBIAN extends Packager {
    * @see https://linux.die.net/man/5/deb-control
    */
   static attributes = {
-    Package: {
+    name: {
       ...pkgbuild_name_attribute,
-      name: "Package",
-      set: v => v.toLowerCase()
+      externalName: "Package",
+      type: types["lowercase-string"]
+    },
+    description: {
+      ...pkgbuild_description_attribute,
+      externalName: "Description",
+      skipEmpty: true
     },
-    Version: { ...pkgbuild_version_attribute, name: "Version" },
-    Maintainer: {
+    version: { ...pkgbuild_version_attribute, externalName: "Version" },
+    maintainer: {
       ...string_attribute_writable,
-      name: "Maintainer",
-      alias: "maintainer",
+      name: "maintainer",
+      externalName: "Maintainer",
       mandatory: true
     },
-    Description: { ...pkgbuild_description_attribute, name: "Description" },
-    Section: { ...string_attribute_writable, name: "Section", alias: "groups" },
-    Priority: { ...string_attribute_writable, name: "Priority" },
-    Essential: { ...yesno_attribute_writable, name: "Essential" },
-    Origin: { ...string_attribute_writable, name: "Origin" },
-    Architecture: {
-      ...string_attribute_writable,
-      name: "Architecture",
-      alias: "arch",
+    arch: {
+      ...arch_attribute_writable,
+      externalName: "Architecture",
       default: "all",
-      mandatory: true,
-      type: architectureType,
-      mapping: { aarch64: "arm64" }
+      mapping: { aarch64: "arm64", any: "all" }
+    },
+    groups: {
+      ...string_attribute_writable,
+      name: "groups",
+      externalName: "Section",
+      skipEmpty: true
+    },
+    Priority: {
+      ...string_attribute_writable,
+      name: "Priority",
+      skipEmpty: true
+    },
+    Essential: {
+      ...yesno_attribute_writable,
+      name: "Essential",
+      skipEmpty: true
     },
-    Homepage: {
+    Origin: { ...string_attribute_writable, name: "Origin", skipEmpty: true },
+    homepage: {
       ...string_attribute_writable,
-      name: "Homepage",
-      alias: "homepage"
+      name: "homepage",
+      externalName: "Homepage",
+      skipEmpty: true
     },
-    Bugs: { ...string_attribute_writable, name: "Bugs", alias: "bugs" },
-    Depends: {
+    Bugs: {
+      ...string_attribute_writable,
+      name: "bugs",
+      externalName: "Bugs",
+      skipEmpty: true
+    },
+    dependencies: {
       ...debian_dependency_attribute_collection_writable,
-      name: "Depends"
+      name: "dependencies",
+      externalName: "Depends",
+      skipEmpty: true
     },
     "Pre-Depends": {
       ...debian_dependency_attribute_collection_writable,
-      name: "Pre-Depends"
+      name: "Pre-Depends",
+      skipEmpty: true
     },
     "Build-Depends": {
       ...debian_dependency_attribute_collection_writable,
-      name: "Build-Depends"
+      name: "Build-Depends",
+      skipEmpty: true
     },
     "Build-Depends-Indep": {
       ...debian_dependency_attribute_collection_writable,
-      name: "Build-Depends-Indep"
+      name: "Build-Depends-Indep",
+      skipEmpty: true
     },
     "Build-Depends-Arch": {
       ...debian_dependency_attribute_collection_writable,
-      name: "Build-Depends-Arch"
+      name: "Build-Depends-Arch",
+      skipEmpty: true
     },
     Recommends: {
       ...debian_dependency_attribute_collection_writable,
-      name: "Recommends"
+      name: "Recommends",
+      skipEmpty: true
     },
     Suggests: {
       ...debian_dependency_attribute_collection_writable,
-      name: "Suggests"
+      name: "Suggests",
+      skipEmpty: true
     },
     Provides: {
       ...debian_dependency_attribute_collection_writable,
-      name: "Provides"
+      name: "Provides",
+      skipEmpty: true
     },
     Breaks: {
       ...debian_dependency_attribute_collection_writable,
-      name: "Breaks"
+      name: "Breaks",
+      skipEmpty: true
     },
     Replaces: {
       ...debian_dependency_attribute_collection_writable,
-      name: "Replaces"
+      name: "Replaces",
+      skipEmpty: true
+    },
+    source: {
+      ...string_attribute_writable,
+      name: "source",
+      externalName: "Source",
+      skipEmpty: true
     },
-    Source: { ...string_attribute_writable, name: "Source", alias: "source" },
     Uploaders: {
       ...string_collection_attribute_writable,
       name: "Uploaders",
-      mandatory: false
+      mandatory: false,
+      skipEmpty: true
     },
-    "Installed-Size": { ...integer_attribute_writable, name: "Installed-Size" }
+    "Installed-Size": {
+      ...integer_attribute_writable,
+      name: "Installed-Size",
+      skipEmpty: true
+    }
   };
 
   /**
@@ -153,13 +194,8 @@ export class DEBIAN extends Packager {
   }
 
   get packageFileName() {
-    const p = this.properties;
-
-    // TODO utility to provide final values
-    const arch = toExternal(p.arch, this.attributes.Architecture);
-
-    // @ts-ignore
-    return `${p.name}_${p.version}_${arch}${this.constructor.fileNameExtension}`;
+    const p = this.externalProperties;
+    return `${p.Package}_${p.Version}_${p.Architecture}${this.constructor.fileNameExtension}`;
   }
 
   /**
@@ -192,10 +228,7 @@ export class DEBIAN extends Packager {
       )
     );
 
-    const depends = this.makeDepends(properties.dependencies);
-    if (depends.length) {
-      properties.Depends = depends;
-    }
+    properties.dependencies = this.makeDepends(properties.dependencies);
 
     const fp = fieldProvider(properties, this.attributes);
 

package/src/output/docker.mjs

@@ -1,7 +1,7 @@
 import { join } from "node:path";
 import { readFile } from "node:fs/promises";
 import { execa } from "execa";
-import { string_attribute } from "pacc";
+import { string_attribute_writable, types } from "pacc";
 import { ContentEntry, IteratorContentEntry } from "content-entry";
 import { transform } from "content-entry-transform";
 import { aggregateFifo } from "aggregate-async-iterator";
@@ -12,10 +12,12 @@ import {
 } from "key-value-transformer";
 import {
   Packager,
+} from "./packager.mjs";
+import {
   pkgbuild_version_attribute,
   pkgbuild_description_attribute,
   pkgbuild_name_attribute
-} from "./packager.mjs";
+} from "../types.mjs";
 import {
   fieldProvider,
   copyEntries,
@@ -62,12 +64,12 @@ export class DOCKER extends Packager {
   static attributes = {
     name: {
       ...pkgbuild_name_attribute,
-      set: value => value.toLowerCase()
+      type: types["lowercase-string"]
     },
     version: pkgbuild_version_attribute,
     description: pkgbuild_description_attribute,
-    author: { ...string_attribute, alias: "maintainer" },
-    workdir: { ...string_attribute, default: "/", mandatory: true }
+    maintainer: { ...string_attribute_writable, name: "maintainer", externalName: "author" },
+    workdir: { ...string_attribute_writable, default: "/", mandatory: true }
   };
 
   /**

package/src/output/packager.mjs

@@ -3,12 +3,11 @@ import { tmpdir } from "node:os";
 import { mkdtemp, mkdir } from "node:fs/promises";
 import { createReadStream } from "node:fs";
 import {
+  asArray,
   expand,
   expandContextDoubbleCurly,
-  string_collection_attribute_writable,
-  name_attribute,
-  description_attribute,
-  version_attribute_writable
+  attributeIterator,
+  iterateToExternal
 } from "pacc";
 import { StringContentEntry } from "content-entry";
 import { publish } from "../publish.mjs";
@@ -58,7 +57,7 @@ export class Packager {
     return false;
   }
 
-  #properties;
+  properties;
   #prepared;
 
   /**
@@ -66,7 +65,21 @@ export class Packager {
    * @param {Object} properties
    */
   constructor(properties) {
-    this.#properties = { ...properties, type: this.constructor.name };
+    this.properties = { ...properties, type: this.constructor.name };
+
+    for (const [path, attribute] of attributeIterator(
+      this.constructor.attributes,
+      attribute => attribute.default !== undefined
+    )) {
+      const name = path.join(".");
+      this.properties[name] ??= attribute.default;
+    }
+  }
+
+  get externalProperties() {
+    return Object.fromEntries([
+      ...iterateToExternal(this.properties, this.constructor.attributes)
+    ]);
   }
 
   /**
@@ -142,14 +155,15 @@ export class Packager {
       return [];
     }
 
-    if (Array.isArray(dependencies)) {
+    if (Array.isArray(dependencies) || dependencies instanceof Set) {
       dependencies = Object.fromEntries(
-        dependencies.map(d => {
+        asArray(dependencies).map(d => {
           const m = d.match(/^([^=<>]+)(.*)/);
           return [m[1], m[2]];
         })
       );
     }
+    
     return Object.entries(dependencies)
       .filter(filterOutUnwantedDependencies())
       .map(([name, expression]) => this.dependencyExpression(name, expression));
@@ -165,41 +179,6 @@ export class Packager {
     return this.constructor.attributes;
   }
 
-  get properties() {
-    const properties = this.#properties;
-
-    for (const [name, field] of Object.entries(this.attributes)) {
-      if (field.set) {
-        if (properties[name] !== undefined) {
-          properties[name] = field.set(properties[name]);
-        } else if (
-          field.alias !== undefined &&
-          properties[field.alias] !== undefined
-        ) {
-          properties[field.alias] = field.set(properties[field.alias]);
-        }
-      }
-
-      const e = properties[field.alias || name];
-
-      if (e !== undefined) {
-        properties[name] = field.set ? field.set(e) : e;
-      } else {
-        if (field.default !== undefined) {
-          const vak = field.alias || name;
-          if (
-            (Array.isArray(properties[vak]) && properties[vak].length === 0) ||
-            properties[vak] === undefined
-          ) {
-            properties[vak] = field.default;
-          }
-        }
-      }
-    }
-
-    return properties;
-  }
-
   /**
    * Create tmp directory.
    * @return {Promise<string>} directory path
@@ -225,9 +204,9 @@ export class Packager {
 
     const out = {
       properties: this.properties,
-      destination: options.destination || tmpdir,
+      destination: options.destination ?? tmpdir,
       tmpdir,
-      staging: options.staging || tmpdir
+      staging: options.staging ?? tmpdir
     };
 
     // @ts-ignore
@@ -273,65 +252,3 @@ export class Packager {
     return publish(artifact, publishingDetails, this.properties, logger);
   }
 }
-
-export const pkgbuild_name_attribute = {
-  ...name_attribute,
-  alias: "name",
-  mandatory: true,
-  pattern: /^[a-z_][a-z0-9_\-]*$/i
-};
-
-export const dependency_type = {
-  name: "dependency",
-  primitive: false,
-  toExternal: (value, attribute) => {
-    switch (typeof value) {
-      case "string":
-      case "undefined":
-        return value;
-    }
-
-    if (Array.isArray(value)) {
-      return value;
-    }
-
-    return Object.entries(value).map(([name, expression]) =>
-      typeof expression === "string" ? `${name}${expression}` : name
-    );
-  }
-};
-
-export const architectureType = {
-  name: "dependency",
-  primitive: true,
-
-  toInternal: (value, attribute) => {
- //   console.log("architectureType toInternal", value);
-    return value;
-  },
-
-  toExternal: (value, attribute) => {
-  //  console.log("architectureType toExternal", value);
-    return attribute.mapping[value] ?? value;
-  }
-};
-
-export const dependency_attribute_collection_writable = {
-  ...string_collection_attribute_writable,
-  type: dependency_type,
-  separator: " ",
-  pattern: /^[a-z_][a-z0-9_\-]*$/i
-};
-
-export const pkgbuild_version_attribute = {
-  ...version_attribute_writable,
-  alias: "version",
-  mandatory: true,
-  set: v => v.replace("-semantic-release", "")
-};
-
-export const pkgbuild_description_attribute = {
-  ...description_attribute,
-  alias: "description",
-  mandatory: true
-};

package/src/output/rpm.mjs

@@ -3,10 +3,9 @@ import { readFile } from "node:fs/promises";
 import { cp } from "node:fs/promises";
 import { execa } from "execa";
 import {
-  integer_attribute,
-  url_attribute,
-  string_attribute,
-  string_collection_attribute_writable
+  integer_attribute_writable,
+  url_attribute_writable,
+  string_attribute_writable
 } from "pacc";
 import { ContentEntry, IteratorContentEntry } from "content-entry";
 import { transform } from "content-entry-transform";
@@ -15,12 +14,14 @@ import {
   colonSeparatedKeyValuePairOptionsDoublingKeys,
   Uint8ArraysToLines
 } from "key-value-transformer";
+import { Packager } from "./packager.mjs";
 import {
-  Packager,
   pkgbuild_version_attribute,
   pkgbuild_description_attribute,
-  pkgbuild_name_attribute
-} from "./packager.mjs";
+  pkgbuild_name_attribute,
+  arch_attribute_writable,
+  dependency_attribute_collection_writable
+} from "../types.mjs";
 import {
   copyEntries,
   fieldProvider,
@@ -61,37 +62,72 @@ export class RPM extends Packager {
    * @see https://rpm-packaging-guide.github.io
    */
   static attributes = {
-    Name: { ...pkgbuild_name_attribute, name: "Name" },
-    Summary: { ...pkgbuild_description_attribute, name: "Summary" },
-    License: {
-      ...string_attribute,
-      name: "License",
-      alias: "license",
+    name: { ...pkgbuild_name_attribute, externalName: "Name" },
+    description: { ...pkgbuild_description_attribute, externalName: "Summary" },
+    license: {
+      ...string_attribute_writable,
+      name: "license",
+      externalName: "License",
       mandatory: true
     },
-    Version: { ...pkgbuild_version_attribute, name: "Version" },
-    Release: {
-      ...integer_attribute,
-      name: "Release",
-      alias: "release",
+    version: { ...pkgbuild_version_attribute, externalName: "Version" },
+    release: {
+      ...integer_attribute_writable,
+      externalName: "Release",
+      name: "release",
       default: 1,
       mandatory: true
     },
-    Source0: { ...string_attribute, name: "Source0", alias: "source" },
-    Group: { ...string_attribute, name: "Group", alias: "groups" },
-    Packager: { ...string_attribute, name: "Packager", alias: "maintainer" },
-    Vendor: { ...string_attribute, name: "Vendor", alias: "vendor" },
-    BuildArch: {
-      ...string_attribute,
-      name: "BuildArch",
-      alias: "arch",
+    source: {
+      ...string_attribute_writable,
+      externalName: "Source0",
+      name: "source",
+      skipEmpty: true
+    },
+    groups: {
+      ...string_attribute_writable,
+      externalName: "Group",
+      name: "groups",
+      skipEmpty: true
+    },
+    maintainer: {
+      ...string_attribute_writable,
+      name: "maintainer",
+      externalName: "Packager"
+    },
+    vendor: {
+      ...string_attribute_writable,
+      externalName: "Vendor",
+      name: "vendor",
+      skipEmpty: true
+    },
+    arch: {
+      ...arch_attribute_writable,
+      externalName: "BuildArch",
       default: "noarch",
-      mandatory: true
+      mapping: { any: "noarch" }
     },
-    URL: { ...url_attribute, name: "URL", alias: "homepage" },
-    Requires: { ...string_collection_attribute_writable, name: "Requires" },
-    Obsoletes: { ...string_collection_attribute_writable, name: "Obsoletes" },
-    Conflicts: { ...string_collection_attribute_writable, name: "Conflicts" }
+    URL: {
+      ...url_attribute_writable,
+      name: "URL",
+      alias: "homepage",
+      skipEmpty: true
+    },
+    dependencies: {
+      ...dependency_attribute_collection_writable,
+      externalName: "Requires",
+      skipEmpty: true
+    },
+    Obsoletes: {
+      ...dependency_attribute_collection_writable,
+      name: "Obsoletes",
+      skipEmpty: true
+    },
+    Conflicts: {
+      ...dependency_attribute_collection_writable,
+      name: "Conflicts",
+      skipEmpty: true
+    }
   };
 
   static get workspaceLayout() {
@@ -152,7 +188,7 @@ export class RPM extends Packager {
     const { properties, tmpdir, staging, destination } =
       await this.prepare(options);
 
-    properties.Requires = this.makeDepends(properties.dependencies);
+    properties.dependencies = this.makeDepends(properties.dependencies);
 
     if (properties.Packager?.length > 1) {
       // TODO how to write several Packages ?

package/src/types.mjs

@@ -0,0 +1,92 @@
+import {
+  asArray,
+  types,
+  string_type,
+  string_collection_attribute_writable,
+  name_attribute,
+  description_attribute,
+  version_attribute_writable,
+  string_attribute_writable
+} from "pacc";
+
+export const dependency_type = {
+  ...string_type,
+  name: "dependency",
+  toExternal: (value, attribute) => {
+    switch (typeof value) {
+      case "string":
+      case "undefined":
+        return value;
+    }
+
+    if (Array.isArray(value)) {
+      if (value.length === 0 && attribute.skipEmpty) {
+        return undefined;
+      }
+
+      return value;
+    }
+
+    if (value instanceof Set) {
+      if (value.size === 0 && attribute.skipEmpty) {
+        return undefined;
+      }
+
+      return asArray(value);
+    }
+
+    return Object.entries(value).map(([name, expression]) =>
+      typeof expression === "string" ? `${name}${expression}` : name
+    );
+  }
+};
+
+export const architecture_type = {
+  ...string_type,
+  name: "architecture",
+
+  toInternal: (value, attribute) => {
+    return value;
+  },
+
+  toExternal: (value, attribute) => {
+    return attribute.mapping?.[value] ?? value;
+  }
+};
+
+export const arch_attribute_writable = {
+  ...string_attribute_writable,
+  name: "arch",
+  default: "all",
+  mandatory: true,
+  type: architecture_type
+};
+
+export const pkgbuild_name_attribute = {
+  ...name_attribute,
+  mandatory: true,
+  pattern: /^[a-z_][a-z0-9_\-]*$/i
+};
+
+export const dependency_attribute_collection_writable = {
+  ...string_collection_attribute_writable,
+  type: dependency_type,
+  separator: undefined,
+  pattern: /^[a-z_][a-z0-9_\-]*$/i
+};
+
+export const pkgbuild_version_attribute = {
+  ...version_attribute_writable,
+  mandatory: true,
+  type: {
+    ...types.string,
+    toExternal: (value, attribute) => {
+      return value.replace("-semantic-release", "");
+    }
+  }
+};
+
+export const pkgbuild_description_attribute = {
+  ...description_attribute,
+  mandatory: true
+};

package/src/util.mjs

@@ -3,7 +3,7 @@ import { mkdir } from "node:fs/promises";
 import { pipeline } from "node:stream/promises";
 import { Readable } from "node:stream";
 import { createWriteStream } from "node:fs";
-import { toExternal, asArray } from "pacc";
+import { asArray, iterateToExternal } from "pacc";
 import { ContentEntry } from "content-entry";
 import { aggregateFifo } from "aggregate-async-iterator";
 
@@ -145,44 +145,21 @@ export function quote(v, qc = "'") {
  * @returns {Function}
  */
 export function fieldProvider(properties, attributes) {
-  function av(attribute, value) {
-    return attribute.collection ? asArray(value) : value;
-  }
+  return function* controlProperties(key, value, presentKeys) {
+    let filter;
 
-  return function* controlProperties(k, v, presentKeys) {
-    if (k === undefined) {
-      for (const [name, attribute] of Object.entries(attributes)) {
-        if (!presentKeys.has(name)) {
-          let value = properties[attribute.alias || name];
-          if (
-            value === undefined ||
-            (attribute.collection && value.size === 0)
-          ) {
-            if (attribute.default === undefined) {
-              if (attribute.mandatory) {
-                console.error(`Missing value for mandatory attribute ${name}`);
-              }
-              if (attribute.skipEmpty) {
-                continue;
-              }
-            } else {
-              yield [name, toExternal(attribute.default, attribute)];
-            }
-          } else {
-            if (attribute.mapping) {
-              const mappedValue = attribute.mapping[value];
-              if (mappedValue !== undefined) {
-                value = mappedValue;
-              }
-            }
-
-            yield [name, av(attribute, toExternal(value, attribute))];
-          }
-        }
-      }
-    } else {
-      const attribute = attributes[k];
-      yield [k, av(attribute, toExternal(properties[k] ?? v, attribute))];
+    if (key) {
+      filter = attribute => attribute.name === key;
+    }
+
+    for (const [name, value] of iterateToExternal(
+      properties,
+      attributes,
+      filter
+    )) {
+      //if (!presentKeys.has(name)) { }
+
+      yield [name, value];
     }
   };
 }