fix: remove subpath

packages/opentelemetry-host-metrics/src/stats/si.ts

@@ -14,10 +14,7 @@
  * limitations under the License.
  */
 
-// Import from network file directly as importing from the root imports the /lib/cpu file,
-// resulting in also having to add osx-temperature-sensor as a dependency for macOS,
-// while /lib/cpu isn't even used by this package (deep-importing not working as expected)
-import { networkStats } from 'systeminformation/lib/network';
+import { networkStats } from 'systeminformation';
 import type { Systeminformation } from 'systeminformation';
 
 export function getNetworkData() {

packages/opentelemetry-host-metrics/src/util.ts

@@ -14,6 +14,6 @@
  * limitations under the License.
  */
 
-export function ObjectKeys<T extends {}>(t: T) {
+export function ObjectKeys<T extends Record<string, unknown>>(t: T) {
   return Object.keys(t) as (keyof T)[];
 }

packages/opentelemetry-host-metrics/test/metric.test.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import * as Network from 'systeminformation/lib/network';
+import * as SI from 'systeminformation';
 import type { Systeminformation } from 'systeminformation';
 import { Attributes } from '@opentelemetry/api';
 import {
@@ -46,21 +46,23 @@ class TestMetricReader extends MetricReader {
 let countSI = 0;
 const mockedSI = {
   networkStats: function () {
-    return new Promise<Systeminformation.NetworkStatsData[]>((resolve, reject) => {
-      countSI++;
-      const stats: any[] = networkJson
-        .slice()
-        .map((obj: any) => Object.assign({}, obj));
-
-      for (let i = 0, j = networkJson.length; i < j; i++) {
-        Object.keys(stats[i]).forEach(key => {
-          if (typeof stats[i][key] === 'number' && stats[i][key] > 0) {
-            stats[i][key] = stats[i][key] * countSI;
-          }
-        });
+    return new Promise<Systeminformation.NetworkStatsData[]>(
+      (resolve, reject) => {
+        countSI++;
+        const stats: any[] = networkJson
+          .slice()
+          .map((obj: any) => Object.assign({}, obj));
+
+        for (let i = 0, j = networkJson.length; i < j; i++) {
+          Object.keys(stats[i]).forEach(key => {
+            if (typeof stats[i][key] === 'number' && stats[i][key] > 0) {
+              stats[i][key] = stats[i][key] * countSI;
+            }
+          });
+        }
+        resolve(stats);
       }
-      resolve(stats);
-    });
+    );
   },
 };
 
@@ -138,12 +140,11 @@ describe('Host Metrics', () => {
       sandbox
         .stub(process.memoryUsage, 'rss')
         .callsFake(mockedProcess.memoryUsage.rss);
-      sandbox.stub(Network, 'networkStats').callsFake(mockedSI.networkStats);
+      sandbox.stub(SI, 'networkStats').callsFake(mockedSI.networkStats);
 
       reader = new TestMetricReader();
 
-      meterProvider = new MeterProvider();
-      meterProvider.addMetricReader(reader);
+      meterProvider = new MeterProvider({ readers: [reader] });
 
       hostMetrics = new HostMetrics({
         meterProvider,

packages/opentelemetry-host-metrics/tsconfig.json

@@ -6,7 +6,6 @@
   },
   "include": [
     "src/**/*.ts",
-    "test/**/*.ts",
-    "global.d.ts"
+    "test/**/*.ts"
   ]
 }