refactor(dicom): unify loading and extract core and vtk modules

.github/workflows/checks.yml

@@ -18,9 +18,11 @@ jobs:
           cache: npm
       - run: npm ci
       - name: Enforce code style
-        run: npx prettier --list-different "src/**/*.[jt]s" "tests/**/*.[jt]s" "src/**/*.vue"
+        run: npx prettier --list-different "src/**/*.[jt]s" "tests/**/*.[jt]s" "src/**/*.vue" "packages/dicom/**/*.{ts,mjs,json,md}"
       - name: Lint
         run: npm run lint
+      - name: Build independent DICOM consumers
+        run: npm run dicom:check
       - name: Quality ratchets
         # A file may not get worse than it is on the pull request's target branch.
         if: github.event_name == 'pull_request'

.github/workflows/e2e-datasets.yml

@@ -22,7 +22,7 @@ jobs:
         uses: actions/cache@v4
         with:
           path: .tmp/datasets
-          key: e2e-datasets-${{ hashFiles('tests/datasets.ts') }}
+          key: e2e-datasets-${{ hashFiles('tests/datasets.ts', 'tests/dicomVolumeCorpus.json', 'tests/dicomCompositionCorpus.json') }}
           restore-keys: e2e-datasets-
       - name: Download missing datasets
         run: node tests/downloadDatasets.ts

.github/workflows/e2e.yml

@@ -28,7 +28,7 @@ jobs:
         uses: actions/cache@v4
         with:
           path: .tmp/datasets
-          key: e2e-datasets-${{ hashFiles('tests/datasets.ts') }}
+          key: e2e-datasets-${{ hashFiles('tests/datasets.ts', 'tests/dicomVolumeCorpus.json', 'tests/dicomCompositionCorpus.json') }}
           restore-keys: e2e-datasets-
       - name: Run E2E on chrome (linux)
         if: matrix.os == 'ubuntu-latest'

.gitignore

@@ -36,3 +36,5 @@ reports
 .github/copilot
 CLAUDE.md
 .claude/
+
+/packages/dicom/dist/

.prettierignore

@@ -1,6 +1,5 @@
 dist
 node_modules
 public
-src/io/itk-dicom
 src/io/resample
 *.json

eslint.config.js

