refactor: add package exports entry

This commit adds the exports entry in each package.json, exposing the public parts of each package and hiding the private ones.

.changeset/red-steaks-brake.md

@@ -0,0 +1,47 @@
+---
+'babel-plugin-extend-docs': minor
+'providence-analytics': minor
+'remark-extend': minor
+'@lion/accordion': minor
+'@lion/ajax': minor
+'@lion/button': minor
+'@lion/calendar': minor
+'@lion/checkbox-group': minor
+'@lion/collapsible': minor
+'@lion/combobox': minor
+'@lion/core': minor
+'@lion/dialog': minor
+'@lion/fieldset': minor
+'@lion/form': minor
+'@lion/form-core': minor
+'@lion/form-integrations': minor
+'@lion/helpers': minor
+'@lion/icon': minor
+'@lion/input': minor
+'@lion/input-amount': minor
+'@lion/input-date': minor
+'@lion/input-datepicker': minor
+'@lion/input-email': minor
+'@lion/input-iban': minor
+'@lion/input-range': minor
+'@lion/input-stepper': minor
+'@lion/listbox': minor
+'@lion/localize': minor
+'@lion/overlays': minor
+'@lion/pagination': minor
+'@lion/progress-indicator': minor
+'@lion/radio-group': minor
+'@lion/select': minor
+'@lion/select-rich': minor
+'singleton-manager': minor
+'@lion/steps': minor
+'@lion/switch': minor
+'@lion/tabs': minor
+'@lion/textarea': minor
+'@lion/tooltip': minor
+'@lion/validate-messages': minor
+---
+
+Add exports field in package.json
+
+Note that some tools can break with this change as long as they respect the exports field. If that is the case, check that you always access the elements included in the exports field, with the same name which they are exported. Any item not exported is considered private to the package and should not be accessed from the outside.

packages-node/babel-plugin-extend-docs/package.json

@@ -30,5 +30,6 @@
   ],
   "publishConfig": {
     "access": "public"
-  }
+  },
+  "exports": "./index.js"
 }

packages-node/providence-analytics/package.json

@@ -65,5 +65,9 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./src/program/providence.js",
+    "./src/cli/index.js": "./src/cli/index.js"
   }
 }

packages-node/remark-extend/package.json

@@ -35,5 +35,6 @@
   ],
   "publishConfig": {
     "access": "public"
-  }
+  },
+  "exports": "./index.js"
 }

packages/accordion/package.json

@@ -42,5 +42,9 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-accordion": "./lion-accordion.js"
   }
 }

packages/ajax/package.json

@@ -39,7 +39,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "exports": {
-    ".": "./index.js"
-  }
+  "exports": "./index.js"
 }

packages/button/package.json

@@ -42,5 +42,9 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-button": "./lion-button.js"
   }
 }

packages/button/src/LionButton.js

@@ -6,7 +6,7 @@ import {
   LitElement,
   SlotMixin,
 } from '@lion/core';
-import '@lion/core/src/differentKeyEventNamesShimIE.js';
+import '@lion/core/differentKeyEventNamesShimIE';
 
 const isKeyboardClickEvent = (/** @type {KeyboardEvent} */ e) => e.key === ' ' || e.key === 'Enter';
 const isSpaceKeyboardClickEvent = (/** @type {KeyboardEvent} */ e) => e.key === ' ';

packages/button/test/lion-button.test.js

@@ -1,7 +1,7 @@
 import { browserDetection } from '@lion/core';
 import { aTimeout, expect, fixture, html, oneEvent, unsafeStatic } from '@open-wc/testing';
 import sinon from 'sinon';
