dirsql 0.4.52__tar.gz → 0.4.54__tar.gz

{dirsql-0.4.52 → dirsql-0.4.54}/Cargo.lock

@@ -552,7 +552,7 @@ checksum = "6184e33543162437515c2e2b48714794e37845ec9851711914eec9d308f6ebe8"
 
 [[package]]
 name = "dirsql"
-version = "0.4.52"
+version = "0.4.54"
 dependencies = [
  "assert_cmd",
  "axum",
@@ -599,7 +599,7 @@ dependencies = [
 
 [[package]]
 name = "dirsql-py-ext"
-version = "0.4.52"
+version = "0.4.54"
 dependencies = [
  "dirsql",
  "pyo3",

{dirsql-0.4.52 → dirsql-0.4.54}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dirsql
-Version: 0.4.52
+Version: 0.4.54
 Requires-Dist: bin-shim>=0.1
 Summary: Ephemeral SQL index over a local directory
 Keywords: sql,filesystem,directory,sqlite,index

{dirsql-0.4.52 → dirsql-0.4.54}/dirsql/cli/main.py

@@ -18,27 +18,21 @@ from .discover_plugins.with_discovered_plugins import with_discovered_plugins
 from .resolve_config_extensions import with_resolved_extensions
 
 
-def _absorb_interrupt(*_args: object) -> None:
-    """Let the core's own shutdown decide the exit code on SIGINT.
-
-    signal-hook (which tokio uses) *chains*: it runs tokio's handler — which
-    drives `dirsql server`'s graceful shutdown, after which `run_cli` returns
-    0 — and then whatever handler was installed before it. CPython's default
-    is `default_int_handler`, which raises `KeyboardInterrupt`; that lands
-    after `run_cli` has already returned 0 and turns a clean shutdown into a
-    130. This handler occupies that slot without raising, so the core's exit
-    code is the one that survives, exactly as it does when the CLI is its own
-    process.
-
-    A signal arriving when the core is NOT handling signals still terminates:
-    `run_cli` is only reached with this installed, and it returns promptly for
-    every non-server command.
+def keep_sigint_fatal(handler=signal.signal):
+    """Give SIGINT its default disposition for the core's run, returning the prior one.
+
+    No Python-level handler can end a run in progress: `run_cli` detaches the
+    GIL for its whole duration, so nothing reaches the eval loop until the
+    core has already finished — CPython's `default_int_handler` included. Only
+    a disposition the kernel acts on by itself works, which is `SIG_DFL`, and
+    it is what the standalone binary runs with.
+
+    `dirsql server` keeps its exit code: signal-hook (which tokio uses)
+    replaces the disposition when the server registers its handlers, and does
+    not re-raise `SIG_DFL` afterwards, so the core's graceful shutdown is the
+    only thing that acts and its 0 survives.
     """
-
-
-def with_core_owned_signals(handler=signal.signal):
-    """Install `_absorb_interrupt` for SIGINT and return the prior handler."""
-    return handler(signal.SIGINT, _absorb_interrupt)
+    return handler(signal.SIGINT, signal.SIG_DFL)
 
 
 def main(argv: list[str] | None = None) -> int:
@@ -56,7 +50,7 @@ def main(argv: list[str] | None = None) -> int:
         print(f"dirsql: {exc}", file=sys.stderr)
         return 1
 
-    previous = with_core_owned_signals()
+    previous = keep_sigint_fatal()
     try:
         return run_in_process(argv=argv, module="dirsql._dirsql")
     except Exception as exc:

{dirsql-0.4.52 → dirsql-0.4.54}/packages/python/Cargo.toml

@@ -4,7 +4,7 @@ name = "dirsql-py-ext"
 # pypi/maturin handler can rewrite it via `write-version` before
 # `maturin build`. `pyproject.toml` declares `dynamic = ["version"]`
 # and maturin reads this field. Mirrors `packages/rust/Cargo.toml`.
-version = "0.4.52"
+version = "0.4.54"
 # abi3 note: the `abi3-py310` pyo3 feature (below) builds ONE stable-ABI
 # `cp310-abi3` wheel per platform that loads on every CPython >= 3.10
 # (matching `pyproject.toml`'s `requires-python = ">=3.10"`), instead of

dirsql-0.4.54/packages/python/changelog.d/2026-09-20-sigint-ends-a-run-in-progress.md

@@ -0,0 +1 @@
+**Fixed** Ctrl-C now ends a `dirsql` run that is still in progress. The launcher installed an empty SIGINT handler for the whole of the core's run, and because the core detaches the GIL for that call no Python handler can run until the work has already finished — so a signal sent during a long directory scan was swallowed and the process was SIGTERM-only. SIGINT is left at its default disposition instead, which the kernel acts on immediately, matching the standalone binary. `dirsql server`'s graceful shutdown still exits 0.

{dirsql-0.4.52 → dirsql-0.4.54}/packages/python/changelog.d/README.md

@@ -14,4 +14,4 @@ Each PR that touches this package's public-facing source adds one file here:
 Fragments are **permanent and append-only** — nothing is ever assembled back
 into a single `CHANGELOG.md`, and the release↔entry mapping comes from
 `git log --tags`. This README is not a fragment and never satisfies the
-changelog gate. See AGENTS.md, "Changelog and Migrations".
+changelog gate. See `agents/reference/changelog-migrations.md`.

dirsql-0.4.54/packages/python/e2e-attestations/claude-1148-clippy-all-targets.json

@@ -0,0 +1,7 @@
+{
+  "command": "just test-e2e",
+  "ran_at": 1789752342,
+  "exit_code": 0,
+  "commit": "a597fe5a5b7f4a27b7132385027bc0b71e447138",
+  "branch": "claude/1148-clippy-all-targets"
+}

dirsql-0.4.54/packages/python/e2e-attestations/claude-1164-sigint-fatal.json

@@ -0,0 +1,7 @@
+{
+  "command": "just test-e2e",
+  "ran_at": 1789910327,
+  "exit_code": 0,
+  "commit": "dfc68b057d52f3d4f586fd0233f292ccc2062dd6",
+  "branch": "claude/1164-sigint-fatal"
+}

dirsql-0.4.54/packages/python/migrations.d/2026-09-20-sigint-ends-a-run-in-progress.md

@@ -0,0 +1,35 @@
+### SIGINT ends a run in progress through the Python launcher
+
+**Summary**
+
+The `dirsql` console script installed an empty SIGINT handler for the duration of the core's run. A signal arriving during a long directory scan was therefore discarded, and the process could only be ended with SIGTERM. SIGINT is now left at its default disposition for that window, so the kernel terminates the process immediately. No API changes; the observable difference is that a signal that used to do nothing now ends the process with the usual 128+SIGINT status.
+
+**Required changes**
+
+| Before | After |
+| --- | --- |
+| `kill -INT <pid>` during a query: ignored; the run continues to completion | `kill -INT <pid>` during a query: the process dies of SIGINT (shell reports `130`) |
+| Ctrl-C while the REPL is executing a statement: ignored | Ctrl-C while the REPL is executing a statement: the session ends |
+
+A script that sent SIGINT to a `dirsql` process and relied on it being ignored must stop sending it. A script that escalated to SIGTERM after an ignored SIGINT can drop the escalation.
+
+`dirsql server` is unchanged: its graceful shutdown still exits `0` on SIGINT and SIGTERM. Ctrl-C at an idle REPL prompt is unchanged too — reedline reads it as a key event, not a signal.
+
+**Deprecations removed**
+
+_None._
+
+**Behavior changes without code changes**
+
+The exit status of an interrupted run changes from "no exit" to death by SIGINT. This aligns the Python launcher with the standalone `dirsql` binary, which has always behaved this way.
+
+**Verification**
+
+Against a directory large enough to take a few seconds to scan:
+
+```bash
+dirsql "SELECT count(*) FROM './'"   # press Ctrl-C while it runs
+echo $?
+```
+
+Expected: the process ends promptly and the shell reports `130`. Before this change it kept running and the SIGINT was discarded. Run it in the foreground: a non-interactive shell sets SIGINT to `SIG_IGN` for `&` background jobs, which hides the difference.

{dirsql-0.4.52 → dirsql-0.4.54}/packages/python/migrations.d/README.md

@@ -15,5 +15,5 @@ the API adds one file here:
   `_None._` where a subsection does not apply.
 
 Fragments are **permanent and append-only** — nothing is assembled back into a
-single `MIGRATIONS.md`. This README is not a fragment. See AGENTS.md,
-"Changelog and Migrations".
+single `MIGRATIONS.md`. This README is not a fragment. See
+`agents/reference/changelog-migrations.md`.

{dirsql-0.4.52 → dirsql-0.4.54}/packages/rust/Cargo.toml

@@ -6,7 +6,7 @@ description = "Ephemeral SQL index over a local directory"
 # version stays for the binding crates (publish = false). The literal
 # intentionally lags the published version: putitoutthere rewrites it to
 # the planned release version at build time and never commits it back.
-version = "0.4.52"
+version = "0.4.54"
 edition.workspace = true
 # Literal `license` (not `license.workspace = true`) because
 # putitoutthere's preflight check reads `[package].license` directly

{dirsql-0.4.52 → dirsql-0.4.54}/packages/rust/src/lib.rs

@@ -2259,7 +2259,7 @@ mod readonly_tests {
         );
         // `watch`, `config`, `matcher` wrap a typed StdError to preserve a
         // `source()` chain.
-        let io = || std::io::Error::new(std::io::ErrorKind::Other, "x");
+        let io = || std::io::Error::other("x");
         let watch_err = DirSqlError::watch(io());
         assert_eq!(watch_err.to_string(), "watcher error: x");
         assert!(StdError::source(&watch_err).is_some());
@@ -4273,8 +4273,7 @@ mod internal_tests {
             &mut sources,
             &mut out,
         )
-        .err()
-        .expect("duplicate function name must error");
+        .expect_err("duplicate function name must error");
         let msg = err.to_string();
         assert!(msg.contains("'dup'"), "got: {msg}");
         assert!(msg.contains("/a/frag.toml"), "got: {msg}");
@@ -4293,8 +4292,7 @@ mod internal_tests {
             &mut sources,
             &mut out,
         )
-        .err()
-        .expect("intra-config duplicate must error");
+        .expect_err("intra-config duplicate must error");
         assert!(err.to_string().contains("'dup'"), "got: {err}");
     }
 

{dirsql-0.4.52 → dirsql-0.4.54}/packages/rust/src/vtab_scaffold.rs

@@ -289,9 +289,7 @@ mod tests {
 
     #[test]
     fn parse_gitignore_rejects_an_unknown_switch() {
-        let err = parse_gitignore("sometimes")
-            .err()
-            .expect("an unknown switch must be rejected");
+        let err = parse_gitignore("sometimes").expect_err("an unknown switch must be rejected");
         assert!(matches!(err, Error::ModuleError(_)), "got {err:?}");
         assert!(err.to_string().contains("no-gitignore"), "got: {err}");
     }