Improve README & misc small fixes#6
Merged
Conversation
Remove developer-specific configuration files that shouldn't be tracked: - .claude/settings.local.json (local IDE permissions) - .roo/mcp.json (old project configuration) Update .gitignore to prevent these directories from being tracked in future.
Standardize project name formatting across all files: - README.md: Fix title "Devonthink" → "DEVONthink" (proper capitalization) - justfile: Update project name "MicroFn" → "DEVONthink" (correct project) - getOpenDatabases.ts: Fix typo "Claude Codet" → "Claude Code" Ensures consistent branding throughout the codebase.
…perience - Add complete prerequisites section (macOS, DEVONthink Pro, Node.js, Claude) - Add three installation options (npm global, npx, local development) - Add Claude Desktop configuration section with file location and all config options - Add comprehensive Claude Code section with CLI setup and scope configuration - Add detailed troubleshooting including real-world issues (tsc not found, restart requirements) - Fix dependency installation documentation to prevent TypeScript build errors - Improve verification and testing instructions for both Claude Desktop and Code
deadmanoz
force-pushed
the
feature/improve-readme-documentation
branch
from
fd573f3 to
a9b0163
Compare
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
ebowman
approved these changes
Aug 19, 2025
dvcrn
reviewed
Aug 22, 2025
Owner
ebowman
pushed a commit
to ebowman/mcp-server-devonthink
that referenced
this pull request
Sep 18, 2025
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
3 participants
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.
Pretty self explanatory changes. Mostly related to the README.
I added
.rooand.claudeto the.gitignorebecause I don't believe they should be there :) And I removed these files withgit rm --cached- we want them to exist but not be committed to the repo?!?!Note that I have personally followed Option 3, and got the tool working with Claude Desktop and Claude Code. I have not tested Options 1 and 2, but they look to be correct.
BTW I noticed that the
package-lock.jsonis still referring to version1.3.0yet project has been bumped to1.4.0. I opted not to do anything about that in this PR!This tool is seriously a game changer for me