fix: correct manifest.json to pass MCPB validation

- Add required 'version' field (1.0.0)
- Change 'author' from string to object with name/url
- Move 'mcp_config' inside 'server' object
- Fix 'user_config' to use proper field-level structure
- Remove invalid 'compatibility' and 'capabilities' top-level fields
- Create bundle icon (512x512 PNG) to satisfy icon requirement
- Remove icon placeholder file

Validation: ✅ mcpb validate passes with warnings only
Tasks: T149 (icon), T158 (validate), T159 (build) marked complete

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

specs/001-obsidian-mcp-bundle/tasks.md

@@ -275,7 +275,7 @@
 
 **Purpose**: Improvements that affect multiple user stories
 
-- [ ] T149 [P] Add bundle icon (icon.png) to assets/ directory
+- [X] T149 [P] Add bundle icon (icon.png) to assets/ directory
 - [ ] T150 [P] Create comprehensive README.md with all 95 tools documented
 - [ ] T151 [P] Add CHANGELOG.md following semver conventions
 - [ ] T152 [P] Update manifest.json tools array with accurate descriptions
@@ -284,8 +284,8 @@
 - [ ] T155 [P] Implement consistent error response structure across all tools
 - [ ] T156 [P] Add comprehensive parameter sanitization for security
 - [ ] T157 [P] Optimize CLI command construction for performance
-- [ ] T158 Verify manifest.json with `mcpb pack --validate`
-- [ ] T159 Run TypeScript build (`npm run build`) and verify no errors
+- [X] T158 Verify manifest.json with `mcpb pack --validate`
+- [X] T159 Run TypeScript build (`npm run build`) and verify no errors
 - [ ] T160 Test bundle packaging with `npm run pack` (creates .mcpb file)
 - [ ] T161 Validate quickstart.md scenarios against implemented tools
 - [ ] T162 [P] Add platform-specific testing (macOS, Windows, Linux)