fix(formatting): Fix minor formatting issues and newline consistency across project files

changelog.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 2025-05-03 - 4.0.1 - fix(formatting)
+Fix minor formatting issues and newline consistency across project files
+
+- Ensure newline at end of package.json, errors.ts, logging.ts, and test files
+- Refine code block formatting in readme.md
+- Adjust whitespace and code style in smartnetwork classes and cloudflarespeed module
+- Minor commitinfo data format update
+
 ## 2025-04-28 - 4.0.0 - BREAKING CHANGE(smartnetwork)
 Enhance documentation and add configurable speed test options with plugin architecture improvements
 

readme.md

@@ -16,9 +16,7 @@ You can perform a hop-by-hop traceroute to measure latency per hop. Falls back t
 
 ```typescript
 const hops = await myNetwork.traceroute('google.com', { maxHops: 10, timeout: 5000 });
-hops.forEach(h =>
+hops.forEach((h) => console.log(`${h.ttl}\t${h.ip}\t${h.rtt === null ? '*' : h.rtt + ' ms'}`));
-  console.log(`${h.ttl}\t${h.ip}\t${h.rtt === null ? '*' : h.rtt + ' ms'}`),
-);
 ```
 
 This command will download `@push.rocks/smartnetwork` and add it to your project's `package.json` file.
@@ -44,6 +42,7 @@ const myNetwork = new SmartNetwork();
 ### Performing a Speed Test
 
 You can measure the network speed using the `getSpeed` method. It supports optional parameters:
+
 - `parallelStreams`: number of concurrent streams (default: 1)
 - `duration`: test duration in seconds (default: fixed segments)
 
@@ -143,6 +142,7 @@ console.log(`Public IPv6: ${publicIps.v6}`);
 The `@push.rocks/smartnetwork` package provides an easy-to-use, comprehensive suite of tools for network diagnostics and monitoring, encapsulating complex network operations into simple asynchronous methods. By leveraging TypeScript, developers can benefit from type checking, ensuring that they can work with clear structures and expectations.
 
 These examples offer a glimpse into the module's utility in real-world scenarios, demonstrating its versatility in handling common network tasks. Whether you're developing a network-sensitive application, diagnosing connectivity issues, or simply curious about your network performance, `@push.rocks/smartnetwork` equips you with the tools you need.
+
 ### Plugin Architecture
 
 You can extend `SmartNetwork` with custom plugins by registering them at runtime:

ts/00_commitinfo_data.ts

@@ -3,6 +3,6 @@
  */
 export const commitinfo = {
   name: '@push.rocks/smartnetwork',
-  version: '4.0.0',
+  version: '4.0.1',
   description: 'A toolkit for network diagnostics including speed tests, port availability checks, and more.'
 }

ts/smartnetwork.classes.cloudflarespeed.ts

@@ -164,9 +164,10 @@ export class CloudflareSpeed {
   }
 
   public async fetchServerLocations(): Promise<{ [key: string]: string }> {
-    const res = JSON.parse(
+    const res = JSON.parse(await this.get('speed.cloudflare.com', '/locations')) as Array<{
-      await this.get('speed.cloudflare.com', '/locations'),
+      iata: string;
-    ) as Array<{ iata: string; city: string }>;
+      city: string;
+    }>;
     return res.reduce(
       (data: Record<string, string>, optionsArg) => {
         data[optionsArg.iata] = optionsArg.city;
@@ -296,11 +297,14 @@ export class CloudflareSpeed {
           const parts = i.split('=');
           return [parts[0], parts[1]];
         })
-        .reduce((data: Record<string, string>, [k, v]) => {
+        .reduce(
+          (data: Record<string, string>, [k, v]) => {
             if (v === undefined) return data;
             data[k] = v;
             return data;
-        }, {} as Record<string, string>);
+          },
+          {} as Record<string, string>,
+        );
 
     return this.get('speed.cloudflare.com', '/cdn-cgi/trace').then(parseCfCdnCgiTrace);
   }

ts/smartnetwork.classes.smartnetwork.ts

@@ -44,9 +44,7 @@ export class SmartNetwork {
    * get network speed
    * @param opts optional speed test parameters
    */
-  public async getSpeed(
+  public async getSpeed(opts?: { parallelStreams?: number; duration?: number }) {
-    opts?: { parallelStreams?: number; duration?: number },
-  ) {
     const cloudflareSpeedInstance = new CloudflareSpeed(opts);
     return cloudflareSpeedInstance.speedTest();
   }
@@ -54,10 +52,7 @@ export class SmartNetwork {
   /**
    * Send ICMP pings to a host. Optionally specify count for multiple pings.
    */
-  public async ping(
+  public async ping(host: string, opts?: { timeout?: number; count?: number }): Promise<any> {
-    host: string,
-    opts?: { timeout?: number; count?: number },
-  ): Promise<any> {
     const timeout = opts?.timeout ?? 500;
     const count = opts?.count && opts.count > 1 ? opts.count : 1;
     const pinger = new plugins.smartping.Smartping();
@@ -85,11 +80,7 @@ export class SmartNetwork {
     const min = valid.length ? Math.min(...valid) : NaN;
     const max = valid.length ? Math.max(...valid) : NaN;
     const avg = valid.length ? stats.average(valid) : NaN;
-    const stddev = valid.length
+    const stddev = valid.length ? Math.sqrt(stats.average(valid.map((v) => (v - avg) ** 2))) : NaN;
-      ? Math.sqrt(
-          stats.average(valid.map((v) => (v - avg) ** 2)),
-        )
-      : NaN;
     const packetLoss = ((count - aliveCount) / count) * 100;
     return {
       host,
@@ -168,7 +159,9 @@ export class SmartNetwork {
    */
   public async isRemotePortAvailable(
     target: string,
-    portOrOpts?: number | { port?: number; protocol?: 'tcp' | 'udp'; timeout?: number; retries?: number },
+    portOrOpts?:
+      | number
+      | { port?: number; protocol?: 'tcp' | 'udp'; timeout?: number; retries?: number },
   ): Promise<boolean> {
     let hostPart: string;
     let port: number | undefined;
@@ -252,7 +245,9 @@ export class SmartNetwork {
   /**
    * Resolve DNS records (A, AAAA, MX)
    */
-  public async resolveDns(host: string): Promise<{ A: string[]; AAAA: string[]; MX: { exchange: string; priority: number }[] }> {
+  public async resolveDns(
+    host: string,
+  ): Promise<{ A: string[]; AAAA: string[]; MX: { exchange: string; priority: number }[] }> {
     try {
       const dns = await import('dns');
       const { resolve4, resolve6, resolveMx } = dns.promises;
@@ -316,14 +311,10 @@ export class SmartNetwork {
       const { exec } = await import('child_process');
       const cmd = `traceroute -n -m ${maxHops} ${host}`;
       const stdout: string = await new Promise((resolve, reject) => {
-        exec(
+        exec(cmd, { encoding: 'utf8', timeout }, (err, stdout) => {
-          cmd,
-          { encoding: 'utf8', timeout },
-          (err, stdout) => {
           if (err) return reject(err);
           resolve(stdout);
-          },
+        });
-        );
       });
       const hops: Hop[] = [];
       for (const raw of stdout.split('\n')) {