labnotes 0.0.62__tar.gz → 0.0.63__tar.gz

{labnotes-0.0.62 → labnotes-0.0.63}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: labnotes
-Version: 0.0.62
+Version: 0.0.63
 Summary: Browse experiment designs, notes, and run data from a folder.
 Project-URL: Repository, https://github.com/thekevinscott/labnotes
 Author-email: Kevin Scott <me@thekevinscott.com>
@@ -101,7 +101,7 @@ To exclude a run, add `"excluded": true` and an optional `"reason"` to its
 replaces it; both show as columns in `runs`.
 
 In a terminal, `run` shows a live table of runs (status and time) and stays
-open when they finish; `q` quits. Elsewhere, or with `--no-tui`, it prints a
+open when they finish; `q` quits. Elsewhere, it prints a
 line per event instead.
 
 - `--concurrency N` runs N at a time (default 1).

{labnotes-0.0.62 → labnotes-0.0.63}/README.md

@@ -80,7 +80,7 @@ To exclude a run, add `"excluded": true` and an optional `"reason"` to its
 replaces it; both show as columns in `runs`.
 
 In a terminal, `run` shows a live table of runs (status and time) and stays
-open when they finish; `q` quits. Elsewhere, or with `--no-tui`, it prints a
+open when they finish; `q` quits. Elsewhere, it prints a
 line per event instead.
 
 - `--concurrency N` runs N at a time (default 1).

{labnotes-0.0.62 → labnotes-0.0.63}/src/labnotes/context.md

@@ -105,7 +105,7 @@ running run under the cursor; `x` again kills it. `+` and
 `-` change how many run at once; lowering it lets running runs
 finish. Enter on a run follows its `stdout.log` and `stderr.log` together, as
 they're written, and Escape returns to the table. An `adhoc` experiment that
-plans a single run opens straight on its output. `q` quits. Elsewhere, or with `--no-tui`, it prints a
+plans a single run opens straight on its output. `q` quits. Elsewhere, it prints a
 line per event instead.
 
 - `--concurrency N` runs N at a time to start with. Without it, `concurrency`

{labnotes-0.0.62 → labnotes-0.0.63}/src/labnotes/create_app.py

@@ -1,6 +1,7 @@
 import asyncio
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
+from datetime import datetime
 from pathlib import Path
 
 from starlette.applications import Starlette
@@ -29,6 +30,7 @@ from labnotes.load_experiment import load_experiment
 from labnotes.models import LabnotesConfig
 from labnotes.notebook_kernel import NotebookKernel
 from labnotes.python_blocks import python_blocks
+from labnotes.save_conflict_copy import save_conflict_copy
 from labnotes.save_note import save_note
 from labnotes.saved_outputs import saved_outputs
 from labnotes.tables.open_tables import open_tables
@@ -127,6 +129,14 @@ def create_app(config: LabnotesConfig) -> Starlette:
             return JSONResponse({"content": current}, status_code=409)
         return Response(status_code=204)
 
+    async def post_conflict(request: Request) -> Response:
+        note = await find_note(request)
+        content = (await request.json()).get("content")
+        if not isinstance(content, str):
+            raise HTTPException(400, "Send content as a string")
+        copy = save_conflict_copy(Path(note["path"]), content, datetime.now())
+        return JSONResponse({"name": copy.name}, status_code=201)
+
     async def run_block(request: Request) -> Response:
         blocks = python_blocks((await find_note(request))["content"])
         key = (request.path_params["name"], request.path_params["note"])
@@ -152,6 +162,11 @@ def create_app(config: LabnotesConfig) -> Starlette:
             Route("/experiments/{name}/notes", post_note, methods=["POST"]),
             Route("/experiments/{name}/notes/{note:path}", get_note),
             Route("/experiments/{name}/notes/{note:path}", put_note, methods=["PUT"]),
+            Route(
+                "/experiments/{name}/notes/{note:path}/conflict",
+                post_conflict,
+                methods=["POST"],
+            ),
             Route(
                 "/experiments/{name}/notes/{note:path}/run",
                 run_block,

{labnotes-0.0.62 → labnotes-0.0.63}/src/labnotes/experiments.py

@@ -39,11 +39,6 @@ from labnotes.tui.run_tui import run_tui
     type=click.Path(exists=True, dir_okay=False, path_type=Path),
     help="With run: read this file instead of the experiment's experiment.toml.",
 )