-import '@lion/core/src/differentKeyEventNamesShimIE.js';
+import '@lion/core/differentKeyEventNamesShimIE';
 import '../lion-button.js';
 
 /**

packages/calendar/index.js

@@ -1 +1,2 @@
+export { isSameDate } from './src/utils/isSameDate.js';
 export { LionCalendar } from './src/LionCalendar.js';

packages/calendar/package.json

@@ -43,5 +43,10 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-calendar": "./lion-calendar.js",
+    "./test-helpers": "./test-helpers.js"
   }
 }

packages/calendar/src/LionCalendar.js

@@ -1,12 +1,13 @@
 import { html, LitElement } from '@lion/core';
-import '@lion/core/src/differentKeyEventNamesShimIE.js';
 import {
   getMonthNames,
   getWeekdayNames,
   localize,
   LocalizeMixin,
   normalizeDateTime,
 } from '@lion/localize';
+
+import '@lion/core/differentKeyEventNamesShimIE';
 import { calendarStyle } from './calendarStyle.js';
 import { createDay } from './utils/createDay.js';
 import { createMultipleMonth } from './utils/createMultipleMonth.js';

packages/calendar/test/lion-calendar.test.js

@@ -1,7 +1,7 @@
 import { html } from '@lion/core';
 import '@lion/core/test-helpers/keyboardEventShimIE.js';
 import { localize } from '@lion/localize';
-import { localizeTearDown } from '@lion/localize/test-helpers.js';
+import { localizeTearDown } from '@lion/localize/test-helpers';
 import { expect, fixture as _fixture } from '@open-wc/testing';
 import sinon from 'sinon';
 import '../lion-calendar.js';

packages/checkbox-group/package.json

@@ -46,5 +46,11 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-checkbox": "./lion-checkbox.js",
+    "./lion-checkbox-group": "./lion-checkbox-group.js",
+    "./lion-checkbox-indeterminate": "./lion-checkbox-indeterminate.js"
   }
 }

packages/checkbox-group/test/lion-checkbox-group.test.js

@@ -1,4 +1,4 @@
-import { localizeTearDown } from '@lion/localize/test-helpers.js';
+import { localizeTearDown } from '@lion/localize/test-helpers';
 import { expect, fixture as _fixture, html } from '@open-wc/testing';
 import '../lion-checkbox-group.js';
 import '../lion-checkbox.js';

packages/collapsible/package.json

@@ -45,5 +45,9 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-collapsible": "./lion-collapsible.js"
   }
 }

packages/combobox/package.json

@@ -55,5 +55,9 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-combobox": "./lion-combobox.js"
   }
 }

packages/combobox/test/lion-combobox.test.js

@@ -1,8 +1,8 @@
-import '@lion/listbox/lion-option.js';
+import '@lion/listbox/lion-option';
 import { expect, fixture, html, defineCE, unsafeStatic } from '@open-wc/testing';
 import sinon from 'sinon';
 import '../lion-combobox.js';
-import { LionOptions } from '@lion/listbox/src/LionOptions.js';
+import { LionOptions } from '@lion/listbox';
 import { browserDetection, LitElement } from '@lion/core';
 import { Required } from '@lion/form-core';
 import { LionCombobox } from '../src/LionCombobox.js';

packages/core/closestPolyfill.js

@@ -0,0 +1 @@
+import './src/closestPolyfill.js';

packages/core/differentKeyEventNamesShimIE.js

@@ -0,0 +1 @@
+import './src/differentKeyEventNamesShimIE.js';

packages/core/package.json

@@ -29,7 +29,10 @@
     "prepublishOnly": "../../scripts/npm-prepublish.js",
     "test": "cd ../../ && npm run test:browser -- --group core"
   },
-  "sideEffects": false,
+  "sideEffects": [
+    "./closestPolyfill.js",
+    "./differentKeyEventNamesShimIE.js"
+  ],
   "dependencies": {
     "@open-wc/dedupe-mixin": "^1.2.18",
     "@open-wc/scoped-elements": "^1.3.3",
@@ -42,5 +45,11 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./test-helpers/*": "./test-helpers/*",
+    "./closestPolyfill": "./src/closestPolyfill.js",
+    "./differentKeyEventNamesShimIE": "./src/differentKeyEventNamesShimIE.js"
   }
 }

packages/dialog/package.json

@@ -44,5 +44,9 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-dialog": "./lion-dialog.js"
   }
 }

packages/fieldset/package.json

@@ -44,5 +44,9 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./lion-fieldset": "./lion-fieldset.js"
   }
 }

packages/form-core/package.json

@@ -44,5 +44,12 @@
   ],
   "publishConfig": {
     "access": "public"
+  },
+  "exports": {
+    ".": "./index.js",
+    "./test-helpers/*": "./test-helpers/*",
+    "./test-suites/*": "./test-suites/*",
+    "./lion-field": "./lion-field.js",
+    "./lion-validation-feedback": "./lion-validation-feedback.js"
   }
 }

packages/form-core/src/FormControlMixin.js

@@ -1,5 +1,4 @@
-import { css, dedupeMixin, html, nothing, SlotMixin } from '@lion/core';
-import { DisabledMixin } from '@lion/core/src/DisabledMixin.js';
+import { css, dedupeMixin, html, nothing, SlotMixin, DisabledMixin } from '@lion/core';
 import { FormRegisteringMixin } from './registration/FormRegisteringMixin.js';
 import { getAriaElementsInRightDomOrder } from './utils/getAriaElementsInRightDomOrder.js';
 import { Unparseable } from './validate/Unparseable.js';

packages/form-core/test-suites/ValidateMixinFeedbackPart.suite.js

@@ -1,6 +1,6 @@
 import { LitElement } from '@lion/core';
 import { localize } from '@lion/localize';
-import { localizeTearDown } from '@lion/localize/test-helpers.js';
+import { localizeTearDown } from '@lion/localize/test-helpers';
 import { defineCE, expect, fixture, html, unsafeStatic } from '@open-wc/testing';
 import sinon from 'sinon';
 import { DefaultSuccess, MinLength, Required, ValidateMixin, Validator } from '../index.js';

packages/form-core/test-suites/choice-group/ChoiceGroupMixin.suite.js

@@ -1,6 +1,6 @@
 import { LitElement } from '@lion/core';
 import { LionInput } from '@lion/input';
-import '@lion/fieldset/lion-fieldset.js';
+import '@lion/fieldset/lion-fieldset';
 import { FormGroupMixin, Required } from '@lion/form-core';
 import { expect, html, fixture, unsafeStatic } from '@open-wc/testing';
 import { ChoiceGroupMixin } from '../../src/choice-group/ChoiceGroupMixin.js';

packages/form-core/test-suites/form-group/FormGroupMixin-input.suite.js

@@ -1,8 +1,8 @@
 import { LitElement } from '@lion/core';
-import { localizeTearDown } from '@lion/localize/test-helpers.js';
+import { localizeTearDown } from '@lion/localize/test-helpers';
 import { defineCE, expect, html, unsafeStatic, fixture } from '@open-wc/testing';
 import { LionInput } from '@lion/input';
-import '@lion/form-core/lion-field.js';
+import '@lion/form-core/lion-field';
 import { FormGroupMixin } from '../../src/form-group/FormGroupMixin.js';
 
 /**

packages/form-core/test-suites/form-group/FormGroupMixin.suite.js

@@ -1,5 +1,5 @@
 import { LitElement } from '@lion/core';
-import { localizeTearDown } from '@lion/localize/test-helpers.js';
+import { localizeTearDown } from '@lion/localize/test-helpers';
 import {
   defineCE,
   expect,
@@ -11,7 +11,7 @@ import {
 } from '@open-wc/testing';
 import sinon from 'sinon';
 import { IsNumber, Validator, LionField } from '@lion/form-core';
-import '@lion/form-core/lion-field.js';
+import '@lion/form-core/lion-field';
 import { FormGroupMixin } from '../../src/form-group/FormGroupMixin.js';
 
 /**

packages/form-core/test/lion-field.test.js

@@ -1,6 +1,6 @@
 import { unsafeHTML } from '@lion/core';
 import { localize } from '@lion/localize';
-import { localizeTearDown } from '@lion/localize/test-helpers.js';
+import { localizeTearDown } from '@lion/localize/test-helpers';
 import { Required, Validator } from '@lion/form-core';
 import {
   expect,

packages/form-integrations/package.json

@@ -64,5 +64,6 @@
   ],
   "publishConfig": {
     "access": "public"
-  }
+  },
+  "exports": "./index.js"
 }

packages/form-integrations/test/helpers/umbrella-form.js

@@ -1,24 +1,24 @@
 import { LitElement, html } from '@lion/core';
 import { Required, MinLength } from '@lion/form-core';
-import '@lion/form/lion-form.js';
-import '@lion/fieldset/lion-fieldset.js';
-import '@lion/input/lion-input.js';
-import '@lion/input-date/lion-input-date.js';
-import '@lion/input-datepicker/lion-input-datepicker.js';
-import '@lion/input-amount/lion-input-amount.js';
-import '@lion/input-iban/lion-input-iban.js';
-import '@lion/input-email/lion-input-email.js';
-import '@lion/checkbox-group/lion-checkbox-group.js';
-import '@lion/checkbox-group/lion-checkbox.js';
-import '@lion/radio-group/lion-radio-group.js';
-import '@lion/radio-group/lion-radio.js';
-import '@lion/select/lion-select.js';
-import '@lion/select-rich/lion-select-rich.js';
-import '@lion/select-rich/lion-options.js';
-import '@lion/select-rich/lion-option.js';
-import '@lion/input-range/lion-input-range.js';
-import '@lion/textarea/lion-textarea.js';
-import '@lion/button/lion-button.js';
+import '@lion/form/lion-form';
+import '@lion/fieldset/lion-fieldset';
+import '@lion/input/lion-input';
+import '@lion/input-date/lion-input-date';
+import '@lion/input-datepicker/lion-input-datepicker';
+import '@lion/input-amount/lion-input-amount';
+import '@lion/input-iban/lion-input-iban';
+import '@lion/input-email/lion-input-email';
+import '@lion/checkbox-group/lion-checkbox-group';
+import '@lion/checkbox-group/lion-checkbox';
+import '@lion/radio-group/lion-radio-group';
+import '@lion/radio-group/lion-radio';
+import '@lion/select/lion-select';
+import '@lion/select-rich/lion-select-rich';
+import '@lion/select-rich/lion-options';
+import '@lion/select-rich/lion-option';
+import '@lion/input-range/lion-input-range';
+import '@lion/textarea/lion-textarea';
+import '@lion/button/lion-button';
 
 export class UmbrellaForm extends LitElement {
   get _lionFormNode() {