ReviewOS

also looking at this

craft-native/craft

fix(sdk): ship declarations that resolve under node16/nodenext

#42
Merged glennmichael123 wants to merge fix/node16-type-resolution into main
43 files +196 -137
packages/ts-maps/src/MapView.tsmodified+7-7
Changes to packages/ts-maps/src/MapView.ts
@@ -10,8 +10,8 @@
1010// package root, so we don't pull the entire `craft-native` surface
1111// (including unrelated stx-templated sidebar files) through the type
1212// checker. The public API of the bridge is stable, so this is fine.
13import type { NativeBridge } from '../../typescript/src/bridge/core'
14import { getBridge } from '../../typescript/src/bridge/core'
13import type { NativeBridge } from '../../typescript/src/bridge/core.js'
14import { getBridge } from '../../typescript/src/bridge/core.js'
1515import type { TsMap } from 'ts-maps'
1616import { createMap, tileLayer } from 'ts-maps'
1717import {
@@ -23,12 +23,12 @@ import {
2323 craftPolylineToPolyline,
2424 craftRegionToBounds,
2525 latLngToCraftCoord,
26} from './adapters'
26} from './adapters.js'
2727import type {
2828 MapBridgeEvent,
2929 TypedMapBridgeRequest,
30} from './bridge-protocol'
31import { BRIDGE_NAMESPACE, mapEventName } from './bridge-protocol'
30} from './bridge-protocol.js'
31import { BRIDGE_NAMESPACE, mapEventName } from './bridge-protocol.js'
3232import type {
3333 Coordinate,
3434 MapCamera,
@@ -38,8 +38,8 @@ import type {
3838 MapPolygon,
3939 MapPolyline,
4040 MapRegion,
41} from './types'
42import { defaultMapCamera, defaultMapConfiguration } from './types'
41} from './types.js'
42import { defaultMapCamera, defaultMapConfiguration } from './types.js'
4343
4444// Monotonic counter — same pattern as `components/native.ts` in
4545// `craft-native` — so multiple map views created in the same tick