-
Notifications
You must be signed in to change notification settings - Fork 18
Release 5.0.0 with plugin API #228
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 56 commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
9dd9621
Release 5.0.0 with plugin API and IIFE entry refactor
jaspk06 d20a657
improve type hints in api
jaspk06 1bbf533
support multiple plugins
jaspk06 ea8e446
separate maps + autocomplete ui kit
jaspk06 f2859de
tsconfig.test.json, and update TypeScript and Jest config for new build
jaspk06 af0e6ed
cleanup type
jaspk06 9698d06
cleanup type
jaspk06 9124fe3
Update GitHub Actions to use Node 24 and latest actions
jaspk06 7a797e0
Update dev dependencies and exports, fix LICENSE typo, remove unused
jaspk06 0cbbc1e
upgrade rollup
jaspk06 e401551
upgrade mock xml http request
jaspk06 7aefb78
upgrade cspell
jaspk06 055247d
typecheck package.json
jaspk06 02f3149
rm unused
jaspk06 e68047c
plugins + plugins release actions
jaspk06 085ee7e
test fix
jaspk06 1f2cc9f
actually fix tests
jaspk06 2a773ad
bump version
jaspk06 90a8a74
revert version
jaspk06 0a76bbc
bump dev deps
jaspk06 850fc1a
rename
jaspk06 a1cdcc7
properly bundle css
jaspk06 150e29f
bump plugin versions
jaspk06 c38d770
plugin types
jaspk06 3ff3b1b
cleanup rollup and tsconfigs
jaspk06 e2f514e
bump versions
jaspk06 429c243
expose RadarError class publically
Luke-zhang-04 855a139
set Radar constant as global in IIFE CDN bundles
Luke-zhang-04 adf30a1
Merge branch 'luke/external-core-sdk' into jasonliu/sdk-split-spike
Luke-zhang-04 6dedba6
fix Radar.ui type union
jaspk06 d678f32
rename radar-map css
jaspk06 50f1521
version bump
jaspk06 f12e647
update agents + symlink claude
jaspk06 87f5ddc
ts rollup, fix build, track fraud param
jaspk06 f1bb14a
docs
jaspk06 370164f
fraud plugin docs
jaspk06 67073a0
revert demo pages
jaspk06 1b81460
update demo pages
jaspk06 ff075cd
tsconfig nits
jaspk06 da22606
update ci
jaspk06 cbfdf96
typecheck failure
jaspk06 b1f4ba4
try again
jaspk06 622f642
radar-map.css -> radar-maps.css
jaspk06 9eae8c4
rename + bump version
jaspk06 6ff4823
jsdoc
jaspk06 8efbbd6
http client v2
jaspk06 d8a6b8e
fix xhr jsdoc
jaspk06 e385f2f
bump version
jaspk06 ebee7c4
rm version jsdoc
jaspk06 8ea7495
bump versions
jaspk06 0bd09a3
lint
jaspk06 4fa1c28
fmt
jaspk06 fa44df6
jsdoc on plugins
jaspk06 ab643c8
updates AGENTS.md
jaspk06 44200ab
consitent type imports
jaspk06 40945fb
lint after build
jaspk06 297057c
update readmes
jaspk06 973700c
address codeql
jaspk06 1e3c0d5
fmt
jaspk06 9618325
check if error was thrown for tests that expect errors
Luke-zhang-04 f58c905
check for response.ok instead of only 200
Luke-zhang-04 c17c323
pass requestId to autocomplete API
Luke-zhang-04 1aff3d9
fix abort controller race condition
Luke-zhang-04 4c4c9a5
export all types from packages
Luke-zhang-04 b6ded6d
narrow types for errors with constants
Luke-zhang-04 5500c6a
handle parse failure in http
Luke-zhang-04 2092038
pass missing location value to trigger test error
Luke-zhang-04 0cbe80d
Merge branch 'luke/sdk-split-fixes' of github.com:radarlabs/radar-sdk…
Luke-zhang-04 e8b51d2
Config.getConfig return promise
jaspk06 ee39267
code comment
jaspk06 358b0bd
narrow types for errors with constants
Luke-zhang-04 d72c162
remove fraud params from trackOnce(), add startTrackingVerified() dem…
nickpatrick c355fdc
Merge branch 'jasonliu/sdk-split-spike' of github.com:radarlabs/radar…
jaspk06 2604622
fix race condition in autocomplete UI
Luke-zhang-04 6a206b5
Merge pull request #232 from radarlabs/luke/sdk-split-fixes
Luke-zhang-04 cfe5ce0
fix config return type, TrackBodyExtension type
jaspk06 4775cab
bump version
jaspk06 995570b
export errors, use Radar.VERSION rather than context
jaspk06 8199c72
bump versions
jaspk06 8d426b5
export all errors
Luke-zhang-04 a5e6e56
bump versions
jaspk06 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,16 +1,101 @@ | ||
| version: 2 | ||
| version: 2.1 | ||
|
|
||
| jobs: | ||
| build: | ||
| install: | ||
| docker: | ||
| - image: cimg/node:16.15 | ||
| - image: cimg/node:24.13.1 | ||
| steps: | ||
| - checkout | ||
| - restore_cache: | ||
| keys: | ||
| - npm-deps-{{ checksum "package-lock.json" }} | ||
| - run: | ||
| name: Install Dependencies | ||
| name: Install dependencies | ||
| command: npm ci | ||
| - save_cache: | ||
| key: npm-deps-{{ checksum "package-lock.json" }} | ||
| paths: | ||
| - node_modules | ||
| - persist_to_workspace: | ||
| root: . | ||
| paths: | ||
| - . | ||
|
|
||
| typecheck: | ||
| docker: | ||
| - image: cimg/node:24.13.1 | ||
| steps: | ||
| - attach_workspace: | ||
| at: . | ||
| - run: | ||
| name: Typecheck | ||
| command: npm run typecheck | ||
|
|
||
| build: | ||
| docker: | ||
| - image: cimg/node:24.13.1 | ||
| steps: | ||
| - attach_workspace: | ||
| at: . | ||
| - run: | ||
| name: Build all packages | ||
| command: npm run build:all | ||
| - persist_to_workspace: | ||
| root: . | ||
| paths: | ||
| - dist | ||
| - packages/maps/dist | ||
| - packages/autocomplete/dist | ||
|
|
||
| fmt: | ||
| docker: | ||
| - image: cimg/node:24.13.1 | ||
| steps: | ||
| - attach_workspace: | ||
| at: . | ||
| - run: | ||
| name: Format check | ||
| command: npm run fmt:check | ||
|
|
||
| lint: | ||
| docker: | ||
| - image: cimg/node:24.13.1 | ||
| steps: | ||
| - attach_workspace: | ||
| at: . | ||
| - run: | ||
| name: Lint | ||
| command: npm run lint | ||
|
|
||
| test: | ||
| docker: | ||
| - image: cimg/node:24.13.1 | ||
| steps: | ||
| - attach_workspace: | ||
| at: . | ||
| - run: | ||
| name: Run Tests | ||
| name: Run tests | ||
| command: npm run test | ||
| - store_artifacts: | ||
| path: coverage | ||
| prefix: coverage | ||
| destination: coverage | ||
|
|
||
| workflows: | ||
| ci: | ||
| jobs: | ||
| - install | ||
| - build: | ||
| requires: | ||
| - install | ||
| - fmt: | ||
| requires: | ||
| - install | ||
| - test: | ||
| requires: | ||
| - install | ||
| - lint: | ||
| requires: | ||
| - build | ||
| - typecheck: | ||
| requires: | ||
| - build | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| { | ||
| "permissions": { | ||
| "allow": [ | ||
| "Bash(npm run build:*)", | ||
| "Bash(npm test:*)", | ||
| "Bash(npm run typecheck:*)", | ||
| "Bash(npm run build:maps:*)", | ||
| "Bash(npm run build:all:*)", | ||
| "Bash(npm run lint:*)", | ||
| "Bash(npm run fmt:check:*)" | ||
| ] | ||
| } | ||
| } |
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
| name: Publish beta plugin package to npmjs | ||
| on: | ||
| release: | ||
| types: [prereleased] | ||
| jobs: | ||
| build: | ||
| if: ${{ !startsWith(github.ref_name, 'v') }} | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v6 | ||
| - uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: '24.x' | ||
| registry-url: 'https://registry.npmjs.org' | ||
| - name: Parse plugin tag | ||
| run: | | ||
| TAG="${{ github.ref_name }}" | ||
| PLUGIN_NAME="${TAG%-v*}" | ||
| PLUGIN_VERSION="${TAG#*-v}" | ||
| echo "PLUGIN_NAME=$PLUGIN_NAME" >> $GITHUB_ENV | ||
| echo "PLUGIN_VERSION=$PLUGIN_VERSION" >> $GITHUB_ENV | ||
| echo "Parsed tag: plugin=$PLUGIN_NAME version=$PLUGIN_VERSION" | ||
| - run: npm ci --force | ||
| - run: npm run build | ||
| - run: npm run build:${{ env.PLUGIN_NAME }} | ||
| - run: node scripts/check-plugin-tag.cjs ${{ env.PLUGIN_NAME }} ${{ env.PLUGIN_VERSION }} beta | ||
| - run: npm publish -w packages/${{ env.PLUGIN_NAME }} --tag beta | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
| - uses: jakejarvis/s3-sync-action@master | ||
| env: | ||
| AWS_S3_BUCKET: js.radar.com | ||
| AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }} | ||
| AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | ||
| AWS_REGION: 'us-east-1' | ||
| SOURCE_DIR: packages/${{ env.PLUGIN_NAME }}/cdn/ | ||
| DEST_DIR: ${{ env.PLUGIN_NAME }}/v${{ env.PLUGIN_VERSION }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,49 @@ | ||
| name: Publish plugin package to npmjs | ||
| on: | ||
| release: | ||
| types: [released] | ||
| jobs: | ||
| build: | ||
| if: ${{ !startsWith(github.ref_name, 'v') }} | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v6 | ||
| - uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: '24.x' | ||
| registry-url: 'https://registry.npmjs.org' | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
| - name: Parse plugin tag | ||
| run: | | ||
| TAG="${{ github.ref_name }}" | ||
| PLUGIN_NAME="${TAG%-v*}" | ||
| PLUGIN_VERSION="${TAG#*-v}" | ||
| MAJOR_VERSION="${PLUGIN_VERSION%%.*}" | ||
| echo "PLUGIN_NAME=$PLUGIN_NAME" >> $GITHUB_ENV | ||
| echo "PLUGIN_VERSION=$PLUGIN_VERSION" >> $GITHUB_ENV | ||
| echo "MAJOR_VERSION=$MAJOR_VERSION" >> $GITHUB_ENV | ||
| echo "Parsed tag: plugin=$PLUGIN_NAME version=$PLUGIN_VERSION major=$MAJOR_VERSION" | ||
| - run: npm ci --force | ||
| - run: npm run build | ||
| - run: npm run build:${{ env.PLUGIN_NAME }} | ||
| - run: node scripts/check-plugin-tag.cjs ${{ env.PLUGIN_NAME }} ${{ env.PLUGIN_VERSION }} stable | ||
| - run: npm publish -w packages/${{ env.PLUGIN_NAME }} | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
| - uses: jakejarvis/s3-sync-action@master | ||
| env: | ||
| AWS_S3_BUCKET: js.radar.com | ||
| AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }} | ||
| AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | ||
| AWS_REGION: 'us-east-1' | ||
| SOURCE_DIR: packages/${{ env.PLUGIN_NAME }}/cdn/ | ||
| DEST_DIR: ${{ env.PLUGIN_NAME }}/v${{ env.PLUGIN_VERSION }} | ||
| - uses: jakejarvis/s3-sync-action@master | ||
| env: | ||
| AWS_S3_BUCKET: js.radar.com | ||
| AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }} | ||
| AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | ||
| AWS_REGION: 'us-east-1' | ||
| SOURCE_DIR: packages/${{ env.PLUGIN_NAME }}/cdn/ | ||
| DEST_DIR: ${{ env.PLUGIN_NAME }}/${{ env.MAJOR_VERSION }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CircleCI workflow persists
cdn/directory from the build job (line 46-48 covers dist folders but not cdn/). If CDN builds are needed for subsequent jobs or artifacts, addcdnandpackages/*/cdnto the persist_to_workspace paths list.