Fix: update rights on duplicate upload, add rights editor UI

smoke_signal_tab.py

@@ -505,9 +505,15 @@ def ingest_pdfs(files, rights_class: str, notes: str) -> tuple:
 
         file_hash = sha256_file(path)
 
-        # Check duplicate
+        # Check duplicate — update rights/notes if changed
         if not df.empty and file_hash in df["sha256"].values:
-            log.append(log_line(f"↩ Duplicate: {path.name}"))
+            existing_rights = df.loc[df["sha256"] == file_hash, "rights_class"].values[0]
+            if rights_class != "unknown" and existing_rights != rights_class:
+                df.loc[df["sha256"] == file_hash, "rights_class"] = rights_class
+                df.loc[df["sha256"] == file_hash, "notes"] = notes
+                log.append(log_line(f"↻ Updated rights for duplicate: {path.name} → {rights_class}"))
+            else:
+                log.append(log_line(f"↩ Duplicate: {path.name} (rights={existing_rights})"))
             dup_count += 1
             continue
 
@@ -568,6 +574,17 @@ def _ingest_status_html(state: str, new=0, dups=0) -> str:
     </div>{alert}"""
 
 
+# ── Rights class updater ──────────────────────────────────────────────────────
+def update_rights(book_id: str, new_rights: str) -> tuple:
+    """Update rights class for an existing book."""
+    df = load_manifest_df()
+    if df.empty or book_id not in df["book_id"].values:
+        return _ingest_status_html("idle"), df, f"Book ID {book_id} not found."
+    df.loc[df["book_id"] == book_id, "rights_class"] = new_rights
+    save_manifest_df(df)
+    return _ingest_status_html("done"), df, f"[{datetime.utcnow().strftime('%H:%M:%S')}] ✓ Updated {book_id} rights → {new_rights}"
+
+
 # ── Step 2: PROFILE ────────────────────────────────────────────────────────────
 def run_profile() -> tuple:
     """Profile all pending PDFs."""
@@ -1151,6 +1168,22 @@ def smoke_signal_tab():
                 refresh_btn = gr.Button("↻ Refresh Manifest", size="sm")
                 refresh_btn.click(lambda: load_manifest_df(), outputs=[manifest_table])
 
+                gr.Markdown("**Update rights class for existing book:**")
+                with gr.Row():
+                    update_book_id = gr.Textbox(label="Book ID", placeholder="SS-BOOK-0001", scale=1)
+                    update_rights_dd = gr.Dropdown(
+                        label="New Rights Class",
+                        choices=["public-domain","licensed-owned","controlled-internal","unknown"],
+                        value="licensed-owned",
+                        scale=1,
+                    )
+                    update_rights_btn = gr.Button("Update Rights", size="sm", scale=1)
+                update_rights_btn.click(
+                    update_rights,
+                    inputs=[update_book_id, update_rights_dd],
+                    outputs=[ingest_status, manifest_table, ingest_log],
+                )
+
                 ingest_btn.click(
                     ingest_pdfs,
                     inputs=[pdf_upload, rights_dd, ingest_notes],