Skip to content

Conversation

@denis-shvets
Copy link
Contributor

@denis-shvets denis-shvets commented Dec 7, 2025

I'm using both systems, so I noticed that Prettier is not working in the Windows environment.

🎯 Changes

  • Fixes Windows compatibility issue with quoted glob patterns in package.json scripts
  • Ensures prettier command works correctly on Windows systems

✅ Checklist

  • 🤷 I have followed the steps in the Contributing guide. - The file does not exist. 🚨
  • I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

No impact.
The command still works on macOS.

- Fixes Windows compatibility issue with quoted glob patterns in package.json scripts
- Ensures prettier command works correctly on Windows systems
@denis-shvets
Copy link
Contributor Author

🤔 I'm not sure how to test this workflow - https://github.com/TanStack/ai/blob/main/.github/workflows/autofix.yml

@nx-cloud
Copy link

nx-cloud bot commented Dec 8, 2025

View your CI Pipeline Execution ↗ for commit 3b62d01

Command Status Duration Result
nx affected --targets=test:sherif,test:knip,tes... ✅ Succeeded 1m 54s View ↗
nx run-many --targets=build --exclude=examples/** ✅ Succeeded 34s View ↗

☁️ Nx Cloud last updated this comment at 2025-12-08 00:26:11 UTC

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 8, 2025

Open in StackBlitz

@tanstack/ai

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai@119

@tanstack/ai-anthropic

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-anthropic@119

@tanstack/ai-client

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-client@119

@tanstack/ai-devtools-core

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-devtools-core@119

@tanstack/ai-gemini

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-gemini@119

@tanstack/ai-ollama

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-ollama@119

@tanstack/ai-openai

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-openai@119

@tanstack/ai-react

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-react@119

@tanstack/ai-react-ui

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-react-ui@119

@tanstack/ai-solid

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-solid@119

@tanstack/ai-solid-ui

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-solid-ui@119

@tanstack/ai-vue

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-vue@119

@tanstack/ai-vue-ui

npm i https://pkg.pr.new/TanStack/ai/@tanstack/ai-vue-ui@119

@tanstack/react-ai-devtools

npm i https://pkg.pr.new/TanStack/ai/@tanstack/react-ai-devtools@119

@tanstack/solid-ai-devtools

npm i https://pkg.pr.new/TanStack/ai/@tanstack/solid-ai-devtools@119

commit: 3b62d01

@AlemTuzlak AlemTuzlak merged commit d991b35 into TanStack:main Dec 8, 2025
5 checks passed
@AlemTuzlak
Copy link
Contributor

i just tested it out on windows and it actually doesn't work without the '' as it tries to lint folders it shouldn't, had to revert

@denis-shvets
Copy link
Contributor Author

i just tested it out on windows and it actually doesn't work without the '' as it tries to lint folders it shouldn't, had to revert

@AlemTuzlak
Do you mean these folders? - https://github.com/TanStack/ai/blob/main/.prettierignore

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