Skip to content

Fixes single import types defintion requirement for docs#195

Open
clairebaire wants to merge 2 commits into
masterfrom
users/clipskey/fix-doc-pipeline
Open

Fixes single import types defintion requirement for docs#195
clairebaire wants to merge 2 commits into
masterfrom
users/clipskey/fix-doc-pipeline

Conversation

@clairebaire
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the packaging build script to ensure the generated root type definitions reference all API-area modules, addressing docs tooling that only reads the root types entry from the package exports map.

Changes:

  • Discovers “API area” subdirectories under src/ (directories containing index.ts) once and reuses the list for multiple packaging steps.
  • Appends type-only, namespace-style re-exports for each API area into the generated root index.d.ts so docs tooling can “see” all areas through a single types entry.
  • Refactors package.json#exports generation to use the shared API-area discovery list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread buildpackage.js Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@craftytrilobite craftytrilobite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of the comments are difficult to understand/verify- e.g. "// Skips Common because src/index.ts already wildcard-re-exports it at the root for backward compat," but I don't see anything terribly dangerous here- approved.

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.

3 participants