-@click.option(
-    "--no-tui",
-    is_flag=True,
-    help="With run: print a line per event instead of the live view.",
-)
 @click.pass_context
 def experiments(
     ctx: click.Context,
@@ -52,7 +47,6 @@ def experiments(
     output_format: str,
     concurrency: int | None,
     experiment_config: Path | None,
-    no_tui: bool,
 ) -> None:
     config = load_config(ctx)
     if config is None:
@@ -68,7 +62,7 @@ def experiments(
             click.echo(f"Created {path / 'experiment.toml'}")
             return
         case str(), "run":
-            live = sys.stdout.isatty() and sys.stderr.isatty() and not no_tui
+            live = sys.stdout.isatty() and sys.stderr.isatty()
             drive = run_tui if live else lambda start, title: run_headless(start)
             try:
                 results = start_experiment(

labnotes-0.0.63/src/labnotes/save_conflict_copy.py

@@ -0,0 +1,16 @@
+from datetime import datetime
+from itertools import chain, count
+from pathlib import Path
+
+
+def save_conflict_copy(path: Path, content: str, now: datetime) -> Path:
+    stamp = f"Conflicted copy {now:%Y%m%d%H%M}"
+    tags = chain([stamp], (f"{stamp} {n}" for n in count(2)))
+    while True:
+        copy = path.with_name(f"{path.stem} ({next(tags)}){path.suffix}")
+        try:
+            with copy.open("x", encoding="utf-8") as file:
+                file.write(content)
+            return copy
+        except FileExistsError:
+            pass

{labnotes-0.0.62 → labnotes-0.0.63}/src/labnotes/static/note-editor.js

@@ -105,9 +105,6 @@ function show(status, message) {
 
 const dmp = new diff_match_patch();
 
-// Three-way: what changed on disk since base, applied onto the editor's text.
-const merge = (base, ours, theirs) => dmp.patch_apply(dmp.patch_make(base, theirs), ours)[0];
-
 // Minimal changes, so the cursor and undo history survive.
 function setText(view, text) {
   const changes = [];
@@ -125,22 +122,52 @@ function setText(view, text) {
 export function edit(note, prose, status) {
   let base = JSON.parse(note.querySelector(".note-source").textContent);
   let timer;
+  let deadline;
   let queue = Promise.resolve();
 
   const host = document.createElement("div");
   host.className = "editor";
   prose.replaceWith(host);
 
+  const notice = (message) => {
+    show(status, message);
+    setTimeout(() => {
+      if (status.textContent === message) status.hidden = true;
+    }, 4000);
+  };
+
   const run = (task) =>
     (queue = queue.then(task).catch((error) => show(status, `Not saved: ${error.message}`)));
 
-  const absorb = (theirs) => {
-    setText(view, merge(base, view.state.doc.toString(), theirs));
+  // Three-way: what changed on disk since base, applied onto the editor's text.
+  // A patch that will not apply leaves the editor alone and keeps the disk
+  // version in a sibling file, as Obsidian Sync does.
+  const absorb = async (theirs) => {
+    const ours = view.state.doc.toString();
+    const [merged, applied] = dmp.patch_apply(dmp.patch_make(base, theirs), ours);
+    const name = note.dataset.note;
+    if (applied.every(Boolean)) {
+      setText(view, merged);
+      if (ours !== base) notice(`${name} was modified externally, merging changes automatically`);
+    } else {
+      const response = await fetch(note.dataset.conflictUrl, {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({ content: theirs }),
+      });
+      if (!response.ok) throw new Error(`${response.status} ${response.statusText}`);
+      const copy = (await response.json()).name;
+      show(
+        status,
+        `${name} was modified externally and could not be merged. The outside version is saved as ${copy}.`,
+      );
+    }
     base = theirs;
   };
 
   const push = async () => {
     clearTimeout(timer);
+    deadline = undefined;
     for (;;) {
       const content = view.state.doc.toString();
       if (content === base) return;
@@ -154,7 +181,7 @@ export function edit(note, prose, status) {
         else show(status, `Not saved: ${response.status} ${response.statusText}`);
         return;
       }
-      absorb((await response.json()).content);
+      await absorb((await response.json()).content);
     }
   };
 
@@ -163,7 +190,7 @@ export function edit(note, prose, status) {
     if (!response.ok) return show(status, `Not synced: ${response.status} ${response.statusText}`);
     const theirs = await response.text();
     if (theirs === base) return;
-    absorb(theirs);
+    await absorb(theirs);
     await push();
   };
 
@@ -194,7 +221,8 @@ export function edit(note, prose, status) {
         EditorView.updateListener.of((update) => {
           if (!update.docChanged) return;
           clearTimeout(timer);
-          timer = setTimeout(save, 500);
+          deadline ??= Date.now() + 2000;
+          timer = setTimeout(save, Math.min(500, deadline - Date.now()));
         }),
       ],
     }),

{labnotes-0.0.62 → labnotes-0.0.63}/src/labnotes/templates/experiment.html

@@ -37,7 +37,7 @@
 <nav class="tabs sub">
   {% for each in e.notes %}<a href="?tab=notebooks&amp;note={{ each.name | urlencode }}"{% if each.name == note.name %} aria-current="page"{% endif %}>{{ each.name }}</a>{% endfor %}<button type="button" class="new-note" data-url="{{ url_for('post_note', name=e.name) }}" aria-label="New note" title="New note">+</button>
 </nav>
-<article class="note" data-run-url="{{ url_for('run_block', name=e.name, note=note.name) }}" data-restart-url="{{ url_for('restart_kernel', name=e.name, note=note.name) }}" data-save-url="{{ url_for('put_note', name=e.name, note=note.name) }}" data-editor="{{ asset('note-editor.js') }}" data-changes-url="{{ url_for('changes', name=e.name) }}" data-note="{{ note.name }}">
+<article class="note" data-run-url="{{ url_for('run_block', name=e.name, note=note.name) }}" data-restart-url="{{ url_for('restart_kernel', name=e.name, note=note.name) }}" data-save-url="{{ url_for('put_note', name=e.name, note=note.name) }}" data-conflict-url="{{ url_for('post_conflict', name=e.name, note=note.name) }}" data-editor="{{ asset('note-editor.js') }}" data-changes-url="{{ url_for('changes', name=e.name) }}" data-note="{{ note.name }}">
   {% if note.runnable %}<header class="note-name"><button type="button" class="restart" title="Stop this note's Python kernel. The next Run starts a fresh one.">Restart</button></header>{% endif %}
   <p class="edit-status" role="status" hidden></p>
   <div class="prose">{{ note.html | safe }}</div>