[PR #2141] [CLOSED] fix: fix bookmark loading state after crawl failure #1995

Closed
opened 2026-03-02 12:00:06 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/karakeep-app/karakeep/pull/2141
Author: @MohamedBassem
Created: 11/16/2025
Status: Closed

Base: mainHead: claude/fix-bookmark-loading-state-01AgF4H2drxwuTCJDB2Xgiu4


📝 Commits (6)

  • 5cf0b02 fix: set tagging/summarization status to 'skipped' on crawling failure
  • c9bbb8e refactor: only skip pending tagging/summarization statuses
  • 56c08f2 refactor: use conditional where clauses instead of reading bookmark
  • e1bea87 refactor: wrap asset preprocessing status updates in transaction
  • 436a5e2 fix: add 'skipped' to taggingStatus and summarizationStatus types
  • f30fb7c chore: run prettier and regenerate OpenAPI spec

📊 Changes

5 files changed (+86 additions, -18 deletions)

View changed files

📝 apps/workers/workers/assetPreprocessingWorker.ts (+30 -1)
📝 apps/workers/workers/crawlerWorker.ts (+36 -7)
📝 packages/db/schema.ts (+2 -2)
📝 packages/open-api/karakeep-openapi-spec.json (+12 -6)
📝 packages/shared/types/bookmarks.ts (+6 -2)

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/karakeep-app/karakeep/pull/2141 **Author:** [@MohamedBassem](https://github.com/MohamedBassem) **Created:** 11/16/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `claude/fix-bookmark-loading-state-01AgF4H2drxwuTCJDB2Xgiu4` --- ### 📝 Commits (6) - [`5cf0b02`](https://github.com/karakeep-app/karakeep/commit/5cf0b02f8260a4dd7f07c02601504af127fb53a6) fix: set tagging/summarization status to 'skipped' on crawling failure - [`c9bbb8e`](https://github.com/karakeep-app/karakeep/commit/c9bbb8e8ed331ae3c5ae10c63eec910061ff47f3) refactor: only skip pending tagging/summarization statuses - [`56c08f2`](https://github.com/karakeep-app/karakeep/commit/56c08f22b97ff552dd16715345f5389eeb5e3738) refactor: use conditional where clauses instead of reading bookmark - [`e1bea87`](https://github.com/karakeep-app/karakeep/commit/e1bea87a219c3d63909aa5ec96453f385bbd7967) refactor: wrap asset preprocessing status updates in transaction - [`436a5e2`](https://github.com/karakeep-app/karakeep/commit/436a5e257015adf2cd31f84666cbe484593e51eb) fix: add 'skipped' to taggingStatus and summarizationStatus types - [`f30fb7c`](https://github.com/karakeep-app/karakeep/commit/f30fb7c072f51853aeb82d55783fc4298d938b7b) chore: run prettier and regenerate OpenAPI spec ### 📊 Changes **5 files changed** (+86 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `apps/workers/workers/assetPreprocessingWorker.ts` (+30 -1) 📝 `apps/workers/workers/crawlerWorker.ts` (+36 -7) 📝 `packages/db/schema.ts` (+2 -2) 📝 `packages/open-api/karakeep-openapi-spec.json` (+12 -6) 📝 `packages/shared/types/bookmarks.ts` (+6 -2) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 12:00:06 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/karakeep#1995
No description provided.