Skip to main content
← Back to list
01Issue
FeatureShippedSwamp ClubPublic
Assigneesstack72

Relationships

#1966 Docs: local_encryption vault secrets are excluded from datastore sync — not documented anywhere

Opened by tagur · 9/2/2026· Shipped 9/3/2026

Problem

Nothing in the manual or skill docs states that local_encryption vault secrets (.swamp/secrets/) are excluded from datastore migration/sync. This exclusion appears to have landed as the fix for #650 (removing secrets from DEFAULT_DATASTORE_SUBDIRS), but the resulting behavior was never documented.

Confirmed on 20260831.202137.0-sha.7401da9d: swamp datastore status --json on a repo with live local_encryption vaults lists directories [auto-definitions, definitions-evaluated, workflows-evaluated, config, data, outputs, workflow-runs, audit, telemetry, logs, files]secrets and vault are absent from the managed set.

Without documentation, users (and agents helping them) reasonably conclude the opposite: that switching to a shared datastore (S3, GCS, etc.) risks syncing vault ciphertext (and per #650's original report, the .key file alongside it) to shared storage. That's exactly backwards from actual behavior, and it's a security-relevant gap to get wrong in either direction.

Proposed documentation

In the vault guide and/or repo/datastore guide, state explicitly:

  • local_encryption vault secrets always live in .swamp/secrets/ and are never included in datastore migration or ongoing sync, regardless of backend (filesystem, S3, GCS, etc.).
  • This is what enables a "shared workflow, per-user local credentials" pattern: a vault's definition (vaults/local_encryption/*.yaml) is git-tracked and shared across teammates, but each person's actual secret value stays local to their machine even when the rest of the repo's state syncs through a shared datastore.
  • Cross-reference #650 as the originating fix/behavior.

Alternatives

None — this is a pure documentation gap, not a behavior change.

02Bog Flow
OPENTRIAGEDIN PROGRESSSHIPPED+ 1 MOREASSIGNED+ 2 MOREREVIEW+ 4 MOREPR_MERGED+ 2 MORESESSION_SUMMARIZED

Shipped

9/3/2026, 12:29:04 AM

Click a lifecycle step above to view its details.

03Sludge Pulse
stack72 assigned stack729/3/2026, 12:11:51 AM
Editable. Press Enter to edit.

stack72 commented 9/3/2026, 12:29:33 AM

Thanks @tagur for reporting this! The fix has been merged and a release is on its way. We appreciate your contribution to swamp.

Sign in to post a ripple.