Lab: Datastore lock not released when interactive command fails in non-TTY context

Description

When swamp model search is run in a non-TTY context (e.g. from an AI agent or script), it acquires the datastore lock, then crashes because Ink's interactive UI requires raw mode on stdin. The lock file (.swamp/.datastore.lock) is never released, blocking all subsequent swamp CLI commands until the lock times out or is manually removed.

Steps to Reproduce

Run swamp model search <query> from a non-TTY context (e.g. piped stdin, or from an AI coding agent's shell tool)
The command crashes with: Raw mode is not supported on the current process.stdin
Run any other swamp command (e.g. swamp model search <query> --json)
The second command hangs and eventually fails with: Lock ".swamp/.datastore.lock" held by <user> (pid <old_pid>) — timed out after 60194ms

Expected Behavior

The datastore lock should be released in a finally block (or equivalent cleanup) so that a crash in the interactive UI layer does not leave a stale lock behind.
Alternatively, interactive commands should detect non-TTY stdin early (before acquiring the lock) and either fall back to --json output or fail with a clear error message.

Actual Behavior

The lock is acquired before the Ink UI is initialized. When Ink crashes due to missing raw mode support, the process exits without releasing the lock. Subsequent commands are blocked for ~60 seconds until the lock times out.

Environment

swamp version: 20260310.230813.0-sha.fcc69c53
OS: macOS (Darwin 25.3.0)
Shell: zsh
Context: Claude Code CLI (non-interactive shell)

Error Output

ERROR Raw mode is not supported on the current process.stdin, which Ink uses
      as input stream by default.

Followed by (on next command):

{
  "error": "Lock ".swamp/.datastore.lock" held by user@host (pid 39643) — timed out after 60194ms"
}

Summary

This bug affects the CLI lifecycle management around the datastore lock. The fix would involve ensuring lock release happens in a cleanup/finally path that runs even when the interactive rendering layer crashes. A secondary improvement would be to detect non-TTY stdin before launching interactive UI components and either fall back gracefully or fail fast with a descriptive error.

yep yep yep

Support named argument syntax for model method run (e.g. --arg key=value)

env var overrides for globalArguments silently change which environment a model targets

deno lint no-import-prefix conflicts with swamp's required npm:/jsr: inline specifiers

AI agents should search community extensions before offering to create custom models

feat: add --content-type filter to extension search

Add a 'reports' feature

Port remaining CLI commands to libswamp + renderer pattern

Persistent runner / server mode to eliminate per-invocation CLI startup overhead

AI agent should prefer fan-out model methods over shell loops for fleet operations

Support unbundled helper scripts in extension packages

Add progress indicator for long-running model methods

Built-in structured output parser for CLI commands

Automatic trace context propagation across workflow steps

Native OpenTelemetry tracing for swamp CLI internals

Claude Code should prefer extending swamp extensions over using external CLI tools

Framework log lines written to stdout pollute model method output

Swamp skill should guide AI agents to create extension models instead of inline shell scripts

Shared team collectives for collaborative extension publishing

Delete methods should return last known state data

Add smoke-test skill for extension models

Workflow run output is unreadable — resource data and errors are buried in log noise

Recommend or manage lockfile strategy for extension model npm dependencies

Output of running a method is not the actual output

Add macOS Keychain vault type

bug: Vault secret shell escaping incomplete — $(), semicolons, and pipes are interpreted

Swamp should support "Scheduled"-type status for execution

feat: add support for Nix via a flake

ModelType.normalize replaces all dots with slashes, corrupting dotted type names

fix: collective validator error message omits drivers and datastores

Skill docs: document configSchema inline requirement and .describe() constraints for vault extensions

It takes 42 seconds to run swamp --help

After today upgrade

Auto-resolver fails when extensions/models/ directory does not exist for vault-only extensions

Azure Key Vault extension vault bundle fails to load in compiled binary

RangeError: Maximum call stack size exceeded when loading extension models with npm dependencies after upgrade to 20260317

Extension Zod schema incompatibility after upgrade to 20260312

Data garbage collection never fully removes expired data from disk

extension push --dry-run requires authentication unnecessarily

Typed context APIs to eliminate no-explicit-any in extension models

AWS extensions fail to load: missing _lib/aws.ts shared dependency

getContent() should resolve vault expressions for sensitive fields

getContent() should return parsed objects, not raw Uint8Array

Add context.readResource() to MethodContext for user extension models

Datastore lock: allow concurrent reads across different models

extension install should be an alias of extension pull

feat: Execution Driver Abstraction — Pluggable Isolation for Model Execution

Support --global-arg on model create

consider a libswamp

Running workflow should not block read-only operations

Update from 20260227 to 20260311 breaks existing model definitions (symlink + path-based type)

loadUserModels ignores --repo-dir, uses cwd instead

datastore lock release --force deadlocks on the stale lock it's trying to release

Datastore lock not released when interactive command fails in non-TTY context

vault put to 1Password stores empty password field

Leaderboard shows 0-event users whose profiles 404

Move AWS, Azure, and 1Password vault providers to extensions

Data instance name uniqueness is global across specs instead of per-spec

Audit timeline misses today's entries in UTC+ timezones

AI agent discoverability: reduce trial-and-error for CLI syntax

GlobalArguments CEL expressions fail when factory inputs are omitted — selective evaluation not working

Document factory pattern for model reuse in skills

Delete method uses identifier as data name instead of instance name

CLI method run ignores required arguments without .default()

Workflow run output mixes data, errors, and logs without clear separation — hard to find what actually executed

data.latest() in workflow step inputs resolved at workflow start, not at step execution time

Update method fails when globalArguments contain cross-model CEL expressions

Track resource deletion in local data model

fix: extension npm packages fail to resolve in compiled binary

swamp repo upgrade replaces entire CLAUDE.md with generic template instead of merging

Add drift detection to reconcile stored state against live cloud resources

Model inputs schema validated on all methods — delete fails requiring create-time inputs

Support wait-until-ready semantics for async resource provisioning

support pushing to organizations

Add data rename/move command for non-destructive refactoring

Schema mismatch warnings lack model context and misrepresent API response shapes

Support pre-flight validate method on models before method execution

Extension models need built-in support for idempotent create operations

Workflow reports succeeded when step outputs indicate failure

Swamp skills don't prevent AI agents from manually generating UUIDs for workflows