feat: local-LLM workflow, dedup UX, queue management, and autosave - #1
feat: local-LLM workflow, dedup UX, queue management, and autosave#1emirikol1 wants to merge 1 commit into
Conversation
cbd5179 to
f247f0d
Compare
LM Studio path with FIFO queue (reorder/cancel), job popover, background dedup/mining, autosave, workflow markers, AI stars, restart confirm. Generic test fixtures only.
f247f0d to
ef900ee
Compare
|
Hi @emirikol1 — thank you for this, and I'm sorry it sat unanswered for five weeks. That's on me, not on the work. I want to be straight with you rather than leave it drifting, so: I can't merge this branch as it stands, and I'd rather explain properly than let it sit open pretending otherwise. Why not, honestly1. It removes the To be clear, local models were never the problem — 2. The branch is behind a rewrite. Since June the codebase has moved a long way — v0.3.0 landed a ProseMirror manuscript editor, track changes, a Codex, a compile engine, and a restructured UI. GitHub already reports this as 3. Size. Even without the drift, 17.5k lines spanning the LLM layer, job queue, dashboard and editor is more than I can review as one unit and honestly say I understood. That's a limitation of my review capacity, not a judgement on the code. What I'd genuinely like to takeSeveral of your instincts were right, and some of them independently ended up in v0.3.0 — which is a decent signal you were reading the problem correctly:
The ones still missing that your PR had and I'd welcome as focused PRs against current
Any one of those as its own PR I can review properly and would likely merge. Small and separate beats comprehensive here. On the note you opened withYou wrote that you might have changed workflows I intended, and offered to rework anything. That was a genuinely good way to open a large PR and I want to acknowledge it rather than let it pass — especially since I then left you waiting over a month for an answer. I'm leaving this open for now in case you want to pull any single piece out of it. If you'd rather not, close it and I'll take no offence — and if you do open a smaller one, I'll review it in days, not weeks. |
Note from the submitter
I built this branch while learning Novel OS on my own install. I may have changed workflows the upstream author intended — in several places I wasn't fully sure what the original design was aiming for, so I optimized for what made sense to me as a daily user (local LLM, background jobs, autosave, queue visibility). Apologies if any of that diverges from your vision. Happy to revert, rename, or rework anything that doesn't fit the project's direction.
What this PR adds (high level)
All test fixtures use generic fiction names only. No user manuscript data, secrets, or home paths are committed.
Workflow markers
Intended workflow (how to use it)
Setup
.env(gitignored).Chapter pipeline
Outline → Draft → Revised → Validated → Approved → Final
Additional chapter tools: Regenerate, Expand placeholders, Outline from notes/text (preview → keep/discard), parallel Mine jobs for plots/characters/bible. Chapter text autosaves.
Dashboard codex
LLM queue (sidebar → AI settings)
Backups & export
Explicit actions only: Backups modal, Export manuscript, Export project.
Feature list (for reviewers)
Security / privacy review (submitter)
.env,*.db,outputs/,projects/, and logs are gitignored — not in this PR.Test plan
pytest tests/from app root (137+ pass; 5 API job tests may need fresh process)