Skip to content

fix(ci): resolve TypeScript composite build failure and clean logger build artifacts - #7

Merged
robo38 merged 3 commits into
mainfrom
@develop/bot
Mar 4, 2026
Merged

robo38 merged 3 commits into
mainfrom
@develop/bot

Conversation

@robo38

@robo38 robo38 commented Mar 4, 2026

Copy link
Copy Markdown
Member

Overview

This pull request fixes the CI TypeScript build failure and cleans up the logger package structure to ensure consistent builds from a completely clean environment.

Changes

CI / Build Fix

  • Removed prepare script from package.json
  • The script executed bun run build during bun install, triggering tsc -p . before referenced projects were built
  • This caused TS6305 errors in CI when dist/ did not exist

Logger Package Cleanup

  • Removed accidentally committed build artifacts from src
  • Deleted the following files:
    • index.js
    • index.d.ts
    • logger.js
    • logger.d.ts
    • utils.js
    • utils.d.ts

These files were generated outputs and should not live next to the source .ts files.

TypeScript Configuration

  • Added libs/logger to the root tsconfig.json project references
  • Ensures proper build ordering when using tsc -b

Result

  • Project builds correctly from a clean environment
  • CI and local builds behave consistently
  • Build artifacts are no longer committed to source directories
image

robo38 added 3 commits March 4, 2026 08:17
remove prepare hook running 'bun run build' during install which triggered tsc before referenced projects were built

delete accidentally committed build artifacts from src (js and d.ts files)

add logger project to root tsconfig references for explicit build ordering

ensure clean builds succeed without dist or tsbuildinfo, matching CI environment
@robo38 robo38 added bug Something isn't working enhancement New feature or request labels Mar 4, 2026
@robo38 robo38 self-assigned this Mar 4, 2026
@robo38
robo38 merged commit feaa6bc into main Mar 4, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant