fix(build): scope tsconfig.build.json to src/ for reproducibility

[ktrz]

Ticket

#351 — tsconfig.build.json compiles scripts/ into dist/ but release workflow excludes them — rebuild reproducibility breaks

Description

• tsconfig.build.json previously used "include": ["**/*.ts"], which compiled scripts/ci_validate_modification.ts into dist/scripts/. The release workflow (scripts/prepare-release.sh) only copies dist/src/ to the release tag, so anyone rebuilding from source got an extra dist/scripts/ci_validate_modification.js not present in the published artifact — failing the Apache Software Foundation's verify-action-build reproducibility checker.
• Tightened include to ["src/**/*.ts"] so npm run build no longer emits dist/scripts/. Set "rootDir": "." explicitly so src/index.ts continues to emit to dist/src/index.js (without rootDir, narrowing the include set lets TypeScript infer src/ as the common root and emit flat under dist/, breaking the release script's dist/src/ layout).
• CI still needs to run scripts/ci_validate_modification.ts. Switched both .github/workflows/ci.yml and .github/workflows/ci-results-repo.yml from node ./dist/scripts/ci_validate_modification.js … to npx tsx scripts/ci_validate_modification.ts … (22 occurrences total). Added tsx to devDependencies.

Fixes#351.

Test scenario

• rm -rf dist && npm run build — verify dist/ contains only src/ (no dist/scripts/).
• ls dist/src/index.js — verify the release-shaped layout is preserved.
• npx tsx scripts/ci_validate_modification.ts (no args) — verify the script runs and exits non-zero with its usage error (proves CI invocation works).
• npm test and npm run lint — verify no regressions.
• CI green on this PR — confirms the tsx-driven validation step still works across all benchmark jobs.

Summary by CodeRabbit

• Chores
  • Updated CI/CD workflows to improve build reliability and consistency.
  • Added development tooling dependency for enhanced script execution.
  • Refined build configuration to streamline the compilation process.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 10353e16-59c9-4377-b480-b3b205ba4aee

📥 Commits

Reviewing files that changed from the base of the PR and between 5aa86d3 and 3fb394b.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (4)

• .github/workflows/ci-results-repo.yml
• .github/workflows/ci.yml
• package.json
• tsconfig.build.json

---

📝 Walkthrough

Walkthrough

The PR fixes a reproducibility issue where scripts/ci_validate_modification.ts was being compiled into dist/ despite being CI-internal tooling. The fix narrows TypeScript compilation to src/ only, adds tsx as a devDependency, and updates all CI workflows to invoke the script directly via npx tsx instead of through the dist/ version.

Changes

Build Reproducibility & CI Script Invocation

Layer / File(s)	Summary
Build Configuration tsconfig.build.json	TypeScript compilation scope narrowed from **/*.ts to src/**/*.ts, excluding scripts/ from the built dist artifact and restoring reproducibility with published releases.
Dependency Addition package.json	Added tsx (^4.21.0) as a devDependency to enable direct execution of TypeScript scripts without pre-compilation.
Workflow Updates .github/workflows/ci-results-repo.yml, .github/workflows/ci.yml	All 11 benchmark workflow steps updated to run npx tsx scripts/ci_validate_modification.ts directly instead of invoking the previously-compiled node ./dist/scripts/ci_validate_modification.js, aligning with the new build scope.

Sequence Diagram

This change does not meet the criteria for a sequence diagram because it consists primarily of configuration and workflow updates without introducing new multi-component interactions or significant control flow modifications.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A script was lost in the dist compile,
But rebuild and publish were out of style—
Now TypeScript runs fresh, no dusty cache,
Reproducible builds, pristine and brash!
The rabbit hops forth with tsx so neat, ✨

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: scoping tsconfig.build.json to src/ for reproducibility, which directly addresses the PR's core objective.
Linked Issues check	✅ Passed	The PR fully addresses issue #351's requirements: it narrows tsconfig.build.json include to src/**/*.ts, sets rootDir to preserve dist/src/ layout, updates CI workflows to use npx tsx for script execution, and adds tsx as a devDependency.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to fixing the reproducibility issue: tsconfig.build.json refinement, CI workflow updates for script execution, and the necessary tsx dependency addition are all aligned with issue #351.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ktrz/351-tsconfig-build-reproducibility-phase-1

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.24%. Comparing base (986b555) to head (3fb394b).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files

@@ Coverage Diff @@## master #352 +/- ##
=======================================
Coverage 89.24% 89.24% =======================================
Files 16 16 Lines 958 958 Branches 205 205 =======================================
Hits 855 855 + Misses 103 99 -4 - Partials 0 4 +4 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.