Added features - re-convert, frontmatter, export, concurrency, notifications, reset - #9
Merged
Merged
Conversation
…ons + reset
Phase 5a client-side features. Highlights:
- Right-click re-convert (replace) with strict visibility — only shown when
selection contains a PDF with a matching .md sibling. Action filters to
those items only; never silently creates fresh .md for items that didn't
have one.
- Dedupe in-flight conversions per attachment id so repeated clicks on the
same PDF don't queue multiple server-side jobs.
- Block concurrent convert batches (addon.data.batchInFlight). Menu rejects
loudly, notifier defers silently with a single 'queued' toast per cycle.
- YAML frontmatter (title, authors, year, doi, url, zotero_key, citation_key)
optionally prepended to every .md output. Default ON.
- Export-to-folder sink (exportFolderPath) alongside the Zotero attachment.
Naming: {citationKey || zoteroKey}.md.
- Parallel concurrency (maxConcurrency 1-8). DB writes serialised through a
global lock so SQLite doesn't race.
- Per-item progress in the toast (N/M counter + truncated filename).
- Managed progress hides on Zotero blur and re-appears on focus with the
current state.
- OS notification when a batch finishes and Zotero isn't focused.
- Reset-to-defaults button in prefs (confirm dialog, clears every pref).
- Plain Error for abort markers instead of DOMException (safer in the Z9
sandbox).
- HTTP error labels use response.statusText ('HTTP 504 Gateway Timeout').
- skipIfExists matches on filename so siblings under the same parent don't
shadow each other.
Removed:
- The cancel button. docling-serve has no per-task cancel API (upstream
issue docling-project/docling-serve#447) and doesn't detect client
disconnects (#401). README 'Known limitations' explains.
- asyncMaxWaitSec pref + UI. Same dishonest pattern — timing out client-
side just abandons a still-running server task.
|
🚀 This ticket has been resolved in v0.2.0. See Release v0.2.0 for release notes. |
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
What
Client-side polish: re-convert, frontmatter, export-to-folder, parallel concurrency, blur-aware toasts, OS notifications, reset-to-defaults, in-flight dedup, batch serialisation.
Removed
maxWaitSec(same dishonest pattern).