added some changes based on my setup experience#5
Open
shinronin wants to merge 1 commit intoapolosan:masterfrom
Open
added some changes based on my setup experience#5shinronin wants to merge 1 commit intoapolosan:masterfrom
shinronin wants to merge 1 commit intoapolosan:masterfrom
Conversation
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
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.
Great MCP Server
Thanks for your work! After getting set up, I ran some recent tooling changes by your tools and it suggested several relevant improvements. Love it!
Summary
This PR improves the first-run setup experience based on issues encountered while setting up the project from scratch. The main changes are:
Add npm setup instructions alongside bun — The project currently requires
bunfor thepreparelifecycle script. Users withoutbunhad no documented path forward. README, QUICKSTART, and CHANGELOG now includenpm install --ignore-scripts && npx tscas an alternative, with individualnode dist/src/cli/*.jscommands for database setup.Fix relative path issues with Cursor and other MCP clients — Cursor does not reliably honor the
cwdfield for stdio MCP servers, causing "Cannot find module" and database errors when using relative paths. All config examples now use absolute paths, with an explicit callout explaining why. The${workspaceFolder}interpolation option is documented for project-level configs.Route seeder warnings through structured logger —
pattern-seeder.tsusedconsole.warn()for missing cross-references during initialization. MCP clients interpret stderr output as errors, which caused noisy false-alarm error banners in Cursor. These are now routed throughlogger.warn()so they respectLOG_LEVELand write to stdout.Remove committed SQLite database from repo — The 7.5 MB
data/design-patterns.dbbinary was tracked in git. Sincedb:setupregenerates it, this removes it and updates.gitignoreto ignore all.dbfiles (removing the!data/design-patterns.dbexception).Remove unused
promptscapability — The MCP server declared an emptyprompts: {}capability that isn't implemented; removed to avoid confusion.Expand troubleshooting section — Added specific entries for common failure modes:
bunnot found duringnpm install, Cursor "Cannot find module" errors, and MCP stderr noise from seeder warnings.Test plan
bun install && bun run build && bun run db:setupstill works end-to-endnpm install --ignore-scripts && npx tscbuilds successfully withoutbun