fix(types): keep prop-types out of the generated public declarations

tsc was inferring the static propTypes objects as PropTypes.Requireable<...>, which emitted `import PropTypes from 'prop-types'` into the shipped .d.ts and forced consumers to install @types/prop-types — a type dependency the v4.5.0 hand-written typings never had. Annotate both classes' propTypes as {[key: string]: unknown} so the inferred PropTypes types stay out of the declarations. (DraggableCore had this annotation originally; it was dropped in an earlier cleanup, and Draggable never had it.)

Add a Contract 3 to scripts/verify-build.cjs that fails the build if any emitted .d.ts references 'prop-types' — the type-compat test compiles against source (where prop-types resolves) so it cannot catch a consumer-facing leak; this guard inspects the actual artifacts. Verified by installing the packed tarball into a clean project with no @types/prop-types and typechecking a consumer under both bundler and node16 resolution.

Author: STRML

lib/Draggable.tsx

@@ -51,7 +51,12 @@ class Draggable extends React.Component<Partial<DraggableProps>, DraggableState>
 
   static displayName?: string = 'Draggable';
 
-  static propTypes = {
+  // The index-signature annotation is load-bearing: without it tsc infers the
+  // PropTypes.Requireable<...> types and emits `import PropTypes from 'prop-types'`
+  // into the generated public .d.ts, forcing consumers to install
+  // @types/prop-types (the v4.5.0 hand-written typings had no prop-types dep).
+  // Do not remove. See lib/DraggableCore.tsx for the same guard.
+  static propTypes: {[key: string]: unknown} = {
     // Accepts all props <DraggableCore> accepts.
     ...DraggableCore.propTypes,
 

lib/DraggableCore.tsx

@@ -73,7 +73,12 @@ export default class DraggableCore extends React.Component<Partial<DraggableCore
 
   static displayName: string | undefined = 'DraggableCore';
 
-  static propTypes = {
+  // The index-signature annotation is load-bearing: without it tsc infers the
+  // PropTypes.Requireable<...> types and emits `import PropTypes from 'prop-types'`
+  // into the generated public .d.ts, forcing consumers to install
+  // @types/prop-types (the v4.5.0 hand-written typings had no prop-types dep).
+  // Do not remove. See lib/Draggable.tsx for the same guard.
+  static propTypes: {[key: string]: unknown} = {
     /**
      * `allowAnyClick` allows dragging using any mouse button.
      * By default, we only accept the left button.

scripts/verify-build.cjs

@@ -56,6 +56,26 @@ assert.ok(
   'UMD bundle must expose the global `ReactDraggable`'
 );
 
+// ── Contract 3: generated declarations must not leak internal-only deps ──────
+// The shipped .d.ts is generated from source. The v4.5.0 hand-written typings
+// depended only on `react`; if a `propTypes`/`defaultProps` static loses its
+// index-signature annotation, tsc infers PropTypes.* types and emits
+// `import ... 'prop-types'` into the public declaration, silently forcing
+// consumers to install @types/prop-types. Fail the build if that creeps back.
+const dtsDir = path.join(root, 'build', 'cjs');
+const dtsFiles = fs
+  .readdirSync(dtsDir)
+  .filter((f) => f.endsWith('.d.ts') || f.endsWith('.d.mts'));
+const leaks = dtsFiles.filter((f) =>
+  /['"]prop-types['"]/.test(fs.readFileSync(path.join(dtsDir, f), 'utf8'))
+);
+assert.equal(
+  leaks.length,
+  0,
+  `Generated declarations leak 'prop-types' (consumers would need @types/prop-types): ${leaks.join(', ')}. ` +
+    `Annotate the offending static (e.g. \`static propTypes: {[key: string]: unknown}\`) so tsc does not emit PropTypes types.`
+);
+
 console.log(
-  '✓ build contract OK: CJS module.exports===Draggable (+.default, .DraggableCore); UMD global ReactDraggable'
+  '✓ build contract OK: CJS module.exports===Draggable (+.default, .DraggableCore); UMD global ReactDraggable; no prop-types leak in .d.ts'
 );