@stackforge-eu/factur-x 1.0.5 → 1.0.7

package/CHANGELOG.md

@@ -7,6 +7,39 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [1.0.7] — 2026-05-27
+
+- **`fx:ConformanceLevel` in the XMP metadata used the raw `Profile` enum
+  identifier instead of the Factur-X / ZUGFeRD spec label.** The XMP packet
+  emitted `EN16931` / `BASIC_WL` where the spec requires `EN 16931` /
+  `BASIC WL`, so strict ZUGFeRD / Factur-X validators (Mustangproject, the
+  FNFE-MPE check tool, veraPDF's Factur-X profile) rejected every PDF this
+  library produced even though the embedded CII XML was correct. Added a
+  `Profile` → XMP-label mapping and routed `buildXmpMetadata` through it in
+  `src/core/embed.ts` (thanks @sco-indy, #2).
+- **`flavor: Flavor.XRECHNUNG` still advertised `EN 16931` in the XMP packet.**
+  Hybrid XRechnung PDFs carry the EN16931 profile internally but per ZUGFeRD
+  2.x §2.4.4 must declare `XRECHNUNG` as the conformance level so the KoSIT /
+  ZUGFeRD validators route them through the XRechnung schematron set rather
+  than the vanilla Factur-X one. `resolveXmpConformanceLevel` in
+  `src/core/embed.ts` now special-cases the XRechnung flavor; the existing
+  `toXRechnung` → XML flow is unaffected. Regression test added in
+  `tests/embed.test.ts` covering all five profiles and the XRechnung flavor.
+
+## [1.0.6] — 2026-05-23
+
+- **`SpecifiedTradeSettlementPaymentMeans` emitted its child elements in the
+  wrong order.** The EN16931 / CII XSD `TradeSettlementPaymentMeansType`
+  sequence requires `PayerPartyDebtorFinancialAccount` before
+  `PayeePartyCreditorFinancialAccount` and
+  `PayeeSpecifiedCreditorFinancialInstitution`, but the builder appended the
+  debtor account last. Any invoice combining a SEPA direct debit
+  (`payment.meansCode = "59"`) with `payment.debtorIban` plus a payee
+  IBAN/BIC failed XSD validation with `Element 'PayerPartyDebtorFinancialAccount':
+  This element is not expected`. Reordered the three blocks in
+  `src/core/xml-builder.ts`; regression test added in
+  `tests/xml-builder.test.ts`.
+
 ## [1.0.5] — 2026-05-12
 
 - **`ChargeIndicator`'s inner `Indicator` element was emitted in the wrong XML
@@ -80,6 +113,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 - Deno compatibility test suite
 
 [Unreleased]: https://github.com/StackForge-EU/factur-x/commits/main
+[1.0.7]: https://github.com/StackForge-EU/factur-x/compare/v1.0.6...v1.0.7
+[1.0.6]: https://github.com/StackForge-EU/factur-x/compare/v1.0.5...v1.0.6
 [1.0.5]: https://github.com/StackForge-EU/factur-x/compare/v1.0.4...v1.0.5
 [1.0.4]: https://github.com/StackForge-EU/factur-x/compare/v1.0.3...v1.0.4
 [1.0.3]: https://github.com/StackForge-EU/factur-x/compare/v1.0.2...v1.0.3

package/dist/index.cjs

@@ -591,6 +591,11 @@ function buildXml(input, profile, flavor) {
     sett += tag("ram:PayeeTradeParty", buildParty(input.payee, profile));
   if (payment?.meansCode && atLeast(profile, "BASIC_WL" /* BASIC_WL */)) {
     let pm = tag("ram:TypeCode", escapeXml(payment.meansCode));
+    if (payment.debtorIban)
+      pm += tag(
+        "ram:PayerPartyDebtorFinancialAccount",
+        tag("ram:IBANID", escapeXml(payment.debtorIban))
+      );
     if (payment.iban || payment.accountId || payment.accountName) {
       let acct = "";
       if (payment.iban) acct += tag("ram:IBANID", escapeXml(payment.iban));
@@ -604,11 +609,6 @@ function buildXml(input, profile, flavor) {
         "ram:PayeeSpecifiedCreditorFinancialInstitution",
         tag("ram:BICID", escapeXml(payment.bic))
       );
-    if (payment.debtorIban)
-      pm += tag(
-        "ram:PayerPartyDebtorFinancialAccount",
-        tag("ram:IBANID", escapeXml(payment.debtorIban))
-      );
     sett += tag("ram:SpecifiedTradeSettlementPaymentMeans", pm);
   }
   if (input.vatBreakdown && atLeast(profile, "BASIC_WL" /* BASIC_WL */))
@@ -1111,6 +1111,17 @@ var AF_RELATIONSHIP_MAP = {
   Data: pdfLib.AFRelationship.Data,
   Source: pdfLib.AFRelationship.Source
 };
+var XMP_CONFORMANCE_LEVEL = {
+  ["MINIMUM" /* MINIMUM */]: "MINIMUM",
+  ["BASIC_WL" /* BASIC_WL */]: "BASIC WL",
+  ["BASIC" /* BASIC */]: "BASIC",
+  ["EN16931" /* EN16931 */]: "EN 16931",
+  ["EXTENDED" /* EXTENDED */]: "EXTENDED"
+};
+function resolveXmpConformanceLevel(profile, flavor) {
+  if (flavor === "xrechnung" /* XRECHNUNG */) return "XRECHNUNG";
+  return XMP_CONFORMANCE_LEVEL[profile];
+}
 function ensureOutputIntent(pdfDoc, iccBytes, identifier) {
   const existing = pdfDoc.catalog.lookup(pdfLib.PDFName.of("OutputIntents"));
   if (existing instanceof pdfLib.PDFArray && existing.size() > 0) return;
@@ -1375,7 +1386,7 @@ async function embedFacturX(options) {
       createDate: isoDate,
       modifyDate: isoDate,
       documentFileName: flavorConfig.attachmentFilename,
-      conformanceLevel: profile
+      conformanceLevel: resolveXmpConformanceLevel(profile, flavor)
     });
     const xmpBytes = new TextEncoder().encode(xmpXml);
     const metadataStream = pdfDoc.context.stream(xmpBytes, {

package/dist/index.js

@@ -561,6 +561,11 @@ function buildXml(input, profile, flavor) {
     sett += tag("ram:PayeeTradeParty", buildParty(input.payee, profile));
   if (payment?.meansCode && atLeast(profile, "BASIC_WL" /* BASIC_WL */)) {
     let pm = tag("ram:TypeCode", escapeXml(payment.meansCode));
+    if (payment.debtorIban)
+      pm += tag(
+        "ram:PayerPartyDebtorFinancialAccount",
+        tag("ram:IBANID", escapeXml(payment.debtorIban))
+      );
     if (payment.iban || payment.accountId || payment.accountName) {
       let acct = "";
       if (payment.iban) acct += tag("ram:IBANID", escapeXml(payment.iban));
@@ -574,11 +579,6 @@ function buildXml(input, profile, flavor) {
         "ram:PayeeSpecifiedCreditorFinancialInstitution",
         tag("ram:BICID", escapeXml(payment.bic))
       );
-    if (payment.debtorIban)
-      pm += tag(
-        "ram:PayerPartyDebtorFinancialAccount",
-        tag("ram:IBANID", escapeXml(payment.debtorIban))
-      );
     sett += tag("ram:SpecifiedTradeSettlementPaymentMeans", pm);
   }
   if (input.vatBreakdown && atLeast(profile, "BASIC_WL" /* BASIC_WL */))
@@ -1081,6 +1081,17 @@ var AF_RELATIONSHIP_MAP = {
   Data: AFRelationship.Data,
   Source: AFRelationship.Source
 };
+var XMP_CONFORMANCE_LEVEL = {
+  ["MINIMUM" /* MINIMUM */]: "MINIMUM",
+  ["BASIC_WL" /* BASIC_WL */]: "BASIC WL",
+  ["BASIC" /* BASIC */]: "BASIC",
+  ["EN16931" /* EN16931 */]: "EN 16931",
+  ["EXTENDED" /* EXTENDED */]: "EXTENDED"
+};
+function resolveXmpConformanceLevel(profile, flavor) {
+  if (flavor === "xrechnung" /* XRECHNUNG */) return "XRECHNUNG";
+  return XMP_CONFORMANCE_LEVEL[profile];
+}
 function ensureOutputIntent(pdfDoc, iccBytes, identifier) {
   const existing = pdfDoc.catalog.lookup(PDFName.of("OutputIntents"));
   if (existing instanceof PDFArray && existing.size() > 0) return;
@@ -1345,7 +1356,7 @@ async function embedFacturX(options) {
       createDate: isoDate,
       modifyDate: isoDate,
       documentFileName: flavorConfig.attachmentFilename,
-      conformanceLevel: profile
+      conformanceLevel: resolveXmpConformanceLevel(profile, flavor)
     });
     const xmpBytes = new TextEncoder().encode(xmpXml);
     const metadataStream = pdfDoc.context.stream(xmpBytes, {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stackforge-eu/factur-x",
-  "version": "1.0.5",
+  "version": "1.0.7",
   "type": "module",
   "description": "Embed Factur-X, ZUGFeRD, and XRechnung structured invoice XML into PDF/A-3 documents",
   "keywords": [