Remove langchain-starter template cruft; fix broken dev/build#23
Merged
Conversation
This repo was scaffolded from `langchain-ts-starter` and carried over config that never applied to the React + Express app it became. Several of those leftovers are exactly what made the build fail in #1, and the recent tsx migration left `npm run dev` broken. - Fix `dev:server` regression: use `tsx` instead of the uninstalled `ts-node` loader, so `npm run dev` starts the server again. - Delete dead template files: `turbo.json` (Turborepo/Next.js config, turbo never installed), `tsup.config.js` (bundler pointed at a nonexistent `src/index.ts` — the `npm run build` failure in #1), and `nodemon.json` (stale, overridden by `dev:server`'s own flags). - Trim `package.json`: drop the broken `build`/`tsup` scripts, the library-publishing `main`/`files` fields, and the now-unused `tsup` and `tsconfig-paths` devDependencies. - Fix `tsconfig.json`: remove `baseUrl: "./src"` (nonexistent) and the contradictory `app` / stale root config includes; scope to `./server`. - Document builds in the README (`npm run dev`, `npm run build:app`, server-runs-via-tsx) and explain `flatten.sh`. Verified: `npm run typecheck`, `npm run build:app`, and the server boot (`tsx server/index.ts`) all pass. Closes #1 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This repo was scaffolded from
langchain-ts-starterand carried over config that never applied to the React + Express app it became. Those leftovers are what made the build fail in #1, and the recenttsxmigration leftnpm run devbroken. This PR removes the dead config and fixes both broken commands.Changes
npm run devregression —dev:serverusednode --loader ts-node/esm, butts-nodeisn't installed (tsxis). Switched totsx, so the server starts again.turbo.json— Turborepo config targeting Next.js.next/**outputs;turbowas never installed or referenced.tsup.config.js— library bundler pointed at a nonexistentsrc/index.ts. This is thenpm run build→Cannot find src/index.tsfailure reported in Project does not build. Also, build steps are not documented. #1.nodemon.json— stale (watch: src,exec: ts-node ./src/index.ts), fully overridden bydev:server's own flags.package.json— removed the brokenbuild/tsupscripts, the library-publishingmain/filesfields, and the now-unusedtsup+tsconfig-pathsdevDependencies.tsconfig.json— droppedbaseUrl: "./src"(nonexistent dir) and the contradictoryapp/ stale rootpostcss/tailwindincludes; scoped to./server(theapp/has its own tsconfig).npm run dev,npm run build:app(the real frontend build), that the server runs directly viatsx, and whatflatten.shis for.Verification
npm run typechecknpm run build:appapp/distdev:serverviatsxServer started at http://localhost:…npm installCloses #1
🤖 Generated with Claude Code