Lab: feat(security): redact sensitive method arg values from audit log

Context

Issue #243 implemented redaction of z.meta({ sensitive: true }) argument fields from per-run log files, result resource attributes, and method summary reports. However, the audit JSONL log is not yet covered.

Problem

When a method executes, the audit log records method arguments verbatim. If a field is marked sensitive: true, its value should be replaced with *** in audit entries just as it is in other output locations.

Acceptance criteria

Audit log entries for sensitive argument fields show *** rather than the plaintext value
Non-sensitive fields remain unredacted
Applies to both string and string-array typed sensitive fields

Issue #243: initial sensitive arg redaction implementation
UAT: swamp-uat#190

Implementation paused mid-stream — leaving a note for future-me / anyone resuming.

Branch: feat/issue-258-audit-log-redaction on stack72/swamp (uncommitted).

Landed and tested (61 unit tests, all passing):

src/domain/audit/audit_command_parser.ts (+ test) — tokenizes, recognizes swamp model method run / swamp model create, extracts --input / --global-arg pairs.
src/domain/audit/audit_command_redactor.ts (+ test) — assembles inputs via setNestedValue, runs extractSensitiveFieldValues against the schema, registers values with SecretRedactor, returns redacted command.
src/domain/audit/test_fixtures.ts — shared Zod schema + buildAuditFixtureDefinition().
src/domain/repo/repo_init_check.ts (+ test) — non-throwing isRepoInitialized probe.
src/cli/commands/audit.ts — extracted pure recordAuditEntry helper; registry-only lookup with null auto-resolver; lazy-load gated on parser match; try/catch around redactor.
src/cli/commands/audit_test.ts — exercises the full pipeline against a temp JsonlAuditRepository.
design/audit.md — new "Sensitive value redaction" section.
design/vaults.md — appended audit log to redaction sites list.

deno check, deno lint, deno fmt clean. 61/61 new tests green.

Open decisions before this can ship:

The original plan included a global-flag pre-walk so leading invocations like swamp --json model method run … would still redact. Implemented and tested, but on review the cost-benefit is poor: Cliffy already accepts global flags at the leaf level, so swamp model method run … --json --input password=x works fine — the parser skips unknown leaf flags. The leading-flag form is stylistic. Recommendation: rip out skipGlobalFlags + the constants + the drift test (~50 lines saved), document the caveat in design/audit.md, accept that leading-flag invocations won't redact.
src/domain/repo/repo_init_check.ts imports RepoMarkerRepository from infrastructure — same pattern as the existing primary_tool.ts violation. The architecture-boundary test (KNOWN_DOMAIN_INFRA_VIOLATIONS=22) needs a bump to 23, OR refactor to inject the repository.

Not done:

Step 10 verification gate (waiting on decision 1)
Step 1 follow-up issues — atomic with the PR open: swamp-club issue (security+enhancement) covering vault put + workflow run --input redaction; swamp-uat issue (adversarial) for audit-log scenario; cross-link comment on swamp-uat#190
PR open + link_pr lifecycle transition

Out of scope (intentional, deferred via the swamp-club follow-up): --input-file paths, swamp vault put, swamp workflow run --input, direct shell commands. Existing JSONL rows are not retroactively redacted (7-day retention sweeps; manual cleanup is rm .swamp/audit/commands-*.jsonl).

extension unyank needs -y/--yes flag for non-interactive use

Step key parsing in execution_service uses naive split(":") and silently truncates colon-containing step names

Workflow-scope report artifacts unreachable via `swamp data get --workflow`

Add --stdin support to method run and workflow run for Unix pipe composition

Docs: add run namespace to CEL expressions reference

Fix invalidate-then-reconcile sequencing in doctor extensions; failure-mode RowStates unreachable in sourceDetails

doctor extensions invalidateAll does not trigger fingerprint recheck for existing Indexed rows

Extension failure recording has dual write paths (legacy buildIndex vs W3 reconcile)

Expose workflow run ID in CEL so resource keys can be run-scoped

Add `swamp doctor workflows` subcommand to surface workflow YAML parse errors during preflight

Add 'swamp issue comment' command for updating existing issues

Close #327 — fixed in 20260511.160514.0-sha.9d03b09a

Nested workflow task fails with 'Bad resource ID' when workflowIdOrName uses @collective/ prefix

Detect stale skill directories and prompt for repo upgrade

Improve skill trigger routing for cross-model edge cases

Add type search for driver, datastore, and report kinds

Surface ReconcileFromDisk dryRun transitions in swamp doctor extensions

