chore(td-tools): enable eslint/strict-boolean-expressions and strictNullChecks

.eslintrc.js

@@ -29,6 +29,7 @@ module.exports = {
         "@typescript-eslint/no-unused-vars": "off", // or "@typescript-eslint/no-unused-vars": "off",
         "no-use-before-define": "off",
         "@typescript-eslint/no-use-before-define": ["error"],
+        "@typescript-eslint/strict-boolean-expressions": ["error"],
         "unused-imports/no-unused-imports": "error",
         "unused-imports/no-unused-vars": [
             "warn",

packages/examples/src/scripts/countdown.ts

@@ -127,7 +127,7 @@ WoT.produce({
             "startCountdown",
             async (params: WoT.InteractionOutput, options): Promise<WoT.InteractionInput> => {
                 let initValue = 100;
-                if (params) {
+                if (params !== undefined) {
                     const value = await params.value();
                     if (typeof value === "number") {
                         initValue = value as number;
@@ -147,7 +147,7 @@ WoT.produce({
         thing.setActionHandler(
             "stopCountdown",
             async (params: WoT.InteractionOutput, options): Promise<WoT.InteractionInput> => {
-                if (params) {
+                if (params !== undefined) {
                     const value = await params.value();
                     if (typeof value === "string" && countdowns.has(value)) {
                         const as = countdowns.get(value);
@@ -170,7 +170,7 @@ WoT.produce({
         thing.setActionHandler(
             "monitorCountdown",
             async (params: WoT.InteractionOutput, options): Promise<WoT.InteractionInput> => {
-                if (params) {
+                if (params !== undefined) {
                     const value = await params.value();
                     if (typeof value === "string" && countdowns.has(value)) {
                         const as = countdowns.get(value);

packages/examples/src/scripts/smart-coffee-machine-client.ts

@@ -61,7 +61,7 @@ WoTHelpers.fetch("http://127.0.0.1:8080/smart-coffee-machine").then(async (td) =
             uriVariables: { drinkId: "latte", size: "l", quantity: 3 },
         });
         const makeCoffeep = (await makeCoffee?.value()) as Record<string, unknown>;
-        if (makeCoffeep.result) {
+        if (makeCoffeep.result !== undefined) {
             log("Enjoy your drink!", makeCoffeep);
         } else {
             log("Failed making your drink:", makeCoffeep);

packages/examples/src/scripts/smart-coffee-machine.ts

@@ -396,7 +396,7 @@ Assumes one medium americano if not specified, but time and mode are mandatory f
             const paramsp = (await params.value()) as Record<string, unknown>; //  : any = await Helpers.parseInteractionOutput(params);
 
             // Check if uriVariables are provided
-            if (paramsp && typeof paramsp === "object" && "time" in paramsp && "mode" in paramsp) {
+            if (paramsp !== undefined && typeof paramsp === "object" && "time" in paramsp && "mode" in paramsp) {
                 // Use default values if not provided
                 paramsp.drinkId = "drinkId" in paramsp ? paramsp.drinkId : "americano";
                 paramsp.size = "size" in paramsp ? paramsp.size : "m";

packages/td-tools/src/td-parser.ts

@@ -158,7 +158,7 @@ export function parseTD(td: string, normalize?: boolean): Thing {
                 throw new Error(`Form of Property '${propName}' has no href field`);
             }
             // check if base field required
-            if (!isAbsoluteUrl(form.href) && !thing.base)
+            if (!isAbsoluteUrl(form.href) && thing.base === undefined)
                 throw new Error(`Form of Property '${propName}' has relative URI while TD has no base field`);
             // add
             allForms.push(form);
@@ -176,7 +176,7 @@ export function parseTD(td: string, normalize?: boolean): Thing {
                 throw new Error(`Form of Action '${actName}' has no href field`);
             }
             // check if base field required
-            if (!isAbsoluteUrl(form.href) && !thing.base)
+            if (!isAbsoluteUrl(form.href) && thing.base === undefined)
                 throw new Error(`Form of Action '${actName}' has relative URI while TD has no base field`);
             // add
             allForms.push(form);
@@ -194,7 +194,7 @@ export function parseTD(td: string, normalize?: boolean): Thing {
                 throw new Error(`Form of Event '${evtName}' has no href field`);
             }
             // check if base field required
-            if (!isAbsoluteUrl(form.href) && !thing.base)
+            if (!isAbsoluteUrl(form.href) && thing.base === undefined)
                 throw new Error(`Form of Event '${evtName}' has relative URI while TD has no base field`);
             // add
             allForms.push(form);
@@ -222,20 +222,20 @@ export function serializeTD(thing: Thing): string {
     const copy = JSON.parse(JSON.stringify(thing));
 
     // clean-ups
-    if (!copy.security || copy.security.length === 0) {
+    if (copy.security === undefined || copy.security.length === 0) {
         copy.securityDefinitions = {
             nosec_sc: { scheme: "nosec" },
         };
         copy.security = ["nosec_sc"];
     }
 
-    if (copy.forms && copy.forms.length === 0) {
+    if (copy.forms !== undefined && copy.forms.length === 0) {
         delete copy.forms;
     }
 
-    if (copy.properties && Object.keys(copy.properties).length === 0) {
+    if (copy.properties !== undefined && Object.keys(copy.properties).length === 0) {
         delete copy.properties;
-    } else if (copy.properties) {
+    } else if (copy.properties !== undefined) {
         // add mandatory fields (if missing): observable, writeOnly, and readOnly
         for (const propName in copy.properties) {
             const prop = copy.properties[propName];
@@ -251,9 +251,9 @@ export function serializeTD(thing: Thing): string {
         }
     }
 
-    if (copy.actions && Object.keys(copy.actions).length === 0) {
+    if (copy.actions !== undefined && Object.keys(copy.actions).length === 0) {
         delete copy.actions;
-    } else if (copy.actions) {
+    } else if (copy.actions !== undefined) {
         // add mandatory fields (if missing): idempotent and safe
         for (const actName in copy.actions) {
             const act = copy.actions[actName];
@@ -265,11 +265,11 @@ export function serializeTD(thing: Thing): string {
             }
         }
     }
-    if (copy.events && Object.keys(copy.events).length === 0) {
+    if (copy.events !== undefined && Object.keys(copy.events).length === 0) {
         delete copy.events;
     }
 
-    if (copy.links && copy.links.length === 0) {
+    if (copy.links !== undefined && copy.links.length === 0) {
         delete copy.links;
     }
 

packages/td-tools/src/thing-description.ts

@@ -67,7 +67,7 @@ export class Form implements TDT.FormElementBase {
 
     constructor(href: string, contentType?: string) {
         this.href = href;
-        if (contentType) this.contentType = contentType;
+        if (contentType !== undefined) this.contentType = contentType;
     }
 }
 export interface ExpectedResponse {

packages/td-tools/src/thing-model-helpers.ts

@@ -110,7 +110,7 @@ export class ThingModelHelpers {
         }
         if ("links" in data && Array.isArray(data.links)) {
             const foundTmExtendsRel = data.links.find((link) => link.rel === "tm:extends");
-            if (foundTmExtendsRel) return true;
+            if (foundTmExtendsRel !== undefined) return true;
         }
 
         if (data.properties !== undefined) {
@@ -138,7 +138,7 @@ export class ThingModelHelpers {
     public static getModelVersion(data: ThingModel): string | undefined {
         if (
             "version" in data &&
-            data.version &&
+            data.version !== undefined &&
             typeof data.version === "object" &&
             "model" in data.version &&
             typeof data.version.model === "string"
@@ -241,7 +241,7 @@ export class ThingModelHelpers {
             case "http": {
                 return new Promise((resolve, reject) => {
                     http.get(uri, (res) => {
-                        if (!res.statusCode || res.statusCode !== 200) {
+                        if (res.statusCode === undefined || res.statusCode !== 200) {
                             reject(new Error(`http status code not 200 but ${res.statusCode} for ${uri}`));
                         }
 
@@ -268,7 +268,7 @@ export class ThingModelHelpers {
                 return new Promise((resolve, reject) => {
                     https
                         .get(uri, (res) => {
-                            if (!res.statusCode || res.statusCode !== 200) {
+                            if (res.statusCode === undefined || res.statusCode !== 200) {
                                 reject(new Error(`https status code not 200 but ${res.statusCode} for ${uri}`));
                             }
 
@@ -343,7 +343,7 @@ export class ThingModelHelpers {
                 for (const aff in affRefs) {
                     const affUri = affRefs[aff] as string;
                     const refObj = this.parseTmRef(affUri);
-                    if (!refObj.uri) {
+                    if (refObj.uri === undefined) {
                         throw new Error(`Missing remote path in ${affUri}`);
                     }
                     let source = await this.fetchModel(refObj.uri);
@@ -376,7 +376,7 @@ export class ThingModelHelpers {
         if (!options) {
             options = {} as CompositionOptions;
         }
-        if (!options.baseUrl) {
+        if (options.baseUrl === undefined) {
             options.baseUrl = ".";
         }
         const newTMHref = this.returnNewTMHref(options.baseUrl, title);
@@ -405,7 +405,7 @@ export class ThingModelHelpers {
 
             for (const key in submodelObj) {
                 const sub = submodelObj[key];
-                if (options.selfComposition) {
+                if (options.selfComposition !== undefined) {
                     if (!data.links) {
                         throw new Error(
                             "You used self composition but links are missing; they are needed to extract the instance name"
@@ -415,7 +415,7 @@ export class ThingModelHelpers {
                     const index = data.links.findIndex((el) => el.href === key);
                     const el = data.links[index];
                     const instanceName = el.instanceName;
-                    if (!instanceName) {
+                    if (instanceName === undefined) {
                         throw new Error("Self composition is not possible without instance names");
                     }
                     // self composition enabled, just one TD expected
@@ -449,7 +449,7 @@ export class ThingModelHelpers {
                 }
             }
         }
-        if (!data.links || options.selfComposition) {
+        if (!data.links || options.selfComposition !== undefined) {
             data.links = [];
         }
         // add reference to the thing model
@@ -475,7 +475,7 @@ export class ThingModelHelpers {
 
     private static getThingModelRef(data: Record<string, unknown>): Record<string, unknown> {
         const refs = {} as Record<string, unknown>;
-        if (!data) {
+        if (data === undefined) {
             return refs;
         }
         for (const key in data) {
@@ -508,7 +508,7 @@ export class ThingModelHelpers {
                 extendedModel.properties = {};
             }
             for (const key in properties) {
-                if (dest.properties && dest.properties[key]) {
+                if (dest.properties && dest.properties[key] !== undefined) {
                     extendedModel.properties[key] = { ...properties[key], ...dest.properties[key] };
                 } else {
                     extendedModel.properties[key] = properties[key];
@@ -625,7 +625,7 @@ export class ThingModelHelpers {
         keys = keys.map((el) => el.replace("{{", "").replace("}}", ""));
         let isValid = true;
         let errors;
-        if (keys && keys.length > 0 && (map === undefined || map === null)) {
+        if (keys !== undefined && keys.length > 0 && (map === undefined || map === null)) {
             isValid = false;
             errors = `No map provided for model ${model.title}`;
         } else if (keys.length > 0) {
@@ -660,7 +660,7 @@ export class ThingModelHelpers {
     }
 
     private removeDependency(dep?: string) {
-        if (dep) {
+        if (dep !== undefined) {
             this.deps = this.deps.filter((el) => el !== dep);
         } else {
             this.deps.pop();

tsconfig.json

@@ -6,6 +6,7 @@
         "module": "commonjs",
         "outDir": "dist",
         "strict": true,
+        "strictNullChecks": true,
         "composite": true,
         "incremental": true,
         "sourceMap": true,