Skip to content

[codex] Fix publish build source tracking#51

Merged
Dhravya merged 1 commit into
mainfrom
codex/fix-publish-build-lib-source
May 8, 2026
Merged

[codex] Fix publish build source tracking#51
Dhravya merged 1 commit into
mainfrom
codex/fix-publish-build-lib-source

Conversation

@Prasanna721
Copy link
Copy Markdown
Member

@Prasanna721 Prasanna721 commented May 7, 2026

What changed

  • Keep lib/validate.ts ignored and explicitly excluded from Biome.
  • Change build:lib to verify the committed lib/validate.js and lib/validate.d.ts artifacts instead of rebuilding from ignored source.
  • Run the full package build in the publish workflow and allow publish to be triggered manually or by lib/workflow changes.

Root cause

The publish workflow was running bun run build:lib, and that script tried to compile lib/validate.ts. That file is intentionally ignored, so a clean GitHub checkout did not contain it and esbuild failed with Could not resolve lib/validate.ts.

Validation

  • bun run build:lib
  • npm run build
  • bun run check-types
  • bun run lint
  • Repeated the same checks from a clean temporary worktree without lib/validate.ts.

@Prasanna721 Prasanna721 force-pushed the codex/fix-publish-build-lib-source branch from 811c9c1 to d5bc36b Compare May 7, 2026 23:52
@Prasanna721 Prasanna721 force-pushed the codex/fix-publish-build-lib-source branch from d5bc36b to 0b0b869 Compare May 8, 2026 00:00
@Prasanna721 Prasanna721 marked this pull request as ready for review May 8, 2026 00:05
@Dhravya Dhravya merged commit 7e817ed into main May 8, 2026
3 checks passed
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.

2 participants