Skip to content

fix: correct kit package files metadata - #1312

Merged
rmi22186 merged 1 commit into
mainfrom
fix/kit-package-files-typos
Aug 6, 2026
Merged

fix: correct kit package files metadata#1312
rmi22186 merged 1 commit into
mainfrom
fix/kit-package-files-typos

Conversation

@rmi22186

@rmi22186 rmi22186 commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

  • Correct three kit package files entries so published package metadata points to the existing CommonJS bundle names.
  • Align the files entries with each package's main and browser fields.

Test plan

  • Ran a lightweight Node JSON parse/alignment check for the three changed package files.
  • No full build run; metadata-only package.json changes.

@rmi22186
rmi22186 requested a review from a team as a code owner August 6, 2026 18:43
@cursor

cursor Bot commented Aug 6, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Metadata-only package.json changes with no runtime or build logic modifications; low risk aside from ensuring the referenced dist files exist at publish time.

Overview
Fixes npm publish packaging for three web kits by correcting typos in each package’s files array so they match the existing main/browser bundle paths.

Adobe Target: dist/AdobeTarget-Kit.common.jssdist/AdobeTarget-Kit.common.js.

AdWords: dist/GoogleAdWordsEventForwarder.common..jsdist/GoogleAdWordsEventForwarder.common.js.

GA4 client: dist/GoogleAnalyticsEvent4ForwarderClientSide-Kit.common.jsdist/GoogleAnalytics4EventForwarderClientSide-Kit.common.js.

Without these fixes, published tarballs could omit the actual CommonJS artifacts consumers resolve via main/browser.

Reviewed by Cursor Bugbot for commit bc041c9. Bugbot is set up for automated code reviews on this repo. Configure here.

@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

@rmi22186
rmi22186 merged commit 105f8a9 into main Aug 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants