Skip to content

feat: implement bookmark update functionality - #14

Open
hiromitsusasaki wants to merge 2 commits into
mainfrom
claude/issue-3-20250604_134406
Open

feat: implement bookmark update functionality#14
hiromitsusasaki wants to merge 2 commits into
mainfrom
claude/issue-3-20250604_134406

Conversation

@hiromitsusasaki

Copy link
Copy Markdown
Owner

Summary

  • Add update-bookmark tool with validation schema
  • Implement PUT /raindrop/{id} API endpoint
  • Support updating title, tags, and collection
  • Add comprehensive tests for API and handlers
  • Include proper error handling and validation

Closes #3

Test plan

  • Unit tests for API method (full/partial updates, error handling)
  • Unit tests for tool handler (individual/multiple fields, validation)
  • Validation schema tests

Generated with Claude Code

- Add update-bookmark tool with validation schema
- Implement PUT /raindrop/{id} API endpoint
- Support updating title, tags, and collection
- Add comprehensive tests for API and handlers
- Include proper error handling and validation

Closes #3

Co-authored-by: hiromitsusasaki <hiromitsusasaki@users.noreply.github.com>
@hiromitsusasaki

Copy link
Copy Markdown
Owner Author

@claude fix lint error

@claude

claude Bot commented Jun 4, 2025

Copy link
Copy Markdown

Claude finished @hiromitsusasaki's task in 1m 34s —— View job


Fixing Lint Errors

  • Check project setup and install dependencies
  • Run linting command to identify errors
  • Fix identified lint errors
  • Commit the fixes to the PR branch

Lint error fixed successfully!

Fixed Issue:

  • Missing beforeEach import in src/__tests__/server.test.ts:1 - The beforeEach function was being used but not imported from @jest/globals

The fix adds beforeEach to the existing import statement from @jest/globals. This should resolve the linting error that was causing the CI failure.

Committed: 8bf686e

Co-authored-by: hiromitsusasaki <hiromitsusasaki@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement bookmark update functionality

1 participant