Docs: update doctor extensions reference for W6 aggregate-state rendering + repair flags

swamp issue: check if reporter is on an outdated binary before opening

Implement W6: swamp doctor extensions aggregate-state rendering + repair surface (extension catalog rearchitecture)

extension pull fails referencing removed source after extension source rm

Investigate whether allExtensionMethodsAttached guard can be removed via registration-path consolidation

Trace context is lost: CLI ignores inbound TRACEPARENT and raw driver doesn't propagate active context into in-process methods

swamp model create --global-arg KEY=VALUE doesn't coerce strings to z.number() schemas

swamp model create --global-arg KEY=VALUE doesn't coerce strings to z.number() schemas

swamp issue get should rate-limit unauthenticated users instead of blocking

swamp issue get should not require authentication

telemetry: emit child entries for follow-up action method invocations

Publish release-candidate / unstable extension versions

extension source rm leaves stale @local/. rows in catalog, blocking later pulls of registry types

extension push drops binaries: field from re-emitted archive manifest.yaml

macOS launchd autoupdate (club.swamp.autoupdate) silently fails — binary stays stale

Clicking @hivemq/honeycomb extension card on /extensions shows 'Something went wrong'

Docs: Update model-definitions.md and workflows.md for direct type execution

Docs: document binaries manifest field in extension-manifest.md

Add an official @swamp/ssh extension for general-purpose SSH (brownfield-friendly)

Accept and display binaries field from extension push metadata

Direct type execution: collapse model create + method run into one command

Per-method telemetry events for workflow runs

Workflow run liveness: orphaned 'running' records when originating CLI process dies mid-run

Provide a CLI-shape primer for AI agents to reduce rediscovery overhead

quality rubric: don't penalize extensions whose upstream constrains them to a single platform

Extension update rejects multiple .ts files extending the same target type within one local extension (regression)

swamp extension push deadlocks when invoked from inside a swamp workflow step

Collective-scoped auth keys + OIDC federation for CI publishing

forEach self.* in modelIdOrName not resolved in runtime execution path

Award leaderboard points for referrals and collective invites

Add agent harness detection and AiTool to telemetry

Workflow-level runtime expressions (env.*, vault.*) not resolved in driverConfig — docker driver receives literal ${{ ... }} strings

Implement W5: Per-fingerprint import URLs + subprocess test harness (extension catalog rearchitecture)

swamp config set crashes with YAML serialization error

datastore compact: VACUUM fails in compiled binary (SQLITE_LIMIT_ATTACHED=0)

Repo-level version gating: minSwampVersion high-water mark for team consistency

Docs: document self.* expressions in modelIdOrName during forEach

Docs: How-to guide for background autoupdating

Manifest version bumps silently ignored for existing local extension aggregates

materialiseExtensions misclassifies pulled rows when manifest name collides with a pulled extension

Local extension edits don't reliably trigger rebundle

Missing unique indexes on user.email and user.username allow duplicate users

Resolve self.* expressions in modelIdOrName during forEach expansion

discord-bot double-sends sign_up notifications

Discord bot sends duplicate signup notifications

Add 'swamp workflow list' as alias for 'swamp workflow search'

Add 'swamp auth status' as alias for 'swamp auth whoami'

Extension bundle cache does not invalidate on source edits

extension pull fails on @local/[email protected] phantom-claim collision when local repo has its own extension

Lab search by numeric issue ID returns no results

W3 sourceToRow writes empty source_mtime — should carry filesystem mtime through Source entity

Warm-start rebundleAndUpdateCatalog should respect terminal RowStates set by reconcile

Implement W4: KindAdapter + unified loader (extension catalog rearchitecture)

Docs: add vault read-secret command to reference manual

Extension layer garbage collection: prune catalog rows + evict orphaned bundles

Docs: document workflow concurrency limits in reference manual

Locally-sourced extension: source_mtime updates without regenerating stale bundle

Extension push: allow shipping executable host helpers (bin/mudroom blocker)

Vault expressions silently deliver __SWAMP_VSEC__ sentinels under the docker driver

First-class shell-shim support for extensions, with registry-level visibility

Local extension model bundles don't rebuild when source changes (no rebuild CLI; manual cache delete breaks the runner)

Configurable concurrency limits for workflow fan-out (forEach, parallel jobs/steps)

feat(security): redact sensitive method arg values from audit log

Workflow-level runtime expressions (env., vault.) not resolved in driverConfig — docker driver receives literal ${{ ... }} strings