@@ -64,6 +64,11 @@ const ownFeatureExceptPure = (feature) => {
 const upperLayerMessage = (feature) =>
   `The ${feature.dir} pure layer must not import stores, components, or upper feature modules. Dependencies point downward only.`;
 
+const dicomPublicSurface = {
+  regex: '^@/packages/dicom/(?!core(?:/index)?$|vtk(?:/index)?$)',
+  message: 'Use the DICOM core or vtk public entry point.',
+};
+
 const featureBoundaries = (features) => {
   const publicSurface = ({ dir }) => ({
     group: [`@/src/${dir}/*`, `@/src/${dir}/*/**`, `!@/src/${dir}/index`],
@@ -72,8 +77,10 @@ const featureBoundaries = (features) => {
   const publicFeatures = features.filter(
     (feature) => feature.publicSurface !== false
   );
-  const otherSurfaces = (feature) =>
-    publicFeatures.filter((other) => other !== feature).map(publicSurface);
+  const otherSurfaces = (feature) => [
+    dicomPublicSurface,
+    ...publicFeatures.filter((other) => other !== feature).map(publicSurface),
+  ];
 
   return [
     {
@@ -82,7 +89,12 @@ const featureBoundaries = (features) => {
       rules: {
         'no-restricted-imports': [
           'error',
-          { patterns: publicFeatures.map(publicSurface) },
+          {
+            patterns: [
+              dicomPublicSurface,
+              ...publicFeatures.map(publicSurface),
+            ],
+          },
         ],
       },
     },
@@ -172,10 +184,10 @@ const E2E_TIMEOUT_DEBT = [
 export default tseslint.config(
   {
     ignores: [
-      'src/io/itk-dicom/emscripten-build/**',
       'src/io/resample/emscripten-build/**',
       '**/*.d.ts',
       'dist/**',
+      'packages/dicom/dist/**',
       'node_modules/**',
     ],
   },
@@ -293,7 +305,7 @@ export default tseslint.config(
     },
   },
   {
-    files: ['src/vtk/**/*.{js,ts}'],
+    files: ['src/vtk/**/*.{js,ts}', 'packages/dicom/vtk/**/*.{js,ts}'],
     rules: {
       'no-param-reassign': [
         'error',
@@ -365,8 +377,9 @@ export default tseslint.config(
   // level because the ratchet only fails a file whose debt grows, so existing
   // files stay over these numbers without failing `npm run lint`.
   {
-    files: ['src/**/*.{js,ts,vue}'],
+    files: ['src/**/*.{js,ts,vue}', 'packages/dicom/**/*.{js,ts}'],
     ignores: [
+      'packages/dicom/**/__tests__/**',
       'src/**/__tests__/**',
       'src/**/*.{spec,test}.{js,ts}',
       'src/**/*.d.ts',
@@ -382,5 +395,26 @@ export default tseslint.config(
       ],
     },
   },
+  ...['core', 'vtk'].map((entry) => ({
+    files: [`packages/dicom/${entry}/**/*.{js,ts}`],
+    ignores: ['**/__tests__/**'],
+    rules: {
+      'no-restricted-imports': [
+        'error',
+        {
+          patterns: [
+            {
+              regex:
+                entry === 'core'
+                  ? '^(@/|@src/|vue(?:/|$)|pinia(?:/|$)|@vueuse/|@kitware/vtk\\.js(?:/|$))'
+                  : '^(@/|@src/|vue(?:/|$)|pinia(?:/|$)|@vueuse/)',
+              message:
+                'DICOM package internals use relative imports and cannot depend on application state or UI frameworks. Core cannot depend on vtk.js.',
+            },
+          ],
+        },
+      ],
+    },
+  })),
   eslintConfigPrettier
 );

package.json

@@ -7,7 +7,11 @@
   "files": [
     "dist",
     "src",
-    "backend-contract"
+    "backend-contract",
+    "packages/dicom/core/**/*.ts",
+    "packages/dicom/vtk/**/*.ts",
+    "packages/dicom/README.md",
+    "!packages/dicom/**/__tests__/**"
   ],
   "scripts": {
     "dev": "cross-env VITE_SHOW_SAMPLE_DATA=true vite",
@@ -21,17 +25,17 @@
     "test:e2e:chrome": "cross-env VITE_SHOW_SAMPLE_DATA=true npm run build && wdio run ./wdio.chrome.conf.ts",
     "test:e2e:chrome:skip-build": "wdio run ./wdio.chrome.conf.ts",
     "test:e2e:dev": "cross-env VITE_SHOW_SAMPLE_DATA=true concurrently -P \"npm run dev\"  \"wdio run ./wdio.dev.conf.ts --watch {@}\"",
-    "lint": "vue-tsc --noEmit && eslint \"src/**/*.{js,ts,vue}\" \"tests/**/*.{js,ts}\" \"scripts/**/*.mjs\"",
-    "build:all": "npm run build:dicom && npm run build:resample && npm run build",
-    "build:dicom": "itk-wasm -s src/io/itk-dicom/ build ",
-    "build:dicom:debug": "itk-wasm -s src/io/itk-dicom/ build -- -DCMAKE_BUILD_TYPE=Debug",
+    "lint": "vue-tsc --noEmit && eslint \"src/**/*.{js,ts,vue}\" \"tests/**/*.{js,ts}\" \"scripts/**/*.mjs\" \"packages/dicom/**/*.{js,ts,mjs}\"",
+    "build:all": "npm run build:resample && npm run build",
     "build:resample": "itk-wasm -s src/io/resample/ build ",
     "build:resample:debug": "itk-wasm -s src/io/resample/ build -- -DCMAKE_BUILD_TYPE=Debug",
     "build:watch": "npm run build -- --watch",
-    "prettify": "prettier --write src tests",
+    "prettify": "prettier --write src tests packages/dicom",
     "docs:dev": "vitepress dev docs",
     "docs:build": "vitepress build docs",
-    "docs:preview": "vitepress preview docs"
+    "docs:preview": "vitepress preview docs",
+    "dicom:build": "node packages/dicom/scripts/build.mjs",
+    "dicom:check": "npm run dicom:build && node packages/dicom/scripts/check-consumers.mjs"
   },
   "devDependencies": {
     "@commitlint/cli": "^21.0.0",
@@ -72,6 +76,7 @@
     "core-js": "3.47.0",
     "cors": "^2.8.5",
     "cross-env": "^10.1.0",
+    "dcmjs": "^0.52.0",
     "dicom-parser": "^1.8.21",
     "dicomweb-client-typed": "^0.8.6",
     "eslint": "^9.39.1",