Skip to content

fix blog:check by adding node types to tsconfig#334

Merged
ElMassimo merged 1 commit into
vite-8from
claude/fix-blog-check-Vy7rZ
Mar 25, 2026
Merged

fix blog:check by adding node types to tsconfig#334
ElMassimo merged 1 commit into
vite-8from
claude/fix-blog-check-Vy7rZ

Conversation

@ElMassimo

Copy link
Copy Markdown
Owner

Summary

  • The blog:check (vue-tsc --noEmit) was failing because process was unresolved in iles.config.ts
  • The playground's tsconfig.json had an explicit types array which restricted auto-inclusion of @types/* packages
  • Adding "node" to the types array allows TypeScript to resolve process.env.DEBUG

Test plan

  • npm run blog:check passes cleanly

https://claude.ai/code/session_01Xn5tftAKYk7pecWHUzrvCD

The tsconfig.json had an explicit `types` array which restricted
auto-inclusion of @types packages. Adding "node" allows TypeScript
to resolve `process.env.DEBUG` used in iles.config.ts.

https://claude.ai/code/session_01Xn5tftAKYk7pecWHUzrvCD
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying iles with  Cloudflare Pages  Cloudflare Pages

Latest commit: b8dd5f1
Status:🚫  Build failed.

View logs

@nx-cloud

nx-cloud Bot commented Mar 25, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit b8dd5f1

Command Status Duration Result
nx run-many --target=build --all --exclude docs... ✅ Succeeded 1s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-25 16:39:32 UTC

@ElMassimo
ElMassimo changed the base branch from main to vite-8 March 25, 2026 16:38
@ElMassimo ElMassimo changed the title Fix blog:check by adding node types to tsconfig fix blog:check by adding node types to tsconfig Mar 25, 2026
@ElMassimo
ElMassimo merged commit 7ed3fc3 into vite-8 Mar 25, 2026
0 of 3 checks passed
@ElMassimo
ElMassimo deleted the claude/fix-blog-check-Vy7rZ branch March 25, 2026 16:38
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