Skip to content

Build#108

Merged
ralfstx merged 2 commits intomainfrom
build
Mar 8, 2026
Merged

Build#108
ralfstx merged 2 commits intomainfrom
build

Conversation

@ralfstx
Copy link
Copy Markdown
Member

@ralfstx ralfstx commented Mar 8, 2026

No description provided.

ralfstx and others added 2 commits March 8, 2026 21:14
The build script copied the contents of `build/api/` flat into `dist/`
instead of preserving the subdirectory structure. This caused
`dist/index.d.ts` to reference `./api/*.ts` paths that didn't exist,
breaking type resolution for consumers.

Split the copy command to place `index.d.ts` and the `api/` directory
separately, so `dist/api/` is created as a proper subdirectory.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
To detect packaging issues where declaration files are missing from the
`dist/` directory, add a smoke test to the build process that verifies
all module references in `dist/index.d.ts` resolve to existing `.d.ts`
files. This will catch any missing files before publishing.

The inline build command in `package.json` had grown long and hard to
read after adding the declaration file copy and dist verification steps.

Move the build steps into `scripts/build.sh` with each step on its own
line and commented. Add `scripts/check-dist.ts` to verify that all
module references in `dist/index.d.ts` resolve to existing `.d.ts`
files, preventing a repeat of the 0.6.0 packaging bug.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ralfstx ralfstx merged commit a9e7001 into main Mar 8, 2026
1 check passed
@ralfstx ralfstx deleted the build branch March 8, 2026 20:17
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.

1 participant