@telorun/pdf 0.2.1 → 0.3.0

package/dist/form-fields-controller.js

@@ -14,10 +14,10 @@ class PdfFormFields {
     }
     async invoke(inputs) {
         const label = `Pdf.FormFields "${this.resource.metadata.name}"`;
+        // An `inputs.scale` is declared on `inputType` with `exclusiveMinimum: 0` and
+        // enforced by the kernel's contract binding; the resource-level fallback is
+        // checked by the config schema. Neither needs re-checking here.
         const scale = inputs?.scale ?? this.resource.scale ?? 1;
-        if (typeof scale !== "number" || !(scale > 0)) {
-            throw new InvokeError("ERR_INVALID_INPUT", `${label}: 'scale' must be a positive number; got ${scale}.`);
-        }
         const data = inputs?.data;
         if (!(data instanceof Uint8Array)) {
             throw new InvokeError("ERR_INVALID_INPUT", `${label}: 'data' must be a Uint8Array of PDF bytes; got ${typeof data}.`);

package/dist/rasterizer-controller.js

@@ -32,10 +32,11 @@ class PdfRasterizer {
         if (!(data instanceof Uint8Array)) {
             throw new InvokeError("ERR_INVALID_INPUT", `Pdf.Rasterizer "${name}": 'data' must be a Uint8Array of PDF bytes; got ${typeof data}.`);
         }
+        // `page` is declared on `inputType` as an integer with `minimum: 1` and
+        // enforced by the kernel's contract binding, so only the range check the
+        // schema cannot express — page vs. the document's actual page count — is
+        // still this controller's to make (below).
         const pageNumber = inputs.page ?? 1;
-        if (!Number.isInteger(pageNumber) || pageNumber < 1) {
-            throw new InvokeError("ERR_INVALID_INPUT", `Pdf.Rasterizer "${name}": 'page' must be a positive integer; got ${pageNumber}.`);
-        }
         const task = loadPdf(data);
         try {
             const doc = await task.promise.catch((err) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@telorun/pdf",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "description": "PDF primitives — rasterize pages to PNG images and author editable AcroForm fields.",
   "keywords": [
     "telo",
@@ -47,7 +47,7 @@
   "devDependencies": {
     "@types/node": "^20.0.0",
     "typescript": "^5.0.0",
-    "@telorun/sdk": "0.48.0"
+    "@telorun/sdk": "0.61.0"
   },
   "peerDependencies": {
     "@telorun/sdk": "*"

package/src/form-fields-controller.ts

@@ -41,13 +41,10 @@ class PdfFormFields implements ResourceInstance<FormFieldsInputs, FormFieldsOutp
 
   async invoke(inputs: FormFieldsInputs): Promise<FormFieldsOutputs> {
     const label = `Pdf.FormFields "${this.resource.metadata.name}"`;
+    // An `inputs.scale` is declared on `inputType` with `exclusiveMinimum: 0` and
+    // enforced by the kernel's contract binding; the resource-level fallback is
+    // checked by the config schema. Neither needs re-checking here.
     const scale = inputs?.scale ?? this.resource.scale ?? 1;
-    if (typeof scale !== "number" || !(scale > 0)) {
-      throw new InvokeError(
-        "ERR_INVALID_INPUT",
-        `${label}: 'scale' must be a positive number; got ${scale}.`,
-      );
-    }
     const data = inputs?.data;
     if (!(data instanceof Uint8Array)) {
       throw new InvokeError(

package/src/rasterizer-controller.ts

@@ -59,13 +59,11 @@ class PdfRasterizer implements ResourceInstance<RasterizerInputs, RasterizerOutp
         `Pdf.Rasterizer "${name}": 'data' must be a Uint8Array of PDF bytes; got ${typeof data}.`,
       );
     }
+    // `page` is declared on `inputType` as an integer with `minimum: 1` and
+    // enforced by the kernel's contract binding, so only the range check the
+    // schema cannot express — page vs. the document's actual page count — is
+    // still this controller's to make (below).
     const pageNumber = inputs.page ?? 1;
-    if (!Number.isInteger(pageNumber) || pageNumber < 1) {
-      throw new InvokeError(
-        "ERR_INVALID_INPUT",
-        `Pdf.Rasterizer "${name}": 'page' must be a positive integer; got ${pageNumber}.`,
-      );
-    }
 
     const task = loadPdf(data);
     try {