Skip to content

Fix docs build: remove NODE_OPTIONS strip-types flag#307

Merged
brianmhunt merged 1 commit into
mainfrom
fix/docs-build-strip-types
Apr 14, 2026
Merged

Fix docs build: remove NODE_OPTIONS strip-types flag#307
brianmhunt merged 1 commit into
mainfrom
fix/docs-build-strip-types

Conversation

@brianmhunt
Copy link
Copy Markdown
Member

@brianmhunt brianmhunt commented Apr 14, 2026

Summary

The docs build has been failing because NODE_OPTIONS=--experimental-strip-types is not allowed when Bun spawns Node for Astro. Bun handles TypeScript natively — the flag is unnecessary.

Also fixes predev to use bun run instead of npm run.

Verified locally

  • cd tko.io && bun run build — succeeds, 57 pages built
  • All 5 example pages in dist/examples/
  • Glossary and agent docs in dist/agents/

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated development workflow to use bun package manager for build scripts.
    • Removed experimental Node.js configuration flags from development and build processes while maintaining existing telemetry settings.

Bun handles TypeScript natively, so the Node flag is unnecessary
and causes "not allowed in NODE_OPTIONS" error in CI. Also fix
predev to use bun run instead of npm run.

Verified: bun run build succeeds, all 5 examples + glossary in dist.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 14, 2026 18:18
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6a59b8bf-9647-4de6-aa1a-c6c6f75f84f4

📥 Commits

Reviewing files that changed from the base of the PR and between 2ac5909 and 58e5ee9.

📒 Files selected for processing (1)
  • tko.io/package.json

📝 Walkthrough

Walkthrough

The pull request modifies tko.io/package.json to switch the predev script from npm to bun for execution, and removes the NODE_OPTIONS=--experimental-strip-types environment variable from the dev, build, preview, and check scripts.

Changes

Cohort / File(s) Summary
Package Configuration
tko.io/package.json
Updated npm scripts: switched predev to use bun instead of npm, and removed NODE_OPTIONS=--experimental-strip-types from dev, build, preview, and check scripts while preserving ASTRO_TELEMETRY_DISABLED=1.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A swift bun hops into place,
Experimental flags now erase,
Scripts refined with simpler grace,
Development speeds up the race! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing the NODE_OPTIONS strip-types flag from the build scripts. This is the primary fix addressed in the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 fix/docs-build-strip-types

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

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

Copy link
Copy Markdown
Contributor

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

Fixes the tko.io (Astro/Starlight) docs site build when run under Bun by removing an incompatible NODE_OPTIONS=--experimental-strip-types flag, and aligns the predev script with the repo’s Bun-first workflow.

Changes:

  • Remove NODE_OPTIONS=--experimental-strip-types from the Astro dev, build, preview, and check scripts.
  • Switch predev from npm run prebuild to bun run prebuild.

@brianmhunt brianmhunt merged commit c917c2a into main Apr 14, 2026
10 of 11 checks passed
@brianmhunt brianmhunt deleted the fix/docs-build-strip-types branch April 14, 2026 18:28
@coderabbitai coderabbitai Bot mentioned this pull request Apr 15, 2026
3 tasks
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