TypeScript error in bridge.d.ts and analytics.d.ts

[cristeahub]

Hi,

I recently stumbled upon the following error when running tsc in my project:

node_modules/@segment/analytics-react-native/build/esm/analytics.d.ts:89:26 - error TS1039: Initializers are not allowed in ambient contexts.

89         readonly ready = false;
                            ~~~~~

node_modules/@segment/analytics-react-native/build/esm/bridge.d.ts:2:17 - error TS2503: Cannot find namespace 'NativeModules'.

2 import bridge = NativeModules.RNAnalytics;
                  ~~~~~~~~~~~~~

node_modules/@segment/analytics-react-native/build/esm/bridge.d.ts:4:31 - error TS2503: Cannot find namespace 'NativeModules'.

4 export declare type JsonMap = NativeModules.RNAnalytics.JsonMap;

This doesn't cause any errors in the app as far as I can see.

[fathyb]

Thanks for the report! The first error is suspicious, what version of TypeScript are you using?

[cristeahub]

I'm currenty on Version 3.0.3.

Update:

Here's my tsconfig.json:

{
  "compilerOptions": {
    "target": "ES2017",
    "module": "es2015",
    "lib": [
      "es2017"
    ],
    "jsx": "react-native",
    "sourceMap": true,
    "outDir": "./build",

    "strict": true,

    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": true,
    "experimentalDecorators": true,

    "moduleResolution": "node",
    "allowSyntheticDefaultImports": true
  },
  "include": [
    "./src/**/*"
  ]
}

[ajmath]

I'm getting a similar error from tsc:

node_modules/@segment/analytics-react-native/src/bridge.ts:3:17 - error TS2503: Cannot find namespace 'NativeModules'.

3 import bridge = NativeModules.RNAnalytics
                  ~~~~~~~~~~~~~

node_modules/@segment/analytics-react-native/src/bridge.ts:10:23 - error TS2503: Cannot find namespace 'NativeModules'.

10 export type JsonMap = NativeModules.RNAnalytics.JsonMap
                         ~~~~~~~~~~~~~

I'm using typescript 3.0.3 with a tsconfig.json that looks like this:

{
  "compilerOptions": {
    "target": "es2017",
    "jsx": "react-native",
    "noEmit": true,
    "moduleResolution": "node",
    "module": "commonjs",
    "strict": true,
    "noUnusedLocals": true,
    "skipLibCheck": true,
    "forceConsistentCasingInFileNames": true,
    "noImplicitReturns": true,
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
    "resolveJsonModule": true,
    "allowJs": true,
    "checkJs": false,
    "outDir": ".tmp",
    "typeRoots": [
      "node_modules/@types",
      "app/types"
    ],
    "plugins": [
      {
        "name": "tslint-language-service",
        "ignoreDefinitionFiles": true
      }
    ],
    "importHelpers": true
  },
  "include" : [
    "app/**/*",
    "__tests__/**/*"
  ],
  "exclude": [
    "node_modules",
    ".tmp",
    "app/native-base-theme"
  ]
}

[cristeahub]

I'll have to add I'm on 0.0.1-beta.1 of this lib. I updated my other comment to include my tsconfig.

I was getting the same error as @ajmath when I was on version 0.0.1-beta.0.

[ajmath]

I was able to resolve this by changing one of my imports from the module from
import { JsonMap } from "@segment/analytics-react-native/src/bridge"; to import { JsonMap } from "@segment/analytics-react-native/build/esm/bridge";.

I'm not sure if this is possible but maybe you can update the declaration file so that all types can be exported into the default index.d.ts?

[fathyb]

Thanks @cristeahub! We've released 0.0.1-beta.2 with new TypeScript tests and the bridge issues fixed.

@ajmath We've also added export to public interfaces so that you can import them using the entry-point:

import analytics, { Configuration, JsonMap } from '@segment/analytics-react-native'