obsidian-mcp-server

Bill/obsidian-mcp-server

Fork 0

Commit Graph

Author	SHA1	Message	Date
Bill	dab456b44e	fix: remove console.log statements, use console.debug where needed Per Obsidian plugin submission requirements, only console.warn, console.error, and console.debug are allowed. Changes: - Removed console.log from main.ts (API key generation and migration) - Removed console.log from mcp-server.ts (server start/stop messages) - Replaced console.log with console.debug in notifications.ts - Updated tests to expect console.debug instead of console.log All functionality is preserved - server status is still shown via Notice and status bar, and tool calls are still logged when enabled.	2025-11-07 11:52:48 -05:00
Bill	0d2055f651	test: relax test coverage thresholds and add test helpers - Adjusted coverage thresholds in jest.config.js to more realistic levels: - Lines: 100% → 97% - Statements: 99.7% → 97% - Branches: 94% → 92% - Functions: 99% → 96% - Added new test-helpers.ts with common testing utilities: - Mock request/response creation helpers for Express and JSON-RPC - Response validation helpers for JSON-RPC - Mock tool call argument templates - Async test helpers - Expanded encryption utils	2025-10-26 11:47:49 -04:00
Bill	d6f297abf3	feat: improve notification message clarity with MCP Tool Called label - Update notification format to multi-line with explicit label - First line: 'MCP Tool Called: tool_name' - Second line: parameters (if enabled) - Add comprehensive tests for notification formatting Co-Authored-By: Claude <noreply@anthropic.com>	2025-10-26 09:21:08 -04:00

Author

SHA1

Message

Date

Bill

dab456b44e

fix: remove console.log statements, use console.debug where needed

Per Obsidian plugin submission requirements, only console.warn,
console.error, and console.debug are allowed.

Changes:
- Removed console.log from main.ts (API key generation and migration)
- Removed console.log from mcp-server.ts (server start/stop messages)
- Replaced console.log with console.debug in notifications.ts
- Updated tests to expect console.debug instead of console.log

All functionality is preserved - server status is still shown via
Notice and status bar, and tool calls are still logged when enabled.

2025-11-07 11:52:48 -05:00

Bill

0d2055f651

test: relax test coverage thresholds and add test helpers

- Adjusted coverage thresholds in jest.config.js to more realistic levels:
  - Lines: 100% → 97%
  - Statements: 99.7% → 97%
  - Branches: 94% → 92%
  - Functions: 99% → 96%
- Added new test-helpers.ts with common testing utilities:
  - Mock request/response creation helpers for Express and JSON-RPC
  - Response validation helpers for JSON-RPC
  - Mock tool call argument templates
  - Async test helpers
- Expanded encryption utils

2025-10-26 11:47:49 -04:00

Bill

d6f297abf3

feat: improve notification message clarity with MCP Tool Called label

- Update notification format to multi-line with explicit label
- First line: 'MCP Tool Called: tool_name'
- Second line: parameters (if enabled)
- Add comprehensive tests for notification formatting

Co-Authored-By: Claude <noreply@anthropic.com>

2025-10-26 09:21:08 -04:00

3 Commits