Skip to content

Conversation

@elmarburke
Copy link
Collaborator

Adding an example an HTTP JSON API of collection items

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment @cursor review or bugbot run to trigger another review on this PR

"private": true,
"type": "module",
"scripts": {
"dev": "node --watch src/index.ts",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Dev script runs TypeScript without TypeScript runtime

The dev script uses node --watch src/index.ts to run a TypeScript file directly, but Node.js cannot execute TypeScript files without either a TypeScript runtime like tsx or experimental flags. The csv-importer example correctly adds tsx to devDependencies for this purpose, but json-api lacks this dependency. Running npm run dev will fail with a syntax error on most Node.js versions.

Fix in Cursor Fix in Web

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works fine with modern node versions. Welcome to the future, LLM.

@elmarburke elmarburke merged commit a4f7122 into main Dec 17, 2025
4 checks passed
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.

3 participants